2023-02-14 09:55:24

by Konrad Dybcio

[permalink] [raw]
Subject: [PATCH 1/3] dt-bindings: power: qcom,rpmpd: Add SA8155P

Add a compatible for SA8155P platforms and relevant defines to the
include file.

Signed-off-by: Konrad Dybcio <[email protected]>
---
Documentation/devicetree/bindings/power/qcom,rpmpd.yaml | 1 +
include/dt-bindings/power/qcom-rpmpd.h | 9 +++++++++
2 files changed, 10 insertions(+)

diff --git a/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml b/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml
index afad3135ed67..f9c211a9a938 100644
--- a/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml
+++ b/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml
@@ -29,6 +29,7 @@ properties:
- qcom,qcm2290-rpmpd
- qcom,qcs404-rpmpd
- qcom,qdu1000-rpmhpd
+ - qcom,sa8155p-rpmhpd
- qcom,sa8540p-rpmhpd
- qcom,sa8775p-rpmhpd
- qcom,sdm660-rpmpd
diff --git a/include/dt-bindings/power/qcom-rpmpd.h b/include/dt-bindings/power/qcom-rpmpd.h
index 1bf8e87ecd7e..5a6dd5ded087 100644
--- a/include/dt-bindings/power/qcom-rpmpd.h
+++ b/include/dt-bindings/power/qcom-rpmpd.h
@@ -4,6 +4,15 @@
#ifndef _DT_BINDINGS_POWER_QCOM_RPMPD_H
#define _DT_BINDINGS_POWER_QCOM_RPMPD_H

+/* SA8155P Power Domain Indexes */
+#define SA8155P_CX 0
+#define SA8155P_CX_AO 1
+#define SA8155P_EBI 2
+#define SA8155P_GFX 3
+#define SA8155P_MSS 4
+#define SA8155P_MX 5
+#define SA8155P_MX_AO 6
+
/* SA8775P Power Domain Indexes */
#define SA8775P_CX 0
#define SA8775P_CX_AO 1
--
2.39.1



2023-02-14 09:57:32

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt-bindings: power: qcom,rpmpd: Add SA8155P

On 14/02/2023 10:54, Konrad Dybcio wrote:
> Add a compatible for SA8155P platforms and relevant defines to the
> include file.
>
> Signed-off-by: Konrad Dybcio <[email protected]>


Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof


2023-02-14 10:00:16

by Bhupesh Sharma

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt-bindings: power: qcom,rpmpd: Add SA8155P

On Tue, 14 Feb 2023 at 15:25, Konrad Dybcio <[email protected]> wrote:
>
> Add a compatible for SA8155P platforms and relevant defines to the
> include file.
>
> Signed-off-by: Konrad Dybcio <[email protected]>
> ---
> Documentation/devicetree/bindings/power/qcom,rpmpd.yaml | 1 +
> include/dt-bindings/power/qcom-rpmpd.h | 9 +++++++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml b/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml
> index afad3135ed67..f9c211a9a938 100644
> --- a/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml
> +++ b/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml
> @@ -29,6 +29,7 @@ properties:
> - qcom,qcm2290-rpmpd
> - qcom,qcs404-rpmpd
> - qcom,qdu1000-rpmhpd
> + - qcom,sa8155p-rpmhpd
> - qcom,sa8540p-rpmhpd
> - qcom,sa8775p-rpmhpd
> - qcom,sdm660-rpmpd
> diff --git a/include/dt-bindings/power/qcom-rpmpd.h b/include/dt-bindings/power/qcom-rpmpd.h
> index 1bf8e87ecd7e..5a6dd5ded087 100644
> --- a/include/dt-bindings/power/qcom-rpmpd.h
> +++ b/include/dt-bindings/power/qcom-rpmpd.h
> @@ -4,6 +4,15 @@
> #ifndef _DT_BINDINGS_POWER_QCOM_RPMPD_H
> #define _DT_BINDINGS_POWER_QCOM_RPMPD_H
>
> +/* SA8155P Power Domain Indexes */
> +#define SA8155P_CX 0
> +#define SA8155P_CX_AO 1
> +#define SA8155P_EBI 2
> +#define SA8155P_GFX 3
> +#define SA8155P_MSS 4
> +#define SA8155P_MX 5
> +#define SA8155P_MX_AO 6
> +
> /* SA8775P Power Domain Indexes */
> #define SA8775P_CX 0
> #define SA8775P_CX_AO 1
> --
> 2.39.1

Tested on my SA8155p-ADP, so for the series:

Reviewed-and-Tested-by: Bhupesh Sharma <[email protected]>

Thanks.

2023-05-25 05:41:49

by Bjorn Andersson

[permalink] [raw]
Subject: Re: (subset) [PATCH 1/3] dt-bindings: power: qcom,rpmpd: Add SA8155P

On Tue, 14 Feb 2023 10:54:33 +0100, Konrad Dybcio wrote:
> Add a compatible for SA8155P platforms and relevant defines to the
> include file.
>
>

Applied, thanks!

[3/3] arm64: dts: qcom: Split out SA8155P and use correct RPMh power domains
commit: 8c67e554754ca669911625412ae9a6cd8cee1c82

Best regards,
--
Bjorn Andersson <[email protected]>