Messages in this thread | | | Subject | Re: [PATCH 4/15] vub300: fix module parameter. | From | Tony Olech <> | Date | Mon, 19 Dec 2011 13:08:40 +0000 |
| |
On Thu, 2011-12-15 at 13:34 +1030, Rusty Russell wrote: > You didn't mean this to be a bool. > > Cc: Tony Olech <tony.olech@elandigitalsystems.com> > Cc: Chris Ball <cjb@laptop.org> > Cc: linux-mmc@vger.kernel.org > Cc: linux-usb@vger.kernel.org > Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> > --- > drivers/mmc/host/vub300.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/vub300.c b/drivers/mmc/host/vub300.c > --- a/drivers/mmc/host/vub300.c > +++ b/drivers/mmc/host/vub300.c > @@ -259,7 +259,7 @@ static int firmware_rom_wait_states = 0x > static int firmware_rom_wait_states = 0x1C; > #endif > > -module_param(firmware_rom_wait_states, bool, 0644); > +module_param(firmware_rom_wait_states, int, 0644); > MODULE_PARM_DESC(firmware_rom_wait_states, > "ROM wait states byte=RRRIIEEE (Reserved Internal External)"); > > -- You are right. I don't know how that slipped through Acked-by: Tony Olech <tony.olech@elandigitalsystems.com> Acked-by: Tony Olech <tony@olech.com>
| |