2023-03-20 13:56:46

by Johan Hovold

[permalink] [raw]
Subject: [PATCH 0/3] 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(-)

--
2.39.2



2023-03-20 13:56:50

by Johan Hovold

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

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")
Signed-off-by: Johan Hovold <[email protected]>
---
arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
index 1411edd42b25..c35e7f6bd657 100644
--- a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
@@ -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>;
--
2.39.2


2023-03-20 13:56:55

by Johan Hovold

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

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

Signed-off-by: Johan Hovold <[email protected]>
---
Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml b/Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml
index adf88245c409..05bf0d76bf5f 100644
--- a/Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml
+++ b/Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml
@@ -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#
--
2.39.2


2023-03-20 13:56:59

by Johan Hovold

[permalink] [raw]
Subject: [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")
Signed-off-by: Johan Hovold <[email protected]>
---
.../bindings/nvmem/qcom,spmi-sdam.yaml | 31 +++++++++++--------
1 file changed, 18 insertions(+), 13 deletions(-)

diff --git a/Documentation/devicetree/bindings/nvmem/qcom,spmi-sdam.yaml b/Documentation/devicetree/bindings/nvmem/qcom,spmi-sdam.yaml
index e08504ef3b6e..4702f10ecca2 100644
--- a/Documentation/devicetree/bindings/nvmem/qcom,spmi-sdam.yaml
+++ b/Documentation/devicetree/bindings/nvmem/qcom,spmi-sdam.yaml
@@ -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>;
+ };
+ };
+ };
...
--
2.39.2


2023-03-20 14:05:28

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH 2/3] arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property



On 20.03.2023 14:57, Johan Hovold wrote:
> 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")
> Signed-off-by: Johan Hovold <[email protected]>
> ---
Reviewed-by: Konrad Dybcio <[email protected]>

Konrad
> arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
> index 1411edd42b25..c35e7f6bd657 100644
> --- a/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc8280xp-pmics.dtsi
> @@ -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>;

2023-03-20 14:11:49

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 2/3] arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

On 20/03/2023 14:57, Johan Hovold wrote:
> The SPMI PMIC register region width is fixed and should not be encoded
> in the devicetree.

It also violates size-cells=0.

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


Best regards,
Krzysztof


2023-03-20 14:12:50

by Krzysztof Kozlowski

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

On 20/03/2023 14:57, Johan Hovold wrote:
> 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.


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



Best regards,
Krzysztof


2023-03-20 14:13:48

by Krzysztof Kozlowski

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

On 20/03/2023 14:57, Johan Hovold wrote:
> Add an 'nvram' pattern property and a reference to the corresponding
> SDAM DT schema.
>


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

Best regards,
Krzysztof


2023-03-20 14:36:13

by Johan Hovold

[permalink] [raw]
Subject: Re: [PATCH 2/3] arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

On Mon, Mar 20, 2023 at 02:57:09PM +0100, Johan Hovold wrote:
> 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

2023-03-20 14:36:55

by Johan Hovold

[permalink] [raw]
Subject: Re: [PATCH 2/3] arm64: dts: qcom: sc8280xp-pmics: fix sdam 'reg' property

On Mon, Mar 20, 2023 at 03:11:39PM +0100, Krzysztof Kozlowski wrote:
> On 20/03/2023 14:57, Johan Hovold wrote:
> > 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

2023-03-20 14:40:11

by Johan Hovold

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

On Mon, Mar 20, 2023 at 03:12:41PM +0100, Krzysztof Kozlowski wrote:
> On 20/03/2023 14:57, Johan Hovold wrote:
> > 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

2023-03-21 13:17:59

by Rob Herring (Arm)

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


On Mon, 20 Mar 2023 14:57:10 +0100, Johan Hovold wrote:
> Add an 'nvram' pattern property and a reference to the corresponding
> SDAM DT schema.
>
> Signed-off-by: Johan Hovold <[email protected]>
> ---
> 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


2023-03-21 13:21:18

by Rob Herring

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

