| Message ID | 20241002022015.867031-1-alexey.klimov@linaro.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | qrb4210-rb2: add HDMI audio playback support | expand |
On Wed, 02 Oct 2024 03:20:08 +0100, Alexey Klimov wrote: > Rebased on top of -master, tested. > > 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. > > This series still keeps "qcom,qrb4210-rb2-sndcard" for sm8250 soundcard. As per > off the list discussion with Srini it was suggested to have it since in future it > may be required to add clocks, workarounds, quirks, model-specific things based on > this compatible. The same as for RB5 compatible in sm8250 snd driver. > > This focuses on HDMI audio playback only hence there are no soundwire and dmic pins, > for instance. The work to enable playback via wcd+wsa8815 amplifier is in progress (it works) > and one of the routes is to merge such two patchsets together. > > Link to prev series: > https://lore.kernel.org/linux-sound/20240628010715.438471-1-alexey.klimov@linaro.org/ > > Alexey Klimov (7): > ASoC: dt-bindings: qcom,sm8250: add qrb4210-rb2-sndcard > ASoC: qcom: sm8250: add qrb4210-rb2-sndcard compatible string > ASoC: qcom: sm8250: add handling of secondary MI2S clock > arm64: dts: qcom: sm6115: add apr and its services > arm64: dts: qcom: sm4250: add LPASS LPI pin controller > arm64: dts: qcom: qrb4210-rb2: add description of lpi_i2s2 pins > arm64: dts: qcom: qrb4210-rb2: add HDMI audio playback support > > .../bindings/sound/qcom,sm8250.yaml | 1 + > arch/arm64/boot/dts/qcom/qrb4210-rb2.dts | 91 +++++++++++++++++++ > arch/arm64/boot/dts/qcom/sm4250.dtsi | 16 ++++ > arch/arm64/boot/dts/qcom/sm6115.dtsi | 72 +++++++++++++++ > sound/soc/qcom/sm8250.c | 9 ++ > 5 files changed, 189 insertions(+) > > -- > 2.45.2 > > > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade New warnings running 'make CHECK_DTBS=y qcom/qrb4210-rb2.dtb' for 20241002022015.867031-1-alexey.klimov@linaro.org: arch/arm64/boot/dts/qcom/qrb4210-rb2.dtb: pinctrl@a7c0000: lpi-i2s2-active-state: 'oneOf' conditional failed, one must be fixed: 'pins' is a required property 'function' is a required property Unevaluated properties are not allowed ('data-pins', 'ext-mclk1', 'sck-pin', 'ws-pins' were unexpected) 'ext-mclk1', 'sck-pin' do not match any of the regexes: '-pins$', 'pinctrl-[0-9]+' from schema $id: http://devicetree.org/schemas/pinctrl/qcom,sm4250-lpass-lpi-pinctrl.yaml#
On Wed, 02 Oct 2024 03:20:08 +0100, Alexey Klimov wrote: > Rebased on top of -master, tested. > > 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. > > [...] Applied to https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next Thanks! [1/7] ASoC: dt-bindings: qcom,sm8250: add qrb4210-rb2-sndcard commit: bbd1e5ea66f6ca88624faefe0a153637f53ad15d [2/7] ASoC: qcom: sm8250: add qrb4210-rb2-sndcard compatible string commit: b97bc0656a66f89f78098d4d72dc04fa9518ab11 All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark
Rebased on top of -master, tested. 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. This series still keeps "qcom,qrb4210-rb2-sndcard" for sm8250 soundcard. As per off the list discussion with Srini it was suggested to have it since in future it may be required to add clocks, workarounds, quirks, model-specific things based on this compatible. The same as for RB5 compatible in sm8250 snd driver. This focuses on HDMI audio playback only hence there are no soundwire and dmic pins, for instance. The work to enable playback via wcd+wsa8815 amplifier is in progress (it works) and one of the routes is to merge such two patchsets together. Link to prev series: https://lore.kernel.org/linux-sound/20240628010715.438471-1-alexey.klimov@linaro.org/ Alexey Klimov (7): ASoC: dt-bindings: qcom,sm8250: add qrb4210-rb2-sndcard ASoC: qcom: sm8250: add qrb4210-rb2-sndcard compatible string ASoC: qcom: sm8250: add handling of secondary MI2S clock arm64: dts: qcom: sm6115: add apr and its services arm64: dts: qcom: sm4250: add LPASS LPI pin controller arm64: dts: qcom: qrb4210-rb2: add description of lpi_i2s2 pins arm64: dts: qcom: qrb4210-rb2: add HDMI audio playback support .../bindings/sound/qcom,sm8250.yaml | 1 + arch/arm64/boot/dts/qcom/qrb4210-rb2.dts | 91 +++++++++++++++++++ arch/arm64/boot/dts/qcom/sm4250.dtsi | 16 ++++ arch/arm64/boot/dts/qcom/sm6115.dtsi | 72 +++++++++++++++ sound/soc/qcom/sm8250.c | 9 ++ 5 files changed, 189 insertions(+)