Message ID | 1641208380-15510-1-git-send-email-quic_srivasam@quicinc.com |
---|---|
Headers | show |
Series | Add soundcard support for sc7280 based platforms. | expand |
On 1/5/2022 3:34 AM, Vincent Knecht wrote: Thanks for Your time Vincent!!! > Le lundi 03 janvier 2022 à 16:42 +0530, Srinivasa Rao Mandadapu a écrit : >> SC7280 has VA, TX and RX macros with SoundWire Controllers to attach with >> codecs like WCD938x, max98360a using soundwire masters and i2s bus. >> Add these nodes for sc7280 based platforms audio use case. >> Add tlmm gpio property in wcd938x node for switching CTIA/OMTP Headset. >> >> Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com> >> Co-developed-by: Venkata Prasad Potturu <quic_potturu@quicinc.com> >> Signed-off-by: Venkata Prasad Potturu <quic_potturu@quicinc.com> >> --- >> arch/arm64/boot/dts/qcom/sc7280-crd.dts | 4 ++ >> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 52 ++++++++++++++ >> arch/arm64/boot/dts/qcom/sc7280.dtsi | 113 +++++++++++++++++++++++++++++++ >> 3 files changed, 169 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/qcom/sc7280-crd.dts b/arch/arm64/boot/dts/qcom/sc7280-crd.dts >> index cd2755c..035cca9 100644 >> --- a/arch/arm64/boot/dts/qcom/sc7280-crd.dts >> +++ b/arch/arm64/boot/dts/qcom/sc7280-crd.dts >> @@ -72,6 +72,10 @@ ap_ts_pen_1v8: &i2c13 { >> pins = "gpio51"; >> }; >> >> +&wcd938x { >> + qcom,us-euro-gpios = <&tlmm 81 0>; >> +}; >> + > Maybe using a defined value is possible, rather than an obscure zero ? > GPIO_ACTIVE_HIGH ? Okay. Will change accordingly!!! > > >
On 1/5/2022 3:53 AM, Vincent Knecht wrote: Thanks for Your team Vincent!!! > Le lundi 03 janvier 2022 à 16:42 +0530, Srinivasa Rao Mandadapu a écrit : >> SC7280 has VA, TX and RX macros with SoundWire Controllers to attach with >> codecs like WCD938x, max98360a using soundwire masters and i2s bus. >> Add these nodes for sc7280 based platforms audio use case. >> Add tlmm gpio property in wcd938x node for switching CTIA/OMTP Headset. >> >> Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com> >> Co-developed-by: Venkata Prasad Potturu <quic_potturu@quicinc.com> >> Signed-off-by: Venkata Prasad Potturu <quic_potturu@quicinc.com> >> --- >> arch/arm64/boot/dts/qcom/sc7280-crd.dts | 4 ++ >> arch/arm64/boot/dts/qcom/sc7280-idp.dtsi | 52 ++++++++++++++ >> arch/arm64/boot/dts/qcom/sc7280.dtsi | 113 +++++++++++++++++++++++++++++++ >> 3 files changed, 169 insertions(+) > [...] > > >> diff --git a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi >> index ddeb508..94614c9 100644 >> --- a/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi >> +++ b/arch/arm64/boot/dts/qcom/sc7280-idp.dtsi >> @@ -636,3 +636,55 @@ >> bias-pull-up; >> }; >> }; >> + >> +&swr0 { >> + wcd_rx: wcd938x-rx{ > Space before brace... Okay. > >> + compatible = "sdw20217010d00"; >> + reg = <0 4>; >> + #sound-dai-cells = <1>; >> + qcom,rx-port-mapping = <1 2 3 4 5>; >> + }; >> +}; >> + >> +&swr1 { >> + wcd_tx: wcd938x-tx{ > Ditto... Okay. > >> + compatible = "sdw20217010d00"; >> + reg = <0 3>; >> + #sound-dai-cells = <1>; >> + qcom,tx-port-mapping = <1 2 3 4>; >> + }; >> +}; >> + >> +&soc { >> + max98360a: audio-codec-0 { >> + compatible = "maxim,max98360a"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&_en>; >> + sdmode-gpios = <&tlmm 63 GPIO_ACTIVE_HIGH>; >> + #sound-dai-cells = <0>; >> + }; >> + >> + wcd938x: codec { >> + compatible = "qcom,wcd9380-codec"; >> + #sound-dai-cells = <1>; >> + >> + reset-gpios = <&tlmm 83 0>; > GPIO_ACTIVE_HIGH ? Okay. Will change accordingly. >