Re: [PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-23 Thread Olof Johansson
On Fri, Jul 5, 2013 at 8:53 AM, Chris Ball wrote: > Hi, > > On Fri, Jul 05 2013, Arnd Bergmann wrote: >> The em_x270_mci_setpower() and em_x270_usb_hub_init() functions >> call regulator_enable(), which may return an error that must >> be checked. >> >> This changes the em_x270_usb_hub_init()

Re: [PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-23 Thread Olof Johansson
On Fri, Jul 5, 2013 at 8:53 AM, Chris Ball c...@laptop.org wrote: Hi, On Fri, Jul 05 2013, Arnd Bergmann wrote: The em_x270_mci_setpower() and em_x270_usb_hub_init() functions call regulator_enable(), which may return an error that must be checked. This changes the em_x270_usb_hub_init()

Re: [PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-08 Thread Hein Tibosch
Hi Arnd, On 7/5/2013 11:51 PM, Arnd Bergmann wrote: > The em_x270_mci_setpower() and em_x270_usb_hub_init() functions > call regulator_enable(), which may return an error that must > be checked. > > This changes the em_x270_usb_hub_init() function to bail out > if it fails, and changes the

Re: [PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-08 Thread Hein Tibosch
Hi Arnd, On 7/5/2013 11:51 PM, Arnd Bergmann wrote: The em_x270_mci_setpower() and em_x270_usb_hub_init() functions call regulator_enable(), which may return an error that must be checked. This changes the em_x270_usb_hub_init() function to bail out if it fails, and changes the

Re: [PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-07 Thread Sachin Kamat
On 5 July 2013 21:21, Arnd Bergmann wrote: > The em_x270_mci_setpower() and em_x270_usb_hub_init() functions > call regulator_enable(), which may return an error that must > be checked. > > This changes the em_x270_usb_hub_init() function to bail out > if it fails, and changes the

Re: [PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-07 Thread Sachin Kamat
On 5 July 2013 21:21, Arnd Bergmann a...@arndb.de wrote: The em_x270_mci_setpower() and em_x270_usb_hub_init() functions call regulator_enable(), which may return an error that must be checked. This changes the em_x270_usb_hub_init() function to bail out if it fails, and changes the

Re: [PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-05 Thread Chris Ball
Hi, On Fri, Jul 05 2013, Arnd Bergmann wrote: > The em_x270_mci_setpower() and em_x270_usb_hub_init() functions > call regulator_enable(), which may return an error that must > be checked. > > This changes the em_x270_usb_hub_init() function to bail out > if it fails, and changes the

[PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-05 Thread Arnd Bergmann
The em_x270_mci_setpower() and em_x270_usb_hub_init() functions call regulator_enable(), which may return an error that must be checked. This changes the em_x270_usb_hub_init() function to bail out if it fails, and changes the pxamci_platform_data->setpower callback so that the a failed

[PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-05 Thread Arnd Bergmann
The em_x270_mci_setpower() and em_x270_usb_hub_init() functions call regulator_enable(), which may return an error that must be checked. This changes the em_x270_usb_hub_init() function to bail out if it fails, and changes the pxamci_platform_data-setpower callback so that the a failed

Re: [PATCH] ARM: pxa: propagate errors from regulator_enable() to pxamci

2013-07-05 Thread Chris Ball
Hi, On Fri, Jul 05 2013, Arnd Bergmann wrote: The em_x270_mci_setpower() and em_x270_usb_hub_init() functions call regulator_enable(), which may return an error that must be checked. This changes the em_x270_usb_hub_init() function to bail out if it fails, and changes the