Thanks for the review,
On 16/02/16 18:58, Mark Rutland wrote:
On Tue, Feb 16, 2016 at 05:32:30PM +0000, Srinivas Kandagatla wrote:
This patch adds DT bindings required for msm8x16 codec which is
integrated in msm8916 and apq8016 SOCs.
Codec IP is divided into two parts, first analog which is integrated
in pmic pm8916 and secondly digital part which is integrated into
application processor. Codec register controls are also split across
pmic an lpass. Analog part is controlled via spmi bus to pmic, registers
on the Application processor are memory mapped.
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@xxxxxxxxxx>
---
.../devicetree/bindings/sound/qcom,msm8x16-wcd.txt | 91 ++++++++++++++++++++++
1 file changed, 91 insertions(+)
create mode 100644 Documentation/devicetree/bindings/sound/qcom,msm8x16-wcd.txt
diff --git a/Documentation/devicetree/bindings/sound/qcom,msm8x16-wcd.txt b/Documentation/devicetree/bindings/sound/qcom,msm8x16-wcd.txt
new file mode 100644
index 0000000..42d3f66
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/qcom,msm8x16-wcd.txt
@@ -0,0 +1,91 @@
+msm8x16 audio CODEC aka Tombak audio CODEC
+
+Codec IP is divided into two parts, first analog which is integrated in pmic pm8916
+and secondly digital part which is integrated into application processor. Codec register
+controls are also split across pmic an lpass. Analog part is controlled via spmi bus to pmic.
+
+## Bindings for codec core on pmic:
+
+Required properties
+ - compatible = "qcom,msm8x16-wcd-codec";
Wildcard bindings are frowned upon generally.
Have separate comaptible strings, perhaps with one as a fallback for the
other.
Ok, Will add soc specific compatible strings without wildcards in next
version.
+ - reg: represents the slave base address provided to the peripheral.
+ - interrupt-parent : The parent interrupt controller.
+ - interrupts: List of interrupts in given SPMI peripheral.
+ - interrupt-names: Names specified to above list of interrupts in same
+ order
You must explicitly describe the set of interrupt-names for the binding.
I will do that.
+ - vddio-supply: phandle to VDD_CDC_IO regulator device tree node.
+ - vdd-tx-rx-supply: phandle to VDD_CDC_TX/RX/CX regulator device tree node.
+ - vdda-cp-supply: phandle of VDD_CP charge pump supply's regulator device tree node.
+ - vdda-hph-supply: phandle of VDD_HPH supply's regulator device tree node.
+ - vdd-micbias-supply: phandle of VDD_MICBIAS supply's regulator device tree
+ node.
+- qcom,lpass-codec-core: phandle to syscon node of lpass code core.
+
+Optional Properties:
+- qcom,micbias1-ext-cap: present if micbias1 has external capacitor connected.
+- qcom,micbias2-ext-cap: present if micbias2 has external capacitor connected.
+
+## Bindings codec core on lpass:
+
+Required properties
+ - compatible: should be "syscon" follwed by "qcom,msm8x16-lpass-codec"
This should be the other way around. "syscon" should be the fallback as
it is the least specific. Likewise for the example.
Yep, will change the order and remove the wildcard in the compatible string.
Again, wildcard bindings are generally frowned upon.
Mark.
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
