Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Aidan MacDonald <aidanmacdonald.0x0@gmail.com>
To: paul@crapouillou.net
Cc: alsa-devel@alsa-project.org, cbranchereau@gmail.com
Subject: Re: Questions about driver implementation (1 I2S controller to 2 codecs)
Date: Sat, 25 Jun 2022 21:19:58 +0100	[thread overview]
Message-ID: <YDzzGSWPBbtkGDRbdAaCsnxpIqqDpyJe@localhost> (raw)
In-Reply-To: <1310ER.AOPVK6422Q8P1@crapouillou.net>


Paul Cercueil <paul@crapouillou.net> writes:

> Hi,
>
> I'm trying to add support for a new board that uses the Ingenic JZ4770 
> SoC.
>
> This SoC has a I2S/AC97 controller, and has an internal codec. The 
> controller has a register switch to select either the internal codec, 
> or an external codec; both cannot be enabled at the same time.
>
> On this board, the external speakers / audio line are wired to the 
> internal codec, while a HDMI chip (ITE66121) is wired as the external 
> codec.
>
> I'm having a hard time trying to figure out how the codec selection 
> switch should be exported. Should it be a regular widget, and the user 
> is responsible for selecting the right codec? Should it be a DAPM, and 
> plugging the HDMI cable auto-enables the switch? Is this configuration 
> (one controller to two codecs) already somehow supported by ALSA?
>
> Right now we're using a "simple-audio-card" in the device tree, and 
> trying to add HDMI sound support. I am not even sure if we should be 
> using one sound card with the two codecs, or one sound card per codec, 
> sharing the same I2S controller?
>
> Any thoughts?
>
> Thanks,
> -Paul

I'm also interested in this, although I'm using a different Ingenic SoC
(also using jz4740-i2s). In my case I have a microphone connected to
the internal codec and an external codec for playback.

Some thoughts: maybe DPCM is what we want? There's three DAIs: the CPU,
the internal codec, and the external codec. The I2S controller selects
which codec DAI the CPU DAI is connected to. DPCM seems to be the only
thing in ALSA that deals with dynamic DAI <-> DAI connections. I can't
figure it out though; the documentation is too sparse and DPCM-enabled
drivers are complicated beasts.

Codec selection should probably be left up to userspace though. ALSA
isn't really in a position to make routing decisions like that, there
is no one-size-fits-all approach that'd be suitable for every board.

What ALSA _can_ do is respond to the user's audio routing choice and
look at the machine state (jacks/connectors, muxes, mixers, etc), and
power things up or down as appropriate.

Regards,
Aidan

      reply	other threads:[~2022-06-25 20:19 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-24 20:33 Questions about driver implementation (1 I2S controller to 2 codecs) Paul Cercueil
2022-06-25 20:19 ` Aidan MacDonald [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YDzzGSWPBbtkGDRbdAaCsnxpIqqDpyJe@localhost \
    --to=aidanmacdonald.0x0@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=cbranchereau@gmail.com \
    --cc=paul@crapouillou.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox