[PATCH 1/2] PalmTreo: generalisation of Treo680 code

[Eric Miao]

Sorry for late response. Applied to 'devel'.

On Sun, Sep 13, 2009 at 5:27 PM, Marek Vasut <marek.vasut at gmail.com> wrote:
> Dne Ne 13. září 2009 11:03:37 Tomáš 'Sleep_Walker' Čech napsal(a):
>> Hi,
>>
>> thanks for reading my patch!
>>
>> Dne So 12. září 2009 21:29:06 Marek Vasut napsal(a):
>> > ...
>> >
>> > > -static struct gpio_led_platform_data gpio_led_info = {
>> > > - .leds           = gpio_leds,
>> > > - .num_leds       = ARRAY_SIZE(gpio_leds),
>> > > +static struct gpio_led_platform_data treo680_gpio_led_info = {
>> > > + .leds           = treo680_gpio_leds,
>> > > + .num_leds       = ARRAY_SIZE(treo680_gpio_leds),
>> > >  };
>> >
>> > Be careful so you dont run into symbol redefinition here (gpio_led_info)
>> > in case the kernel supports more than this one device ...
>>
>> That's why I change the name. It's called now 'treo680_gpio_led_info' as
>>  you can see.
>>
>> > ...
>> >
>> > > +#endif /* CONFIG_MACH_TREO680 */
>> > > +
>> > >  /* setup udc GPIOs initial state */
>> > > -static void __init treo680_udc_init(void)
>> > > +static void __init treo_udc_init(void)
>> > >  {
>> > > - if (!gpio_request(GPIO_NR_TREO680_USB_PULLUP, "UDC Vbus")) {
>> > > -         gpio_direction_output(GPIO_NR_TREO680_USB_PULLUP, 1);
>> > > -         gpio_free(GPIO_NR_TREO680_USB_PULLUP);
>> > > + if (!gpio_request(GPIO_NR_TREO_USB_PULLUP, "UDC Vbus")) {
>> > > +         gpio_direction_output(GPIO_NR_TREO_USB_PULLUP, 1);
>> > > +         gpio_free(GPIO_NR_TREO_USB_PULLUP);
>> > >   }
>> > >  }
>> >
>> > Is this really needed or is it just a copy-paste from other Palm device?
>>
>> Unfortunately yes. There is no copy-paste in this code which wasn't checked
>>  and tested. It doesn't work correctly without that.
>>
>> Best regards,
>>
>> Tomas Cech
>>
>
> Oh ok, you have my Ack on both then.
>