linux-kernel - Re: [PATCH v4] arm64: dts: qcom: sc7280: Fix Dmic no sound on villager-r1

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Yw07xSlVpQcXDqe7@google.com>
Date:   Mon, 29 Aug 2022 15:20:53 -0700
From:   Matthias Kaehlcke <mka@...omium.org>
To:     Doug Anderson <dianders@...omium.org>
Cc:     Judy Hsiao <judyhsiao@...omium.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        Srini Kandagatla <srinivas.kandagatla@...aro.org>,
        Jimmy Cheng-Yi Chiang <cychiang@...gle.com>,
        Judy Hsiao <judyhsiao@...gle.com>,
        Stephen Boyd <swboyd@...omium.org>,
        Linux ARM <linux-arm-kernel@...ts.infradead.org>,
        linux-arm-msm <linux-arm-msm@...r.kernel.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>, LKML <linux-kernel@...r.kernel.org>,
        Srinivasa Rao Mandadapu <quic_srivasam@...cinc.com>,
        Satya Priya <quic_c_skakit@...cinc.com>
Subject: Re: [PATCH v4] arm64: dts: qcom: sc7280: Fix Dmic no sound on
 villager-r1

On Fri, Aug 26, 2022 at 10:14:58AM -0700, Doug Anderson wrote:
> Hi,
> 
> On Thu, Aug 25, 2022 at 11:56 PM Judy Hsiao <judyhsiao@...omium.org> wrote:
> >
> > Fix the DMIC no sound issue of villager-r1 by using "PP1800_L2C" as the
> > DMIC power source to match the hardware schematic.
> >
> > This patch:
> >    1. set vdd-micb-supply to PP1800_L2C as the MIC Bias voltage regulator.
> >    2. In audio-routing, set VA DMIC01~VA DMIC03 to use the vdd-micb-supply
> >       setting.
> >
> > Co-developed-by: Srinivasa Rao Mandadapu <quic_srivasam@...cinc.com>
> > Signed-off-by: Srinivasa Rao Mandadapu <quic_srivasam@...cinc.com>
> > Signed-off-by: Judy Hsiao <judyhsiao@...omium.org>
> > ---
> > Changes since V3:
> >   -- Update the commit message and fix extra blank line.
> > Changes since V2:
> >   -- Update the commit message.
> > Changes since V1:
> >   -- Update the commit message.
> >
> > This patch depends on:
> > arm64: dts: qcom: sc7280: Add herobrine-villager-r1. [1]
> >
> > [1] https://patchwork.kernel.org/patch/12929106
> 
> After applying your [1] then your patch applies without merge
> conflicts. However, it still doesn't compile. You also depend on
> whatever patch adds the nodes "lpass_va_macro" and "sound" since
> you're referencing them in your patch. I haven't been keeping track of
> the latest status of all the audio patches, so I myself can't point to
> exactly what patches you depend on and whether those patches are ready
> to land.
> 
> In any case, the contents of this patch seem OK assuming the dependent
> patches cause something that looks like the current (downstream)
> chromeos-5.15 tree.
> 
> Reviewed-by: Douglas Anderson <dianders@...omium.org>

To my best knowledge the dependencies of the audio mess are (roughly in
order):

Add support for audio clock gating resets for SC7280 (v8)
https://patchwork.kernel.org/project/linux-arm-msm/list/?series=671887
  ready to land? (patch 2 has a minor comment about SoB tag)

Add DT support for audio clock gating resets for SC7280 (v4)
https://patchwork.kernel.org/project/linux-arm-msm/list/?series=666480
  ready to land after dependencies land

Add soundcard support for sc7280 based platforms (v14)
https://patchwork.kernel.org/project/linux-arm-msm/list/?series=657417
  ready to land after dependencies land

Add dtsi for sc7280 herobrine boards that using rt5682 codec (v3)
https://patchwork.kernel.org/project/linux-arm-msm/list/?series=644812
  ready to land after dependencies land

Add new board revision and LTE SKUs for sc7280-villager family (v8)
https://patchwork.kernel.org/project/linux-arm-msm/list/?series=663014
  ready to land after dependencies land

arm64: dts: qcom: sc7280: Fix Dmic no sound on villager-r1 (v4)
https://patchwork.kernel.org/project/linux-arm-msm/patch/20220826065621.2255795-1-judyhsiao@chromium.org/
  ready to land after dependencies land

So it looks like things are generally ready to land, unless [1] needs a
re-spin to fix the SoB tag.

m.

[1] https://patchwork.kernel.org/project/linux-arm-msm/patch/1661754153-14813-3-git-send-email-quic_c_skakit@quicinc.com/