lkml.org 
[lkml]   [2022]   [Jan]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v5] ASoC: dt-bindings: davinci-mcasp: convert McASP bindings to yaml schema
From
Date


On 03/01/22 7:28 pm, Rob Herring wrote:
> On Mon, 03 Jan 2022 13:14:27 +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',
>> '#sound-dai-cells' and 'port' which were missing from the txt file.
>> Removes properties 'sram-size-playback' and 'sram-size-capture'
>> since they are not used.
>> Adds 'dmas' and 'dma-names' in the example which were missing from
>> the txt file.
>> Changes 'interrupts' and 'interrupt-names' from optional to
>> required properties.
>> Changes 'op-modes', 'serial-dir' to optional properties as they are
>> not needed if the McASP is used only as GPIO.
>> Changes 'tdm-slots' to required property only for I2S operation mode.
>>
>> Adds the yaml file in the 'MAINTAINERS' under the heading 'TEXAS
>> INSTRUMENTS ASoC DRIVERS'
>>
>> Signed-off-by: Jayesh Choudhary <j-choudhary@ti.com>
>> ---
>> .../bindings/sound/davinci-mcasp-audio.txt | 86 --------
>> .../bindings/sound/davinci-mcasp-audio.yaml | 201 ++++++++++++++++++
>> MAINTAINERS | 1 +
>> 3 files changed, 202 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/1574696
>
>
> 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
>

This is an error in the dts file. It has not been updated.
As pointed out by Peter, this property was removed by the commit ID:
1427e660b49e87cd842dba94158b0fc73030c17e

> mcasp@0: 'rt-num-evt' does not match any of the regexes: 'pinctrl-[0-9]+'
> arch/arm/boot/dts/am335x-wega-rdk.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: '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: '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
>

Patches to fix the above errors have already been applied to linux-next
tree.

Links:
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=277ee96f89d806f110e3011ea324155dd69e798f

https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=29a5e8496b3ac0d400dfe32288c26c774beb8cc8

I re-ran the checks with the latest dt_schema. I didn't see these errors.

- Jayesh

\
 
 \ /
  Last update: 2022-01-10 08:22    [W:0.107 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site