On 07/25/2012 07:48 AM, Antti Palosaari wrote: > On 07/25/2012 04:24 PM, Devin Heitmueller wrote: >> On Wed, Jul 25, 2012 at 9:15 AM, Tim Gardner >> <tim.gardner@xxxxxxxxxxxxx> wrote: >>> This will make modinfo more useful with regard >>> to discovering necessary firmware files. >>> >>> Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> >>> Cc: Michael Krufky <mkrufky@xxxxxxxxxxxxxx> >>> Cc: Eddi De Pieri <eddi@xxxxxxxxxxx> >>> Cc: linux-media@xxxxxxxxxxxxxxx >>> Signed-off-by: Tim Gardner <tim.gardner@xxxxxxxxxxxxx> >>> --- >>> drivers/media/common/tuners/xc5000.c | 8 ++++++-- >>> 1 file changed, 6 insertions(+), 2 deletions(-) >>> >>> diff --git a/drivers/media/common/tuners/xc5000.c >>> b/drivers/media/common/tuners/xc5000.c >>> index dcca42c..4d33f86 100644 >>> --- a/drivers/media/common/tuners/xc5000.c >>> +++ b/drivers/media/common/tuners/xc5000.c >>> @@ -210,13 +210,15 @@ struct xc5000_fw_cfg { >>> u16 size; >>> }; >>> >>> +#define XC5000A_FIRMWARE "dvb-fe-xc5000-1.6.114.fw" >>> static const struct xc5000_fw_cfg xc5000a_1_6_114 = { >>> - .name = "dvb-fe-xc5000-1.6.114.fw", >>> + .name = XC5000A_FIRMWARE, >>> .size = 12401, >>> }; >>> >>> +#define XC5000C_FIRMWARE "dvb-fe-xc5000c-41.024.5.fw" >>> static const struct xc5000_fw_cfg xc5000c_41_024_5 = { >>> - .name = "dvb-fe-xc5000c-41.024.5.fw", >>> + .name = XC5000C_FIRMWARE, >>> .size = 16497, >>> }; >>> >>> @@ -1253,3 +1255,5 @@ EXPORT_SYMBOL(xc5000_attach); >>> MODULE_AUTHOR("Steven Toth"); >>> MODULE_DESCRIPTION("Xceive xc5000 silicon tuner driver"); >>> MODULE_LICENSE("GPL"); >>> +MODULE_FIRMWARE(XC5000A_FIRMWARE); >>> +MODULE_FIRMWARE(XC5000C_FIRMWARE); >>> -- >> >> Hi Tim, >> >> I'm just eyeballing the patch and I'm not familiar with this new >> functionality, but where are the new macros you're specifying actually >> defined? You're swapping out the filename for XC5000A_FIRMWARE, but >> where is the actual reference to "dvb-fe-xc5000-1.6.114.fw"? >> >> Also, Mauro, can I merge this into my tree first rather than you >> pulling it direct? I've got a whole patch series for xc5000 that I'm >> slated to issue a PULL for this weekend, and I *really* don't want to >> rebase the series for a four line change (which will definitely cause >> a conflict). >> >> Devin > > Also this issue has been spoken earlier and nacked. It was 2009 when Ben > Hutchings sends large patch serie adding MODULE_FIRMWARE for every > Linux-Media driver. I am not sure if arguments are changed after that to > allow it now. > http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg11373.html > > regards > Antti > That discussion went completely off the rails. MODULE_FIRMWARE() is purely informational and describes the firmware files that your driver _might_ need. I'm using this information to pare down firmware files to those that are actually used by the kernel. It makes quite a difference in space required for Live CDs and on users disks. rtg -- Tim Gardner tim.gardner@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html