Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1422278 > unrolled thread

Re: [PATCH v2] backlight: lp855x: Add enable regulator

Started byRob Herring <robh@kernel.org>
First post2016-06-14 22:10 +0200
Last post2016-06-14 22:10 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v2] backlight: lp855x: Add enable regulator Rob Herring <robh@kernel.org> - 2016-06-14 22:10 +0200

#1422278 — Re: [PATCH v2] backlight: lp855x: Add enable regulator

FromRob Herring <robh@kernel.org>
Date2016-06-14 22:10 +0200
SubjectRe: [PATCH v2] backlight: lp855x: Add enable regulator
Message-ID<rK2OS-1aP-21@gated-at.bofh.it>
On Fri, Jun 10, 2016 at 12:39:57PM -0700, Brian Norris wrote:
> The LP8556 datasheet describes an EN/VDDIO input, which serves "both as
> a chip enable and as a power supply reference for PWM, SDA, and SCL
> inputs." The LP8556 that I'm testing doesn't respond properly if I try
> to talk I2C to it too quickly after enabling VDDIO, and the LP8555
> datasheet mentions a t_RESPONSE delay of up to 1 millisecond.
> 
> Support this EN/VDDIO by adding a regulator property to the binding;
> enabling this regulator at probe time; and sleeping for 1 to 2ms, if the
> EN/VDDIO regulator was provided.
> 
> Signed-off-by: Brian Norris <briannorris@chromium.org>
> ---
> v2: s/TS8555/LP8555/ in comment (typo)
> 
>  .../devicetree/bindings/leds/backlight/lp855x.txt  |  2 ++

I would have picked 'vddio' as the name, but it's fine.

Acked-by: Rob Herring <robh@kernel.org>

>  drivers/video/backlight/lp855x_bl.c                | 29 ++++++++++++++++++++++
>  2 files changed, 31 insertions(+)

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web