| Message ID | 20240210-msm8974-rpmpd-v2-0-595e2ff80ea1@z3ntu.xyz |
|---|---|
| Headers | show |
| Series | Add RPMPD support for MSM8974 | expand |
On Sonntag, 11. Februar 2024 12:03:15 CET Stephan Gerhold wrote: > 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". FWIW if we'd do that it should be qcom,msm8974-pm8841-rpmpd (so pm8841 instead of pm8941) But also in the same vain, it was maybe a bit of a bad decision originally to make the compatibles SoC-specific and not SoC+PMIC-specific - though in nearly all cases this combo is fixed for a given SoC? Anyways, I'll wait for more comments about this, I'm open to changing it either way. Regards Luca > > 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
On 10/02/2024 17:38, 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> > --- Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Sat, 10 Feb 2024 at 17:39, Luca Weiss <luca@z3ntu.xyz> wrote: > > Add driver support for the RPM power domains found on the different > MSM8974 devices. > > Devicetree integration will come at a later point since also some > mostly remoteproc drivers need to be adjusted. > > Also the MX power domains on this SoC seems to work quite a bit > differently, we'd need to send raw voltages to it, so these are ignored > in this series. > > Signed-off-by: Luca Weiss <luca@z3ntu.xyz> Applied for next, thanks! Note that patch1 is also available at the immutable dt branch, if soc maintainers need to pull it in too. Kind regards Uffe > --- > Changes in v2: > - Drop MSM8974_VDDGFX_AO in all patches > - Link to v1: https://lore.kernel.org/r/20240210-msm8974-rpmpd-v1-0-de9355e6842a@z3ntu.xyz > > --- > Luca Weiss (3): > dt-bindings: power: rpmpd: Add MSM8974 power domains > pmdomain: qcom: rpmpd: Add MSM8974+PM8841 power domains > pmdomain: qcom: rpmpd: Add MSM8974PRO+PMA8084 power domains > > .../devicetree/bindings/power/qcom,rpmpd.yaml | 2 + > drivers/pmdomain/qcom/rpmpd.c | 83 ++++++++++++++++++++++ > include/dt-bindings/power/qcom-rpmpd.h | 7 ++ > 3 files changed, 92 insertions(+) > --- > base-commit: 6e3fa474051f3d276ea708bdb8e8e1f66d1d3ee5 > change-id: 20240210-msm8974-rpmpd-6e48fe374275 > > Best regards, > -- > Luca Weiss <luca@z3ntu.xyz> >
Add driver support for the RPM power domains found on the different MSM8974 devices. Devicetree integration will come at a later point since also some mostly remoteproc drivers need to be adjusted. Also the MX power domains on this SoC seems to work quite a bit differently, we'd need to send raw voltages to it, so these are ignored in this series. Signed-off-by: Luca Weiss <luca@z3ntu.xyz> --- Changes in v2: - Drop MSM8974_VDDGFX_AO in all patches - Link to v1: https://lore.kernel.org/r/20240210-msm8974-rpmpd-v1-0-de9355e6842a@z3ntu.xyz --- Luca Weiss (3): dt-bindings: power: rpmpd: Add MSM8974 power domains pmdomain: qcom: rpmpd: Add MSM8974+PM8841 power domains pmdomain: qcom: rpmpd: Add MSM8974PRO+PMA8084 power domains .../devicetree/bindings/power/qcom,rpmpd.yaml | 2 + drivers/pmdomain/qcom/rpmpd.c | 83 ++++++++++++++++++++++ include/dt-bindings/power/qcom-rpmpd.h | 7 ++ 3 files changed, 92 insertions(+) --- base-commit: 6e3fa474051f3d276ea708bdb8e8e1f66d1d3ee5 change-id: 20240210-msm8974-rpmpd-6e48fe374275 Best regards,