linux-kernel - Re: [PATCH] ALSA: machine: update documentation

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: <CABCoZhAV0+MK6FEDRpojHoW4opfqNaYCtTrBTjA8O8+yurLr2A@mail.gmail.com>
Date: Wed, 6 Nov 2024 20:31:14 -0800
From: anish kumar <yesanishhere@...il.com>
To: Bagas Sanjaya <bagasdotme@...il.com>
Cc: lgirdwood@...il.com, broonie@...nel.org, perex@...ex.cz, tiwai@...e.com, 
	corbet@....net, linux-kernel@...r.kernel.org, linux-sound@...r.kernel.org, 
	linux-doc@...r.kernel.org
Subject: Re: [PATCH] ALSA: machine: update documentation

Mark,

Do you have any comments, or should I
go ahead and respin the patch to address
the review comments?

On Sat, Nov 2, 2024 at 9:30 PM Bagas Sanjaya <bagasdotme@...il.com> wrote:
>
> On Sat, Nov 02, 2024 at 12:37:18AM -0700, anish kumar wrote:
> > +Additionaly below example macro can be used to register cpu, codec and
> > +platform dai.
> > +
> > +SND_SOC_DAILINK_DEFS(wm2200_cpu_dsp,
> > +     DAILINK_COMP_ARRAY(COMP_CPU("samsung-i2s.0")),
> > +     DAILINK_COMP_ARRAY(COMP_CODEC("spi0.0", "wm0010-sdi1")),
> > +     DAILINK_COMP_ARRAY(COMP_PLATFORM("samsung-i2s.0")));
> > +
>
> Make the macro usage a literal code block:
>
> ---- >8 ----
> diff --git a/Documentation/sound/soc/machine.rst b/Documentation/sound/soc/machine.rst
> index af7bad7ba13455..8788953cd160f5 100644
> --- a/Documentation/sound/soc/machine.rst
> +++ b/Documentation/sound/soc/machine.rst
> @@ -76,9 +76,9 @@ either dai name or device tree node but not both. Also, names used here
>  for cpu/codec/platform dais should be globally unique.
>
>  Additionaly below example macro can be used to register cpu, codec and
> -platform dai.
> +platform dai::
>
> -SND_SOC_DAILINK_DEFS(wm2200_cpu_dsp,
> +  SND_SOC_DAILINK_DEFS(wm2200_cpu_dsp,
>         DAILINK_COMP_ARRAY(COMP_CPU("samsung-i2s.0")),
>         DAILINK_COMP_ARRAY(COMP_CODEC("spi0.0", "wm0010-sdi1")),
>         DAILINK_COMP_ARRAY(COMP_PLATFORM("samsung-i2s.0")));
>
> >  Machine Power Map
> > ------------------
> > +---------------------------
>
> Keep the section underline length as-is.
>
> >  Machine Controls
> > -----------------
> > +-----------------------
>
> Likewise.
>
> > +Clocking Controls:
> > +—-----------------
> > +
>
> Fix the underline to render the section name as a heading:
>
> ---- >8 ----
> diff --git a/Documentation/sound/soc/machine.rst b/Documentation/sound/soc/machine.rst
> index 8788953cd160f5..bdb9dfc5d88e45 100644
> --- a/Documentation/sound/soc/machine.rst
> +++ b/Documentation/sound/soc/machine.rst
> @@ -113,8 +113,8 @@ Machine Controls
>  Machine specific audio mixer controls can be added in the DAI init function.
>
>
> -Clocking Controls:
> -—-----------------
> +Clocking Controls
> +-----------------
>
>  As previously noted, clock configuration is handled within the machine driver.
>  For details on the clock APIs that the machine driver can utilize for
>
> Thanks.
>
> --
> An old man doll... just what I always wanted! - Clara