mbox series

[v2,0/2] Enable audio output for Genio 700 EVK board

Message ID 20241210-genio700-audio-output-v2-0-c50886ae0be2@collabora.com (mailing list archive)
Headers show
Series Enable audio output for Genio 700 EVK board | expand

Message

Nícolas F. R. A. Prado Dec. 10, 2024, 8:54 p.m. UTC
This series enables audio output support for the Genio 700 EVK board.
Patch 1 adds the missing #sound-dai-cells property in the MT6359 node as
a prerequisite, and patch 2 describes the audio DT nodes to get audio
output working on the two audio jacks, Earphone and Speaker, present on
the board.

Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
---
Changes in v2:
- Dropped merged/unnecessary patches (2, 3, 4)
- Removed unnecessary mediatek,dai-link property
- Link to v1: https://lore.kernel.org/r/20241205-genio700-audio-output-v1-0-0e955c78c29e@collabora.com

---
Nícolas F. R. A. Prado (2):
      arm64: dts: mt6359: Add #sound-dai-cells property
      arm64: dts: mediatek: mt8390-genio-700-evk: Add sound output support

 arch/arm64/boot/dts/mediatek/mt6359.dtsi           |  1 +
 .../boot/dts/mediatek/mt8390-genio-700-evk.dts     | 48 ++++++++++++++++++++++
 2 files changed, 49 insertions(+)
---
base-commit: d1486dca38afd08ca279ae94eb3a397f10737824
change-id: 20241204-genio700-audio-output-fba8f2ebad10

Best regards,

Comments

Rob Herring (Arm) Dec. 11, 2024, 3:54 p.m. UTC | #1
On Tue, 10 Dec 2024 17:54:27 -0300, Nícolas F. R. A. Prado wrote:
> This series enables audio output support for the Genio 700 EVK board.
> Patch 1 adds the missing #sound-dai-cells property in the MT6359 node as
> a prerequisite, and patch 2 describes the audio DT nodes to get audio
> output working on the two audio jacks, Earphone and Speaker, present on
> the board.
> 
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
> ---
> Changes in v2:
> - Dropped merged/unnecessary patches (2, 3, 4)
> - Removed unnecessary mediatek,dai-link property
> - Link to v1: https://lore.kernel.org/r/20241205-genio700-audio-output-v1-0-0e955c78c29e@collabora.com
> 
> ---
> Nícolas F. R. A. Prado (2):
>       arm64: dts: mt6359: Add #sound-dai-cells property
>       arm64: dts: mediatek: mt8390-genio-700-evk: Add sound output support
> 
>  arch/arm64/boot/dts/mediatek/mt6359.dtsi           |  1 +
>  .../boot/dts/mediatek/mt8390-genio-700-evk.dts     | 48 ++++++++++++++++++++++
>  2 files changed, 49 insertions(+)
> ---
> base-commit: d1486dca38afd08ca279ae94eb3a397f10737824
> change-id: 20241204-genio700-audio-output-fba8f2ebad10
> 
> Best regards,
> --
> Nícolas F. R. A. Prado <nfraprado@collabora.com>
> 
> 


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 mediatek/mt8390-genio-700-evk.dtb' for 20241210-genio700-audio-output-v2-0-c50886ae0be2@collabora.com:

arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dtb: pmic: '#sound-dai-cells', 'mt6359codec', 'mt6359rtc' do not match any of the regexes: 'pinctrl-[0-9]+'
	from schema $id: http://devicetree.org/schemas/mfd/mediatek,mt6397.yaml#
arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dtb: jpeg-decoder@1a040000: iommus: [[124, 685], [124, 686], [124, 690], [124, 691], [124, 692], [124, 693]] is too long
	from schema $id: http://devicetree.org/schemas/media/mediatek-jpeg-decoder.yaml#
AngeloGioacchino Del Regno Dec. 12, 2024, 10:04 a.m. UTC | #2
On Tue, 10 Dec 2024 17:54:27 -0300, Nícolas F. R. A. Prado wrote:
> This series enables audio output support for the Genio 700 EVK board.
> Patch 1 adds the missing #sound-dai-cells property in the MT6359 node as
> a prerequisite, and patch 2 describes the audio DT nodes to get audio
> output working on the two audio jacks, Earphone and Speaker, present on
> the board.
> 
> 
> [...]

Applied to v6.13-next/dts64, thanks!

[1/2] arm64: dts: mt6359: Add #sound-dai-cells property
      commit: 9bc8353be720ca1f9cb6e03825929bc172e1157d
[2/2] arm64: dts: mediatek: mt8390-genio-700-evk: Add sound output support
      commit: b8457716eeee45ed5cb6b8fdc5a5b459f7ebe819

Cheers,
Angelo