On Tue, Mar 21, 2023 at 8:17 AM Rob Herring <[email protected]> wrote:
>
>
> On Mon, 20 Mar 2023 14:57:10 +0100, Johan Hovold wrote:
> > Add an 'nvram' pattern property and a reference to the corresponding
> > SDAM DT schema.
> >
> > Signed-off-by: Johan Hovold <[email protected]>
> > ---
> > 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]
>

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

>
> 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
>

2023-03-21 13:27:20

by Johan Hovold

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

On Tue, Mar 21, 2023 at 08:17:45AM -0500, Rob Herring wrote:
>
> On Mon, 20 Mar 2023 14:57:10 +0100, Johan Hovold wrote:
> > Add an 'nvram' pattern property and a reference to the corresponding
> > SDAM DT schema.
> >
> > Signed-off-by: Johan Hovold <[email protected]>
> > ---
> > 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.

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

Johan

2023-03-21 13:37:34

by Johan Hovold

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

On Tue, Mar 21, 2023 at 08:20:55AM -0500, Rob Herring wrote:
> On Tue, Mar 21, 2023 at 8:17 AM Rob Herring <[email protected]> wrote:
> >
> >
> > On Mon, 20 Mar 2023 14:57:10 +0100, Johan Hovold wrote:
> > > Add an 'nvram' pattern property and a reference to the corresponding
> > > SDAM DT schema.
> > >
> > > Signed-off-by: Johan Hovold <[email protected]>
> > > ---
> > > 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]
> >
>
> 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

2023-03-21 15:28:15

by Rob Herring

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

On Tue, Mar 21, 2023 at 8:37 AM Johan Hovold <[email protected]> wrote:
>
> On Tue, Mar 21, 2023 at 08:20:55AM -0500, Rob Herring wrote:
> > On Tue, Mar 21, 2023 at 8:17 AM Rob Herring <[email protected]> wrote:
> > >
> > >
> > > On Mon, 20 Mar 2023 14:57:10 +0100, Johan Hovold wrote:
> > > > Add an 'nvram' pattern property and a reference to the corresponding
> > > > SDAM DT schema.
> > > >
> > > > Signed-off-by: Johan Hovold <[email protected]>
> > > > ---
> > > > 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]
> > >
> >
> > 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

2023-03-21 19:53:49

by Rob Herring (Arm)

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


On Mon, 20 Mar 2023 14:57:10 +0100, Johan Hovold wrote:
> Add an 'nvram' pattern property and a reference to the corresponding
> SDAM DT schema.
>
> Signed-off-by: Johan Hovold <[email protected]>
> ---
> Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>

Acked-by: Rob Herring <[email protected]>


2023-03-22 07:13:48

by Johan Hovold

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

On Tue, Mar 21, 2023 at 10:27:55AM -0500, Rob Herring wrote:
> On Tue, Mar 21, 2023 at 8:37 AM Johan Hovold <[email protected]> wrote:
> > On Tue, Mar 21, 2023 at 08:20:55AM -0500, Rob Herring wrote:
> > > On Tue, Mar 21, 2023 at 8:17 AM Rob Herring <[email protected]> wrote:
> > > > On Mon, 20 Mar 2023 14:57:10 +0100, Johan Hovold wrote:
> > > > > Add an 'nvram' pattern property and a reference to the corresponding
> > > > > SDAM DT schema.
> > > > >
> > > > > Signed-off-by: Johan Hovold <[email protected]>
> > > > > ---
> > > > > 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]
> > > >
> > >
> > > 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.

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.

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

Understood.

Johan

2023-03-24 18:33:13

by Bjorn Andersson

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

On Mon, 20 Mar 2023 14:57:07 +0100, Johan Hovold wrote:
> 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,
--
Bjorn Andersson <[email protected]>

2023-03-30 13:06:03

by Lee Jones

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

On Mon, 20 Mar 2023, Johan Hovold wrote:

> Add an 'nvram' pattern property and a reference to the corresponding
> SDAM DT schema.
>
> Signed-off-by: Johan Hovold <[email protected]>
> ---
> Documentation/devicetree/bindings/mfd/qcom,spmi-pmic.yaml | 4 ++++
> 1 file changed, 4 insertions(+)

Applied, thanks

--
Lee Jones [李琼斯]