On Thu, 2012-07-26 at 12:26 -0600, Tim Gardner wrote: > Cc: Andy Walls <[email protected]> > Cc: Mauro Carvalho Chehab <[email protected]> > Cc: [email protected] > Cc: [email protected] > Signed-off-by: Tim Gardner <[email protected]> > --- > drivers/media/video/ivtv/ivtv-firmware.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/media/video/ivtv/ivtv-firmware.c > b/drivers/media/video/ivtv/ivtv-firmware.c > index 02c5ade..6ec7705 100644 > --- a/drivers/media/video/ivtv/ivtv-firmware.c > +++ b/drivers/media/video/ivtv/ivtv-firmware.c > @@ -396,3 +396,7 @@ int ivtv_firmware_check(struct ivtv *itv, char *where) > > return res; > } > + > +MODULE_FIRMWARE(CX2341X_FIRM_ENC_FILENAME); > +MODULE_FIRMWARE(CX2341X_FIRM_DEC_FILENAME); > +MODULE_FIRMWARE(IVTV_DECODE_INIT_MPEG_FILENAME);
Only the PVR-350, based on the iTVC-15/CX23415 chip, needs the CX2341X_FIRM_DEC_FILENAME and IVTV_DECODE_INIT_MPEG_FILENAME. (And even in the case of that card, not having the IVTV_DECODE_INIT_MPEG_FILENAME file is non-fatal.) I would not want anything in user-space or kernel space preventing the ivtv module from loading, if some of those files don't exist. Regards, Andy -- 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/
