Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-09 Thread Arnd Bergmann
On Tuesday 09 December 2014 12:46:15 Jingoo Han wrote: > On Tuesday, December 09, 2014 12:04 PM, Kim, Milo wrote: > > > > Hi Arnd, > > > > Good to talk to you again. It's been a long time since I met you in > > Linaro Connect HK few years ago > > > > On 12/3/2014 11:11 PM, Arnd Bergmann wrote:

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-08 Thread Jingoo Han
On Tuesday, December 09, 2014 12:04 PM, Kim, Milo wrote: > > Hi Arnd, > > Good to talk to you again. It's been a long time since I met you in > Linaro Connect HK few years ago :) > > On 12/3/2014 11:11 PM, Arnd Bergmann wrote: > > While your two patches are both correct (and applied already), I

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-08 Thread Kim, Milo
Hi Arnd, Good to talk to you again. It's been a long time since I met you in Linaro Connect HK few years ago :) On 12/3/2014 11:11 PM, Arnd Bergmann wrote: While your two patches are both correct (and applied already), I took a look at the platform_data header and noticed that all users of th

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-03 Thread Sean Paul
On Wed, Dec 3, 2014 at 6:11 AM, Arnd Bergmann wrote: > On Tuesday 02 December 2014 17:39:12 Sean Paul wrote: >> >> .../devicetree/bindings/video/backlight/lp855x.txt | 2 ++ >> drivers/video/backlight/lp855x_bl.c| 18 >> ++ >> include/linux/platform_data/

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-03 Thread Arnd Bergmann
On Tuesday 02 December 2014 17:39:12 Sean Paul wrote: > > .../devicetree/bindings/video/backlight/lp855x.txt | 2 ++ > drivers/video/backlight/lp855x_bl.c| 18 > ++ > include/linux/platform_data/lp855x.h | 2 ++ > 3 files changed, 22 in

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-03 Thread Lee Jones
On Tue, 02 Dec 2014, Sean Paul wrote: > This patch adds a supply regulator to the lp855x platform data to facilitate > powering on/off the 3V rail attached to the controller. > > Cc: Stéphane Marchesin > Cc: Aaron Durbin > Acked-by: Milo Kim > Signed-off-by: Sean Paul > --- > > Changes in v2

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-02 Thread Bryan Wu
On Tue, Dec 2, 2014 at 5:39 PM, Sean Paul wrote: > This patch adds a supply regulator to the lp855x platform data to facilitate > powering on/off the 3V rail attached to the controller. > > Cc: Stéphane Marchesin > Cc: Aaron Durbin > Acked-by: Milo Kim > Signed-off-by: Sean Paul Acked-by: Bry

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-02 Thread Jingoo Han
On Wednesday, December 03, 2014 10:39 AM, Sean Paul wrote: > > This patch adds a supply regulator to the lp855x platform data to facilitate > powering on/off the 3V rail attached to the controller. > > Cc: Stéphane Marchesin > Cc: Aaron Durbin > Acked-by: Milo Kim > Signed-off-by: Sean Paul

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-02 Thread Kim, Milo
On 12/3/2014 10:39 AM, Sean Paul wrote: This patch adds a supply regulator to the lp855x platform data to facilitate powering on/off the 3V rail attached to the controller. Cc: Stéphane Marchesin Cc: Aaron Durbin Acked-by: Milo Kim Signed-off-by: Sean Paul Acked-by: Milo Kim -- To unsubscribe

[PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-02 Thread Sean Paul
This patch adds a supply regulator to the lp855x platform data to facilitate powering on/off the 3V rail attached to the controller. Cc: Stéphane Marchesin Cc: Aaron Durbin Acked-by: Milo Kim Signed-off-by: Sean Paul --- Changes in v2: - Add NULL check in lp855x_remove .../devicetr

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-02 Thread Bryan Wu
ease submit two patches in series like below. > > [PATCH v2 1/2] backlight/lp855x: Refactor dt parsing code > [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x > It looks good to me too. Please follow Milo's advice and I will give my Ack then. T

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-02 Thread Kim, Milo
code [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x Thanks! Best regards, Milo -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-02 Thread Sean Paul
On Tue, Dec 2, 2014 at 4:54 PM, Kim, Milo wrote: > On 12/2/2014 6:07 AM, Sean Paul wrote: >> >> This patch adds a supply regulator to the lp855x platform data to >> facilitate >> powering on/off the 3V rail attached to the controller. >> >> Cc: Stéphane Marchesin >> Cc: Aaron Durbin >> Signed-off-

Re: [PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-02 Thread Kim, Milo
On 12/2/2014 6:07 AM, Sean Paul wrote: This patch adds a supply regulator to the lp855x platform data to facilitate powering on/off the 3V rail attached to the controller. Cc: Stéphane Marchesin Cc: Aaron Durbin Signed-off-by: Sean Paul Acked-by: Milo Kim For effective power consumption, reg

[PATCH v2 2/2] backlight/lp855x: Add supply regulator to lp855x

2014-12-01 Thread Sean Paul
This patch adds a supply regulator to the lp855x platform data to facilitate powering on/off the 3V rail attached to the controller. Cc: Stéphane Marchesin Cc: Aaron Durbin Signed-off-by: Sean Paul --- Changes in v2: - Add NULL check in lp855x_remove .../devicetree/bindings/video/ba