2021-09-13 12:50:49

by Rajendra Nayak

[permalink] [raw]
Subject: [PATCH] arm64: dts: qcom: sc7280-idp: Add vcc-supply for qfprom

Add vcc-supply for the IDP boards that was missed when the
qfprom device tree properties were added for the sc7280 SoC.

Fixes: c1b2189a19cf ("arm64: dts: qcom: sc7280: Add qfprom node")
Reported-by: satya priya <[email protected]>
Signed-off-by: Rajendra Nayak <[email protected]>
---
arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
index 371a2a9..99f9ee5 100644
--- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
@@ -207,6 +207,10 @@
};
};

+&qfprom {
+ vcc-supply = <&vreg_l1c_1p8>;
+};
+
&qupv3_id_0 {
status = "okay";
};
--
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member
of Code Aurora Forum, hosted by The Linux Foundation


2021-09-14 01:03:22

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: sc7280-idp: Add vcc-supply for qfprom

Quoting Rajendra Nayak (2021-09-13 03:58:55)
> Add vcc-supply for the IDP boards that was missed when the
> qfprom device tree properties were added for the sc7280 SoC.
>
> Fixes: c1b2189a19cf ("arm64: dts: qcom: sc7280: Add qfprom node")
> Reported-by: satya priya <[email protected]>
> Signed-off-by: Rajendra Nayak <[email protected]>
> ---

Reviewed-by: Stephen Boyd <[email protected]>

> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> index 371a2a9..99f9ee5 100644
> --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> @@ -207,6 +207,10 @@
> };
> };
>
> +&qfprom {

Maybe that node should also be marked status = "disabled" by default so
that it can only be marked OK if the board has setup the regulator
properly?

> + vcc-supply = <&vreg_l1c_1p8>;
> +};
> +
> &qupv3_id_0 {
> status = "okay";
> };

2021-09-14 01:13:29

by Doug Anderson

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: sc7280-idp: Add vcc-supply for qfprom

Hi,

On Mon, Sep 13, 2021 at 12:57 PM Stephen Boyd <[email protected]> wrote:
>
> Quoting Rajendra Nayak (2021-09-13 03:58:55)
> > Add vcc-supply for the IDP boards that was missed when the
> > qfprom device tree properties were added for the sc7280 SoC.
> >
> > Fixes: c1b2189a19cf ("arm64: dts: qcom: sc7280: Add qfprom node")
> > Reported-by: satya priya <[email protected]>
> > Signed-off-by: Rajendra Nayak <[email protected]>
> > ---
>
> Reviewed-by: Stephen Boyd <[email protected]>
>
> > arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> > index 371a2a9..99f9ee5 100644
> > --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi
> > @@ -207,6 +207,10 @@
> > };
> > };
> >
> > +&qfprom {
>
> Maybe that node should also be marked status = "disabled" by default so
> that it can only be marked OK if the board has setup the regulator
> properly?

I don't think that's a good idea. You still want to be able to use the
nvmem in "read only" mode and it doesn't seem to need the regulator
for that. This is only for burning.

I believe that actually everything will work out fine-ish for boards
that leave this regulator off. They'll get a dummy regulator. When
they start trying to burn fuses and the call the
regulator_set_voltage() happens then that'll fail (you can't do that
on a dummy) and we should error out of writing. Reading will work
fine. It would probably be better if qfprom_probe() used
devm_regulator_get_optional() though and fell back to read-only mode.
I'm happy to review a patch that does that.

-Doug

2021-09-14 01:14:30

by Doug Anderson

[permalink] [raw]
Subject: Re: [PATCH] arm64: dts: qcom: sc7280-idp: Add vcc-supply for qfprom

Hi,

On Mon, Sep 13, 2021 at 4:00 AM Rajendra Nayak <[email protected]> wrote:
>
> Add vcc-supply for the IDP boards that was missed when the
> qfprom device tree properties were added for the sc7280 SoC.
>
> Fixes: c1b2189a19cf ("arm64: dts: qcom: sc7280: Add qfprom node")
> Reported-by: satya priya <[email protected]>
> Signed-off-by: Rajendra Nayak <[email protected]>
> ---
> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 4 ++++
> 1 file changed, 4 insertions(+)

Reviewed-by: Douglas Anderson <[email protected]>