lkml.org 
[lkml]   [2018]   [Jul]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH v2 01/10] ASoC: dt-bindings: add dt bindings for wcd9335 audio codec
    On Fri, Jul 27, 2018 at 01:17:57PM +0100, Srinivas Kandagatla wrote:
    > This patch adds bindings for wcd9335 audio codec which can support both SLIMbus
    > and I2S/I2C interface.
    >
    > Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
    > ---
    > .../devicetree/bindings/sound/qcom,wcd9335.txt | 123 +++++++++++++++++++++
    > 1 file changed, 123 insertions(+)
    > create mode 100644 Documentation/devicetree/bindings/sound/qcom,wcd9335.txt
    >
    > diff --git a/Documentation/devicetree/bindings/sound/qcom,wcd9335.txt b/Documentation/devicetree/bindings/sound/qcom,wcd9335.txt
    > new file mode 100644
    > index 000000000000..1d8d49e30af7
    > --- /dev/null
    > +++ b/Documentation/devicetree/bindings/sound/qcom,wcd9335.txt
    > @@ -0,0 +1,123 @@
    > +QCOM WCD9335 Codec
    > +
    > +Qualcomm WCD9335 Codec is a standalone Hi-Fi audio codec IC, supports
    > +Qualcomm Technologies, Inc. (QTI) multimedia solutions, including
    > +the MSM8996, MSM8976, and MSM8956 chipsets. It has in-built
    > +Soundwire controller, interrupt mux. It supports both I2S/I2C and
    > +SLIMbus audio interfaces.
    > +
    > +Required properties with SLIMbus Interface:
    > +
    > +- compatible:
    > + Usage: required
    > + Value type: <stringlist>
    > + Definition: For SLIMbus interface it should be "slimMID,PID",
    > + textual representation of Manufacturer ID, Product Code,
    > + shall be in lower case hexadecimal with leading zeroes
    > + suppressed. Refer to slimbus/bus.txt for details.
    > + Should be:
    > + "slim217,1a0" for MSM8996 and APQ8096 SoCs with SLIMbus.
    > +
    > +- reg
    > + Usage: required
    > + Value type: <u32 u32>
    > + Definition: Should be ('Device index', 'Instance ID')
    > +
    > +- interrupts
    > + Usage: required
    > + Value type: <prop-encoded-array>
    > + Definition: Interrupts via WCD INTR1 and INTR2 pins
    > +
    > +- interrupt-names:
    > + Usage: required
    > + Value type: <String array>
    > + Definition: Interrupt names of WCD INTR1 and INTR2
    > + Should be: "intr1", "intr2"
    > +
    > +- reset-gpio:

    reset-gpios

    > + Usage: required
    > + Value type: <String Array>
    > + Definition: Reset gpio line
    > +
    > +- qcom,ifd:
    > + Usage: required
    > + Value type: <phandle>
    > + Definition: SLIM interface device

    Wouldn't this be the parent?

    > +
    > +- clocks:
    > + Usage: required
    > + Value type: <prop-encoded-array>
    > + Definition: See clock-bindings.txt section "consumers". List of
    > + three clock specifiers for mclk, mclk2 and slimbus clock.
    > +
    > +- clock-names:
    > + Usage: required
    > + Value type: <string>
    > + Definition: Must contain "mclk", "mclk2" and "slimbus" strings.
    > +
    > +- vdd-buck-supply:
    > + Usage: required
    > + Value type: <phandle>
    > + Definition: Should contain a reference to the 1.8V buck supply
    > +
    > +- vdd-buck-sido-supply:
    > + Usage: required
    > + Value type: <phandle>
    > + Definition: Should contain a reference to the 1.8V SIDO buck supply
    > +
    > +- vdd-rx-supply:
    > + Usage: required
    > + Value type: <phandle>
    > + Definition: Should contain a reference to the 1.8V rx supply
    > +
    > +- vdd-tx-supply:
    > + Usage: required
    > + Value type: <phandle>
    > + Definition: Should contain a reference to the 1.8V tx supply
    > +
    > +- vdd-vbat-supply:
    > + Usage: Optional
    > + Value type: <phandle>
    > + Definition: Should contain a reference to the vbat supply
    > +
    > +- vdd-micbias-supply:
    > + Usage: required
    > + Value type: <phandle>
    > + Definition: Should contain a reference to the micbias supply
    > +
    > +- vdd-io-supply:
    > + Usage: required
    > + Value type: <phandle>
    > + Definition: Should contain a reference to the 1.8V io supply
    > +
    > +- interrupt-controller:
    > + Usage: required
    > + Definition: Indicating that this is a interrupt controller
    > +
    > +- #interrupt-cells:
    > + Usage: required
    > + Value type: <int>
    > + Definition: should be 1
    > +
    > +#sound-dai-cells
    > + Usage: required
    > + Value type: <u32>
    > + Definition: Must be 1
    > +
    > +codec@1{

    audio-codec@1

    > + compatible = "slim217,1a0";
    > + reg = <1 0>;
    > + interrupts = <&msmgpio 54 IRQ_TYPE_LEVEL_HIGH>;
    > + interrupt-names = "intr2"
    > + reset-gpio = <&msmgpio 64 0>;
    > + qcom,ifd = <&wc9335_ifd>;
    > + clock-names = "mclk", "native";
    > + clocks = <&rpmcc RPM_SMD_DIV_CLK1>,
    > + <&rpmcc RPM_SMD_BB_CLK1>;
    > + vdd-buck-supply = <&pm8994_s4>;
    > + vdd-rx-supply = <&pm8994_s4>;
    > + vdd-buck-sido-supply = <&pm8994_s4>;
    > + vdd-tx-supply = <&pm8994_s4>;
    > + vdd-io-supply = <&pm8994_s4>;
    > + #sound-dai-cells = <1>;
    > +}
    > --
    > 2.16.2
    >

    \
     
     \ /
      Last update: 2018-07-31 22:44    [W:4.181 / U:0.320 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site