[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