Hello, this patch fix 3 calls to module_param_string() in driver/media/video/tuner-core.c and drivers/media/video/tda9887.c. In all three places, the len and the perm parameter was switched.
Patch is against 2.6.12-rc1. Signed-off-by: Bert Wesarg <[EMAIL PROTECTED]> diff -uprN linux-2.6.12-rc1.orig/drivers/media/video/tda9887.c linux-2.6.12-rc1/drivers/media/video/tda9887.c --- linux-2.6.12-rc1.orig/drivers/media/video/tda9887.c 2005-03-27 14:44:23.000000000 +0200 +++ linux-2.6.12-rc1/drivers/media/video/tda9887.c 2005-03-27 14:48:06.000000000 +0200 @@ -478,9 +478,9 @@ static int tda9887_set_pinnacle(struct t /* ---------------------------------------------------------------------- */ static char pal[] = "-"; -module_param_string(pal, pal, 0644, sizeof(pal)); +module_param_string(pal, pal, sizeof(pal), 0644); static char secam[] = "-"; -module_param_string(secam, secam, 0644, sizeof(secam)); +module_param_string(secam, secam, sizeof(secam), 0644); static int tda9887_fixup_std(struct tda9887 *t) { diff -uprN linux-2.6.12-rc1.orig/drivers/media/video/tuner-core.c linux-2.6.12-rc1/drivers/media/video/tuner-core.c --- linux-2.6.12-rc1.orig/drivers/media/video/tuner-core.c 2005-03-27 14:44:23.000000000 +0200 +++ linux-2.6.12-rc1/drivers/media/video/tuner-core.c 2005-03-27 14:47:17.000000000 +0200 @@ -162,7 +162,7 @@ static void set_type(struct i2c_client * } static char pal[] = "-"; -module_param_string(pal, pal, 0644, sizeof(pal)); +module_param_string(pal, pal, sizeof(pal), 0644); static int tuner_fixup_std(struct tuner *t) { - 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/