mbox series

[v3,0/3] arm64: dts: qcom: sc8280xp: add audio support

Message ID 20221121130403.161817-1-srinivas.kandagatla@linaro.org
Headers
Series arm64: dts: qcom: sc8280xp: add audio support |

Message

Srinivas Kandagatla Nov. 21, 2022, 1:04 p.m. UTC 
  This patchset adds audio support for sc8280xp Lenovo x13s.
Support for Headset Playback/Capture, Speaker Playback and DMIC is
tested.

A prebuit ASoC topology file available at
https://git.linaro.org/people/srinivas.kandagatla/audioreach-topology.git/tree/prebuilt/SC8280XP-LENOVO-X13S-tplg.bin


Thanks to Johan and Kryz for reviewing v2.

Changes since v2:
	- removed lots of stray lines.
	- ordered include files.
	- moved all the nodes before pinctrl
	- fixed subject line to include x13s.
	- rebased to latest qcom dts branch.

Thanks,
Srini

Srinivas Kandagatla (3):
  arm64: dts: qcom: sc8280xp/sa8540p: add gpr node
  arm64: dts: qcom: sc8280xp/sa8540p: add SoundWire and LPASS
  arm64: dts: qcom: sc8280xp-x13s: Add soundcard support

 .../qcom/sc8280xp-lenovo-thinkpad-x13s.dts    | 213 +++++++++++
 arch/arm64/boot/dts/qcom/sc8280xp.dtsi        | 359 ++++++++++++++++++
 2 files changed, 572 insertions(+)

Comments

Johan Hovold Nov. 21, 2022, 2:22 p.m. UTC | #1 
On Mon, Nov 21, 2022 at 01:04:00PM +0000, Srinivas Kandagatla wrote:
> This patchset adds audio support for sc8280xp Lenovo x13s.
> Support for Headset Playback/Capture, Speaker Playback and DMIC is
> tested.
> 
> A prebuit ASoC topology file available at
> https://git.linaro.org/people/srinivas.kandagatla/audioreach-topology.git/tree/prebuilt/SC8280XP-LENOVO-X13S-tplg.bin
> 
> 
> Thanks to Johan and Kryz for reviewing v2.
> 
> Changes since v2:
> 	- removed lots of stray lines.
> 	- ordered include files.
> 	- moved all the nodes before pinctrl
> 	- fixed subject line to include x13s.
> 	- rebased to latest qcom dts branch.
> 
> Thanks,
> Srini
> 
> Srinivas Kandagatla (3):
>   arm64: dts: qcom: sc8280xp/sa8540p: add gpr node
>   arm64: dts: qcom: sc8280xp/sa8540p: add SoundWire and LPASS

Only noticed now, but the commit prefix here should just be:

	arm64: dts: qcom: sc8280xp: ...

sa8540p is based on sc8280xp but you don't have to point that out in
every commit (there was one exception where I was also updating the
sa8540p dtsi due to platform differences).

>   arm64: dts: qcom: sc8280xp-x13s: Add soundcard support
> 
>  .../qcom/sc8280xp-lenovo-thinkpad-x13s.dts    | 213 +++++++++++
>  arch/arm64/boot/dts/qcom/sc8280xp.dtsi        | 359 ++++++++++++++++++
>  2 files changed, 572 insertions(+)

Johan