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

Konrad Dybcio
2023-03-20T17:29:42Z

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.

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 {