On Wed, 2025-05-14 at 12:54 +0200, Krzysztof Kozlowski wrote:

External email : Please do not click links or open attachments until
you have verified the sender or the content.


On 14/05/2025 10:11, Darren.Ye wrote:

quoted

From: Darren Ye <redacted>

Add mt8196 audio AFE document.

A nit, subject: drop second/last, redundant "document".
See also:

https://urldefense.com/v3/__https://elixir.bootlin.com/linux/v6.7-rc8/source/Documentation/devicetree/bindings/submitting-patches.rst*L18__;Iw!!CTRNKA9wMg0ARbw!iSQl6_WMX8Q6wsBj4pVe7Tdnbd2zpPwSmHMSm4hlhkD0lO7h5JSAr28-dUj_DwLrXhU_IByy_QWQGQ$

quoted

Signed-off-by: Darren Ye <redacted>

...

quoted

+
+required:
+  - compatible
+  - reg
+  - interrupts
+  - mediatek,vlpcksys
+  - power-domains
+  - memory-region
+  - clocks
+  - clock-names

Keep the same order as in properties:.

ok, thanks.

Best regards,
Darren

Reviewed-by: Krzysztof Kozlowski <redacted>

<form letter>
This is an automated instruction, just in case, because many review
tags
are being ignored. If you know the process, you can skip it (please
do
not feel offended by me posting it here - no bad intentions
intended).
If you do not know the process, here is a short explanation:

Please add Acked-by/Reviewed-by/Tested-by tags when posting new
versions
of patchset, under or above your Signed-off-by tag, unless patch
changed
significantly (e.g. new properties added to the DT bindings). Tag is
"received", when provided in a message replied to you on the mailing
list. Tools like b4 can help here. However, there's no need to repost
patches *only* to add the tags. The upstream maintainer will do that
for
tags received on the version they apply.

Full context and explanation:

https://urldefense.com/v3/__https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst*L577__;Iw!!CTRNKA9wMg0ARbw!iSQl6_WMX8Q6wsBj4pVe7Tdnbd2zpPwSmHMSm4hlhkD0lO7h5JSAr28-dUj_DwLrXhU_IBzvkGpWKg$

</form letter>


Best regards,
Krzysztof