lkml.org 
[lkml]   [2018]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [RESEND PATCH v2 15/15] arm64: dts: msm8996: db820c: Add sound card support
From
Date
Thanks for the comments,

On 03/01/18 00:22, Bjorn Andersson wrote:
> On Thu 14 Dec 09:34 PST 2017, srinivas.kandagatla@linaro.org wrote:
>
>> From: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>>
>> This patch adds hdmi sound card support to db820c via qdsp.
>>
>> Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
>> ---
>> arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 5 +++++
>> arch/arm64/boot/dts/qcom/msm8996.dtsi | 33 ++++++++++++++++++++++++++++
>> 2 files changed, 38 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
>> index 9769053957af..b955769b100d 100644
>> --- a/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi
>> @@ -190,6 +190,11 @@
>> };
>> };
>>
>> + snd {
>> + compatible = "qcom,apq8096-sndcard";
>> + qcom,model = "DB820c";
>> + iommus = <&lpass_q6_smmu 1>;
>> + };
>>
>> gpio_keys {
>> compatible = "gpio-keys";
>> diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
>> index a144cec7bb71..25c43fb8ab49 100644
>> --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
>> +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
>> @@ -1262,6 +1262,7 @@
>>
>> phys = <&hdmi_phy>;
>> phy-names = "hdmi_phy";
>> + #sound-dai-cells = <0>;
>>
>> ports {
>> #address-cells = <1>;
>> @@ -1297,6 +1298,33 @@
>> "ref_clk";
>> };
>> };
>> +
>> + lpass_q6_smmu: arm,smmu-lpass_q6@1600000 {
>
> name this node "iommu"
will rename it to arm,smmu@1600000

>
>> + compatible = "qcom,msm8996-smmu-v2";
>> + reg = <0x1600000 0x20000>;
>> + #iommu-cells = <1>;
>> + power-domains = <&gcc HLOS1_VOTE_LPASS_CORE_GDSC>;
>
> Indentation
sure.

>
>> +
>> + #global-interrupts = <1>;
>> + interrupts = <GIC_SPI 404 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 393 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 394 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 395 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 396 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 397 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 398 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 399 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 400 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 401 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 402 IRQ_TYPE_LEVEL_HIGH>,
>> + <GIC_SPI 403 IRQ_TYPE_LEVEL_HIGH>;
>> +
>> + clocks = <&gcc GCC_HLOS1_VOTE_LPASS_CORE_SMMU_CLK>,
>> + <&gcc GCC_HLOS1_VOTE_LPASS_ADSP_SMMU_CLK>;
>> + clock-names = "iface", "bus";
>> + status = "okay";
>> + };
>> };
>>
>> adsp-pil {
>> @@ -1325,6 +1353,11 @@
>> qcom,ipc = <&apcs 16 8>;
>> qcom,smd-edge = <1>;
>> qcom,remote-pid = <2>;
>> +
>> + apr {
>
> "apr-audio-svc", as this is not the only apr channel on this edge.

yep.

>
>> + compatible = "qcom,apr-msm8996";
>> + qcom,smd-channels = "apr_audio_svc";
>> + };
>> };
>
> Regards,
> Bjorn
>

\
 
 \ /
  Last update: 2018-01-03 17:28    [W:2.253 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site