Messages in this thread | | | From | Shenwei Wang <> | Subject | Re: [PATCH 1/2] dt-bindings: power: Add regulator-pd yaml file | Date | Thu, 24 Aug 2023 16:35:16 +0000 |
| |
> -----Original Message----- > From: Ulf Hansson <ulf.hansson@linaro.org> > Sent: Thursday, August 24, 2023 4:27 AM > To: Shenwei Wang <shenwei.wang@nxp.com> > Cc: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski > <krzysztof.kozlowski+dt@linaro.org>; Conor Dooley <conor+dt@kernel.org>; > Liam Girdwood <lgirdwood@gmail.com>; Mark Brown <broonie@kernel.org>; > imx@lists.linux.dev; devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; > dl-linux-imx <linux-imx@nxp.com> > Subject: [EXT] Re: [PATCH 1/2] dt-bindings: power: Add regulator-pd yaml file > > Caution: This is an external email. Please take care when clicking links or > opening attachments. When in doubt, report the message using the 'Report this > email' button > > > On Fri, 18 Aug 2023 at 17:35, Shenwei Wang <shenwei.wang@nxp.com> wrote: > > > > Documenting the regulator power domain properties and usage examples. > > As Rob and Krzysztof already pointed out, I agree that this binding looks a bit > questionable. > > Rather than adding a new DT binding, why can't we just use the existing way of > describing a platform specific power-domain provider?
Can you please provide more details on how you thought we should implement this feature using the existing way? Very appreciate if you could provide a simple example.
> This still looks platform specific to me.
What does platform specific exactly mean here? I want to make sure I understand what you were referring to.
Thanks, Shenwei
> > Kind regards > Uffe > > > > > Signed-off-by: Shenwei Wang <shenwei.wang@nxp.com> > > --- > > .../bindings/power/regulator-pd.yaml | 71 +++++++++++++++++++ > > 1 file changed, 71 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/power/regulator-pd.yaml > > > > diff --git a/Documentation/devicetree/bindings/power/regulator-pd.yaml > > b/Documentation/devicetree/bindings/power/regulator-pd.yaml > > new file mode 100644 > > index 000000000000..181d2fa83f8a
| |