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@mediatek.com
.../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-nau8825model: $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