On Tue, Dec 2, 2014 at 4:54 PM, Kim, Milo <milo.kim@xxxxxx> 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<marcheu@xxxxxxxxxxxx> >> Cc: Aaron Durbin<adurbin@xxxxxxxxxxxx> >> Signed-off-by: Sean Paul<seanpaul@xxxxxxxxxxxx> > > > Acked-by: Milo Kim <milo.kim@xxxxxx> > > For effective power consumption, regulator could be enabled/disabled based > on backlight control flow. > For example, when the brightness is 0, then call regulator_disable() > when the brightness is > 0 and regulator is off, then call > regulator_enable(). > However, I'd like to send ACK to this patch. > > Sean, > Could you send the patch-set with the first one - 'backlight/lp855x: > Refactor dt parsing code'? I'll send my ACK as well. Hi Milo, Thanks for looking at these. There aren't any changes to PATCH 1/2, is there any value in resending the same patches? Sean > For the backlight maintainers, it would be better to apply them together. > > Thanks! > > Best regards, > Milo -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html