Re: [PATCH v5 08/10] ASoC: dt-bindings: mediatek,mt8196-afe: add support for MT8196 audio AFE controller

[Darren Ye (叶飞)]

On Sun, 2025-06-22 at 11:43 +0200, Krzysztof Kozlowski wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> 
> 
> On 20/06/2025 11:40, Darren.Ye wrote:
> > From: Darren Ye <darren.ye@xxxxxxxxxxxx>
> > 
> > This patch adds initial support for the audio AFE(Audio Front End)
> > controller
> 
> Why this was changed to undesired 'This patch' (see submitting
> patches)?
> I think you are circling back to previous versions, reintroducing
> issues
> fixed in between.

Thanks for you feedback. Just to confirm, for the next version, should
I revert the subject/description back to previous version and only drop
the document part, whitout making other changes?


Please see the following change:

ASoC: dt-bindings: mediatek,mt8196-afe: add audio AFE

Add mt8196 audio AFE.


Please let me know if this is correct.

Best regards,
Darren

> 
> > on the mediatek MT8196 platform.
> > 
> > Signed-off-by: Darren Ye <darren.ye@xxxxxxxxxxxx>
> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> 
> 
> Subject: I asked to drop document, because the rest was correct and
> you
> totally rewritten it to unnecessary too long subject.
> 
> Well, at least the rest you implemented, but I just don't get why,
> when
> asked to change something you change several things to less desired
> style.
> 
> Best regards,
> Krzysztof