| Message ID | 20240210-msm8974-rpmpd-v2-1-595e2ff80ea1@z3ntu.xyz |
|---|---|
| State | Accepted |
| Commit | 05d103950352d35c99142fe52ab50a8280dff7aa |
| Headers | show |
| Series | Add RPMPD support for MSM8974 | expand |
On Sat, Feb 10, 2024 at 05:38:56PM +0100, Luca Weiss wrote: > Add the compatibles and indexes for the rpmpd in MSM8974, both with the > standard PM8841+PM8941 PMICs but also devices found with PMA8084. > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> > --- > Documentation/devicetree/bindings/power/qcom,rpmpd.yaml | 2 ++ > include/dt-bindings/power/qcom-rpmpd.h | 7 +++++++ > 2 files changed, 9 insertions(+) > > diff --git a/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml b/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml > index 2ff246cf8b81..929b7ef9c1bc 100644 > --- a/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml > +++ b/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml > @@ -24,6 +24,8 @@ properties: > - qcom,msm8917-rpmpd > - qcom,msm8939-rpmpd > - qcom,msm8953-rpmpd > + - qcom,msm8974-rpmpd > + - qcom,msm8974pro-pma8084-rpmpd > - qcom,msm8976-rpmpd > - qcom,msm8994-rpmpd > - qcom,msm8996-rpmpd This is maybe more something for the DT reviewers to decide but I wonder if it is a bit confusing/misleading to describe one particular PMIC with a generic compatible, and the other with a more specific one. Perhaps it would be clearer to include the PMIC name in both compatibles, i.e. "qcom,msm8974-pm8941-rpmpd" instead of "qcom,msm8974-rpmpd". The "qcom,msm8974-rpmpd" compatible could be maybe added as fallback. While it wouldn't be used for matching in the (Linux) driver the DT binding itself *is* "compatible" between the two PMICs because they both have the same power domain indexes. i.e. compatible = "qcom,msm8974-pm8941-rpmpd", "qcom,msm8974-rpmpd"; compatible = "qcom,msm8974pro-pma8084-rpmpd", "qcom,msm8974-rpmpd"; Thanks, Stephan
diff --git a/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml b/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml index 2ff246cf8b81..929b7ef9c1bc 100644 --- a/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml +++ b/Documentation/devicetree/bindings/power/qcom,rpmpd.yaml @@ -24,6 +24,8 @@ properties: - qcom,msm8917-rpmpd - qcom,msm8939-rpmpd - qcom,msm8953-rpmpd + - qcom,msm8974-rpmpd + - qcom,msm8974pro-pma8084-rpmpd - qcom,msm8976-rpmpd - qcom,msm8994-rpmpd - qcom,msm8996-rpmpd diff --git a/include/dt-bindings/power/qcom-rpmpd.h b/include/dt-bindings/power/qcom-rpmpd.h index 7f4e2983a4c5..608087fb9a3d 100644 --- a/include/dt-bindings/power/qcom-rpmpd.h +++ b/include/dt-bindings/power/qcom-rpmpd.h @@ -308,6 +308,13 @@ #define MSM8953_VDDMX 5 #define MSM8953_VDDMX_AO 6 +/* MSM8974 Power Domain Indexes */ +#define MSM8974_VDDCX 0 +#define MSM8974_VDDCX_AO 1 +#define MSM8974_VDDCX_VFC 2 +#define MSM8974_VDDGFX 3 +#define MSM8974_VDDGFX_VFC 4 + /* MSM8976 Power Domain Indexes */ #define MSM8976_VDDCX 0 #define MSM8976_VDDCX_AO 1
Add the compatibles and indexes for the rpmpd in MSM8974, both with the standard PM8841+PM8941 PMICs but also devices found with PMA8084. Signed-off-by: Luca Weiss <luca@z3ntu.xyz> --- Documentation/devicetree/bindings/power/qcom,rpmpd.yaml | 2 ++ include/dt-bindings/power/qcom-rpmpd.h | 7 +++++++ 2 files changed, 9 insertions(+)