From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
To: Rob Herring <robh@kernel.org>
Cc: broonie@kernel.org, linus.walleij@linaro.org,
lee.jones@linaro.org, vinod.koul@linaro.org,
alsa-devel@alsa-project.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, spapothi@codeaurora.org,
bgoswami@codeaurora.org, linux-gpio@vger.kernel.org
Subject: Re: [PATCH v2 01/11] ASoC: dt-bindings: add dt bindings for WCD9340/WCD9341 audio codec
Date: Mon, 28 Oct 2019 12:40:42 +0000 [thread overview]
Message-ID: <90b2d83b-f2b2-3a5d-4deb-589f4b48b208@linaro.org> (raw)
In-Reply-To: <20191025204338.GA25892@bogus>
Thanks Rob for the review.
On 25/10/2019 21:43, Rob Herring wrote:
> On Fri, Oct 18, 2019 at 01:18:39AM +0100, Srinivas Kandagatla wrote:
>> This patch adds bindings for wcd9340/wcd9341 audio codec which can
>> support both SLIMbus and I2S/I2C interface.
>>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> ---
>> .../bindings/sound/qcom,wcd934x.yaml | 169 ++++++++++++++++++
>> 1 file changed, 169 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml
>>
>> diff --git a/Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml b/Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml
>> new file mode 100644
>> index 000000000000..299d6b96c339
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/sound/qcom,wcd934x.yaml
>> @@ -0,0 +1,169 @@
>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/sound/qcom,wcd934x.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Bindings for Qualcomm WCD9340/WCD9341 Audio Codec
>> +
>> +maintainers:
>> + - Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> +
>> +description: |
>> + Qualcomm WCD9340/WCD9341 Codec is a standalone Hi-Fi audio codec IC.
>> + It has in-built Soundwire controller, pin controller, interrupt mux and
>> + supports both I2S/I2C and SLIMbus audio interfaces.
>> +
>> +properties:
>> + compatible:
>> + const: slim217,250
>> +
>> + reg:
>> + maxItems: 1
>> +
>> + interrupts:
>> + maxItems: 1
>> +
>> + reset-gpios:
>> + description: GPIO spec for reset line to use
>> + maxItems: 1
>> +
>> + slim-ifc-dev:
>> + description: SLIMBus Interface device phandle
>
> phandle or...
>
Its Phandle.
something like this is okay?
slim-ifc-dev:
$ref: '/schemas/types.yaml#/definitions/phandle-array'
description: SLIMBus Interface device phandle
>> + maxItems: 1
>
> array?
>
> Needs a type if a phandle.
>
>> +
>> + clocks:
>> + maxItems: 1
>> +
>> + clock-names:
>> + const: extclk
>> +
>> + vdd-buck-supply:
>> + description: A reference to the 1.8V buck supply
>> +
>> + vdd-buck-sido-supply:
>> + description: A reference to the 1.8V SIDO buck supply
>> +
>> + vdd-rx-supply:
>> + description: A reference to the 1.8V rx supply
>> +
>> + vdd-tx-supply:
>> + description: A reference to the 1.8V tx supply
>> +
>> + vdd-vbat-supply:
>> + description: A reference to the vbat supply
>> +
>> + vdd-io-supply:
>> + description: A reference to the 1.8V I/O supply
>> +
>> + vdd-micbias-supply:
>> + description: A reference to the micbias supply
>> +
>> + qcom,micbias1-millivolt:
>
> The standard unit is '-microvolt'
>
>> + description: Voltage betwee 1800mv-2850mv for micbias1 output
>
> typo...
>
Yep, will fix this and also move to standard unit microvolt
> Sounds like constraints.
>
>> + allOf:
>> + - $ref: /schemas/types.yaml#/definitions/uint32
>
> With standard units, you can drop the type.
>
>> +
>> + qcom,micbias2-millivolt:
>> + description: Voltage betwee 1800mv-2850mv for micbias2 output
>> + allOf:
>> + - $ref: /schemas/types.yaml#/definitions/uint32
>> +
>> + qcom,micbias3-millivolt:
>> + description: Voltage betwee 1800mv-2850mv for micbias3 output
>> + allOf:
>> + - $ref: /schemas/types.yaml#/definitions/uint32
>> +
>> + qcom,micbias4-millivolt:
>> + description: Voltage betwee 1800mv-2850mv for micbias4 output
>> + allOf:
>> + - $ref: /schemas/types.yaml#/definitions/uint32
>> +
>> + clock-output-names:
>> + const: mclk
>> +
>> + clock-frequency:
>> + description: Clock frequency of output clk in Hz
>> +
>> + interrupt-controller: true
>> +
>> + '#interrupt-cells':
>> + const: 1
>> +
>> + '#clock-cells':
>> + const: 0
>> +
>> + '#sound-dai-cells':
>> + const: 1
>> +
>> + "#address-cells":
>> + const: 1
>> +
>> + "#size-cells":
>> + const: 1
>> +
>> +patternProperties:
>> + "^.*@[0-9a-f]+$":
>> + type: object
>> + description: |
>> + WCD934x subnode for each slave devices. Bindings of each subnodes
>> + depends on the specific driver providing the functionality and
>> + documented in there respective bindings.
>
> s/there/their/
will fix this in next version!
thanks,
srini
next prev parent reply other threads:[~2019-10-28 12:40 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-18 0:18 [PATCH v2 00/11] ASoC: Add support to WCD9340/WCD9341 codec Srinivas Kandagatla
2019-10-18 0:18 ` [PATCH v2 01/11] ASoC: dt-bindings: add dt bindings for WCD9340/WCD9341 audio codec Srinivas Kandagatla
2019-10-25 20:43 ` Rob Herring
2019-10-28 12:40 ` Srinivas Kandagatla [this message]
2019-10-28 12:45 ` Srinivas Kandagatla
2019-10-29 20:47 ` Rob Herring
2019-10-30 9:55 ` Srinivas Kandagatla
2019-11-05 19:08 ` Rob Herring
2019-11-06 18:09 ` Srinivas Kandagatla
2019-11-06 19:20 ` Rob Herring
2019-10-18 0:18 ` [PATCH v2 02/11] mfd: wcd934x: add support to wcd9340/wcd9341 codec Srinivas Kandagatla
2019-10-21 10:46 ` Lee Jones
2019-10-21 11:25 ` Srinivas Kandagatla
2019-10-21 11:45 ` Lee Jones
2019-10-21 12:14 ` Srinivas Kandagatla
2019-10-21 13:19 ` Lee Jones
2019-10-18 0:18 ` [PATCH v2 03/11] ASoC: " Srinivas Kandagatla
2019-10-18 0:18 ` [PATCH v2 04/11] ASoC: wcd934x: add basic controls Srinivas Kandagatla
2019-10-18 0:18 ` [PATCH v2 05/11] ASoC: wcd934x: add playback dapm widgets Srinivas Kandagatla
2019-10-20 20:05 ` Cezary Rojewski
2019-10-21 9:46 ` Srinivas Kandagatla
2019-10-18 0:18 ` [PATCH v2 06/11] ASoC: wcd934x: add capture " Srinivas Kandagatla
2019-10-18 0:18 ` [PATCH v2 07/11] ASoC: wcd934x: add audio routings Srinivas Kandagatla
2019-10-18 0:18 ` [PATCH v2 08/11] dt-bindings: pinctrl: qcom-wcd934x: Add bindings for gpio Srinivas Kandagatla
2019-10-25 21:00 ` Rob Herring
2019-10-28 12:41 ` Srinivas Kandagatla
2019-10-18 0:18 ` [PATCH v2 09/11] pinctrl: qcom-wcd934x: Add support to wcd934x pinctrl driver Srinivas Kandagatla
2019-10-18 0:18 ` [PATCH v2 10/11] ASoC: dt-bindings: Add compatible for DB845c and Lenovo Yoga Srinivas Kandagatla
2019-10-25 21:01 ` Rob Herring
2019-10-18 0:18 ` [PATCH v2 11/11] ASoC: qcom: sdm845: add support to " Srinivas Kandagatla
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=90b2d83b-f2b2-3a5d-4deb-589f4b48b208@linaro.org \
--to=srinivas.kandagatla@linaro.org \
--cc=alsa-devel@alsa-project.org \
--cc=bgoswami@codeaurora.org \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=lee.jones@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=spapothi@codeaurora.org \
--cc=vinod.koul@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).