lkml.org 
[lkml]   [2021]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [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.
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2021-12-13 17:35    [W:0.201 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site