[PATCH 2/2] ASoC: meson: axg-fifo: convert Amlogic FIFO controller to yaml
Rob Herring
robh at kernel.org
Sat Jan 15 09:22:30 PST 2022
On Sat, 15 Jan 2022 10:35:57 +0100, Alexander Stein wrote:
> Convert Amlogic FIFO controller documentation to yaml format.
>
> Signed-off-by: Alexander Stein <alexander.stein at mailbox.org>
> ---
> Things to note:
> First of, Jerome, sorry for adding you as maintainer, but
> 1) it's mandatory
> 2) your are the (only) author of amlogic,axg-fifo.txt
>
> Please add your Signed-off-by if that is okay with you.
>
> License is mandated by checkpath, not my choice.
>
> I'm not so sure about the compatible check. Essentially it is either
> * 'amlogic,axg-frddr' OR
> * 'amlogic,g12a-frddr' + 'amlogic,axg-frddr'
> (or 'sm1' instead of 'g12a')
>
> Same goes for *-toddr. Is this schema correct in that regard? At least I
> got no warnings on existing device trees.
>
> .../bindings/sound/amlogic,axg-fifo.txt | 34 -------
> .../bindings/sound/amlogic,axg-fifo.yaml | 97 +++++++++++++++++++
> 2 files changed, 97 insertions(+), 34 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/sound/amlogic,axg-fifo.txt
> create mode 100644 Documentation/devicetree/bindings/sound/amlogic,axg-fifo.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/1580333
audio-controller at 100: 'reset-names' is a required property
arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml
audio-controller at 140: 'reset-names' is a required property
arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml
audio-controller at 180: 'reset-names' is a required property
arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml
audio-controller at 1c0: 'reset-names' is a required property
arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml
audio-controller at 200: 'reset-names' is a required property
arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml
audio-controller at 240: 'reset-names' is a required property
arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml
More information about the linux-amlogic
mailing list