From: Tony Lindgren <tony@atomide.com>
To: Milo Kim <milo.kim@ti.com>
Cc: Lee Jones <lee.jones@linaro.org>, Rob Herring <robh@kernel.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 2/2] mfd: add TI LMU driver
Date: Tue, 28 Feb 2017 16:44:41 -0800 [thread overview]
Message-ID: <20170301004441.GH20572@atomide.com> (raw)
In-Reply-To: <20170228064515.7320-3-milo.kim@ti.com>
* Milo Kim <milo.kim@ti.com> [170227 22:47]:
> TI LMU (Lighting Management Unit) driver supports lighting devices below.
>
> LM3532, LM3631, LM3632, LM3633, LM3695 and LM3697.
>
> LMU devices have common features.
> - I2C interface for accessing device registers
> - Hardware enable pin control
> - Backlight brightness control
> - Notifier for hardware fault monitoring
> - Regulators for LCD display bias
>
> It contains fault monitor, backlight, LED and regulator driver.
>
> LMU fault monitor
> -----------------
> LM3633 and LM3697 provide hardware monitoring feature.
> It enables open or short circuit detection.
> After monitoring is done, each device should be re-initialized.
> Notifier is used for this case.
> Separate patch for 'ti-lmu-fault-monitor' will be sent later.
>
> Backlight
> ---------
> It's handled by TI LMU backlight consolidated driver and
> chip dependent data. Separate patchset will be sent later.
>
> LED indicator
> -------------
> LM3633 has 6 indicator LEDs. Programmable dimming pattern is also
> supported. Separate patch for 'leds-lm3633' will be sent later.
>
> Regulator
> ---------
> LM3631 has 5 regulators for the display bias.
> LM3632 supports 3 regulators. One consolidated driver enables it.
> The lm363x regulator driver is already upstreamed.
>
> Acked-by: Lee Jones <lee.jones@linaro.org>
> Signed-off-by: Milo Kim <milo.kim@ti.com>
This works for me on droid 4 with LM3632 using your earlier
"[v2,7/9] backlight: add TI LMU backlight driver" patch at
https://patchwork.kernel.org/patch/7704601/
I had to update the dts file according to the binding and then
enable backlight after booting with:
# echo 180 > /sys/class/backlight/lcd/brightness
So please feel free to add:
Tested-by: Tony Lindgren <tony@atomide.com>
next prev parent reply other threads:[~2017-03-01 2:40 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-28 6:45 [PATCH v3 0/2] Support TI LMU devices Milo Kim
2017-02-28 6:45 ` [PATCH 1/2] Documentation: dt-bindings: mfd: add TI LMU device binding information Milo Kim
2017-03-01 0:37 ` Tony Lindgren
2017-03-14 16:44 ` Lee Jones
2017-02-28 6:45 ` [PATCH 2/2] mfd: add TI LMU driver Milo Kim
2017-03-01 0:44 ` Tony Lindgren [this message]
2017-03-14 16:44 ` Lee Jones
2017-03-01 0:36 ` [PATCH v3 0/2] Support TI LMU devices Tony Lindgren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170301004441.GH20572@atomide.com \
--to=tony@atomide.com \
--cc=lee.jones@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=milo.kim@ti.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.