Re: [PATCH 6/7] ASoC: dt-bindings: mediatek,mt79xx-wm8960: add mt79xx-wm8960 document

[Maso Huang (黃加竹)]

On Wed, 2023-06-14 at 10:22 +0200, Krzysztof Kozlowski wrote:
>  	 
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>  On 14/06/2023 09:40, Maso Huang (黃加竹) wrote:
> >>> Signed-off-by: Maso Huang <maso.huang@xxxxxxxxxxxx>
> >>> ---
> >>>  .../sound/mediatek,mt79xx-wm8960.yaml         | 53
> >> +++++++++++++++++++
> >>>  1 file changed, 53 insertions(+)
> >>>  create mode 100644
> >> Documentation/devicetree/bindings/sound/mediatek,mt79xx-
> wm8960.yaml
> >>>
> >>> diff --git
> >> a/Documentation/devicetree/bindings/sound/mediatek,mt79xx-
> wm8960.yaml 
> >> b/Documentation/devicetree/bindings/sound/mediatek,mt79xx-
> wm8960.yaml
> >>> new file mode 100644
> >>> index 000000000000..26b38bb629da
> >>> --- /dev/null
> >>> +++ b/Documentation/devicetree/bindings/sound/mediatek,mt79xx-
> >> wm8960.yaml
> >>> @@ -0,0 +1,53 @@
> >>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> >>> +%YAML 1.2
> >>> +---
> >>> +$id: 
> >> http://devicetree.org/schemas/sound/mediatek,mt79xx-wm8960.yaml#
> >>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> >>> +
> >>> +title: MediaTek MT79xx ASOC sound card with WM8960 codec
> >>
> >> What is a MT79xx ASOC? Is it some specific SoC name? What does "A"
> >> stands for in SoC? XX also looks odd, I thought Mediatek uses only
> >> numbers.
> >>
> > 
> > I'll use mt7986 instead of mt79xx in v2 patch.
> > And ASoC means ALSA SoC.
> 
> ALSA is Linux stuff, so does not belong to bindings. I have no clue
> what
> is "ALSA SoC" (as SoC means System on Chip).
> 
> Please describe real hardware not Linux or your configuration or some
> SW.
> 

Got it, I'll remove it in v2 patch.

> 
> Best regards,
> Krzysztof
> 

Best regards,
Maso