Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-12 Thread Jingoo Han
On Saturday, July 11, 2015 6:11 AM, Kim, Milo wrote: > On 7/11/2015 5:52 AM, Kim, Milo wrote: > > Hi Paul, > > > > On 7/11/2015 5:49 AM, Sean Paul wrote: > >> On Fri, Jul 10, 2015 at 4:43 PM, Kim, Milo wrote: > >>> Hi Paul, > >>> > >>> > >>> On 7/11/2015 12:01 AM, Sean Paul wrote: > >

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-12 Thread Jingoo Han
On Saturday, July 11, 2015 6:11 AM, Kim, Milo wrote: On 7/11/2015 5:52 AM, Kim, Milo wrote: Hi Paul, On 7/11/2015 5:49 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:43 PM, Kim, Milo milo@ti.com wrote: Hi Paul, On 7/11/2015 12:01 AM, Sean Paul wrote: On Fri, Jul 10, 2015

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Kim, Milo
Hi Paul, On 7/11/2015 5:52 AM, Kim, Milo wrote: Hi Paul, On 7/11/2015 5:49 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:43 PM, Kim, Milo wrote: Hi Paul, On 7/11/2015 12:01 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim wrote: LP855x backlight device can be enabled

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Kim, Milo
Hi Paul, On 7/11/2015 5:49 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:43 PM, Kim, Milo wrote: Hi Paul, On 7/11/2015 12:01 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim wrote: LP855x backlight device can be enabled by external VDD input. The 'supply' data is used

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Sean Paul
On Fri, Jul 10, 2015 at 4:43 PM, Kim, Milo wrote: > Hi Paul, > > > On 7/11/2015 12:01 AM, Sean Paul wrote: >> >> On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim wrote: >>> >>> LP855x backlight device can be enabled by external VDD input. >>> The 'supply' data is used for this purpose. >>> It's kind of

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Kim, Milo
Hi Paul, On 7/11/2015 12:01 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim wrote: LP855x backlight device can be enabled by external VDD input. The 'supply' data is used for this purpose. It's kind of private data which runs internally, so there is no reason to expose to the

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Sean Paul
On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim wrote: > LP855x backlight device can be enabled by external VDD input. > The 'supply' data is used for this purpose. > It's kind of private data which runs internally, so there is no reason to > expose to the platform data. > > And devm_regulator_get() is

[PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Milo Kim
LP855x backlight device can be enabled by external VDD input. The 'supply' data is used for this purpose. It's kind of private data which runs internally, so there is no reason to expose to the platform data. And devm_regulator_get() is moved from _parse_dt() to _probe(). Regulator

[PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Milo Kim
LP855x backlight device can be enabled by external VDD input. The 'supply' data is used for this purpose. It's kind of private data which runs internally, so there is no reason to expose to the platform data. And devm_regulator_get() is moved from _parse_dt() to _probe(). Regulator

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Sean Paul
On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim milo@ti.com wrote: LP855x backlight device can be enabled by external VDD input. The 'supply' data is used for this purpose. It's kind of private data which runs internally, so there is no reason to expose to the platform data. And

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Kim, Milo
Hi Paul, On 7/11/2015 12:01 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim milo@ti.com wrote: LP855x backlight device can be enabled by external VDD input. The 'supply' data is used for this purpose. It's kind of private data which runs internally, so there is no reason to

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Kim, Milo
Hi Paul, On 7/11/2015 5:49 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:43 PM, Kim, Milo milo@ti.com wrote: Hi Paul, On 7/11/2015 12:01 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim milo@ti.com wrote: LP855x backlight device can be enabled by external VDD

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Kim, Milo
Hi Paul, On 7/11/2015 5:52 AM, Kim, Milo wrote: Hi Paul, On 7/11/2015 5:49 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:43 PM, Kim, Milo milo@ti.com wrote: Hi Paul, On 7/11/2015 12:01 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim milo@ti.com wrote: LP855x

Re: [PATCH v2] backlight: lp855x: use private data for regulator control

2015-07-10 Thread Sean Paul
On Fri, Jul 10, 2015 at 4:43 PM, Kim, Milo milo@ti.com wrote: Hi Paul, On 7/11/2015 12:01 AM, Sean Paul wrote: On Fri, Jul 10, 2015 at 4:26 AM, Milo Kim milo@ti.com wrote: LP855x backlight device can be enabled by external VDD input. The 'supply' data is used for this purpose.