Re: [PATCH v2 1/9] ASoC: dt-bindings: rt5659: Update binding doc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Rob,

On 2/5/2021 8:59 PM, Rob Herring wrote:
External email: Use caution opening links or attachments


On Fri, Jan 29, 2021 at 11:57:38PM +0530, Sameer Pujar wrote:
Update following in rt5659.txt binding doc
   - Add JD source for Intel HDA header: Commit 041e74b71491
     ("ASoC: rt5659: Add the support of Intel HDA Header")
     added driver support. Add missing info here.

   - sound-name-prefix: Used to prefix component widgets/kcontrols
     with given prefix.

   - ports: Helps to use the Codec with audio graph card

Signed-off-by: Sameer Pujar <spujar@xxxxxxxxxx>
Reported-by: Jon Hunter <jonathanh@xxxxxxxxxx>
Cc: Oder Chiou <oder_chiou@xxxxxxxxxxx>
Cc: Bard Liao <bardliao@xxxxxxxxxxx>
---
  Documentation/devicetree/bindings/sound/rt5659.txt | 11 +++++++++++
  1 file changed, 11 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/rt5659.txt b/Documentation/devicetree/bindings/sound/rt5659.txt
index 56788f5..c473df5 100644
--- a/Documentation/devicetree/bindings/sound/rt5659.txt
+++ b/Documentation/devicetree/bindings/sound/rt5659.txt
@@ -37,10 +37,21 @@ Optional properties:
  - realtek,jd-src
    0: No JD is used
    1: using JD3 as JD source
+  2: JD source for Intel HDA header

  - realtek,ldo1-en-gpios : The GPIO that controls the CODEC's LDO1_EN pin.
  - realtek,reset-gpios : The GPIO that controls the CODEC's RESET pin.

+- sound-name-prefix: Please refer to name-prefix.txt
+
+- ports: A Codec may have a single or multiple I2S interfaces. These
+  interfaces on Codec side can be described under 'ports' or 'port'.
+  When the SoC or host device is connected to multiple interfaces of
+  the Codec, the connectivity can be described using 'ports' property.
+  If a single interface is used, then 'port' can be used. The usage
+  depends on the platform or board design.
+  Please refer to Documentation/devicetree/bindings/graph.txt
Please read what that file says now.

Sorry, I did not understand above comment. Can you please elaborate a bit more?




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux