Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Message-ID: <16c03446-cc1c-4f71-8c0d-fe6c6f1bdfd0@quicinc.com>
Date:   Mon, 6 Nov 2023 09:07:34 -0800
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 1/2] arm64: dts: qcom: sm8550-qrd: add sound card
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Andy Gross <agross@kernel.org>,
        Bjorn Andersson <andersson@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        <linux-arm-msm@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
References: <20230612173758.286411-1-krzysztof.kozlowski@linaro.org>
 <17535d7a-73e2-465b-941c-99db4092327a@quicinc.com>
 <7371257e-3fb0-4538-ad0f-07bd0a827120@linaro.org>
From:   Elliot Berman <quic_eberman@quicinc.com>
In-Reply-To: <7371257e-3fb0-4538-ad0f-07bd0a827120@linaro.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 11/5/2023 4:55 AM, Krzysztof Kozlowski wrote:
> On 04/11/2023 00:10, Elliot Berman wrote:
>> Hi Krzysztof,
>>
>> On 6/12/2023 10:37 AM, Krzysztof Kozlowski wrote:
>>> Add the sound card node with tested playback over WSA8845 speakers and
>>> WCD9385 headset over USB Type-C.  The recording links were not tested,
>>> but should be similar to previous platforms.
>>>
>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>>
>>> ---
>>>
>>> Bindings for the sound card were not sent to LKML. Work-in-progress is
>>> available here:
>>> https://github.com/krzk/linux/tree/n/audio-wsa884x-on-top-of-wip-sm8450-audio-on-next
>>
>> Do you plan to send the bindings up? I can send this patch out if you don't have the cycle:
>>
>> https://github.com/krzk/linux/commit/f678691570386a11eb75dceca7291b4e05d981da
> 
> The patch itself does not make sense without dependencies. It should be
> sent after dependencies are sent.
> 

I think the patch makes sense now that the v6.6 sm8550-mtp.dtb is using the compatible:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/qcom/sm8550-mtp.dts?h=v6.6#n100

- Elliot