Re: [PATCH v2 09/15] arm64: dts: qcom: sa8775p: add the Power On device node

[Bartosz Golaszewski <brgl@xxxxxxxx>]

On Mon, Mar 20, 2023 at 6:25 PM Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> wrote:
>
>
>
> On 20.03.2023 18:23, Konrad Dybcio wrote:
> >
> >
> > On 20.03.2023 16:48, Bartosz Golaszewski wrote:
> >> From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
> >>
> >> Add the PON node to PMIC #0 for sa8775p platforms.
> >>
> >> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
> >> ---
> > Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>
> >
> > Konrad
> Hold up, I am not sure if PBS is there on PMM8654AU. Check the
> -pmic-overlay.dtsi.
>

Yep, it's there alright.

Bartosz

> Konrad
> >>  arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi | 8 ++++++++
> >>  1 file changed, 8 insertions(+)
> >>
> >> diff --git a/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi b/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi
> >> index afe220b374c2..dbc596e32253 100644
> >> --- a/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi
> >> +++ b/arch/arm64/boot/dts/qcom/sa8775p-pmics.dtsi
> >> @@ -12,6 +12,14 @@ pmm8654au_0: pmic@0 {
> >>              reg = <0x0 SPMI_USID>;
> >>              #address-cells = <1>;
> >>              #size-cells = <0>;
> >> +
> >> +            pmm8654au_0_pon: pon@1200 {
> >> +                    compatible = "qcom,pmk8350-pon";
> >> +                    reg = <0x1200>, <0x800>;
> >> +                    reg-names = "hlos", "pbs";
> >> +                    mode-recovery = <0x1>;
> >> +                    mode-bootloader = <0x2>;
> >> +            };
> >>      };
> >>
> >>      pmm8654au_1: pmic@2 {