From mboxrd@z Thu Jan 1 00:00:00 1970 From: maxime.ripard@free-electrons.com (Maxime Ripard) Date: Mon, 24 Oct 2016 14:34:49 +0200 Subject: [linux-sunxi] [PATCH v5 4/7] ASoC: sunxi: Add sun8i I2S driver In-Reply-To: <20161023094503.fbb22dd0701e3f2785828609@free.fr> References: <20161023094503.fbb22dd0701e3f2785828609@free.fr> Message-ID: <20161024123449.npo4grdlbrj4tcsj@lukather> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On Sun, Oct 23, 2016 at 09:45:03AM +0200, Jean-Francois Moine wrote: > On Sun, 23 Oct 2016 09:33:16 +0800 > Chen-Yu Tsai wrote: > > > > Note: This driver is closed to the sun4i-i2s except that: > > > - it handles the H3 > > > > If it's close to sun4i-i2s, you should probably rework that one to support > > the newer SoCs. > > > > > - it creates the sound card (with sun4i-i2s, the sound card is created > > > by the CODECs) > > > > I think this is wrong. I2S is only the DAI. You typically have a separate > > platform driver for the whole card, or just use simple-card. > > An other device is not needed. The layout is simple: > I2S_controller (CPU DAI) <-> HDMI_CODEC (CODEC DAI) > The HDMI CODEC is supported by the HDMI video driver (only one device), > so, it cannot be the card device. > ASoC does not use the CPU DAI device (I2S_controller), so, it is > natural to use it to handle the card. Still, duplicating the driver is not the solution. I agree with Chen-Yu that we want to leverage the driver that is already there. Maxime -- Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: