Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-10-05 Thread Simon Guinot
On Mon, Sep 28, 2015 at 09:58:44AM +0200, Jacek Anaszewski wrote: > Hi Rob. > > If you are satisfied with Simon's explanation, could you confirm > that with your ack, please? Hi Rob, Please let us know your answer. Thanks in advance. Simon > On 09/22/2015 11:30 AM, Simon Guinot wrote: > >On M

Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-28 Thread Jacek Anaszewski
Hi Rob. If you are satisfied with Simon's explanation, could you confirm that with your ack, please? Best Regards, Jacek Anaszewski On 09/22/2015 11:30 AM, Simon Guinot wrote: On Mon, Sep 21, 2015 at 10:38:45AM -0500, Rob Herring wrote: Hi Rob, Thanks for your feedback. Please, see my answ

Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-22 Thread Simon Guinot
On Mon, Sep 21, 2015 at 10:38:45AM -0500, Rob Herring wrote: Hi Rob, Thanks for your feedback. Please, see my answers below. > On 09/17/2015 10:59 AM, Simon Guinot wrote: > > This patch adds device tree support for the netxbig LEDs. > > > > This also introduces a additionnal DT binding for the

Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-21 Thread Rob Herring
On 09/17/2015 10:59 AM, Simon Guinot wrote: > This patch adds device tree support for the netxbig LEDs. > > This also introduces a additionnal DT binding for the GPIO extension bus > (netxbig-gpio-ext) used to configure the LEDs. Since this bus could also > be used to control other devices, then i

Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-21 Thread Jacek Anaszewski
On 09/18/2015 03:10 PM, Simon Guinot wrote: On Fri, Sep 18, 2015 at 12:49:28PM +0200, Jacek Anaszewski wrote: Moreover, if you switched to using devm* prefixed version of gpio_request_one and led_classdev_reqister, you could simplify the error paths in the driver. Yes, I have a pending patch

Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-18 Thread Simon Guinot
On Fri, Sep 18, 2015 at 12:49:28PM +0200, Jacek Anaszewski wrote: > >> > >>Moreover, if you switched to using devm* prefixed version of > >>gpio_request_one and led_classdev_reqister, you could simplify > >>the error paths in the driver. > > > >Yes, I have a pending patch for this conversion. But s

Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-18 Thread Jacek Anaszewski
On 09/18/2015 12:30 PM, Simon Guinot wrote: On Fri, Sep 18, 2015 at 11:16:41AM +0200, Jacek Anaszewski wrote: Hi Simon, Please find my comments in the code below. Hi Jacek, Thanks for the quick review. You're welcome. On 09/17/2015 05:59 PM, Simon Guinot wrote: This patch adds device t

Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-18 Thread Simon Guinot
On Fri, Sep 18, 2015 at 11:16:41AM +0200, Jacek Anaszewski wrote: > Hi Simon, > > Please find my comments in the code below. Hi Jacek, Thanks for the quick review. > > On 09/17/2015 05:59 PM, Simon Guinot wrote: > >This patch adds device tree support for the netxbig LEDs. > > > >This also intr

Re: [PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-18 Thread Jacek Anaszewski
Hi Simon, Please find my comments in the code below. On 09/17/2015 05:59 PM, Simon Guinot wrote: This patch adds device tree support for the netxbig LEDs. This also introduces a additionnal DT binding for the GPIO extension bus (netxbig-gpio-ext) used to configure the LEDs. Since this bus coul

[PATCH v4 1/3] leds: netxbig: add device tree binding

2015-09-17 Thread Simon Guinot
This patch adds device tree support for the netxbig LEDs. This also introduces a additionnal DT binding for the GPIO extension bus (netxbig-gpio-ext) used to configure the LEDs. Since this bus could also be used to control other devices, then it seems more suitable to have it in a separate DT bind