[PATCH 1/2] PalmTreo: generalisation of Treo680 code
Marek Vasut
marek.vasut at gmail.com
Sun Sep 13 05:27:05 EDT 2009
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.
More information about the linux-arm-kernel
mailing list