devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v5 0/2] ASoC: codecs: add support for ES8389
@ 2025-03-07  9:19 Zhang Yi
  2025-03-07  9:19 ` [PATCH v5 1/2] " Zhang Yi
  2025-03-07  9:19 ` [PATCH v5 2/2] ASoC: dt-bindings: Add Everest ES8389 audio CODEC Zhang Yi
  0 siblings, 2 replies; 7+ messages in thread
From: Zhang Yi @ 2025-03-07  9:19 UTC (permalink / raw)
  To: alsa-devel, broonie, devicetree
  Cc: tiwai, amadeuszx.slawinski, robh+dt, krzysztof.kozlowski+dt,
	conor+dt, Zhang Yi

The driver is for codec ES8389 of everest-semi.

Zhang Yi (2):
  ASoC: codecs: add support for ES8389
  ASoC: dt-bindings: Add Everest ES8389 audio CODEC

 .../bindings/sound/everest,es8389.yaml        |  50 +
 sound/soc/codecs/Kconfig                      |   6 +-
 sound/soc/codecs/Makefile                     |   2 +
 sound/soc/codecs/es8389.c                     | 966 ++++++++++++++++++
 sound/soc/codecs/es8389.h                     | 140 +++
 5 files changed, 1163 insertions(+), 1 deletion(-)
 create mode 100644 Documentation/devicetree/bindings/sound/everest,es8389.yaml
 create mode 100644 sound/soc/codecs/es8389.c
 create mode 100644 sound/soc/codecs/es8389.h

-- 
2.17.1


^ permalink raw reply	[flat|nested] 7+ messages in thread
* RE: [PATCH v5 2/2] ASoC: dt-bindings: Add Everest ES8389 audio CODEC
@ 2025-03-09  9:52 Zhang Yi
  0 siblings, 0 replies; 7+ messages in thread
From: Zhang Yi @ 2025-03-09  9:52 UTC (permalink / raw)
  To: krzysztof.kozlowski+dt
  Cc: tiwai, amadeuszx.slawinski, robh+dt, conor+dt, alsa-devel,
	broonie, devicetree

> >  1 file changed, 50 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/sound/everest,es8389.yaml
> > 
> > diff --git a/Documentation/devicetree/bindings/sound/everest,es8389.yaml b/Documentation/devicetree/bindings/sound/everest,es8389.yaml
> 
> 
> You got feedback how to create CC list. Two times.
> 
> You still create it incorrectly and Cc address which does not exist in
> the kernel and is a proof you work on wrong tree, which I asked to correct.
> 
> What's more, there is still no changelog and you never responded to
> review comments.
> 
> Can you slow down with resubmits and rethink the received feedback? I
> expect that feedback to be responded to, so I am not wasting time here
> repeating the same every time.

I'm sorry for the trouble I've caused you.
And I have responded to the comments at v3 and v4.

^ permalink raw reply	[flat|nested] 7+ messages in thread
* RE: [PATCH v5 2/2] ASoC: dt-bindings: Add Everest ES8389 audio CODEC
@ 2025-03-10  7:51 Zhang Yi
  0 siblings, 0 replies; 7+ messages in thread
From: Zhang Yi @ 2025-03-10  7:51 UTC (permalink / raw)
  To: krzysztof.kozlowski
  Cc: tiwai, conor+dt, broonie, devicetree, amadeuszx.slawinski

> > Signed-off-by: Zhang Yi <zhangyi@everest-semi.com>
> > ---
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> 
> 
> ---
> 
> <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.

Thank you for the explanation.I will do it in a following patch

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2025-03-10  7:51 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-07  9:19 [PATCH v5 0/2] ASoC: codecs: add support for ES8389 Zhang Yi
2025-03-07  9:19 ` [PATCH v5 1/2] " Zhang Yi
2025-03-07  9:19 ` [PATCH v5 2/2] ASoC: dt-bindings: Add Everest ES8389 audio CODEC Zhang Yi
2025-03-07 13:12   ` Krzysztof Kozlowski
2025-03-10  7:36   ` Krzysztof Kozlowski
  -- strict thread matches above, loose matches on Subject: below --
2025-03-09  9:52 Zhang Yi
2025-03-10  7:51 Zhang Yi

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).