linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzk@kernel.org>
To: "Darren Ye (叶飞)" <Darren.Ye@mediatek.com>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-mediatek@lists.infradead.org"
	<linux-mediatek@lists.infradead.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"linus.walleij@linaro.org" <linus.walleij@linaro.org>,
	"linux-sound@vger.kernel.org" <linux-sound@vger.kernel.org>,
	"linux-gpio@vger.kernel.org" <linux-gpio@vger.kernel.org>,
	"broonie@kernel.org" <broonie@kernel.org>,
	"brgl@bgdev.pl" <brgl@bgdev.pl>,
	"conor+dt@kernel.org" <conor+dt@kernel.org>,
	"tiwai@suse.com" <tiwai@suse.com>,
	"robh@kernel.org" <robh@kernel.org>,
	"lgirdwood@gmail.com" <lgirdwood@gmail.com>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"matthias.bgg@gmail.com" <matthias.bgg@gmail.com>,
	"krzk+dt@kernel.org" <krzk+dt@kernel.org>,
	"perex@perex.cz" <perex@perex.cz>,
	AngeloGioacchino Del Regno
	<angelogioacchino.delregno@collabora.com>
Subject: Re: [PATCH v4 08/10] ASoC: dt-bindings: mediatek,mt8196-afe: add audio AFE document
Date: Wed, 11 Jun 2025 11:26:54 +0200	[thread overview]
Message-ID: <7e197546-4bae-441c-a69f-9ae2566e8600@kernel.org> (raw)
In-Reply-To: <bed13d4bba5923e0d71a5e32e61163c2e7958d68.camel@mediatek.com>

On 11/06/2025 11:09, Darren Ye (叶飞) wrote:
> On Wed, 2025-06-11 at 10:27 +0200, Krzysztof Kozlowski wrote:
>> External email : Please do not click links or open attachments until
>> you have verified the sender or the content.
>>
>>
>> On Tue, Jun 10, 2025 at 05:27:25PM GMT, Darren.Ye wrote:
>>> From: Darren Ye <darren.ye@mediatek.com>
>>>
>>> Add mt8196 audio AFE document.
>>>
>>> Signed-off-by: Darren Ye <darren.ye@mediatek.com>
>>> ---
>>
>>
>> I gave you detailed instruction last time. Did you read it or you
>> decided to just ignore my review?
>>
>> Best regards,
>> Krzysztof
>>
> 
> Hi Krzysztof,
> 
> I would never ignore your suggestions, on the contrary, I really
> appreciate them. I may have missed your advice, and I'm very sorry for
> causing any confusion. However, I haven't noticed it so far. Could you
> please point it out or directly paste what needs to be revised? Thank
> you.
You ignored all comments, so how can I point what was missing? Everything.

If this helps you to understand the comments, then go to previous email,
read each comment and respond to each how did you implement it. If you
cannot come with the answer how you implement it, then you did not
implement it...

I don't know how to be helpful? I even wrote long detailed instruction,
with references, explaining you what to do.

Best regards,
Krzysztof

  reply	other threads:[~2025-06-11  9:27 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-10  9:27 [PATCH v4 00/10] ASoC: mediatek: Add support for MT8196 SoC Darren.Ye
2025-06-10  9:27 ` [PATCH v4 01/10] ASoC: mediatek: common: modify mtk afe platform driver for mt8196 Darren.Ye
2025-06-11 11:12   ` Louis-Alexis Eyraud
2025-06-11 11:14   ` AngeloGioacchino Del Regno
2025-06-10  9:27 ` [PATCH v4 02/10] ASoC: mediatek: mt8196: add common header Darren.Ye
2025-06-10  9:27 ` [PATCH v4 03/10] ASoC: mediatek: mt8196: support audio clock control Darren.Ye
2025-06-11  3:03   ` kernel test robot
2025-06-10  9:27 ` [PATCH v4 04/10] ASoC: mediatek: mt8196: support ADDA in platform driver Darren.Ye
2025-06-10  9:27 ` [PATCH v4 05/10] ASoC: mediatek: mt8196: support I2S " Darren.Ye
2025-06-10  9:27 ` [PATCH v4 06/10] ASoC: mediatek: mt8196: support TDM " Darren.Ye
2025-06-10  9:27 ` [PATCH v4 07/10] ASoC: mediatek: mt8196: add " Darren.Ye
2025-06-10  9:27 ` [PATCH v4 08/10] ASoC: dt-bindings: mediatek,mt8196-afe: add audio AFE document Darren.Ye
2025-06-11  8:27   ` Krzysztof Kozlowski
2025-06-11  9:09     ` Darren Ye (叶飞)
2025-06-11  9:26       ` Krzysztof Kozlowski [this message]
2025-06-10  9:27 ` [PATCH v4 09/10] ASoC: mediatek: mt8196: add machine driver with nau8825 Darren.Ye
2025-06-10  9:27 ` [PATCH v4 10/10] ASoC: dt-bindings: mediatek,mt8196-nau8825: add mt8196-nau8825 document Darren.Ye

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7e197546-4bae-441c-a69f-9ae2566e8600@kernel.org \
    --to=krzk@kernel.org \
    --cc=Darren.Ye@mediatek.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=brgl@bgdev.pl \
    --cc=broonie@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=lgirdwood@gmail.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=matthias.bgg@gmail.com \
    --cc=perex@perex.cz \
    --cc=robh@kernel.org \
    --cc=tiwai@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).