Re: [PATCH v4 0/4] regulator: output-supply DT support

From: Rob Herring
Date: Thu Jul 14 2022 - 10:17:18 EST


On Mon, Jul 11, 2022 at 08:11:02PM +0100, Mark Brown wrote:
> On Thu, 7 Jul 2022 10:18:22 +0200, Naresh Solanki wrote:
> > v4:
> > - Add 9elements as vendor.
> > - Update dt binding.
> >
> > Add devicetree support of the output-supply driver.
> > The supply names for this driver is provided through DT
> > regulator-names & regulator handle can be acquired.
> > Driver events can be received from sysfs.
> >
> > [...]
>
> Applied to
>
> https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next
>
> Thanks!
>
> [1/4] dt-bindings: vendor-prefixes: add 9elements
> commit: bd9b7998b4816b3d604253a774d83b6736474283
> [2/4] dt-bindings: regulator: add bindings for output-supply
> commit: df9c96b69a1d2629519a5d8a9dc4b39f775ebe2d
> [3/4] regulator: output-supply: Add devicetree support
> commit: 34e5700e1e64077ede50eb60d04e7604dc4f508a
> [4/4] regulator: output-supply: Add Notification support
> commit: 490a15324ce6d55b950fce0eb9e95c793fac0dff

Please revert this. The vendor prefix is not even appropriate here and
the binding has lots of issues. The first being whether it should even
be in DT. It's not been reviewed as none of it has gone to the DT list
and the only thing I got was '[PATCH v4 1/4] dt-bindings:
vendor-prefixes: add 9elements'.

Rob