Adrian Bunk wrote: > On Tue, Nov 28, 2006 at 08:45:56PM -0800, Trent Piepho wrote: > > On Wed, 29 Nov 2006, Adrian Bunk wrote: > > > On Tue, Nov 28, 2006 at 01:06:02PM -0800, Trent Piepho wrote: > > > > On Sun, 26 Nov 2006, Adrian Bunk wrote: > > > > > DVB_AV7110_FIRMWARE was (except for some OSS drivers) the only option > > > > > that was still compiling a binary-only user-supplied firmware file at > > > > > build-time into the kernel. > > > > > > > > > > This patch changes the driver to always use the standard > > > > > request_firmware() way for firmware by removing DVB_AV7110_FIRMWARE. > > > > > > > > Doesn't this also prevent the AV7110 module from getting compiled > > > > into the kernel? Shouldn't the Kconfig file be adjusted so > > > > that 'y' can't be selected anymore and it depends on MODULES? > > > > > > No. > > > No. > > > > > > request_firmware() works fine for built-in drivers. > > > > Wouldn't that require loading the firmware file before the filesystems are > > mounted? > > Sure.
And you have to create an initrd for the firmware! As I wrote before: I NAK any attempt to remove this option. The option _is_ useful because it allows a user to build an av7110 driver without hotplug, initrd etc. Nobody has to use this option, but it should be possible to do so. CU Oliver -- -------------------------------------------------------- VDR Remote Plugin 0.3.8 available at http://www.escape-edv.de/endriss/vdr/ -------------------------------------------------------- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/