On Fri 15 Mar 2024 at 02:21, Jan Dakinevich <jan.dakinevich@xxxxxxxxxxxxxxxxx> wrote: > Add "amlogic,a1-tdm-iface" compatible string alias to > "amlogic,axg-tdm-iface". > > Signed-off-by: Jan Dakinevich <jan.dakinevich@xxxxxxxxxxxxxxxxx> > --- > .../devicetree/bindings/sound/amlogic,axg-tdm-iface.yaml | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/sound/amlogic,axg-tdm-iface.yaml b/Documentation/devicetree/bindings/sound/amlogic,axg-tdm-iface.yaml > index 45955d8a26d1..7c1af85b52b4 100644 > --- a/Documentation/devicetree/bindings/sound/amlogic,axg-tdm-iface.yaml > +++ b/Documentation/devicetree/bindings/sound/amlogic,axg-tdm-iface.yaml > @@ -14,7 +14,11 @@ allOf: > > properties: > compatible: > - const: amlogic,axg-tdm-iface > + oneOf: > + - const: amlogic,axg-tdm-iface > + - items: > + - const: amlogic,a1-tdm-iface > + - const: amlogic,axg-tdm-iface Same as the card driver. I could have named it "amlogic,tdm-iface" This is purely a SW component, which help agregate clocks and tdm-formatters. It is analog to a "gpio-leds" or a "pwm-clock" driver. We would add a compatible for every SoC for these, would we ? I don't think it makes a lot of sense to add this. It is not going to hurt but this is just adding useless compatible to the doc that will never be used > > "#sound-dai-cells": > const: 0 -- Jerome
