On 17/05/2023 13:15, Trevor Wu wrote: > Add compatible string "mediatek,mt8188-nau8825" to support new board > with nau8825 codec. Subject: everything can be update of properties. Rephrase to focus on actual change, e.g. on adding NAU8825. > > ADDA_BE is used to connect to mt6359. For the machine, it must be > fixed to the same codec and configured on the machine driver. Remove > ADDA_BE from items of link-name. I don't understand the justification for this. If this is not a correct link name for MT6359 setup, then it should be removed in its own patch. If this is removed because of adding NAU8825, then why does it affect MT6359? > > Introduce two properties "dai-format" and "mediatek,clk-provider" under > dai-link subnode to configure dai-link parameters via dts. > > "codec" property is removed from required property of dai-link subnode. > For co-clock case, it's possible two dai-links should be configured to > the same format, but only one dai-link builds link with codec. > > Signed-off-by: Trevor Wu <trevor.wu@xxxxxxxxxxxx> > --- > .../sound/mediatek,mt8188-mt6359.yaml | 27 ++++++++++++++++--- > 1 file changed, 23 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/sound/mediatek,mt8188-mt6359.yaml b/Documentation/devicetree/bindings/sound/mediatek,mt8188-mt6359.yaml > index 6640272b3f4f..05e532b5d50a 100644 > --- a/Documentation/devicetree/bindings/sound/mediatek,mt8188-mt6359.yaml > +++ b/Documentation/devicetree/bindings/sound/mediatek,mt8188-mt6359.yaml > @@ -11,7 +11,9 @@ maintainers: > > properties: > compatible: > - const: mediatek,mt8188-mt6359-evb > + enum: > + - mediatek,mt8188-mt6359-evb > + - mediatek,mt8188-nau8825 > > model: > $ref: /schemas/types.yaml#/definitions/string > @@ -42,7 +44,6 @@ patternProperties: > we are going to update parameters in this node. > items: > enum: > - - ADDA_BE Best regards, Krzysztof