From: Mohammad Rafi Shaik <quic_mohs@quicinc.com>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: <srinivas.kandagatla@linaro.org>, <bgoswami@quicinc.com>,
<broonie@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>,
<andersson@kernel.org>, <konrad.dybcio@linaro.org>,
<lgirdwood@gmail.com>, <tiwai@suse.com>,
<quic_rohkumar@quicinc.com>, <linux-kernel@vger.kernel.org>,
<alsa-devel@alsa-project.org>, <linux-sound@vger.kernel.org>,
<linux-arm-msm@vger.kernel.org>
Subject: Re: [PATCH v1 2/2] ASoC: qcom: qcm6490: Add machine driver for qcm6490
Date: Tue, 2 Apr 2024 14:50:56 +0530 [thread overview]
Message-ID: <7cd89939-e0d0-c43b-fa2a-224bf60f4166@quicinc.com> (raw)
In-Reply-To: <CAA8EJprLB6+i4H5tK9LbJYOEH8v9QF78u-+HMmqPqfpvz5J8zw@mail.gmail.com>
On 3/30/2024 12:09 AM, Dmitry Baryshkov wrote:
> Hi,
>
> On Fri, 29 Mar 2024 at 17:20, Mohammad Rafi Shaik <quic_mohs@quicinc.com> wrote:
>>
>>
>> On 3/27/2024 6:39 PM, Dmitry Baryshkov wrote:
>>
>> On Wed, 27 Mar 2024 at 13:34, Mohammad Rafi Shaik <quic_mohs@quicinc.com> wrote:
>
> Please turn off HTML mail composition and never ever send HTML email
> to OSS mailing lists.
> Now quotation levels are broken...
>
>>
>> Add machine driver for qcm6490 SoC.
>>
>> This initial supports which includes WSA883x Speakers with onboard DMIC
>> connected to internal LPASS codec via VA macros respectively and also
>> WCD937x based headset.
>>
>> Add compatible for sound card on Qualcomm qcs6490 boards.
>>
>> Granted that qcm6490 is similar to sc7280, is there any reason why you
>> can not use one of the existing sound card drivers? Like sc7280.c or
>> sc8280xp.c?
>>
>> The qcs6490/qcm6490 has additional interfaces like MI2S, which requires additional machine driver modification.
>>
>> Example : Required new DAPM widget change in machine driver for MI2S interface which is specific to qcs6490/qcm6490.
>>
>> So, using new qcm6490 machine driver.
>>
>> Will try to use existing sc8280xp.c machine driver.
>
> What about sc7280? In the end, qcm6490 is sc7280 with different firmware / spin.
>
>
The sc7280 is tightly linked with realtek codec. which will only support
for chrome.
The sc8280xp.c machine driver is more suitable for qcm6490, will use
same sc8280xp.
next prev parent reply other threads:[~2024-04-02 9:21 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-27 11:32 [PATCH v1 0/2] ASoC: qcom: add support for qcm6490 machine Mohammad Rafi Shaik
2024-03-27 11:32 ` [PATCH v1 1/2] ASoC: dt-bindings: qcom,qcm6490: Add qcm6490 snd qcs6490 sound card Mohammad Rafi Shaik
2024-03-27 12:16 ` Krzysztof Kozlowski
2024-03-27 18:37 ` Krzysztof Kozlowski
2024-03-27 11:32 ` [PATCH v1 2/2] ASoC: qcom: qcm6490: Add machine driver for qcm6490 Mohammad Rafi Shaik
2024-03-27 12:22 ` Krzysztof Kozlowski
2024-03-27 13:09 ` Dmitry Baryshkov
2024-03-29 15:19 ` Mohammad Rafi Shaik
2024-03-29 15:58 ` Krzysztof Kozlowski
2024-03-29 18:39 ` Dmitry Baryshkov
2024-04-02 9:20 ` Mohammad Rafi Shaik [this message]
2024-04-23 13:05 ` Konrad Dybcio
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7cd89939-e0d0-c43b-fa2a-224bf60f4166@quicinc.com \
--to=quic_mohs@quicinc.com \
--cc=alsa-devel@alsa-project.org \
--cc=andersson@kernel.org \
--cc=bgoswami@quicinc.com \
--cc=broonie@kernel.org \
--cc=dmitry.baryshkov@linaro.org \
--cc=konrad.dybcio@linaro.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=quic_rohkumar@quicinc.com \
--cc=srinivas.kandagatla@linaro.org \
--cc=tiwai@suse.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).