On Tue, 8 Oct 2024 13:37:18 +0100
Andre Przywara <andre.przywara@xxxxxxx> wrote:
Hi,
> On Sun, 29 Sep 2024 23:06:07 +1300
> Ryan Walklin <ryan@xxxxxxxxxxxxx> wrote:
>
> Hi Ryan,
>
> > Now that the sun4i codec driver supports the H616, add a node in the
> > device tree for it.
>
> Can you please add another patch that actually enables the codec for at
> least one board? Is that really just status = "okay";? Or do we need
> simple-soundcard nodes still?
For the records: We don't need simple-soundcard, but we seem to need an
audio routing entry. So I tested this successfully on the X96 Mate with:
&codec {
allwinner,audio-routing = "Line Out", "LINEOUT";
status = "okay";
};
So can you add a patch adding those properties to the boards with a 3.5mm
jack? The OrangePi Zero2 and Zero3 do not have such a jack, but route the
LINEOUT signals to dedicated pins on the 13-pin header. Not sure that counts?
Cheers,
Andre
> I will try to give it a test on the H61* boards I have, and would then
> like those boards to be enabled as well, as part of this series. Otherwise
> it's somewhat of a dead feature, isn't it?
>
> Cheers,
> Andre
>
> > Signed-off-by: Ryan Walklin <ryan@xxxxxxxxxxxxx>
> > ---
> > arch/arm64/boot/dts/allwinner/sun50i-h616.dtsi | 15 +++++++++++++++
> > 1 file changed, 15 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h616.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h616.dtsi
> > index e88c1fbac6acc..006fdb7e7e0ae 100644
> > --- a/arch/arm64/boot/dts/allwinner/sun50i-h616.dtsi
> > +++ b/arch/arm64/boot/dts/allwinner/sun50i-h616.dtsi
> > @@ -645,6 +645,21 @@ spdif: spdif@5093000 {
> > status = "disabled";
> > };
> >
> > + codec: codec@05096000 {
> > + #sound-dai-cells = <0>;
> > + compatible = "allwinner,sun50i-h616-codec";
> > + reg = <0x05096000 0x31c>;
> > + interrupts = <GIC_SPI 58 IRQ_TYPE_LEVEL_HIGH>;
> > + clocks = <&ccu CLK_BUS_AUDIO_CODEC>,
> > + <&ccu CLK_AUDIO_CODEC_1X>,
> > + <&ccu CLK_AUDIO_CODEC_4X>;
> > + clock-names = "apb", "codec", "audio-codec-4x";
> > + resets = <&ccu RST_BUS_AUDIO_CODEC>;
> > + dmas = <&dma 6>;
> > + dma-names = "tx";
> > + status = "disabled";
> > + };
> > +
> > gpadc: adc@5070000 {
> > compatible = "allwinner,sun50i-h616-gpadc",
> > "allwinner,sun20i-d1-gpadc";
>
