Re: [PATCH 00/14] arm64/dt-bindings: mfd: qcom: SPMI PMIC fixes

David Heidelberg
2022-08-29T14:55:20Z

Patches 1 ‒ 3 and 7 ‒ 13

Reviewed-by: David Heidelberg <david@ixit.cz>

On 28/08/2022 10:43, Krzysztof Kozlowski wrote:
> Hi,
>
> The Qualcomm SPMI PMIC DT schema conversion was not really tested and several
> issues in the bindings and DTS should be corrected.
>
> 1. The DTS patches can go independently.
> 2. The binding change continuous work of PWM reg fix and depends on it in
> context (diff hunk):
> https://lore.kernel.org/all/20220827145640.3530878-1-bryan.odonoghue@linaro.org/
> Binidings changes and above, can be taken via MFD tree (fixed commit was
> merged in v6.0-rc1).
>
> Best regards,
> Krzysztof
>
> Krzysztof Kozlowski (14):
> ARM: dts: qcom: align SPMI PMIC ADC node name with dtschema
> ARM: dts: qcom: pm8941: align SPMI PMIC LPG node name with dtschema
> ARM: dts: qcom: pmx55: align SPMI PMIC Power-on node name with
> dtschema
> arm64: dts: qcom: pmk8350: drop incorrect io-channel-ranges
> arm64: dts: qcom: pmk8350: drop interrupt-names from ADC
> arm64: dts: qcom: align SPMI PMIC ADC node name with dtschema
> arm64: dts: qcom: align SPMI PMIC regulators node name with dtschema
> arm64: dts: qcom: align SPMI PMIC LPG node name with dtschema
> arm64: dts: qcom: align SPMI PMIC Power-on node name with dtschema
> arm64: dts: qcom: align PMIC GPIO pin configuration with DT schema
> arm64: dts: qcom: sc7280-idp: correct ADC channel node name and unit
> address
> arm64: dts: qcom: sdm845-mtp: correct ADC settle time
> dt-bindings: mfd: qcom,spmi-pmic: fix regulator node schema
> dt-bindings: mfd: qcom,spmi-pmic: fix TM ADC node schema on PM8998
>
> .../bindings/mfd/qcom,spmi-pmic.yaml | 20 +++++++++++++++++--
> arch/arm/boot/dts/qcom-pm8941.dtsi | 4 ++--
> arch/arm/boot/dts/qcom-pma8084.dtsi | 2 +-
> arch/arm/boot/dts/qcom-pmx55.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/pm660.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/pm660l.dtsi | 4 ++--
> arch/arm64/boot/dts/qcom/pm8150.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/pm8150b.dtsi | 4 ++--
> arch/arm64/boot/dts/qcom/pm8150l.dtsi | 4 ++--
> arch/arm64/boot/dts/qcom/pm8953.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/pm8994.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/pmi8994.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/pmi8998.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/pmk8350.dtsi | 3 ---
> arch/arm64/boot/dts/qcom/pmm8155au_1.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/pmm8155au_2.dtsi | 2 +-
> .../boot/dts/qcom/sc7280-herobrine-crd.dts | 2 +-
> arch/arm64/boot/dts/qcom/sc7280-idp.dts | 2 +-
> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 2 +-
> arch/arm64/boot/dts/qcom/sdm845-mtp.dts | 12 +++++------
> .../boot/dts/qcom/sdm845-xiaomi-polaris.dts | 2 +-
> 21 files changed, 46 insertions(+), 33 deletions(-)
>
--
David Heidelberg
Consultant Software Engineer

Matrix: @okias:matrix.org