[PATCH 1/3] arm64: dts: qcom: pm8150: use qcom,pm8998-pon binding

[PATCH 1/3] arm64: dts: qcom: pm8150: use qcom,pm8998-pon binding

[Dmitry Baryshkov]

Change pm8150 to use the qcom,pm8998-pon compatible string for the pon
in order to pass reboot mode properly.

Fixes: 5101f22a5c37 ("arm64: dts: qcom: pm8150: Add base dts file")
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm64/boot/dts/qcom/pm8150.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/pm8150.dtsi b/arch/arm64/boot/dts/qcom/pm8150.dtsi
index c566a64b1373..00385b1fd358 100644
--- a/arch/arm64/boot/dts/qcom/pm8150.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8150.dtsi
@@ -48,7 +48,7 @@ pm8150_0: pmic@0 {
 		#size-cells = <0>;
 
 		pon: power-on@800 {
-			compatible = "qcom,pm8916-pon";
+			compatible = "qcom,pm8998-pon";
 			reg = <0x0800>;
 
 			pon_pwrkey: pwrkey {
-- 
2.33.0

[PATCH 2/3] arm64: dts: qcom: pm8150: specify reboot mode magics

[Dmitry Baryshkov]

Specify recovery and bootloader magic values to be programmed by the
qcom-pon driver. This allows the bootloader to handle
reboot-to-bootloader functionality.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm64/boot/dts/qcom/pm8150.dtsi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/pm8150.dtsi b/arch/arm64/boot/dts/qcom/pm8150.dtsi
index 00385b1fd358..0df76f7b1cc1 100644
--- a/arch/arm64/boot/dts/qcom/pm8150.dtsi
+++ b/arch/arm64/boot/dts/qcom/pm8150.dtsi
@@ -50,6 +50,8 @@ pm8150_0: pmic@0 {
 		pon: power-on@800 {
 			compatible = "qcom,pm8998-pon";
 			reg = <0x0800>;
+			mode-bootloader = <0x2>;
+			mode-recovery = <0x1>;
 
 			pon_pwrkey: pwrkey {
 				compatible = "qcom,pm8941-pwrkey";
-- 
2.33.0

[PATCH 3/3] arm64: dts: qcom:qrb5165-rb5: enabled pwrkey and resin nodes

[Dmitry Baryshkov]

Enable powerkey and resin nodes to let the board handle POWER and
Volume- keys properly.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
index 8ac96f8e79d4..28d5b5528516 100644
--- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
+++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
@@ -804,6 +804,16 @@ lt9611_rst_pin: lt9611-rst-pin {
 	};
 };
 
+&pon_pwrkey {
+	status = "okay";
+};
+
+&pon_resin {
+	status = "okay";
+
+	linux,code = <KEY_VOLUMEDOWN>;
+};
+
 &qupv3_id_0 {
 	status = "okay";
 };
-- 
2.33.0

Re: [PATCH 1/3] arm64: dts: qcom: pm8150: use qcom,pm8998-pon binding

[Amit Pundir]

On Thu, 16 Sept 2021 at 20:43, Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
>
> Change pm8150 to use the qcom,pm8998-pon compatible string for the pon
> in order to pass reboot mode properly.
>
> Fixes: 5101f22a5c37 ("arm64: dts: qcom: pm8150: Add base dts file")
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Smoke tested the series on RB5.
"adb reboot bootloader" and Power, Volume_Down keys work as expected.
So for the whole series:

Tested-by: Amit Pundir <amit.pundir@linaro.org>

> ---
>  arch/arm64/boot/dts/qcom/pm8150.dtsi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/pm8150.dtsi b/arch/arm64/boot/dts/qcom/pm8150.dtsi
> index c566a64b1373..00385b1fd358 100644
> --- a/arch/arm64/boot/dts/qcom/pm8150.dtsi
> +++ b/arch/arm64/boot/dts/qcom/pm8150.dtsi
> @@ -48,7 +48,7 @@ pm8150_0: pmic@0 {
>                 #size-cells = <0>;
>
>                 pon: power-on@800 {
> -                       compatible = "qcom,pm8916-pon";
> +                       compatible = "qcom,pm8998-pon";
>                         reg = <0x0800>;
>
>                         pon_pwrkey: pwrkey {
> --
> 2.33.0
>