* [PATCH] ASoC: dt-bindings: add missing pins
@ 2021-12-11 21:51 David Heidelberg
2021-12-13 16:34 ` Mark Brown
0 siblings, 1 reply; 2+ messages in thread
From: David Heidelberg @ 2021-12-11 21:51 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown, Rob Herring, Oder Chiou
Cc: devicetree, alsa-devel, Mark Brown, linux-kernel,
David Heidelberg, ~okias/devicetree
Add pins missing in documentation, but present in the hardware.
Fixes: 0e826e867264 ("ASoC: add RT5677 CODEC driver")
Signed-off-by: David Heidelberg <david@ixit.cz>
---
Documentation/devicetree/bindings/sound/rt5677.txt | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/Documentation/devicetree/bindings/sound/rt5677.txt b/Documentation/devicetree/bindings/sound/rt5677.txt
index da2430099181..74b6359b802d 100644
--- a/Documentation/devicetree/bindings/sound/rt5677.txt
+++ b/Documentation/devicetree/bindings/sound/rt5677.txt
@@ -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
* LOUT1
* LOUT2
* LOUT3
+ * PDM1L
+ * PDM1R
+ * PDM2L
+ * PDM2R
Example:
--
2.33.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ASoC: dt-bindings: add missing pins
2021-12-11 21:51 [PATCH] ASoC: dt-bindings: add missing pins David Heidelberg
@ 2021-12-13 16:34 ` Mark Brown
0 siblings, 0 replies; 2+ messages in thread
From: Mark Brown @ 2021-12-13 16:34 UTC (permalink / raw)
To: David Heidelberg
Cc: Oder Chiou, devicetree, alsa-devel, Liam Girdwood, linux-kernel,
Rob Herring, ~okias/devicetree
[-- Attachment #1: Type: text/plain, Size: 1147 bytes --]
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.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-12-13 16:35 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-11 21:51 [PATCH] ASoC: dt-bindings: add missing pins David Heidelberg
2021-12-13 16:34 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).