[PATCH] USB: gadget: pxa2xx: allow -1 for invalid gpio

Steve Bennett steveb at workware.net.au
Wed Apr 21 08:29:08 EDT 2010


On 21/04/2010, at 4:51 PM, Eric Miao wrote:

> On Wed, Apr 21, 2010 at 2:30 PM, Steve Bennett <steveb at workware.net.au> wrote:
>> gpio must be int, not u16, otherwise -1 isn't recognised
>> by gpio_is_valid().
>> 
>> Signed-off-by: Steve Bennett <steveb at workware.net.au>
> 
> Ack.
> 
> And better submit a patch as well for pxa2xx_udc.c to allow -1 as no GPIO.

Both pxa25x_udc.c and pxa_27x_udc.c already use gpio_is_valid() which checks >= 0.

Unless I'm missing something, there is nothing else to do there.

> 
>> ---
>>  arch/arm/include/asm/mach/udc_pxa2xx.h |    4 ++--
>>  1 files changed, 2 insertions(+), 2 deletions(-)
>> 
>> diff --git a/arch/arm/include/asm/mach/udc_pxa2xx.h b/arch/arm/include/asm/mach/udc_pxa2xx.h
>> index f3eabf1..833306e 100644
>> --- a/arch/arm/include/asm/mach/udc_pxa2xx.h
>> +++ b/arch/arm/include/asm/mach/udc_pxa2xx.h
>> @@ -21,8 +21,8 @@ struct pxa2xx_udc_mach_info {
>>         * here.  Note that sometimes the signals go through inverters...
>>         */
>>        bool    gpio_vbus_inverted;
>> -       u16     gpio_vbus;                      /* high == vbus present */
>> +       int     gpio_vbus;                      /* high == vbus present */
>>        bool    gpio_pullup_inverted;
>> -       u16     gpio_pullup;                    /* high == pullup activated */
>> +       int     gpio_pullup;                    /* high == pullup activated */
>>  };
>> 
>> 
>> 
>> --
>> µWeb: Embedded Web Framework - http://uweb.workware.net.au/
>> WorkWare Systems Pty Ltd
>> W: www.workware.net.au      P: 0434 921 300
>> E: steveb at workware.net.au   F: 07 3102 9221




More information about the linux-arm-kernel mailing list