Re: [linux-sunxi] Re: [PATCH 2/2] ASoC: sunxi: add support for the on-chip codec on early Allwinner SoCs

2015-09-17 Thread Mark Brown
On Thu, Sep 17, 2015 at 11:10:41PM +0800, Chen-Yu Tsai wrote: > On Thu, Sep 17, 2015 at 9:31 PM, Maxime Ripard > >> > + .formats= SNDRV_PCM_FMTBIT_S16_LE | > >> > + SNDRV_PCM_FMTBIT_S32_LE, > >> > + .sig_bits = 24, > >> So presumably a

Re: [linux-sunxi] Re: [PATCH 2/2] ASoC: sunxi: add support for the on-chip codec on early Allwinner SoCs

2015-09-17 Thread Chen-Yu Tsai
On Thu, Sep 17, 2015 at 9:31 PM, Maxime Ripard wrote: > Hi Mark, > > On Wed, Sep 16, 2015 at 08:16:12PM +0100, Mark Brown wrote: >> On Sat, Sep 12, 2015 at 03:26:24PM +0200, Maxime Ripard wrote: >> >> This looks pretty good, there's a few minor things below but I'll apply >> anyway - please send f

Re: [PATCH 2/2] ASoC: sunxi: add support for the on-chip codec on early Allwinner SoCs

2015-09-17 Thread Maxime Ripard
Hi Mark, On Wed, Sep 16, 2015 at 08:16:12PM +0100, Mark Brown wrote: > On Sat, Sep 12, 2015 at 03:26:24PM +0200, Maxime Ripard wrote: > > This looks pretty good, there's a few minor things below but I'll apply > anyway - please send followup patches fixing these. Sure, I will. Thanks! > > > +

Re: [PATCH 2/2] ASoC: sunxi: add support for the on-chip codec on early Allwinner SoCs

2015-09-16 Thread Mark Brown
On Sat, Sep 12, 2015 at 03:26:24PM +0200, Maxime Ripard wrote: This looks pretty good, there's a few minor things below but I'll apply anyway - please send followup patches fixing these. > + if (clk_set_rate(scodec->clk_module, clk_freq)) > + return -EINVAL; Better to pass back t

[PATCH 2/2] ASoC: sunxi: add support for the on-chip codec on early Allwinner SoCs

2015-09-12 Thread Maxime Ripard
From: Emilio López The sun4i, sun5i and sun7i SoC families have a built-in codec, capable of both audio capture and playback. While this is called a codec by Allwinner, it really is an in-SoC combination of a codec and a DAI, with its own DAC/ADC and amplifiers in a single memory-mapped controll