Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-28 Thread Mark Brown
On Fri, Feb 28, 2014 at 09:15:37AM +0100, Lars-Peter Clausen wrote: > Yes, I think that's almost all of them. si476x is missing, but I > think that one is currently broken, as it doesn't call > snd_soc_codec_set_cache_io() at all. Probably, yeah - there were other problems with that driver that m

Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-28 Thread Lars-Peter Clausen
On 02/28/2014 08:50 AM, li.xi...@freescale.com wrote: A quick grep reveals: mc13783.c: codec->control_data = dev_get_regmap(codec->dev->parent, NULL); si476x.c: codec->control_data = dev_get_regmap(codec->dev->parent, NULL); wm5102.c: codec->control_data = priv->core.arizona->

RE: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread li.xi...@freescale.com
> > A quick grep reveals: > mc13783.c:codec->control_data = dev_get_regmap(codec->dev->parent, NULL); > si476x.c: codec->control_data = dev_get_regmap(codec->dev->parent, NULL); > wm5102.c: codec->control_data = priv->core.arizona->regmap; > wm5110.c: codec->control_data = priv->c

RE: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread li.xi...@freescale.com
> > A quick grep reveals: > mc13783.c:codec->control_data = dev_get_regmap(codec->dev->parent, NULL); > si476x.c: codec->control_data = dev_get_regmap(codec->dev->parent, NULL); > wm5102.c: codec->control_data = priv->core.arizona->regmap; > wm5110.c: codec->control_data = priv->c

Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread Lars-Peter Clausen
On 02/28/2014 08:08 AM, li.xi...@freescale.com wrote: Subject: Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. On Fri, Feb 28, 2014 at 04:00:38AM +, li.xi...@freescale.com wrote: I'll send another patches to applied to use this for another CODEC drivers. And there a

RE: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread li.xi...@freescale.com
> >>>> Subject: Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. > >>>> > >>>> On Fri, Feb 28, 2014 at 04:00:38AM +, li.xi...@freescale.com wrote: > >>>> > >>>>> I'll send another patches to

Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread Lars-Peter Clausen
On 02/28/2014 07:40 AM, li.xi...@freescale.com wrote: Subject: Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. On 02/28/2014 06:56 AM, li.xi...@freescale.com wrote: Subject: Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. On Fri, Feb 28, 2014 at 04

RE: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread li.xi...@freescale.com
> Subject: Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try > regmap. > > On 02/28/2014 06:56 AM, li.xi...@freescale.com wrote: > >> Subject: Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. > >> > >> On Fri,

Re: [alsa-devel] [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread Lars-Peter Clausen
On 02/28/2014 06:56 AM, li.xi...@freescale.com wrote: Subject: Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. On Fri, Feb 28, 2014 at 04:00:38AM +, li.xi...@freescale.com wrote: I'll send another patches to applied to use this for another CODEC drivers. And there almo

RE: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread li.xi...@freescale.com
> Subject: Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. > > On Fri, Feb 28, 2014 at 04:00:38AM +, li.xi...@freescale.com wrote: > > > I'll send another patches to applied to use this for another CODEC drivers. > > And there almost 80 files, Sh

Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread Mark Brown
On Fri, Feb 28, 2014 at 04:00:38AM +, li.xi...@freescale.com wrote: > I'll send another patches to applied to use this for another CODEC drivers. > And there almost 80 files, Should I send them in one patch or split them into > individual patch for each CODEC driver ? I'd suggest doing one pa

RE: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread li.xi...@freescale.com
> Subject: Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap. > > On Thu, Feb 27, 2014 at 05:49:52PM +0800, Xiubo Li wrote: > > For most CODEC drivers which the REGMAP is used, the soc_probe_codec() > > will do the stuff work of snd_soc_codec_set_cache_

Re: [PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread Mark Brown
On Thu, Feb 27, 2014 at 05:49:52PM +0800, Xiubo Li wrote: > For most CODEC drivers which the REGMAP is used, the soc_probe_codec() > will do the stuff work of snd_soc_codec_set_cache_io(), which the CODEC > drivers' ASoC probe will do too, and almost at the same time. Applied, thanks. I did a che

[PATCH 2/3] ASoC: core: Set the default I/O up try regmap.

2014-02-27 Thread Xiubo Li
For most CODEC drivers which the REGMAP is used, the soc_probe_codec() will do the stuff work of snd_soc_codec_set_cache_io(), which the CODEC drivers' ASoC probe will do too, and almost at the same time. This patch set the default I/O up try regmap, and then the CODEC drivers' stuff work of snd_s