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

Bartosz Golaszewski
2023-03-21T14:18:00Z

On Mon, Mar 20, 2023 at 6:25 PM Konrad Dybcio <konrad.dybcio@linaro.org> wrote:
>
>
>
> On 20.03.2023 18:23, Konrad Dybcio wrote:
> >
> >
> > On 20.03.2023 16:48, Bartosz Golaszewski wrote:
> >> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> >>
> >> Add the PON node to PMIC #0 for sa8775p platforms.
> >>
> >> Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
> >> ---
> > Reviewed-by: Konrad Dybcio <konrad.dybcio@linaro.org>
> >
> > 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 {