mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Srinivasa Rao Mandadapu <quic_srivasam@quicinc.com>
To: Matthias Kaehlcke <mka@chromium.org>
Cc: <agross@kernel.org>, <bjorn.andersson@linaro.org>,
	<robh+dt@kernel.org>, <linux-arm-msm@vger.kernel.org>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<quic_rohkumar@quicinc.com>, <srinivas.kandagatla@linaro.org>,
	<dianders@chromium.org>, <swboyd@chromium.org>,
	<judyhsiao@chromium.org>, <quic_tdas@quicinc.com>
Subject: Re: [PATCH v12 00/12] Add soundcard support for sc7280 based platforms.
Date: Tue, 7 Jun 2022 22:08:38 +0530	[thread overview]
Message-ID: <5427aa79-b87c-74eb-def5-9b4e3299a331@quicinc.com> (raw)
In-Reply-To: <Yp5yUSvC05wOxtei@google.com>


On 6/7/2022 3:02 AM, Matthias Kaehlcke wrote:
Thanks for your time Matthias!!!
> On Fri, May 20, 2022 at 05:48:32PM +0530, Srinivasa Rao Mandadapu wrote:
>> This patch set is to add bolero digital macros, WCD and maxim codecs nodes
>> for audio on sc7280 based platforms.
>>
>> This patch set depends on:
>>      -- https://patchwork.kernel.org/project/linux-arm-msm/list/?series=638776
>>      -- https://patchwork.kernel.org/project/linux-arm-msm/list/?series=634597
>>      -- https://patchwork.kernel.org/project/linux-clk/list/?series=637999
>>      -- https://patchwork.kernel.org/project/linux-arm-msm/list/?series=638002
> Another dependency (at least in terms of functionality) is:
>
> ASoC: qcom: soundwire: Add support for controlling audio CGCR from HLOS
> https://patchwork.kernel.org/patch/12853622/
This is landed today.
>
> And then there is this:
>
> arm64: dts: qcom: sc7280: Add lpasscore & lpassaudio clock controllers
> https://patchwork.kernel.org/project/linux-arm-msm/patch/20220523100058.26241-1-quic_tdas@quicinc.com/
>
> A previous version (v3) of that patch already landed (9499240d15f2
> "arm64: dts: qcom: sc7280: Add lpasscore & lpassaudio clock controllers"),
> it is not clear to me why it is still evolving as if that weren't the
> case.

I too have same doubt. the changes should be incremental and new patch. 
May be Taniya considered

the status in patchwork.kernel org, where still it's showing new.

>
>  From the newer version of the patch at least marking the 'lpasscc' node
> as disabled is needed.

yes, agree. And the node name changed to lpasscore to lpass_core.

Included Taniya in the mail chain for further discussion.


  reply	other threads:[~2022-06-07 16:38 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-20 12:18 Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 01/12] arm64: dts: qcom: sc7280: Add nodes for soundwire and va tx rx digital macro codecs Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 02/12] arm64: dts: qcom: sc7280: Enable digital codecs and soundwire for CRD 1.0/2.0 and IDP boards Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 03/12] arm64: dts: qcom: sc7280: Enable digital codecs and soundwire for CRD 3.0/3.1 Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 04/12] arm64: dts: qcom: sc7280: Add wcd9385 codec node for CRD 1.0/2.0 and IDP boards Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 05/12] arm64: dts: qcom: sc7280: Add wcd9385 codec node for CRD 3.0/3.1 Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 06/12] arm64: dts: qcom: sc7280: Add max98360a codec for CRD 1.0/2.0 and IDP boards Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 07/12] arm64: dts: qcom: sc7280: herobrine: Add max98360a codec node Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 08/12] arm64: dts: qcom: sc7280: Add lpass cpu node Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 09/12] arm64: dts: qcom: sc7280: Enable lpass cpu node for CRD 1.0/2.0 and IDP boards Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 10/12] arm64: dts: qcom: sc7280: Enable lpass cpu node for CRD 3.0/3.1 Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 11/12] arm64: dts: qcom: sc7280: Add sound node for CRD 1.0/2.0 and IDP boards Srinivasa Rao Mandadapu
2022-05-20 12:18 ` [PATCH v12 12/12] arm64: dts: qcom: sc7280: Add sound node for CRD 3.0/3.1 Srinivasa Rao Mandadapu
2022-06-06 21:32 ` [PATCH v12 00/12] Add soundcard support for sc7280 based platforms Matthias Kaehlcke
2022-06-07 16:38   ` Srinivasa Rao Mandadapu [this message]
2022-09-13 15:04 ` Bjorn Andersson

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=5427aa79-b87c-74eb-def5-9b4e3299a331@quicinc.com \
    --to=quic_srivasam@quicinc.com \
    --cc=agross@kernel.org \
    --cc=bjorn.andersson@linaro.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dianders@chromium.org \
    --cc=judyhsiao@chromium.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mka@chromium.org \
    --cc=quic_rohkumar@quicinc.com \
    --cc=quic_tdas@quicinc.com \
    --cc=robh+dt@kernel.org \
    --cc=srinivas.kandagatla@linaro.org \
    --cc=swboyd@chromium.org \
    /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

all inboxes | Powered by JetHome®