Hi Wolfram, On Wed, Sep 25, 2024 at 11:08 AM Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > Meanwhile, bindings for the audio codec exist. Add dai-cells property > and limit max frequency accordingly: > > codec@0: '#sound-dai-cells' is a required property > codec@0: spi-max-frequency: 5000000 is greater than the maximum of 526000 > > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> i.e. will queue in renesas-devel for v6.13, with s/bindings/device node/ in the one-line summary. > --- a/arch/arm/boot/dts/renesas/r7s72100-genmai.dts > +++ b/arch/arm/boot/dts/renesas/r7s72100-genmai.dts > @@ -248,9 +248,10 @@ &spi4 { > status = "okay"; > > codec: codec@0 { > + #sound-dai-cells = <0>; I'll move this to the end, to match generic property order. > compatible = "wlf,wm8978"; > reg = <0>; > - spi-max-frequency = <5000000>; > + spi-max-frequency = <500000>; > }; > }; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds