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@xxxxxxxxxxx> > --- > 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@100: 'reset-names' is a required property arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml audio-controller@140: 'reset-names' is a required property arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml audio-controller@180: 'reset-names' is a required property arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml audio-controller@1c0: 'reset-names' is a required property arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml audio-controller@200: 'reset-names' is a required property arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml audio-controller@240: 'reset-names' is a required property arch/arm64/boot/dts/amlogic/meson-axg-s400.dt.yaml