linux-kernel - Re: [PATCH v2 6/6] ASoC: dt-binding: Convert MediaTek mt8183-mt6358 bindings to YAML

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] [day] [month] [year] [list]

Message-ID: <57d67b983f527e9a202422dcc329622cceac6b89.camel@collabora.com>
Date: Mon, 25 Aug 2025 18:05:27 +0200
From: Julien Massot <julien.massot@...labora.com>
To: Rob Herring <robh@...nel.org>
Cc: kernel@...labora.com, Michael Turquette <mturquette@...libre.com>, 
 Stephen Boyd <sboyd@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>,  Matthias Brugger
 <matthias.bgg@...il.com>, AngeloGioacchino Del Regno
 <angelogioacchino.delregno@...labora.com>,  Ikjoon Jang
 <ikjn@...omium.org>, Enric Balletbo i Serra <eballetbo@...nel.org>, Chen-Yu
 Tsai	 <wenst@...omium.org>, Weiyi Lu <weiyi.lu@...iatek.com>, Eugen Hristev
	 <eugen.hristev@...aro.org>, Liam Girdwood <lgirdwood@...il.com>, Mark
 Brown	 <broonie@...nel.org>, Julien Massot <jmassot@...labora.com>, Sean
 Wang	 <sean.wang@...nel.org>, Linus Walleij <linus.walleij@...aro.org>, 
	linux-clk@...r.kernel.org, devicetree@...r.kernel.org, 
	linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, 
	linux-mediatek@...ts.infradead.org, linux-sound@...r.kernel.org, 
	linux-gpio@...r.kernel.org
Subject: Re: [PATCH v2 6/6] ASoC: dt-binding: Convert MediaTek mt8183-mt6358
 bindings to YAML

On Fri, 2025-08-22 at 09:46 -0500, Rob Herring wrote:
> On Wed, Aug 20, 2025 at 03:44:57PM +0200, Julien Massot wrote:
> > Convert the existing text-based DT binding for MT8183 sound cards
> > using
> > MT6358 and various other codecs to a YAML schema.
> 
> In the subject (on all the patches), avoid saying 'binding' twice and I 
> prefer 'DT schema' over YAML. Lot's of things are YAML, but only one 
> thing is DT schema.

Ok, I fixed all the commit messages and replaced 'YAML' by 'DT schema'.

> 
> > 
> > Reviewed-by: AngeloGioacchino Del Regno
> > <angelogioacchino.delregno@...labora.com>
> > Signed-off-by: Julien Massot <julien.massot@...labora.com>
> > ---
> >  .../sound/mediatek,mt8183_mt6358_ts3a227.yaml      | 59
> > ++++++++++++++++++++++
> >  .../sound/mt8183-mt6358-ts3a227-max98357.txt       | 25 ---------
> >  2 files changed, 59 insertions(+), 25 deletions(-)
> > 
> > diff --git
> > a/Documentation/devicetree/bindings/sound/mediatek,mt8183_mt6358_ts3a2
> > 27.yaml
> > b/Documentation/devicetree/bindings/sound/mediatek,mt8183_mt6358_ts3a2
> > 27.yaml
> > new file mode 100644
> > index
> > 0000000000000000000000000000000000000000..048fe62715d67d44daa08e75a63c
> > 782238815689
> > --- /dev/null
> > +++
> > b/Documentation/devicetree/bindings/sound/mediatek,mt8183_mt6358_ts3a2
> > 27.yaml
> > @@ -0,0 +1,59 @@
> > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id:
> > http://devicetree.org/schemas/sound/mediatek,mt8183_mt6358_ts3a227.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: MediaTek MT8183 sound card with MT6358, TS3A227, and
> > MAX98357/RT1015 codecs
> > +
> > +maintainers:
> > +  - Julien Massot <julien.massot@...labora.com>
> > +
> > +description:
> > +  Binding for MediaTek MT8183 SoC-based sound cards using the MT6358
> > codec,
> 
> Drop 'Binding for '.
> 
> Sure
> Reviewed-by: Rob Herring (Arm) <robh@...nel.org>
> 
> And thank you for jumping on all these Mediatek bindings.
> 
> Rob
No problem, Thanks for your time.

Regards,
Julien