| Message ID | 20230625202547.174647-26-dmitry.baryshkov@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
| Series | ARM: qcom: apq8064: support CPU frequency scaling | expand |
On 25.06.2023 22:25, Dmitry Baryshkov wrote: > The SAW2 device should describe the regulator constraints rather than > just declaring that it has the regulator. > > Drop the 'regulator' property. If/when CPU voltage scaling is > implemented for this platform, proper regulator nodes show be added > instead. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org> Konrad > arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi > index f0ef86fadc9d..ad3c922843c7 100644 > --- a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi > +++ b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi > @@ -350,31 +350,26 @@ acc3: power-manager@b0b8000 { > saw0: regulator@b089000 { > compatible = "qcom,saw2"; > reg = <0x0b089000 0x1000>, <0x0b009000 0x1000>; > - regulator; > }; > > saw1: regulator@b099000 { > compatible = "qcom,saw2"; > reg = <0x0b099000 0x1000>, <0x0b009000 0x1000>; > - regulator; > }; > > saw2: regulator@b0a9000 { > compatible = "qcom,saw2"; > reg = <0x0b0a9000 0x1000>, <0x0b009000 0x1000>; > - regulator; > }; > > saw3: regulator@b0b9000 { > compatible = "qcom,saw2"; > reg = <0x0b0b9000 0x1000>, <0x0b009000 0x1000>; > - regulator; > }; > > saw_l2: regulator@b012000 { > compatible = "qcom,saw2"; > reg = <0xb012000 0x1000>; > - regulator; > }; > > blsp1_uart1: serial@78af000 {
diff --git a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi index f0ef86fadc9d..ad3c922843c7 100644 --- a/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi +++ b/arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi @@ -350,31 +350,26 @@ acc3: power-manager@b0b8000 { saw0: regulator@b089000 { compatible = "qcom,saw2"; reg = <0x0b089000 0x1000>, <0x0b009000 0x1000>; - regulator; }; saw1: regulator@b099000 { compatible = "qcom,saw2"; reg = <0x0b099000 0x1000>, <0x0b009000 0x1000>; - regulator; }; saw2: regulator@b0a9000 { compatible = "qcom,saw2"; reg = <0x0b0a9000 0x1000>, <0x0b009000 0x1000>; - regulator; }; saw3: regulator@b0b9000 { compatible = "qcom,saw2"; reg = <0x0b0b9000 0x1000>, <0x0b009000 0x1000>; - regulator; }; saw_l2: regulator@b012000 { compatible = "qcom,saw2"; reg = <0xb012000 0x1000>; - regulator; }; blsp1_uart1: serial@78af000 {
The SAW2 device should describe the regulator constraints rather than just declaring that it has the regulator. Drop the 'regulator' property. If/when CPU voltage scaling is implemented for this platform, proper regulator nodes show be added instead. Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> --- arch/arm/boot/dts/qcom/qcom-ipq4019.dtsi | 5 ----- 1 file changed, 5 deletions(-)