linux-kernel - Re: [PATCH] ASoC: dt-bindings: add missing pins

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: <Ybd2LMZ8+UqcUZTS@sirena.org.uk>
Date:   Mon, 13 Dec 2021 16:34:52 +0000
From:   Mark Brown <broonie@...nel.org>
To:     David Heidelberg <david@...t.cz>
Cc:     Liam Girdwood <lgirdwood@...il.com>,
        Rob Herring <robh+dt@...nel.org>,
        Oder Chiou <oder_chiou@...ltek.com>,
        ~okias/devicetree@...ts.sr.ht, alsa-devel@...a-project.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ASoC: dt-bindings: add missing pins

On Sat, Dec 11, 2021 at 10:51:19PM +0100, David Heidelberg wrote:

> Add pins missing in documentation, but present in the hardware.

> Fixes: 0e826e867264 ("ASoC: add RT5677 CODEC driver")

Please don't just add nonsense fixes tags for the sake of it, this just
creates noise and gets in the way of people trying to use the tags to
work out if thy need fixes.  The commit you're mentioning doesn't even
contain any changes to the binding document.

> @@ -54,9 +54,21 @@ Pins on the device (for linking into audio routes):
>    * DMIC2
>    * DMIC3
>    * DMIC4
> +  * DMIC L1
> +  * DMIC L2
> +  * DMIC L3
> +  * DMIC L4
> +  * DMIC R1
> +  * DMIC R2
> +  * DMIC R3
> +  * DMIC R4

These clearly aren't pins you're adding, digital microphones use PDM
and carry a stereo pair on a single physical pin (which is what should
be and already is in the bindings).  This is adding an extra set of pins
to the binding duplicating the existing ones.

>    * LOUT1
>    * LOUT2
>    * LOUT3
> +  * PDM1L
> +  * PDM1R
> +  * PDM2L
> +  * PDM2R

I'd expect these to be just PDM1 and PDM2, assuming that that's what the
pins are called in the chip datasheet.

Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)