Re: [PATCH] cpia2: use request_firmware()

From: Mauro Carvalho Chehab
Date: Mon Jun 23 2008 - 08:17:22 EST


Hi David,

On Mon, 23 Jun 2008 12:03:33 +0100
David Woodhouse <dwmw2@xxxxxxxxxxxxx> wrote:

> diff --git a/drivers/media/video/cpia2/Kconfig b/drivers/media/video/cpia2/Kconfig
> index e39a961..64d9866 100644
> --- a/drivers/media/video/cpia2/Kconfig
> +++ b/drivers/media/video/cpia2/Kconfig
> @@ -1,9 +1,17 @@
> config VIDEO_CPIA2
> tristate "CPiA2 Video For Linux"
> depends on VIDEO_DEV && USB && VIDEO_V4L1
> + select FW_LOADER

Since select doesn't check for dependencies, on all places you select
FW_LOADER, you also need to add "depends on HOTPLUG" since FW_LOADER is
defined as(at drivers/base/Kconfig):

config FW_LOADER
tristate "Userspace firmware loading support"
depends on HOTPLUG

Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/