On Tue 01 Dec 09:37 CST 2020, Srinivas Kandagatla wrote:

> This patch add support for two WSA881X smart speakers attached via Soundwire
> and a DMIC0 on the main board.
> 
> Signed-off-by: Srinivas Kandagatla <srinivas.kandaga...@linaro.org>
> ---
>  arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 125 +++++++++++++++++++++++
>  1 file changed, 125 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts 
> b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> index ce22d4fa383e..03229d5cb9d3 100644
> --- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> +++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> @@ -7,6 +7,8 @@
>  
>  #include <dt-bindings/gpio/gpio.h>
>  #include <dt-bindings/regulator/qcom,rpmh-regulator.h>
> +#include <dt-bindings/sound/qcom,q6afe.h>
> +#include <dt-bindings/sound/qcom,q6asm.h>
>  #include "sm8250.dtsi"
>  #include "pm8150.dtsi"
>  #include "pm8150b.dtsi"
> @@ -120,6 +122,11 @@
>       };
>  };
>  
> +&adsp {
> +     status = "okay";
> +     firmware-name = "qcom/sm8250/adsp.mdt";

Rather than adding all the mdt + bXX files to linux-firmware (when that
day comes) can we please make this qcom/sm8250/adsp.mbn from the start?

The mbn can be generated from the existing files using
https://github.com/andersson/pil-squasher, or for testing purposes the
.mdt can simply be renamed .mbn and the mdt loader will find the
remaining .bXX files.

> +};
> +
>  &apps_rsc {
>       pm8009-rpmh-regulators {
>               compatible = "qcom,pm8009-rpmh-regulators";
> @@ -483,6 +490,35 @@
>       status = "okay";
>  };
>  
> +&q6afedai {
> +     qi2s@16 {
> +             reg = <16>;
> +             qcom,sd-lines = <0 1 2 3>;
> +     };
> +};
> +
> +/* TERT I2S Uses 1 I2S SD Lines for audio on LT9611 HDMI Bridge */
> +&q6afedai {
> +     qi2s@20 {
> +             reg = <20>;
> +             qcom,sd-lines = <0>;
> +     };
> +};
> +
> +&q6asmdai {
> +     dai@0 {
> +             reg = <0>;
> +     };
> +
> +     dai@1 {
> +             reg = <1>;
> +     };
> +
> +     dai@2 {
> +             reg = <2>;
> +     };
> +};
> +
>  &sdhc_2 {
>       status = "okay";
>       pinctrl-names = "default";
> @@ -497,6 +533,88 @@
>       no-emmc;
>  };
>  
> +&swr0 {
> +

Unnecessary empty line.

> +     left_spkr: wsa8810-left{
> +             compatible = "sdw10217211000";
> +             reg = <0 3>;
> +             powerdown-gpios = <&tlmm 130 GPIO_ACTIVE_HIGH>;
> +             #thermal-sensor-cells = <0>;
> +             sound-name-prefix = "SpkrLeft";
> +             #sound-dai-cells = <0>;
> +     };
> +

Ditto.

Regards,
Bjorn

> +
> +     right_spkr: wsa8810-right{
> +             compatible = "sdw10217211000";
> +             reg = <0 4>;
> +             powerdown-gpios = <&tlmm 130 GPIO_ACTIVE_HIGH>;
> +             #thermal-sensor-cells = <0>;
> +             sound-name-prefix = "SpkrRight";
> +             #sound-dai-cells = <0>;
> +     };
> +};
> +
> +&sound {
> +     compatible = "qcom,qrb5165-rb5";
> +     pinctrl-0 = <&tert_mi2s_sck_active
> +                      &tert_mi2s_sd0_active
> +                      &tert_mi2s_ws_active>;
> +     pinctrl-names = "default";
> +     model = "Qualcomm-RB5-WSA8815-Speakers-DMIC0";
> +     audio-routing =
> +             "SpkrLeft IN", "WSA_SPK1 OUT",
> +             "SpkrRight IN", "WSA_SPK2 OUT",
> +             "VA DMIC0", "vdd-micb",
> +                "VA DMIC1", "vdd-micb",
> +             "MM_DL1",  "MultiMedia1 Playback",
> +             "MultiMedia3 Capture", "MM_UL3";
> +
> +     mm1-dai-link {
> +             link-name = "MultiMedia1";
> +             cpu {
> +                     sound-dai = <&q6asmdai  MSM_FRONTEND_DAI_MULTIMEDIA1>;
> +             };
> +     };
> +
> +     mm3-dai-link {
> +             link-name = "MultiMedia3";
> +             cpu {
> +                     sound-dai = <&q6asmdai  MSM_FRONTEND_DAI_MULTIMEDIA3>;
> +             };
> +     };
> +
> +     dma-dai-link {
> +             link-name = "WSA Playback";
> +             cpu {
> +                     sound-dai = <&q6afedai WSA_CODEC_DMA_RX_0>;
> +             };
> +
> +             platform {
> +                     sound-dai = <&q6routing>;
> +             };
> +
> +             codec {
> +                     sound-dai = <&left_spkr>, <&right_spkr>, <&swr0 0>, 
> <&wsamacro 0>;
> +             };
> +     };
> +
> +     va-dai-link {
> +             link-name = "VA Capture";
> +             cpu {
> +                     sound-dai = <&q6afedai VA_CODEC_DMA_TX_0>;
> +             };
> +
> +             platform {
> +                     sound-dai = <&q6routing>;
> +             };
> +
> +             codec {
> +                     sound-dai = <&vamacro 0>;
> +             };
> +     };
> +};
> +
>  /* CAN */
>  &spi0 {
>       status = "okay";
> @@ -792,3 +910,10 @@
>       vdda-phy-supply = <&vreg_l9a_1p2>;
>       vdda-pll-supply = <&vreg_l18a_0p92>;
>  };
> +
> +&vamacro {
> +     pinctrl-0 = <&cdc_dmic01_clk_active &cdc_dmic01_data_active>;
> +     pinctrl-names = "default";
> +     vdd-micb-supply = <&vreg_s4a_1p8>;
> +     qcom,dmic-sample-rate = <600000>;
> +};
> -- 
> 2.21.0
> 

Reply via email to