2020-04-21 16:44:47

by Jerome Brunet

[permalink] [raw]
Subject: [PATCH 0/6] arm64: dts: meson-gx: add initial playback support

This patchset is adding the aiu support in DT and well as basic card
support for the p230/q200 and libretech boards

I was hoping to provide the internal codec support with this series but
this is still blocked on the reset dt-bindings of the DAC [0].

So far, things are fairly stable on these boards. I have experienced
a few glitches on rare occasions. I have not been able to precisely found
out why. It seems to be linked the AIU resets and 8ch support. Maybe more
eyes (and ears) on this will help. If things get annoying and no solution
is found, I'll submit a change to restrict the output to i2s 2ch.

[0]: https://lore.kernel.org/r/[email protected]

Jerome Brunet (6):
arm64: defconfig: enable meson gx audio as module
arm64: dts: meson-gx: add aiu support
arm64: dts: meson: p230-q200: add initial audio playback support
arm64: dts: meson: libretech-cc: add initial audio playback support
arm64: dts: meson: libretech-ac: add initial audio playback support
arm64: dts: meson: libretech-pc: add initial audio playback support

.../dts/amlogic/meson-gx-libretech-pc.dtsi | 40 ++++++++++++
.../boot/dts/amlogic/meson-gx-p23x-q20x.dtsi | 63 +++++++++++++++++++
arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 13 ++++
arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 23 +++++++
.../amlogic/meson-gxl-s805x-libretech-ac.dts | 40 ++++++++++++
.../amlogic/meson-gxl-s905x-libretech-cc.dts | 40 ++++++++++++
arch/arm64/boot/dts/amlogic/meson-gxl.dtsi | 23 +++++++
arch/arm64/configs/defconfig | 2 +
8 files changed, 244 insertions(+)

--
2.25.2


2020-04-30 01:56:19

by Christian Hewitt

[permalink] [raw]
Subject: Re: [PATCH 0/6] arm64: dts: meson-gx: add initial playback support


> On 21 Apr 2020, at 8:39 pm, Jerome Brunet <[email protected]> wrote:
>
> This patchset is adding the aiu support in DT and well as basic card
> support for the p230/q200 and libretech boards
>
> I was hoping to provide the internal codec support with this series but
> this is still blocked on the reset dt-bindings of the DAC [0].
>
> So far, things are fairly stable on these boards. I have experienced
> a few glitches on rare occasions. I have not been able to precisely found
> out why. It seems to be linked the AIU resets and 8ch support. Maybe more
> eyes (and ears) on this will help. If things get annoying and no solution
> is found, I'll submit a change to restrict the output to i2s 2ch.
>
> [0]: https://lore.kernel.org/r/[email protected]

I have a reproducible way to provoke one glitch using Kodi but we can talk
about that off-list. Overall the series (and changes in [0] above) work
well and it’s great to see progress.

Tested-by: Christian Hewitt <[email protected]>

2020-04-30 14:58:54

by Kevin Hilman

[permalink] [raw]
Subject: Re: [PATCH 0/6] arm64: dts: meson-gx: add initial playback support

On Tue, 21 Apr 2020 18:39:29 +0200, Jerome Brunet wrote:
> This patchset is adding the aiu support in DT and well as basic card
> support for the p230/q200 and libretech boards
>
> I was hoping to provide the internal codec support with this series but
> this is still blocked on the reset dt-bindings of the DAC [0].
>
> So far, things are fairly stable on these boards. I have experienced
> a few glitches on rare occasions. I have not been able to precisely found
> out why. It seems to be linked the AIU resets and 8ch support. Maybe more
> eyes (and ears) on this will help. If things get annoying and no solution
> is found, I'll submit a change to restrict the output to i2s 2ch.
>
> [...]

Queued for v5.8, thanks!

[1/6] arm64: defconfig: enable meson gx audio as module
commit: cba26aef31ec791bc4d5ea13cb2c1288fcd95c0e
[2/6] arm64: dts: meson-gx: add aiu support
commit: c531ca35ad54abcf6ceb35767ff25355557aa7ca
[3/6] arm64: dts: meson: p230-q200: add initial audio playback support
commit: ace867239dd8059f2e92556895e7e13aea7b2e8a
[4/6] arm64: dts: meson: libretech-cc: add initial audio playback support
commit: 9644eb9dc8bd1ac1112e267d771c6b492af1ff79
[5/6] arm64: dts: meson: libretech-ac: add initial audio playback support
commit: 74c601fa46d109049472bb66adee88b3290b4bb8
[6/6] arm64: dts: meson: libretech-pc: add initial audio playback support
commit: 1c75a81473153c211478e37152223aa585631b87

Best regards,
--
Kevin Hilman <[email protected]>