Skip to contentSkip to conversation

arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

PATCH Applied Johan Hovold 8 participants 20 messages Mar 20, 2023 – Mar 30, 2023

Subsystems (5)
lore
Share & tools
lore
#0 lore raw

cover arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

This series addresses the dt-schema validation warnings that were introduced when enabling the PMIC SDAM block (and RTC) on sc8280xp.

I guess this should go through the nvmem, qcom, and mfd trees, respectively.

Johan

Johan Hovold (3):
dt-bindings: nvmem: qcom,spmi-sdam: fix example 'reg' property
arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property
dt-bindings: mfd: qcom,spmi-pmic: add nvram function

 .../bindings/mfd/qcom,spmi-pmic.yaml          |  4 +++
 .../bindings/nvmem/qcom,spmi-sdam.yaml        | 31 +++++++++++--------
 arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi  |  2 +-
 3 files changed, 23 insertions(+), 14 deletions(-)
Signature · 1 line
2.39.2
#1 lore raw

PATCH 1/3 dt-bindings: nvmem: qcom,spmi-sdam: fix example 'reg' property

The SPMI PMIC register region width is fixed and should not be encoded in the devicetree.

Amend the example with a parent pmic node with the expected '#address-cells' and '#size-cells' and fix up the 'reg' property.

Fixes 9664a6b54c57 ("dt-bindings: nvmem: add binding for QTI SPMI SDAM")
---
 .../bindings/nvmem/qcom,spmi-sdam.yaml        | 31 +++++++++++--------
 1 file changed, 18 insertions(+), 13 deletions(-)
Documentation/devicetree/bindings/nvmem/qcom,spmi-sdam.yaml +18 -13
index e08504ef3b6e..4702f10ecca2 100644
@@ -42,17 +42,22 @@ unevaluatedProperties: false
 
 examples:
   - |
-      sdam_1: nvram@b000 {
-          #address-cells = <1>;
-          #size-cells = <1>;
-          compatible = "qcom,spmi-sdam";
-          reg = <0xb000 0x100>;
-          ranges = <0 0xb000 0x100>;
-
-          /* Data cells */
-          restart_reason: restart@50 {
-              reg = <0x50 0x1>;
-              bits = <6 2>;
-          };
-      };
+    pmic {
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        sdam_1: nvram@b000 {
+            compatible = "qcom,spmi-sdam";
+            reg = <0xb000>;
+            #address-cells = <1>;
+            #size-cells = <1>;
+            ranges = <0 0xb000 0x100>;
+
+            /* Data cells */
+            restart_reason: restart@50 {
+                reg = <0x50 0x1>;
+                bits = <6 2>;
+            };
+        };
+    };
 ...
Signature · 1 line
2.39.2
Reviewed-by on v1 1/3
#2 lore raw

Re: [PATCH 1/3] dt-bindings: nvmem: qcom,spmi-sdam: fix example 'reg' property

Quoted 7 lines · Johan Hovold

The SPMI PMIC register region width is fixed and should not be encoded in the devicetree.

Amend the example with a parent pmic node with the expected '#address-cells' and '#size-cells' and fix up the 'reg' property.

Fixes 9664a6b54c57 ("dt-bindings: nvmem: add binding for QTI SPMI SDAM")

I would argue there is nothing to fix - the original example in that context was correct, because examples have size-cells=1.

Best regards,
Krzysztof

#3 lore raw

Re: [PATCH 1/3] dt-bindings: nvmem: qcom,spmi-sdam: fix example 'reg' property

Quoted 11 lines · Krzysztof Kozlowski
Quoted 7 lines · Johan Hovold

The SPMI PMIC register region width is fixed and should not be encoded in the devicetree.

Amend the example with a parent pmic node with the expected '#address-cells' and '#size-cells' and fix up the 'reg' property.

Fixes 9664a6b54c57 ("dt-bindings: nvmem: add binding for QTI SPMI SDAM")

I would argue there is nothing to fix - the original example in that context was correct, because examples have size-cells=1.

But you'll never find these PMIC functions on a memory-mapped or other bus with size-cells != 0 so I'd still say that the example is broken (and it is definitely misleading as I found out the hard way).

Johan

#4 lore raw

PATCH 2/3 arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

In reply to Johan Hovold

The SPMI PMIC register region width is fixed and should not be encoded in the devicetree.

Fixes 42f45cc655d0 ("arm64: dts: qcom: sc8280xp-pmics: add pmk8280 sdam nvram")
---
 arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi +1 -1
