Re: [PATCH v3 1/3] devicetree: bindings: bd718x7: document HW state related ROHM specific properties

From: Lee Jones
Date: Mon Mar 25 2019 - 02:28:38 EST


On Thu, 21 Mar 2019, Vaittinen, Matti wrote:

> On Thu, 2019-03-21 at 08:23 +0000, Lee Jones wrote:
> > On Wed, 20 Mar 2019, Matti Vaittinen wrote:
> >
> > > Hello Lee, All
> > >
> > > On Wed, Mar 20, 2019 at 01:02:59PM +0000, Lee Jones wrote:
> > > > On Thu, 14 Feb 2019, Matti Vaittinen wrote:
> > > >
> > > > > Add ROHM BD71837 / BD71847 specific device tree bindings for
> > > > > controlling the PMIC shutdown/reset states and voltages for
> > > > > different HW states. The PMIC was designed to be used with NXP
> > > > > i.MX8 SoC and it supports SNVS low power state which seems to
> > > > > be typical for NXP i.MX SoCs. However, when SNVS is used we
> > > > > must
> > > > > not allow SW to control enabling/disabling those regulators
> > > > > which
> > > > > are crucial for system to boot as there is a HW limitation
> > > > > which
> > > > > causes SW controlled regulators to be kept shut down after SNVS
> > > > > reset.
> > > > >
> > > > > Allow setting the SNVS to be used as reset target state and
> > > > > allow
> > > > > marking those regulators which are critical for boot.
> > > > >
> > > > > Signed-off-by: Matti Vaittinen <
> > > > > matti.vaittinen@xxxxxxxxxxxxxxxxx>
> > > > > ---
> > > > > .../devicetree/bindings/mfd/rohm,bd71837-pmic.txt | 17
> > > > > ++++++++++
> > > > > .../bindings/regulator/rohm,bd71837-regulator.txt | 38
> > > > > ++++++++++++++++++++++
> > > > > 2 files changed, 55 insertions(+)
> > > >
> > > > Needs a DT Ack.
> > >
> > > I think this was already applied to regulator tree by Mark:
> > >
> https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git/commit/?h=for-next&id=f43d1b388f9be4aa47ed42c33659243a675c5c76
> > > Does it go in via his tree or do we need to apply it in Lee's tree
> > > too?
> >
> > Why don't I see an email from Mark?
>
> https://lore.kernel.org/lkml/20190214160025.65B8E1126F78@xxxxxxxxxxxxxxxxxxxxxxx/
>
> Maybe that's what you're looking for?

It is, but I need this in my inbox.

Mark, could you please take the time to fix your scripts please?

--
Lee Jones [æçæ]
Linaro Services Technical Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog