From: Jayesh Choudhary <j-choudhary@ti.com>
To: Rob Herring <robh@kernel.org>
Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org,
lgirdwood@gmail.com, robh+dt@kernel.org,
linux-kernel@vger.kernel.org, broonie@kernel.org, kishon@ti.com
Subject: Re: [PATCH] ASoC: dt-bindings: davinci-mcasp: convert McASP bindings to yaml schema
Date: Thu, 25 Nov 2021 15:37:36 +0530 [thread overview]
Message-ID: <d72bac88-8f56-35cd-a953-270a1ed4d34b@ti.com> (raw)
In-Reply-To: <1637685269.740254.3442929.nullmailer@robh.at.kernel.org>
On 23/11/21 10:04 pm, Rob Herring wrote:
> On Mon, 22 Nov 2021 14:45:25 +0530, Jayesh Choudhary wrote:
>> Convert the bindings for McASP controllers for TI SOCs
>> from txt to YAML schema.
>>
>> Adds additional properties 'clocks', 'clock-names', 'power-domains'
>> and '#sound-dai-cells' which were not there in txt file.
>> Adds 'dmas' and 'dma-names' in the example which were not there in
>> txt file.
>> Changes 'interrupts' and 'interrupt-names' from optional to
>> required properties.
>>
>> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
>> ---
>> .../bindings/sound/davinci-mcasp-audio.txt | 86 ----------
>> .../bindings/sound/davinci-mcasp-audio.yaml | 161 ++++++++++++++++++
>> 2 files changed, 161 insertions(+), 86 deletions(-)
>> delete mode 100644 Documentation/devicetree/bindings/sound/davinci-mcasp-audio.txt
>> create mode 100644 Documentation/devicetree/bindings/sound/davinci-mcasp-audio.yaml
>>
>
> 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/patch/1557927
>
>
> mcasp@0: clock-names: Additional items are not allowed ('ahclkx', 'ahclkr' were unexpected)
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clock-names: Additional items are not allowed ('ahclkx' was unexpected)
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clock-names: ['fck', 'ahclkx', 'ahclkr'] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clock-names: ['fck', 'ahclkx'] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[155, 0, 0], [155, 0, 24], [155, 0, 28]] is too long
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
>
> mcasp@0: clocks: [[156, 0, 0], [156, 0, 24], [156, 0, 28]] is too long
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/dra71-evm.dt.yaml
>
> mcasp@0: clocks: [[157, 0, 0], [157, 0, 24], [157, 0, 28]] is too long
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra76-evm.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[158, 0, 0], [158, 0, 24], [158, 0, 28]] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
>
> mcasp@0: clocks: [[91, 340, 0], [156, 0, 24], [91, 340, 28]] is too long
> arch/arm/boot/dts/dra71-evm.dt.yaml
>
> mcasp@0: clocks: [[91, 348, 0], [91, 348, 24]] is too long
> arch/arm/boot/dts/dra71-evm.dt.yaml
>
> mcasp@0: clocks: [[91, 364, 0], [91, 364, 24]] is too long
> arch/arm/boot/dts/dra71-evm.dt.yaml
>
> mcasp@0: clocks: [[91, 388, 0], [91, 388, 24]] is too long
> arch/arm/boot/dts/dra71-evm.dt.yaml
>
> mcasp@0: clocks: [[91, 396, 0], [91, 396, 24]] is too long
> arch/arm/boot/dts/dra71-evm.dt.yaml
>
> mcasp@0: clocks: [[91, 504, 0], [91, 504, 24]] is too long
> arch/arm/boot/dts/dra71-evm.dt.yaml
>
> mcasp@0: clocks: [[91, 508, 0], [91, 508, 24]] is too long
> arch/arm/boot/dts/dra71-evm.dt.yaml
>
> mcasp@0: clocks: [[92, 340, 0], [155, 0, 24], [92, 340, 28]] is too long
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
>
> mcasp@0: clocks: [[92, 340, 0], [156, 0, 24], [92, 340, 28]] is too long
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
>
> mcasp@0: clocks: [[92, 340, 0], [157, 0, 24], [92, 340, 28]] is too long
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[92, 340, 0], [158, 0, 24], [92, 340, 28]] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
>
> mcasp@0: clocks: [[92, 348, 0], [92, 348, 24]] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[92, 364, 0], [92, 364, 24]] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[92, 388, 0], [92, 388, 24]] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[92, 396, 0], [92, 396, 24]] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[92, 504, 0], [92, 504, 24]] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[92, 508, 0], [92, 508, 24]] is too long
> arch/arm/boot/dts/am571x-idk.dt.yaml
> arch/arm/boot/dts/am5729-beagleboneai.dt.yaml
> arch/arm/boot/dts/am572x-idk.dt.yaml
> arch/arm/boot/dts/am574x-idk.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revb1.dt.yaml
> arch/arm/boot/dts/am57xx-beagle-x15-revc.dt.yaml
> arch/arm/boot/dts/am57xx-cl-som-am57x.dt.yaml
> arch/arm/boot/dts/am57xx-sbc-am57x.dt.yaml
> arch/arm/boot/dts/dra72-evm.dt.yaml
> arch/arm/boot/dts/dra72-evm-revc.dt.yaml
> arch/arm/boot/dts/dra7-evm.dt.yaml
>
> mcasp@0: clocks: [[94, 340, 0], [157, 0, 24], [94, 340, 28]] is too long
> arch/arm/boot/dts/dra76-evm.dt.yaml
>
> mcasp@0: clocks: [[94, 348, 0], [94, 348, 24]] is too long
> arch/arm/boot/dts/dra76-evm.dt.yaml
>
> mcasp@0: clocks: [[94, 364, 0], [94, 364, 24]] is too long
> arch/arm/boot/dts/dra76-evm.dt.yaml
>
> mcasp@0: clocks: [[94, 388, 0], [94, 388, 24]] is too long
> arch/arm/boot/dts/dra76-evm.dt.yaml
>
> mcasp@0: clocks: [[94, 396, 0], [94, 396, 24]] is too long
> arch/arm/boot/dts/dra76-evm.dt.yaml
>
> mcasp@0: clocks: [[94, 504, 0], [94, 504, 24]] is too long
> arch/arm/boot/dts/dra76-evm.dt.yaml
>
> mcasp@0: clocks: [[94, 508, 0], [94, 508, 24]] is too long
> arch/arm/boot/dts/dra76-evm.dt.yaml
>
> mcasp@0: dma-names: ['tx'] is too short
> arch/arm/boot/dts/omap4-droid4-xt894.dt.yaml
> arch/arm/boot/dts/omap4-droid-bionic-xt875.dt.yaml
> arch/arm/boot/dts/omap4-duovero-parlor.dt.yaml
> arch/arm/boot/dts/omap4-kc1.dt.yaml
> arch/arm/boot/dts/omap4-panda-a4.dt.yaml
> arch/arm/boot/dts/omap4-panda.dt.yaml
> arch/arm/boot/dts/omap4-panda-es.dt.yaml
> arch/arm/boot/dts/omap4-sdp.dt.yaml
> arch/arm/boot/dts/omap4-sdp-es23plus.dt.yaml
> arch/arm/boot/dts/omap4-var-dvk-om44.dt.yaml
> arch/arm/boot/dts/omap4-var-stk-om44.dt.yaml
>
> mcasp@0: dmas: [[119, 8]] is too short
> arch/arm/boot/dts/omap4-droid4-xt894.dt.yaml
> arch/arm/boot/dts/omap4-droid-bionic-xt875.dt.yaml
> arch/arm/boot/dts/omap4-kc1.dt.yaml
>
> mcasp@0: dmas: [[124, 8]] is too short
> arch/arm/boot/dts/omap4-duovero-parlor.dt.yaml
>
> mcasp@0: dmas: [[127, 8]] is too short
> arch/arm/boot/dts/omap4-panda-a4.dt.yaml
> arch/arm/boot/dts/omap4-panda.dt.yaml
> arch/arm/boot/dts/omap4-sdp.dt.yaml
> arch/arm/boot/dts/omap4-sdp-es23plus.dt.yaml
>
> mcasp@0: dmas: [[130, 8]] is too short
> arch/arm/boot/dts/omap4-var-dvk-om44.dt.yaml
> arch/arm/boot/dts/omap4-var-stk-om44.dt.yaml
>
> mcasp@0: dmas: [[132, 8]] is too short
> arch/arm/boot/dts/omap4-panda-es.dt.yaml
>
> mcasp@0: interrupt-names: ['tx'] is too short
> arch/arm/boot/dts/omap4-droid4-xt894.dt.yaml
> arch/arm/boot/dts/omap4-droid-bionic-xt875.dt.yaml
> arch/arm/boot/dts/omap4-duovero-parlor.dt.yaml
> arch/arm/boot/dts/omap4-kc1.dt.yaml
> arch/arm/boot/dts/omap4-panda-a4.dt.yaml
> arch/arm/boot/dts/omap4-panda.dt.yaml
> arch/arm/boot/dts/omap4-panda-es.dt.yaml
> arch/arm/boot/dts/omap4-sdp.dt.yaml
> arch/arm/boot/dts/omap4-sdp-es23plus.dt.yaml
> arch/arm/boot/dts/omap4-var-dvk-om44.dt.yaml
> arch/arm/boot/dts/omap4-var-stk-om44.dt.yaml
>
> mcasp@0: interrupts: [[0, 109, 4]] is too short
> arch/arm/boot/dts/omap4-droid4-xt894.dt.yaml
> arch/arm/boot/dts/omap4-droid-bionic-xt875.dt.yaml
> arch/arm/boot/dts/omap4-duovero-parlor.dt.yaml
> arch/arm/boot/dts/omap4-kc1.dt.yaml
> arch/arm/boot/dts/omap4-panda-a4.dt.yaml
> arch/arm/boot/dts/omap4-panda.dt.yaml
> arch/arm/boot/dts/omap4-panda-es.dt.yaml
> arch/arm/boot/dts/omap4-sdp.dt.yaml
> arch/arm/boot/dts/omap4-sdp-es23plus.dt.yaml
> arch/arm/boot/dts/omap4-var-dvk-om44.dt.yaml
> arch/arm/boot/dts/omap4-var-stk-om44.dt.yaml
>
> mcasp@0: 'num-serializer' does not match any of the regexes: 'pinctrl-[0-9]+'
> arch/arm/boot/dts/am335x-cm-t335.dt.yaml
> arch/arm/boot/dts/am335x-sbc-t335.dt.yaml
>
> mcasp@0: 'port' does not match any of the regexes: 'pinctrl-[0-9]+'
> arch/arm/boot/dts/am335x-sl50.dt.yaml
>
> mcasp@0: 'rt-num-evt' does not match any of the regexes: 'pinctrl-[0-9]+'
> arch/arm/boot/dts/am335x-wega-rdk.dt.yaml
>
This particular error looks like a typo error in the dts file. I will
send a seperate patch to fix it.
> mcasp@100000: interrupt-names:0: 'tx' was expected
> arch/arm/boot/dts/da850-enbw-cmc.dt.yaml
> arch/arm/boot/dts/da850-evm.dt.yaml
> arch/arm/boot/dts/da850-lcdk.dt.yaml
> arch/arm/boot/dts/da850-lego-ev3.dt.yaml
>
> mcasp@100000: interrupt-names: ['common'] is too short
> arch/arm/boot/dts/da850-enbw-cmc.dt.yaml
> arch/arm/boot/dts/da850-evm.dt.yaml
> arch/arm/boot/dts/da850-lcdk.dt.yaml
> arch/arm/boot/dts/da850-lego-ev3.dt.yaml
>
> mcasp@100000: interrupts: [[54]] is too short
> arch/arm/boot/dts/da850-enbw-cmc.dt.yaml
> arch/arm/boot/dts/da850-evm.dt.yaml
> arch/arm/boot/dts/da850-lcdk.dt.yaml
> arch/arm/boot/dts/da850-lego-ev3.dt.yaml
> v2 patch will take care of rest of the arm SoC errors as well.
> mcasp@2b00000: 'op-mode' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
> mcasp@2b00000: 'serial-dir' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
> mcasp@2b00000: 'tdm-slots' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
> mcasp@2b10000: 'op-mode' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
> mcasp@2b10000: 'serial-dir' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
> mcasp@2b10000: 'tdm-slots' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
> mcasp@2b20000: 'op-mode' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
> mcasp@2b20000: 'serial-dir' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
> mcasp@2b20000: 'tdm-slots' is a required property
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6528-iot2050-basic-pg2.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced.dt.yaml
> arch/arm64/boot/dts/ti/k3-am6548-iot2050-advanced-pg2.dt.yaml
>
For these arm64 SoC errors, a patch has already been sent that disables
the nodes in the dtsi file since some required properties are added via
overlay.
Patch link:
https://lore.kernel.org/all/20211117053806.10095-1-j-choudhary@ti.com/
Warm Regards,
Jayesh
next prev parent reply other threads:[~2021-11-25 10:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-22 9:15 [PATCH] ASoC: dt-bindings: davinci-mcasp: convert McASP bindings to yaml schema Jayesh Choudhary
2021-11-23 16:34 ` Rob Herring
2021-11-25 10:07 ` Jayesh Choudhary [this message]
2021-11-25 11:47 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d72bac88-8f56-35cd-a953-270a1ed4d34b@ti.com \
--to=j-choudhary@ti.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=kishon@ti.com \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox