Thanks for review comments,
On 14/06/16 17:12, Kenneth Westfield wrote:
On Fri, Jun 10, 2016 at 11:18:44AM -0700, Srinivas Kandagatla wrote:
diff --git a/Documentation/devicetree/bindings/sound/qcom,msm8916-wcd.txt
b/Documentation/devicetree/bindings/sound/qcom,msm8916-wcd.txt
new file mode 100644
index 0000000..0559c1f
--- /dev/null
+++ b/Documentation/devicetree/bindings/sound/qcom,msm8916-wcd.txt
@@ -0,0 +1,103 @@
+msm8916 audio CODEC 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.
Cleaning up the description:
msm8916 audio CODEC
Codec IP is divided into two parts:
* analog, which is integrated into pmic pm8916
* digital, which is integrated into LPASS on the SOC
Codec registers are also split across pmic and LPASS.
Analog codec is controlled via spmi bus to pmic.
+## Bindings for codec core on pmic:
+
+Required properties
+ - compatible = "qcom,msm8916-pmic-wcd-codec";
OK. Looking at this again, I think its better just to drop the pmic.
Yep..
+ "spk_cnp_int" - Speaker click and pop interrupt
+ "spk_clip_int" - Speaker clip interrupt
+ "spk_ocp_int" - Speaker over current protect interrupt.
+ "ins_rem_det1" - jack insert removal detect interrupt 1.
+ "but_rel_det" - button release interrupt
+ "but_press_det" - button press event
+ "ins_rem_det" - jack insert removal detect interrup
+ "mbhc_int" - multi button headset interrupt.
+ "ear_ocp_int" - Earphone over current protect interrupt.
+ "hphr_ocp_int" - Headphone R over current protect interrupt.
+ "hphl_ocp_det" - Headphone L over current protect interrupt
+ "ear_cnp_int" - earphone cnp interrupt.
+ "hphr_cnp_int" - hphr click and pop interrupt.
+ "hphl_cnp_int" - hphl click and pop interrupt
Please use labels that more closely match the HW spec:
"cdc_spk_cnp_int"
"cdc_spk_clip_int"
"cdc_spk_ocp_int"
"mbhc_ins_rem_det1"
"mbhc_but_rel_det"
"mbhc_but_press_det"
"mbhc_ins_rem_det"
"mbhc_switch_int"
"cdc_ear_ocp_int"
"cdc_hphr_ocp_int"
"cdc_hphl_ocp_int"
"cdc_ear_cnp_int"
"cdc_hphr_cnp_int"
"cdc_hphl_cnp_int"
Yes, we can do that to match the specs.
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html