[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <af58b6b2-25b2-e968-73c9-d87e1a9e2746@nvidia.com>
Date: Mon, 10 Feb 2020 17:00:56 +0530
From: Sameer Pujar <spujar@...dia.com>
To: Rob Herring <robh@...nel.org>
CC: <spujar@...dia.com>, <perex@...ex.cz>, <tiwai@...e.com>,
<broonie@...nel.org>, <lgirdwood@...il.com>,
<thierry.reding@...il.com>, <jonathanh@...dia.com>,
<digetx@...il.com>, <alsa-devel@...a-project.org>,
<devicetree@...r.kernel.org>, <linux-tegra@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <sharadg@...dia.com>,
<mkumard@...dia.com>, <viswanathl@...dia.com>,
<rlokhande@...dia.com>, <dramesh@...dia.com>,
<atalambedu@...dia.com>
Subject: Re: [PATCH v2 1/9] dt-bindings: sound: tegra: add DT binding for AHUB
On 2/6/2020 11:40 PM, Rob Herring wrote:
> External email: Use caution opening links or attachments
>
>
> On Thu, Jan 30, 2020 at 04:03:34PM +0530, Sameer Pujar wrote:
>> Audio Hub (AHUB) comprises a collection of hardware accelerators for audio
>> pre-processing and post-processing and a programmable full crossbar for
>> audio routing across these accelerators. This patch adds YAML schema for DT
>> binding of AHUB and few of its following components. These devices will be
>> registered as ASoC components.
>> * ADMAIF
>> * I2S
>> * DMIC
>> * DSPK
>>
>> Signed-off-by: Sameer Pujar <spujar@...dia.com>
>> ---
>> .../bindings/sound/nvidia,tegra186-dspk.yaml | 105 +++++++++++++
>> .../bindings/sound/nvidia,tegra210-admaif.yaml | 165 +++++++++++++++++++++
>> .../bindings/sound/nvidia,tegra210-ahub.yaml | 130 ++++++++++++++++
>> .../bindings/sound/nvidia,tegra210-dmic.yaml | 105 +++++++++++++
>> .../bindings/sound/nvidia,tegra210-i2s.yaml | 112 ++++++++++++++
>> 5 files changed, 617 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra186-dspk.yaml
>> create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra210-admaif.yaml
>> create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra210-ahub.yaml
>> create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra210-dmic.yaml
>> create mode 100644 Documentation/devicetree/bindings/sound/nvidia,tegra210-i2s.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/sound/nvidia,tegra186-dspk.yaml b/Documentation/devicetree/bindings/sound/nvidia,tegra186-dspk.yaml
>> new file mode 100644
>> index 0000000..dc9fef3
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/sound/nvidia,tegra186-dspk.yaml
>> @@ -0,0 +1,105 @@
>> +# SPDX-License-Identifier: (GPL-2.0)
. . .
>> + items:
>> + - description: initial module clock rate
>> +
>> + "#sound-dai-cells":
>> + const: 1
>> +
>> + sound-name-prefix:
>> + $ref: /schemas/types.yaml#/definitions/string
>> + description:
>> + Used as prefix for sink/source names of the component. Must be a
>> + unique string among multiple instances of the same component.
>> + The name can be "DSPK1" or "DSPKx", where x depends on the maximum
> Sounds like a constraint.
Is there a better way to convey the recommended strings above?
All the other comments I will fix in next revision. Thanks.
. . .
Powered by blists - more mailing lists