index 1411edd42b25..c35e7f6bd657 100644
@@ -107,7 +107,7 @@ pmk8280_rtc: rtc@6100 {
 
 		pmk8280_sdam_6: nvram@8500 {
 			compatible = "qcom,spmi-sdam";
-			reg = <0x8500 0x100>;
+			reg = <0x8500>;
 			#address-cells = <1>;
 			#size-cells = <1>;
 			ranges = <0 0x8500 0x100>;
Signature · 1 line
2.39.2
Reviewed-by on v1 2/3
#5 lore raw
Quoted 6 lines · Johan Hovold

The SPMI PMIC register region width is fixed and should not be encoded in the devicetree.

Fixes 42f45cc655d0 ("arm64: dts: qcom: sc8280xp-pmics: add pmk8280 sdam nvram")

---

Konrad

Quoted 16 lines
 arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi +1 -1
index 1411edd42b25..c35e7f6bd657 100644
@@ -107,7 +107,7 @@ pmk8280_rtc: rtc@6100 {
 
 		pmk8280_sdam_6: nvram@8500 {
 			compatible = "qcom,spmi-sdam";
-			reg = <0x8500 0x100>;
+			reg = <0x8500>;
 			#address-cells = <1>;
 			#size-cells = <1>;
 			ranges = <0 0x8500 0x100>;
#7 lore raw
Quoted 5 lines · Krzysztof Kozlowski
Quoted 2 lines · Johan Hovold

The SPMI PMIC register region width is fixed and should not be encoded in the devicetree.

It also violates size-cells=0.

That's the same thing.

Johan

#8 lore raw
In reply to Johan Hovold
Quoted 4 lines · Johan Hovold

The SPMI PMIC register region width is fixed and should not be encoded in the devicetree.

Fixes 42f45cc655d0 ("arm64: dts: qcom: sc8280xp-pmics: add pmk8280 sdam nvram")

Bah, that should have been:

Fixes d6dbbda37ab5 ("arm64: dts: qcom: sc8280xp-pmics: add pmk8280 sdam nvram")

Can you fix that up when applying, Bjorn?

Johan

#9 lore raw

PATCH 3/3 dt-bindings: mfd: qcom,spmi-pmic: add nvram function

In reply to Johan Hovold

Add an 'nvram' pattern property and a reference to the corresponding SDAM DT schema.

---
 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
 1 file changed, 4 insertions(+)
Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml +4 -0
index adf88245c409..05bf0d76bf5f 100644
@@ -135,6 +135,10 @@ patternProperties:
     type: object
     $ref: /schemas/pinctrl/qcom,pmic-gpio.yaml#
 
+  "^nvram@[0-9a-f]+$":
+    type: object
+    $ref: /schemas/nvmem/qcom,spmi-sdam.yaml#
+
   "pon@[0-9a-f]+$":
     type: object
     $ref: /schemas/power/reset/qcom,pon.yaml#
Signature · 1 line
2.39.2
maintainer
#11 lore raw

Re: [PATCH 3/3] dt-bindings: mfd: qcom,spmi-pmic: add nvram function

In reply to Johan Hovold
Quoted 8 lines · Johan Hovold

Add an 'nvram' pattern property and a reference to the corresponding SDAM DT schema.

---
 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
 1 file changed, 4 insertions(+)

Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/[email protected]

pm8941@0: gpio@c000: 'otg' does not match any of the regexes: '-state$', 'pinctrl-[0-9]+'
arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dtb

pm8941@0: 'pwrkey@800' does not match any of the regexes: '(.*)?(wled|leds)@[0-9a-f]+$', '^adc-tm@[0-9a-f]+$', '^adc@[0-9a-f]+$', '^audio-codec@[0-9a-f]+$', '^charger@[0-9a-f]+$', '^mpps@[0-9a-f]+$', '^nvram@[0-9a-f]+$', '^rtc@[0-9a-f]+$', '^temp-alarm@[0-9a-f]+$', '^usb-detect@[0-9a-f]+$', '^usb-vbus-regulator@[0-9a-f]+$', '^vibrator@[0-9a-f]+$', 'gpio@[0-9a-f]+$', 'pinctrl-[0-9]+', 'pon@[0-9a-f]+$'

	arch/arm/boot/dts/qcom-apq8074-dragonboard.dtb
	arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dtb
	arch/arm/boot/dts/qcom-msm8974pro-fairphone-fp2.dtb
	arch/arm/boot/dts/qcom-msm8974pro-oneplus-bacon.dtb
	arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dtb
	arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine-amami.dtb
	arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine-honami.dtb

pma8084@0: 'pwrkey@800' does not match any of the regexes: '(.*)?(wled|leds)@[0-9a-f]+$', '^adc-tm@[0-9a-f]+$', '^adc@[0-9a-f]+$', '^audio-codec@[0-9a-f]+$', '^charger@[0-9a-f]+$', '^mpps@[0-9a-f]+$', '^nvram@[0-9a-f]+$', '^rtc@[0-9a-f]+$', '^temp-alarm@[0-9a-f]+$', '^usb-detect@[0-9a-f]+$', '^usb-vbus-regulator@[0-9a-f]+$', '^vibrator@[0-9a-f]+$', 'gpio@[0-9a-f]+$', 'pinctrl-[0-9]+', 'pon@[0-9a-f]+$'

	arch/arm/boot/dts/qcom-apq8084-ifc6540.dtb
	arch/arm/boot/dts/qcom-apq8084-mtp.dtb
	arch/arm/boot/dts/qcom-msm8974pro-samsung-klte.dtb

pmic@0: 'extcon@1300' does not match any of the regexes: '(.*)?(wled|leds)@[0-9a-f]+$', '^adc-tm@[0-9a-f]+$', '^adc@[0-9a-f]+$', '^audio-codec@[0-9a-f]+$', '^charger@[0-9a-f]+$', '^mpps@[0-9a-f]+$', '^nvram@[0-9a-f]+$', '^rtc@[0-9a-f]+$', '^temp-alarm@[0-9a-f]+$', '^usb-detect@[0-9a-f]+$', '^usb-vbus-regulator@[0-9a-f]+$', '^vibrator@[0-9a-f]+$', 'gpio@[0-9a-f]+$', 'pinctrl-[0-9]+', 'pon@[0-9a-f]+$'

	arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
	arch/arm64/boot/dts/qcom/msm8916-acer-a1-724.dtb
	arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dtb
	arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dtb
	arch/arm64/boot/dts/qcom/msm8916-gplus-fl8005a.dtb
	arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dtb
	arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dtb
	arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dtb
	arch/arm64/boot/dts/qcom/msm8916-mtp.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-a3u-eur.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-a5u-eur.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-e5.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-e7.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-grandmax.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-gt510.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-gt58.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-j5x.dtb
	arch/arm64/boot/dts/qcom/msm8916-samsung-serranove.dtb
	arch/arm64/boot/dts/qcom/msm8916-thwc-uf896.dtb
	arch/arm64/boot/dts/qcom/msm8916-thwc-ufi001c.dtb
	arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dtb
	arch/arm/boot/dts/qcom-apq8016-sbc.dtb
	arch/arm/boot/dts/qcom-msm8916-samsung-e5.dtb
	arch/arm/boot/dts/qcom-msm8916-samsung-e7.dtb
	arch/arm/boot/dts/qcom-msm8916-samsung-grandmax.dtb
	arch/arm/boot/dts/qcom-msm8916-samsung-serranove.dtb

pmic@2: adc@3100:adc-chan@8:qcom,pre-scaling: 'oneOf' conditional failed, one must be fixed:

	arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@2: adc@3100: 'oneOf' conditional failed, one must be fixed:

	arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@2: adc@4500:compatible: 'oneOf' conditional failed, one must be fixed:

	arch/arm64/boot/dts/qcom/msm8998-oneplus-cheeseburger.dtb
	arch/arm64/boot/dts/qcom/msm8998-oneplus-dumpling.dtb
	arch/arm64/boot/dts/qcom/sdm845-db845c.dtb
	arch/arm64/boot/dts/qcom/sdm845-db845c-navigation-mezzanine.dtb
	arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dtb
	arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dtb
	arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-ebbg.dtb
	arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-tianma.dtb

pmic@2: adc@4500: 'oneOf' conditional failed, one must be fixed:

	arch/arm64/boot/dts/qcom/msm8998-oneplus-cheeseburger.dtb
	arch/arm64/boot/dts/qcom/msm8998-oneplus-dumpling.dtb
	arch/arm64/boot/dts/qcom/sdm845-db845c.dtb
	arch/arm64/boot/dts/qcom/sdm845-db845c-navigation-mezzanine.dtb
	arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dtb
	arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dtb
	arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-ebbg.dtb
	arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-tianma.dtb

pmic@2: gpio@c000:button-backlight-state: 'oneOf' conditional failed, one must be fixed:
arch/arm64/boot/dts/qcom/msm8998-oneplus-cheeseburger.dtb

pmic@3: leds@d800:interrupt-names:0: 'ovp' was expected

	arch/arm64/boot/dts/qcom/msm8953-motorola-potter.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-daisy.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-mido.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-tissot.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-vince.dtb
	arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-kugo.dtb
	arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-suzu.dtb
	arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dtb

pmic@3: leds@d800:interrupt-names: ['ovp'] is too short

	arch/arm64/boot/dts/qcom/sda660-inforce-ifc6560.dtb
	arch/arm64/boot/dts/qcom/sdm630-sony-xperia-ganges-kirin.dtb
	arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-discovery.dtb
	arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-pioneer.dtb
	arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-voyager.dtb
	arch/arm64/boot/dts/qcom/sdm636-sony-xperia-ganges-mermaid.dtb
	arch/arm64/boot/dts/qcom/sdm660-xiaomi-lavender.dtb
	arch/arm64/boot/dts/qcom/sdm670-google-sargo.dtb

pmic@3: leds@d800:interrupt-names: ['short'] is too short

	arch/arm64/boot/dts/qcom/msm8953-motorola-potter.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-daisy.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-mido.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-tissot.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-vince.dtb
	arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-kugo.dtb
	arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-suzu.dtb
	arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dtb

pmic@3: leds@d800:interrupts: [[3, 216, 1, 1]] is too short

	arch/arm64/boot/dts/qcom/sda660-inforce-ifc6560.dtb
	arch/arm64/boot/dts/qcom/sdm630-sony-xperia-ganges-kirin.dtb
	arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-discovery.dtb
	arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-pioneer.dtb
	arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-voyager.dtb
	arch/arm64/boot/dts/qcom/sdm636-sony-xperia-ganges-mermaid.dtb
	arch/arm64/boot/dts/qcom/sdm660-xiaomi-lavender.dtb
	arch/arm64/boot/dts/qcom/sdm670-google-sargo.dtb

pmic@3: leds@d800:interrupts: [[3, 216, 2, 1]] is too short

	arch/arm64/boot/dts/qcom/msm8953-motorola-potter.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-daisy.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-mido.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-tissot.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-vince.dtb
	arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-kugo.dtb
	arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-suzu.dtb
	arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dtb

pmic@3: leds@d800: Unevaluated properties are not allowed ('interrupt-names' was unexpected)

	arch/arm64/boot/dts/qcom/msm8953-motorola-potter.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-daisy.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-tissot.dtb
	arch/arm64/boot/dts/qcom/msm8953-xiaomi-vince.dtb
	arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dtb

pmic@3: regulators: Unevaluated properties are not allowed ('#address-cells', '#size-cells', 's2@1700' were unexpected)

	arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dtb
	arch/arm64/boot/dts/qcom/apq8096-db820c.dtb
	arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dtb
	arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb

pmic@3: regulators: Unevaluated properties are not allowed ('#address-cells', '#size-cells' were unexpected)

	arch/arm64/boot/dts/qcom/apq8096-ifc6640.dtb
	arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dtb
	arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dtb
	arch/arm64/boot/dts/qcom/msm8992-xiaomi-libra.dtb
	arch/arm64/boot/dts/qcom/msm8996-oneplus3.dtb
	arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dtb
	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-scorpio.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
	arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800:interrupt-names:0: 'ovp' was expected

	arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dtb
	arch/arm64/boot/dts/qcom/apq8096-db820c.dtb
	arch/arm64/boot/dts/qcom/apq8096-ifc6640.dtb
	arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dtb
	arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dtb
	arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dtb
	arch/arm64/boot/dts/qcom/msm8992-xiaomi-libra.dtb
	arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb
	arch/arm64/boot/dts/qcom/msm8996-oneplus3.dtb
	arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dtb
	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-scorpio.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
	arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800:interrupt-names: ['short'] is too short

	arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dtb
	arch/arm64/boot/dts/qcom/apq8096-db820c.dtb
	arch/arm64/boot/dts/qcom/apq8096-ifc6640.dtb
	arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dtb
	arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dtb
	arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dtb
	arch/arm64/boot/dts/qcom/msm8992-xiaomi-libra.dtb
	arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb
	arch/arm64/boot/dts/qcom/msm8996-oneplus3.dtb
	arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dtb
	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-scorpio.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
	arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800:interrupts: [[3, 216, 2, 1]] is too short

	arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dtb
	arch/arm64/boot/dts/qcom/apq8096-db820c.dtb
	arch/arm64/boot/dts/qcom/apq8096-ifc6640.dtb
	arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dtb
	arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dtb
	arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dtb
	arch/arm64/boot/dts/qcom/msm8992-xiaomi-libra.dtb
	arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb
	arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb
	arch/arm64/boot/dts/qcom/msm8996-oneplus3.dtb
	arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dtb
	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-scorpio.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
	arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800: 'label' is a required property

	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
	arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800: Unevaluated properties are not allowed ('interrupt-names' was unexpected)

	arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
	arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
	arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@5: 'led-controller@d300' does not match any of the regexes: '(.*)?(wled|leds)@[0-9a-f]+$', '^adc-tm@[0-9a-f]+$', '^adc@[0-9a-f]+$', '^audio-codec@[0-9a-f]+$', '^charger@[0-9a-f]+$', '^mpps@[0-9a-f]+$', '^nvram@[0-9a-f]+$', '^rtc@[0-9a-f]+$', '^temp-alarm@[0-9a-f]+$', '^usb-detect@[0-9a-f]+$', '^usb-vbus-regulator@[0-9a-f]+$', '^vibrator@[0-9a-f]+$', 'gpio@[0-9a-f]+$', 'pinctrl-[0-9]+', 'pon@[0-9a-f]+$'

	arch/arm64/boot/dts/qcom/sc7180-idp.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r2.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-auo.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-auo.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-wifi.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-parade.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-ti.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-parade.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-ti.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-inx.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb
	arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@5: leds@d800:interrupt-names: ['ovp'] is too short

	arch/arm64/boot/dts/qcom/sc7180-idp.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r2.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-auo.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-auo.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-wifi.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-parade.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-ti.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-parade.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-ti.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-inx.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb
	arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@5: leds@d800:interrupts: [[5, 216, 1, 1]] is too short

	arch/arm64/boot/dts/qcom/sc7180-idp.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r2.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r4.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-kb.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-auo.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-auo.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-wifi.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-parade.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-ti.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-parade.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-ti.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-r1-lte.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-inx.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx.dtb
	arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb
	arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@7: compatible:0: 'qcom,pm8450' is not one of ['qcom,pm6125', 'qcom,pm6150', 'qcom,pm6150l', 'qcom,pm6350', 'qcom,pm660', 'qcom,pm660l', 'qcom,pm7250b', 'qcom,pm7325', 'qcom,pm8004', 'qcom,pm8005', 'qcom,pm8009', 'qcom,pm8010', 'qcom,pm8019', 'qcom,pm8028', 'qcom,pm8110', 'qcom,pm8150', 'qcom,pm8150b', 'qcom,pm8150c', 'qcom,pm8150l', 'qcom,pm8226', 'qcom,pm8350', 'qcom,pm8350b', 'qcom,pm8350c', 'qcom,pm8550', 'qcom,pm8550b', 'qcom,pm8550ve', 'qcom,pm8550vs', 'qcom,pm8841', 'qcom,pm8909', 'qcom,pm8916', 'qcom,pm8941', 'qcom,pm8950', 'qcom,pm8953', 'qcom,pm8994', 'qcom,pm8998', 'qcom,pma8084', 'qcom,pmd9635', 'qcom,pmi8950', 'qcom,pmi8962', 'qcom,pmi8994', 'qcom,pmi8998', 'qcom,pmk8002', 'qcom,pmk8350', 'qcom,pmk8550', 'qcom,pmm8155au', 'qcom,pmp8074', 'qcom,pmr735a', 'qcom,pmr735b', 'qcom,pmr735d', 'qcom,pms405', 'qcom,pmx55', 'qcom,pmx65', 'qcom,smb2351']

	arch/arm64/boot/dts/qcom/sm8450-hdk.dtb
	arch/arm64/boot/dts/qcom/sm8450-qrd.dtb
	arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara-pdx223.dtb
	arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara-pdx224.dtb
#12 lore raw

Re: [PATCH 3/3] dt-bindings: mfd: qcom,spmi-pmic: add nvram function

Quoted 12 lines · Rob Herring
Quoted 8 lines · Johan Hovold

Add an 'nvram' pattern property and a reference to the corresponding SDAM DT schema.

---
 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
 1 file changed, 4 insertions(+)
Quoted 9 lines

Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/[email protected]

This is a long list of warnings. Can you all fix some of them before adding on to this binding.

Quoted 409 lines

pm8941@0: gpio@c000: 'otg' does not match any of the regexes: '-state$', 'pinctrl-[0-9]+'
arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dtb

pm8941@0: 'pwrkey@800' does not match any of the regexes: '(.*)?(wled|leds)@[0-9a-f]+$', '^adc-tm@[0-9a-f]+$', '^adc@[0-9a-f]+$', '^audio-codec@[0-9a-f]+$', '^charger@[0-9a-f]+$', '^mpps@[0-9a-f]+$', '^nvram@[0-9a-f]+$', '^rtc@[0-9a-f]+$', '^temp-alarm@[0-9a-f]+$', '^usb-detect@[0-9a-f]+$', '^usb-vbus-regulator@[0-9a-f]+$', '^vibrator@[0-9a-f]+$', 'gpio@[0-9a-f]+$', 'pinctrl-[0-9]+', 'pon@[0-9a-f]+$'

        arch/arm/boot/dts/qcom-apq8074-dragonboard.dtb
        arch/arm/boot/dts/qcom-msm8974-lge-nexus5-hammerhead.dtb
        arch/arm/boot/dts/qcom-msm8974pro-fairphone-fp2.dtb
        arch/arm/boot/dts/qcom-msm8974pro-oneplus-bacon.dtb
        arch/arm/boot/dts/qcom-msm8974pro-sony-xperia-shinano-castor.dtb
        arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine-amami.dtb
        arch/arm/boot/dts/qcom-msm8974-sony-xperia-rhine-honami.dtb

pma8084@0: 'pwrkey@800' does not match any of the regexes: '(.*)?(wled|leds)@[0-9a-f]+$', '^adc-tm@[0-9a-f]+$', '^adc@[0-9a-f]+$', '^audio-codec@[0-9a-f]+$', '^charger@[0-9a-f]+$', '^mpps@[0-9a-f]+$', '^nvram@[0-9a-f]+$', '^rtc@[0-9a-f]+$', '^temp-alarm@[0-9a-f]+$', '^usb-detect@[0-9a-f]+$', '^usb-vbus-regulator@[0-9a-f]+$', '^vibrator@[0-9a-f]+$', 'gpio@[0-9a-f]+$', 'pinctrl-[0-9]+', 'pon@[0-9a-f]+$'

        arch/arm/boot/dts/qcom-apq8084-ifc6540.dtb
        arch/arm/boot/dts/qcom-apq8084-mtp.dtb
        arch/arm/boot/dts/qcom-msm8974pro-samsung-klte.dtb

pmic@0: 'extcon@1300' does not match any of the regexes: '(.*)?(wled|leds)@[0-9a-f]+$', '^adc-tm@[0-9a-f]+$', '^adc@[0-9a-f]+$', '^audio-codec@[0-9a-f]+$', '^charger@[0-9a-f]+$', '^mpps@[0-9a-f]+$', '^nvram@[0-9a-f]+$', '^rtc@[0-9a-f]+$', '^temp-alarm@[0-9a-f]+$', '^usb-detect@[0-9a-f]+$', '^usb-vbus-regulator@[0-9a-f]+$', '^vibrator@[0-9a-f]+$', 'gpio@[0-9a-f]+$', 'pinctrl-[0-9]+', 'pon@[0-9a-f]+$'

        arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
        arch/arm64/boot/dts/qcom/msm8916-acer-a1-724.dtb
        arch/arm64/boot/dts/qcom/msm8916-alcatel-idol347.dtb
        arch/arm64/boot/dts/qcom/msm8916-asus-z00l.dtb
        arch/arm64/boot/dts/qcom/msm8916-gplus-fl8005a.dtb
        arch/arm64/boot/dts/qcom/msm8916-huawei-g7.dtb
        arch/arm64/boot/dts/qcom/msm8916-longcheer-l8150.dtb
        arch/arm64/boot/dts/qcom/msm8916-longcheer-l8910.dtb
        arch/arm64/boot/dts/qcom/msm8916-mtp.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-a3u-eur.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-a5u-eur.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-e5.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-e7.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-grandmax.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-gt510.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-gt58.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-j5.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-j5x.dtb
        arch/arm64/boot/dts/qcom/msm8916-samsung-serranove.dtb
        arch/arm64/boot/dts/qcom/msm8916-thwc-uf896.dtb
        arch/arm64/boot/dts/qcom/msm8916-thwc-ufi001c.dtb
        arch/arm64/boot/dts/qcom/msm8916-wingtech-wt88047.dtb
        arch/arm/boot/dts/qcom-apq8016-sbc.dtb
        arch/arm/boot/dts/qcom-msm8916-samsung-e5.dtb
        arch/arm/boot/dts/qcom-msm8916-samsung-e7.dtb
        arch/arm/boot/dts/qcom-msm8916-samsung-grandmax.dtb
        arch/arm/boot/dts/qcom-msm8916-samsung-serranove.dtb

pmic@2: adc@3100:adc-chan@8:qcom,pre-scaling: 'oneOf' conditional failed, one must be fixed:

        arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@2: adc@3100: 'oneOf' conditional failed, one must be fixed:

        arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@2: adc@4500:compatible: 'oneOf' conditional failed, one must be fixed:

        arch/arm64/boot/dts/qcom/msm8998-oneplus-cheeseburger.dtb
        arch/arm64/boot/dts/qcom/msm8998-oneplus-dumpling.dtb
        arch/arm64/boot/dts/qcom/sdm845-db845c.dtb
        arch/arm64/boot/dts/qcom/sdm845-db845c-navigation-mezzanine.dtb
        arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dtb
        arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dtb
        arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-ebbg.dtb
        arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-tianma.dtb

pmic@2: adc@4500: 'oneOf' conditional failed, one must be fixed:

        arch/arm64/boot/dts/qcom/msm8998-oneplus-cheeseburger.dtb
        arch/arm64/boot/dts/qcom/msm8998-oneplus-dumpling.dtb
        arch/arm64/boot/dts/qcom/sdm845-db845c.dtb
        arch/arm64/boot/dts/qcom/sdm845-db845c-navigation-mezzanine.dtb
        arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dtb
        arch/arm64/boot/dts/qcom/sdm845-oneplus-fajita.dtb
        arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-ebbg.dtb
        arch/arm64/boot/dts/qcom/sdm845-xiaomi-beryllium-tianma.dtb

pmic@2: gpio@c000:button-backlight-state: 'oneOf' conditional failed, one must be fixed:
arch/arm64/boot/dts/qcom/msm8998-oneplus-cheeseburger.dtb

pmic@3: leds@d800:interrupt-names:0: 'ovp' was expected

        arch/arm64/boot/dts/qcom/msm8953-motorola-potter.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-daisy.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-mido.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-tissot.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-vince.dtb
        arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-kugo.dtb
        arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-suzu.dtb
        arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dtb

pmic@3: leds@d800:interrupt-names: ['ovp'] is too short

        arch/arm64/boot/dts/qcom/sda660-inforce-ifc6560.dtb
        arch/arm64/boot/dts/qcom/sdm630-sony-xperia-ganges-kirin.dtb
        arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-discovery.dtb
        arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-pioneer.dtb
        arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-voyager.dtb
        arch/arm64/boot/dts/qcom/sdm636-sony-xperia-ganges-mermaid.dtb
        arch/arm64/boot/dts/qcom/sdm660-xiaomi-lavender.dtb
        arch/arm64/boot/dts/qcom/sdm670-google-sargo.dtb

pmic@3: leds@d800:interrupt-names: ['short'] is too short

        arch/arm64/boot/dts/qcom/msm8953-motorola-potter.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-daisy.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-mido.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-tissot.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-vince.dtb
        arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-kugo.dtb
        arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-suzu.dtb
        arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dtb

pmic@3: leds@d800:interrupts: [[3, 216, 1, 1]] is too short

        arch/arm64/boot/dts/qcom/sda660-inforce-ifc6560.dtb
        arch/arm64/boot/dts/qcom/sdm630-sony-xperia-ganges-kirin.dtb
        arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-discovery.dtb
        arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-pioneer.dtb
        arch/arm64/boot/dts/qcom/sdm630-sony-xperia-nile-voyager.dtb
        arch/arm64/boot/dts/qcom/sdm636-sony-xperia-ganges-mermaid.dtb
        arch/arm64/boot/dts/qcom/sdm660-xiaomi-lavender.dtb
        arch/arm64/boot/dts/qcom/sdm670-google-sargo.dtb

pmic@3: leds@d800:interrupts: [[3, 216, 2, 1]] is too short

        arch/arm64/boot/dts/qcom/msm8953-motorola-potter.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-daisy.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-mido.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-tissot.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-vince.dtb
        arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-kugo.dtb
        arch/arm64/boot/dts/qcom/msm8956-sony-xperia-loire-suzu.dtb
        arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dtb

pmic@3: leds@d800: Unevaluated properties are not allowed ('interrupt-names' was unexpected)

        arch/arm64/boot/dts/qcom/msm8953-motorola-potter.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-daisy.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-tissot.dtb
        arch/arm64/boot/dts/qcom/msm8953-xiaomi-vince.dtb
        arch/arm64/boot/dts/qcom/sdm450-motorola-ali.dtb

pmic@3: regulators: Unevaluated properties are not allowed ('#address-cells', '#size-cells', 's2@1700' were unexpected)

        arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dtb
        arch/arm64/boot/dts/qcom/apq8096-db820c.dtb
        arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dtb
        arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb

pmic@3: regulators: Unevaluated properties are not allowed ('#address-cells', '#size-cells' were unexpected)

        arch/arm64/boot/dts/qcom/apq8096-ifc6640.dtb
        arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dtb
        arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dtb
        arch/arm64/boot/dts/qcom/msm8992-xiaomi-libra.dtb
        arch/arm64/boot/dts/qcom/msm8996-oneplus3.dtb
        arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dtb
        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-scorpio.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
        arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800:interrupt-names:0: 'ovp' was expected

        arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dtb
        arch/arm64/boot/dts/qcom/apq8096-db820c.dtb
        arch/arm64/boot/dts/qcom/apq8096-ifc6640.dtb
        arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dtb
        arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dtb
        arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dtb
        arch/arm64/boot/dts/qcom/msm8992-xiaomi-libra.dtb
        arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb
        arch/arm64/boot/dts/qcom/msm8996-oneplus3.dtb
        arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dtb
        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-scorpio.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
        arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800:interrupt-names: ['short'] is too short

        arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dtb
        arch/arm64/boot/dts/qcom/apq8096-db820c.dtb
        arch/arm64/boot/dts/qcom/apq8096-ifc6640.dtb
        arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dtb
        arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dtb
        arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dtb
        arch/arm64/boot/dts/qcom/msm8992-xiaomi-libra.dtb
        arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb
        arch/arm64/boot/dts/qcom/msm8996-oneplus3.dtb
        arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dtb
        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-scorpio.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
        arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800:interrupts: [[3, 216, 2, 1]] is too short

        arch/arm64/boot/dts/qcom/apq8094-sony-xperia-kitakami-karin_windy.dtb
        arch/arm64/boot/dts/qcom/apq8096-db820c.dtb
        arch/arm64/boot/dts/qcom/apq8096-ifc6640.dtb
        arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-101.dtb
        arch/arm64/boot/dts/qcom/msm8992-lg-bullhead-rev-10.dtb
        arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dtb
        arch/arm64/boot/dts/qcom/msm8992-xiaomi-libra.dtb
        arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-ivy.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-karin.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-satsuki.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-sumire.dtb
        arch/arm64/boot/dts/qcom/msm8994-sony-xperia-kitakami-suzuran.dtb
        arch/arm64/boot/dts/qcom/msm8996-oneplus3.dtb
        arch/arm64/boot/dts/qcom/msm8996-oneplus3t.dtb
        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-scorpio.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
        arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800: 'label' is a required property

        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
        arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@3: wled@d800: Unevaluated properties are not allowed ('interrupt-names' was unexpected)

        arch/arm64/boot/dts/qcom/msm8996pro-xiaomi-natrium.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-dora.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-kagura.dtb
        arch/arm64/boot/dts/qcom/msm8996-sony-xperia-tone-keyaki.dtb
        arch/arm64/boot/dts/qcom/msm8996-xiaomi-gemini.dtb

pmic@5: 'led-controller@d300' does not match any of the regexes: '(.*)?(wled|leds)@[0-9a-f]+$', '^adc-tm@[0-9a-f]+$', '^adc@[0-9a-f]+$', '^audio-codec@[0-9a-f]+$', '^charger@[0-9a-f]+$', '^mpps@[0-9a-f]+$', '^nvram@[0-9a-f]+$', '^rtc@[0-9a-f]+$', '^temp-alarm@[0-9a-f]+$', '^usb-detect@[0-9a-f]+$', '^usb-vbus-regulator@[0-9a-f]+$', '^vibrator@[0-9a-f]+$', 'gpio@[0-9a-f]+$', 'pinctrl-[0-9]+', 'pon@[0-9a-f]+$'

        arch/arm64/boot/dts/qcom/sc7180-idp.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r2.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-auo.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-auo.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-wifi.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-parade.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-ti.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-parade.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-ti.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-inx.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb
        arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@5: leds@d800:interrupt-names: ['ovp'] is too short

        arch/arm64/boot/dts/qcom/sc7180-idp.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r2.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-auo.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-auo.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-wifi.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-parade.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-ti.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-parade.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-ti.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-inx.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb
        arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@5: leds@d800:interrupts: [[5, 216, 1, 1]] is too short

        arch/arm64/boot/dts/qcom/sc7180-idp.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-coachz-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r2.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r4.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-kb.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-r9-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-auo.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev0-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-auo.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-mrbland-rev1-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel360-wifi.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-parade.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-lte-ti.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-parade.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pazquel-ti.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r2-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-pompom-r3-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-quackingstick-r0-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-r1.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-r1-lte.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev0-inx.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx.dtb
        arch/arm64/boot/dts/qcom/sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb
        arch/arm64/boot/dts/qcom/sm7225-fairphone-fp4.dtb

pmic@7: compatible:0: 'qcom,pm8450' is not one of ['qcom,pm6125', 'qcom,pm6150', 'qcom,pm6150l', 'qcom,pm6350', 'qcom,pm660', 'qcom,pm660l', 'qcom,pm7250b', 'qcom,pm7325', 'qcom,pm8004', 'qcom,pm8005', 'qcom,pm8009', 'qcom,pm8010', 'qcom,pm8019', 'qcom,pm8028', 'qcom,pm8110', 'qcom,pm8150', 'qcom,pm8150b', 'qcom,pm8150c', 'qcom,pm8150l', 'qcom,pm8226', 'qcom,pm8350', 'qcom,pm8350b', 'qcom,pm8350c', 'qcom,pm8550', 'qcom,pm8550b', 'qcom,pm8550ve', 'qcom,pm8550vs', 'qcom,pm8841', 'qcom,pm8909', 'qcom,pm8916', 'qcom,pm8941', 'qcom,pm8950', 'qcom,pm8953', 'qcom,pm8994', 'qcom,pm8998', 'qcom,pma8084', 'qcom,pmd9635', 'qcom,pmi8950', 'qcom,pmi8962', 'qcom,pmi8994', 'qcom,pmi8998', 'qcom,pmk8002', 'qcom,pmk8350', 'qcom,pmk8550', 'qcom,pmm8155au', 'qcom,pmp8074', 'qcom,pmr735a', 'qcom,pmr735b', 'qcom,pmr735d', 'qcom,pms405', 'qcom,pmx55', 'qcom,pmx65', 'qcom,smb2351']

        arch/arm64/boot/dts/qcom/sm8450-hdk.dtb
        arch/arm64/boot/dts/qcom/sm8450-qrd.dtb
        arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara-pdx223.dtb
        arch/arm64/boot/dts/qcom/sm8450-sony-xperia-nagara-pdx224.dtb
#13 lore raw

Re: [PATCH 3/3] dt-bindings: mfd: qcom,spmi-pmic: add nvram function

Quoted 17 lines · Rob Herring
Quoted 16 lines · Rob Herring
Quoted 8 lines · Johan Hovold

Add an 'nvram' pattern property and a reference to the corresponding SDAM DT schema.

---
 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
 1 file changed, 4 insertions(+)

Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings.

Quoted 8 lines
Quoted 5 lines

Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/[email protected]

This is a long list of warnings. Can you all fix some of them before adding on to this binding.

Note that this patch is only making the list of warnings shorter so I don't think addressing the remaining issues should block this one.

Johan

#14 lore raw

Re: [PATCH 3/3] dt-bindings: mfd: qcom,spmi-pmic: add nvram function

Quoted 22 lines · Johan Hovold
Quoted 20 lines · Rob Herring
Quoted 19 lines · Rob Herring
Quoted 8 lines · Johan Hovold

Add an 'nvram' pattern property and a reference to the corresponding SDAM DT schema.

---
 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
 1 file changed, 4 insertions(+)

Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future.

Quoted 8 lines
Quoted 5 lines
Quoted 2 lines

Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/[email protected]

This is a long list of warnings. Can you all fix some of them before adding on to this binding.

Note that this patch is only making the list of warnings shorter so I don't think addressing the remaining issues should block this one.

How can you tell sorting thru the 100s of warnings? It sounded like a new feature to me, not a fix for something missing. Anyways, I've gotten 2 patches today for this binding and I don't see patches for fixes. Which is a bit surprising because you all generally are fixing warnings on the QCom stuff.

I didn't really mean to block this one, but I don't really want to see more.

Rob

#15 lore raw

Re: [PATCH 3/3] dt-bindings: mfd: qcom,spmi-pmic: add nvram function

Quoted 25 lines · Rob Herring
Quoted 24 lines · Johan Hovold
Quoted 22 lines · Rob Herring
Quoted 9 lines · Rob Herring
Quoted 7 lines · Johan Hovold

Add an 'nvram' pattern property and a reference to the corresponding SDAM DT schema.

---
 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
 1 file changed, 4 insertions(+)
Quoted 9 lines

Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future.

Full log is available here: https://patchwork.ozlabs.org/project/devicetree-bindings/patch/[email protected]

This is a long list of warnings. Can you all fix some of them before adding on to this binding.

Quoted 8 lines
Quoted 2 lines

Note that this patch is only making the list of warnings shorter so I don't think addressing the remaining issues should block this one.

How can you tell sorting thru the 100s of warnings? It sounded like a new feature to me, not a fix for something missing. Anyways, I've gotten 2 patches today for this binding and I don't see patches for fixes. Which is a bit surprising because you all generally are fixing warnings on the QCom stuff.

Indeed, it was not obvious and that's why I mentioned it.

The SDAM binding has been there since 2020 and 9664a6b54c57 ("dt-bindings: nvmem: add binding for QTI SPMI SDAM") but updating the "parent" pmic binding was apparently overlooked.

I'm generally not cleaning up DT checker warnings, but I want to avoid adding new ones as I unknowingly did in this case when enabling the RTC on the X13s.

Quoted 1 line

I didn't really mean to block this one, but I don't really want to see more.

Understood.

Johan

#16 lore raw

Re: [PATCH 3/3] dt-bindings: mfd: qcom,spmi-pmic: add nvram function

In reply to Rob Herring
Quoted 11 lines · Rob Herring
Quoted 8 lines · Johan Hovold

Add an 'nvram' pattern property and a reference to the corresponding SDAM DT schema.

---
 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
 1 file changed, 4 insertions(+)
Quoted 6 lines

Running 'make dtbs_check' with the schema in this patch gives the following warnings. Consider if they are expected or the schema is incorrect. These may not be new warnings.

Note that it is not yet a requirement to have 0 warnings for dtbs_check. This will change in the future.

All of these warnings appear to be due to existing issues unrelated to this patch.

Johan

maintainer Applied
#18 lore raw

Re: [PATCH 3/3] dt-bindings: mfd: qcom,spmi-pmic: add nvram function

In reply to Johan Hovold
Quoted 7 lines · Johan Hovold

Add an 'nvram' pattern property and a reference to the corresponding SDAM DT schema.

---
 Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
 1 file changed, 4 insertions(+)

Applied, thanks

--
Lee Jones [李琼斯]

maintainer Applied
#19 lore raw

Re: (subset) [PATCH 0/3] arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

In reply to Johan Hovold
Quoted 9 lines · Johan Hovold

This series addresses the dt-schema validation warnings that were introduced when enabling the PMIC SDAM block (and RTC) on sc8280xp.

I guess this should go through the nvmem, qcom, and mfd trees, respectively.

Johan

[...]

Applied, thanks!

[2/3] arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

      commit: 1681ce3735483ab6897e403a20a7faf5c34c1f09

Best regards,

Signature · 1 line
Bjorn Andersson <[email protected]>