[PATCH v5 0/4] qrb4210-rb2: add HDMI audio playback support

Alexey Klimov posted 4 patches 1 year, 2 months ago
arch/arm64/boot/dts/qcom/qrb4210-rb2.dts | 59 ++++++++++++++++
arch/arm64/boot/dts/qcom/sm4250.dtsi     | 39 +++++++++++
arch/arm64/boot/dts/qcom/sm6115.dtsi     | 87 ++++++++++++++++++++++++
3 files changed, 185 insertions(+)
[PATCH v5 0/4] qrb4210-rb2: add HDMI audio playback support
Posted by Alexey Klimov 1 year, 2 months ago
Rebased on top of today's -next and re-tested.

Changes in v5:
-- drop MI2S clock in sm8250 patch (seems to be merged);
-- fix gpio-ranges in lpass_tlmm (Dmitry);
-- added reviewed-by tags from Konrad and Dmitry.

URL to previous series:
https://lore.kernel.org/linux-sound/20241101005925.186696-1-alexey.klimov@linaro.org/

Changes in v4 (or since v3):
-- cleaned doubled spaces in the last patch as suggested by Konrad;
-- added Reviewed-by tag to patch 4.

Changes since v2:
-- added tags, updated commit messages, added Cc;
-- updated LT9611 -> LT9611UXC comment in qrb4210-rb2.dts;
-- updated addresses in DT to 8 hex digits as requested by Dmitry;
-- added lpass pinctrl to sm6115.dtsi as suggested by Dmitry;
-- added lpass pinctrl override and pins description to sm4250.dtsi,
pins are the property of sm4250;
-- verified with make dtbs_check as suggested by Krzysztof and Rob's bot.
-- dropped two patches (they seem to be merged):
[PATCH v2 1/7] ASoC: dt-bindings: qcom,sm8250: add qrb4210-rb2-sndcard
[PATCH v2 2/7] ASoC: qcom: sm8250: add qrb4210-rb2-sndcard compatible string
-- stopped Cc-ing out-of-date emails.

Changes since v1:
-- removed handling of MI2S clock in sm2450_snd_shutdown(): setting clock rate
   and disabling it causes audio delay on playback start;
-- removed empty sound { } node from sm6115.dtsi as suggested by Krzysztof;
-- moved lpi_i2s2_active pins description to qrb423310 board-specific file
   as suggested by Dmitry Baryshkov;
-- moved q6asmdai DAIs to apr soc node as suggested by Konrad Dybcio;
-- lpass_tlmm is not disabled;
-- lpass_tlmm node moved to sm4250.dtsi;
-- kept MultiMedia DAIs as is, without them the sound card driver doesn't initialise;
-- added some reviewed-by tags.

Alexey Klimov (4):
  arm64: dts: qcom: sm6115: add apr and its services
  arm64: dts: qcom: sm6115: add LPASS LPI pin controller
  arm64: dts: qcom: sm4250: add LPASS LPI pin controller
  arm64: dts: qcom: qrb4210-rb2: add HDMI audio playback support

 arch/arm64/boot/dts/qcom/qrb4210-rb2.dts | 59 ++++++++++++++++
 arch/arm64/boot/dts/qcom/sm4250.dtsi     | 39 +++++++++++
 arch/arm64/boot/dts/qcom/sm6115.dtsi     | 87 ++++++++++++++++++++++++
 3 files changed, 185 insertions(+)

-- 
2.45.2
Re: [PATCH v5 0/4] qrb4210-rb2: add HDMI audio playback support
Posted by Bjorn Andersson 1 year, 1 month ago
On Tue, 12 Nov 2024 02:53:02 +0000, Alexey Klimov wrote:
> Rebased on top of today's -next and re-tested.
> 
> Changes in v5:
> -- drop MI2S clock in sm8250 patch (seems to be merged);
> -- fix gpio-ranges in lpass_tlmm (Dmitry);
> -- added reviewed-by tags from Konrad and Dmitry.
> 
> [...]

Applied, thanks!

[1/4] arm64: dts: qcom: sm6115: add apr and its services
      commit: c722e3ce278826f29a2a8500d685130dd0b6a297
[2/4] arm64: dts: qcom: sm6115: add LPASS LPI pin controller
      commit: 4541a5f11e59015d2f4b39864e421bf9e804097d
[3/4] arm64: dts: qcom: sm4250: add LPASS LPI pin controller
      commit: 6624d17a8142776e43bcd632c227ebf0bbe9d590
[4/4] arm64: dts: qcom: qrb4210-rb2: add HDMI audio playback support
      commit: 1caf6149c3bf41a2ee07869449c4ea1ec8bbc2f8

Best regards,
-- 
Bjorn Andersson <andersson@kernel.org>
Re: [PATCH v5 0/4] qrb4210-rb2: add HDMI audio playback support
Posted by Alexey Klimov 1 year, 1 month ago
Hello Bjorn,

On Fri Dec 27, 2024 at 4:40 AM GMT, Bjorn Andersson wrote:
>
> On Tue, 12 Nov 2024 02:53:02 +0000, Alexey Klimov wrote:
> > Rebased on top of today's -next and re-tested.
> > 
> > Changes in v5:
> > -- drop MI2S clock in sm8250 patch (seems to be merged);
> > -- fix gpio-ranges in lpass_tlmm (Dmitry);
> > -- added reviewed-by tags from Konrad and Dmitry.
> > 
> > [...]
>
> Applied, thanks!
>
> [1/4] arm64: dts: qcom: sm6115: add apr and its services
>       commit: c722e3ce278826f29a2a8500d685130dd0b6a297
> [2/4] arm64: dts: qcom: sm6115: add LPASS LPI pin controller
>       commit: 4541a5f11e59015d2f4b39864e421bf9e804097d
> [3/4] arm64: dts: qcom: sm4250: add LPASS LPI pin controller
>       commit: 6624d17a8142776e43bcd632c227ebf0bbe9d590
> [4/4] arm64: dts: qcom: qrb4210-rb2: add HDMI audio playback support
>       commit: 1caf6149c3bf41a2ee07869449c4ea1ec8bbc2f8

It seems you applied v5 of this patchset.
There is also v6 of this patchset that was sent here:
https://lore.kernel.org/linux-arm-msm/20241204140027.2198763-1-alexey.klimov@linaro.org/

Could you please merge v6 instead?

Thanks,
Alexey