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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <3562cdd32d6f723a35ed063b65560a8e25801255.camel@mediatek.com>
Date: Mon, 23 Jun 2025 06:52:49 +0000
From: Darren Ye (叶飞) <Darren.Ye@...iatek.com>
To: "krzk@...nel.org" <krzk@...nel.org>, "linus.walleij@...aro.org"
	<linus.walleij@...aro.org>, "broonie@...nel.org" <broonie@...nel.org>,
	"brgl@...ev.pl" <brgl@...ev.pl>, "conor+dt@...nel.org" <conor+dt@...nel.org>,
	"tiwai@...e.com" <tiwai@...e.com>, "robh@...nel.org" <robh@...nel.org>,
	"lgirdwood@...il.com" <lgirdwood@...il.com>, "matthias.bgg@...il.com"
	<matthias.bgg@...il.com>, "krzk+dt@...nel.org" <krzk+dt@...nel.org>,
	"perex@...ex.cz" <perex@...ex.cz>, AngeloGioacchino Del Regno
	<angelogioacchino.delregno@...labora.com>
CC: "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "linux-mediatek@...ts.infradead.org"
	<linux-mediatek@...ts.infradead.org>, "linux-sound@...r.kernel.org"
	<linux-sound@...r.kernel.org>, "linux-gpio@...r.kernel.org"
	<linux-gpio@...r.kernel.org>, "krzysztof.kozlowski@...aro.org"
	<krzysztof.kozlowski@...aro.org>, "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>
Subject: Re: [PATCH v5 08/10] ASoC: dt-bindings: mediatek,mt8196-afe: add
 support for MT8196 audio AFE controller

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@...iatek.com>
> > 
> > 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@...iatek.com>
> > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> 
> 
> 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