From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752846Ab1LSNI5 (ORCPT ); Mon, 19 Dec 2011 08:08:57 -0500 Received: from moutng.kundenserver.de ([212.227.17.9]:62255 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752346Ab1LSNIx (ORCPT ); Mon, 19 Dec 2011 08:08:53 -0500 Subject: Re: [PATCH 4/15] vub300: fix module parameter. From: Tony Olech To: Rusty Russell Cc: lkml - Kernel Mailing List , Pawel Moll , Chris Ball , linux-mmc@vger.kernel.org, linux-usb@vger.kernel.org In-Reply-To: <1324062122.2302.5.camel@study-2> References: <871us6tu0d.fsf@rustcorp.com.au> <1324062122.2302.5.camel@study-2> Content-Type: text/plain; charset="UTF-8" Date: Mon, 19 Dec 2011 13:08:40 +0000 Message-ID: <1324300120.4283.6.camel@study-2> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 Content-Transfer-Encoding: 7bit X-Provags-ID: V02:K0:EFkOPt1EP7SkozkrDdoBl5H0G70JdquizsblOee6+uE kReTQaLrT6+RvrSep5YUk0OO07llzJQo4kMK6pAo1YXRj0scIW VO83EQMypVmPaI3vN+4BmxxgWRvtSvu1O0iy00+lzXAplf1nAZ jEO0LnRhM4Wh7HapsC0tfPWRx52LolZBSeVSyR2cNpw/AClvuQ EDI+I2nPYT8pcb52hmXXcYt0+KjmbLHW4sl/UjdB3aut+3JvF5 ABzl2PPTB6papfODUivsJqhpjs94OnSFDzm5LvLu+g870oBozx meGuC18+9HGTylZq5RtR1tmOoEz9SZOg10mgDdf+xIpPetS1wK H0V/h9swidDBuKhr4qCAnWLQ/3zV/wIbQQ34q6wu8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 2011-12-15 at 13:34 +1030, Rusty Russell wrote: > You didn't mean this to be a bool. > > Cc: Tony Olech > Cc: Chris Ball > Cc: linux-mmc@vger.kernel.org > Cc: linux-usb@vger.kernel.org > Signed-off-by: Rusty Russell > --- > 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 Acked-by: Tony Olech