Hi, On Mon, Oct 15, 2018 at 08:02:30PM -0700, Vasily Khoruzhick wrote: > This series adds Allwinner A64 audiocodec support into sun4i-i2s, > sun8i-codec drivers, introduces new sun50i-codec-analog driver and enables > sound on Pine64, SoPine boards and Pinebook. > > I2S for audiocodec in A64 is different from other 3 I2S modules but > similar to one in A10, digital part of codec is compatible with A33 and > analog controls part is completely different from other SoCs - it shares > only few bits in few registers, so adding support for it into existing > sun8i-codec-analog would mean duplicating all the widgets, controls and > some routes and making it hard to read. Therefore it makes sense to > introduce new driver. > > v2: - Use simple-amplifier for speaker amp on Pinebook > - Rename sun50i-a64-i2s to sun50i-a64-codec-i2s to preserve compatible > string for other 3 I2S modules in A64 in case if there's any > incompatibility with H3 > v3: - renamed sunxi-adda-pr-regmap to sun8i-adda-pr-regmap > - use ilog2() to calculate reg value for LRCK div instead of using a > table > v4: - dts: don't use 'Mic' and 'Headset Mic' widgets from sun8i-codec, > define our board-level widgets instead. You also need to collect the tags that are given by the various people involved in that review :) Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
