linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc
@ 2013-08-31 13:07 Mark Brown
  2013-09-01 14:48 ` Samuel Ortiz
  0 siblings, 1 reply; 5+ messages in thread
From: Mark Brown @ 2013-08-31 13:07 UTC (permalink / raw)
  To: Miguel Aguilar, Liam Girdwood, Lee Jones, Samuel Ortiz
  Cc: davinci-linux-open-source, alsa-devel, linaro-kernel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1127 bytes --]

This patch series removes the use of the ASoC-level read and write
functions from the cq93vc driver as part of a wider push to remove them
completely and just use regmap for all register I/O.  Since the driver
is essentially doing what regmap-mmio is doing this is done by adding a
MMIO regmap to the core device and using that.  This is compile tested
only, I don't have any hardware to run on.

Since I anticipate doing more cleanups over the CODEC drivers during the
next release cycle it'd be good to merge via ASoC, though there
shouldn't be any overlap with the first patch.

Mark Brown (5):
      mfd: davinci_voicecodec: Remove unused read and write functions
      mfd: davinci_voicecodec: Provide a regmap for register I/O
      ASoC: cq93vc: Use core I/O functions
      ASoC: cq93vc: Don't use control data for core driver data
      ASoC: cq93vc: Use regmap for I/O

 drivers/mfd/davinci_voicecodec.c       | 23 +++++++++++--------
 include/linux/mfd/davinci_voicecodec.h |  3 +++
 sound/soc/codecs/cq93vc.c              | 42 +++++++++++-----------------------
 3 files changed, 29 insertions(+), 39 deletions(-)


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc
  2013-08-31 13:07 [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc Mark Brown
@ 2013-09-01 14:48 ` Samuel Ortiz
  2013-09-01 15:08   ` Mark Brown
  0 siblings, 1 reply; 5+ messages in thread
From: Samuel Ortiz @ 2013-09-01 14:48 UTC (permalink / raw)
  To: Mark Brown
  Cc: Miguel Aguilar, Liam Girdwood, Lee Jones,
	davinci-linux-open-source, alsa-devel, linaro-kernel,
	linux-kernel

Hi Mark,

On Sat, Aug 31, 2013 at 02:07:41PM +0100, Mark Brown wrote:
> This patch series removes the use of the ASoC-level read and write
> functions from the cq93vc driver as part of a wider push to remove them
> completely and just use regmap for all register I/O.  Since the driver
> is essentially doing what regmap-mmio is doing this is done by adding a
> MMIO regmap to the core device and using that.  This is compile tested
> only, I don't have any hardware to run on.
> 
> Since I anticipate doing more cleanups over the CODEC drivers during the
> next release cycle it'd be good to merge via ASoC, though there
> shouldn't be any overlap with the first patch.
As I'd prefer to carry the MFD ones (Including the twl6040 one) through
mfd-next, I can build a branch for you to pull from. Would that be fine
with you ?

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc
  2013-09-01 14:48 ` Samuel Ortiz
@ 2013-09-01 15:08   ` Mark Brown
  2013-09-02  8:55     ` Samuel Ortiz
  0 siblings, 1 reply; 5+ messages in thread
From: Mark Brown @ 2013-09-01 15:08 UTC (permalink / raw)
  To: Samuel Ortiz
  Cc: Miguel Aguilar, Liam Girdwood, Lee Jones,
	davinci-linux-open-source, alsa-devel, linaro-kernel,
	linux-kernel

[-- Attachment #1: Type: text/plain, Size: 327 bytes --]

On Sun, Sep 01, 2013 at 04:48:47PM +0200, Samuel Ortiz wrote:

> As I'd prefer to carry the MFD ones (Including the twl6040 one) through
> mfd-next, I can build a branch for you to pull from. Would that be fine
> with you ?

That's fine by me, I just want the patches in ASoC - obviously the
branch couldn't be rebased though.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc
  2013-09-01 15:08   ` Mark Brown
@ 2013-09-02  8:55     ` Samuel Ortiz
  2013-09-02  9:48       ` Mark Brown
  0 siblings, 1 reply; 5+ messages in thread
From: Samuel Ortiz @ 2013-09-02  8:55 UTC (permalink / raw)
  To: Mark Brown
  Cc: Miguel Aguilar, Liam Girdwood, Lee Jones,
	davinci-linux-open-source, alsa-devel, linaro-kernel,
	linux-kernel

Hi Mark,

On Sun, Sep 01, 2013 at 04:08:53PM +0100, Mark Brown wrote:
> On Sun, Sep 01, 2013 at 04:48:47PM +0200, Samuel Ortiz wrote:
> 
> > As I'd prefer to carry the MFD ones (Including the twl6040 one) through
> > mfd-next, I can build a branch for you to pull from. Would that be fine
> > with you ?
> 
> That's fine by me, I just want the patches in ASoC - obviously the
> branch couldn't be rebased though.
I put a topic/for-asoc branch upstream for you to pull from:

The following changes since commit 5ae90d8e467e625e447000cb4335c4db973b1095:

  Linux 3.11-rc3 (2013-07-28 20:53:33 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/sameo/mfd-next.git topic/for-asoc

for you to fetch changes up to c6f39257c952bc7da974bf93255936ff2ece2c34:

  mfd: twl6040: Use regmap for register cache (2013-09-02 10:30:14
+0200)

----------------------------------------------------------------
Mark Brown (3):
      mfd: davinci_voicecodec: Remove unused read and write functions
      mfd: davinci_voicecodec: Provide a regmap for register I/O
      mfd: twl6040: Use regmap for register cache

 drivers/mfd/davinci_voicecodec.c       |   23 +++++++++++++----------
 drivers/mfd/twl6040.c                  |   43 ++++++++++++++++++++++++++++++-------------
 include/linux/mfd/davinci_voicecodec.h |    3 +++
 include/linux/mfd/twl6040.h            |    1 -
 4 files changed, 46 insertions(+), 24 deletions(-)

-- 
Intel Open Source Technology Centre
http://oss.intel.com/

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc
  2013-09-02  8:55     ` Samuel Ortiz
@ 2013-09-02  9:48       ` Mark Brown
  0 siblings, 0 replies; 5+ messages in thread
From: Mark Brown @ 2013-09-02  9:48 UTC (permalink / raw)
  To: Samuel Ortiz
  Cc: Miguel Aguilar, Liam Girdwood, Lee Jones,
	davinci-linux-open-source, alsa-devel, linaro-kernel,
	linux-kernel

[-- Attachment #1: Type: text/plain, Size: 142 bytes --]

On Mon, Sep 02, 2013 at 10:55:34AM +0200, Samuel Ortiz wrote:

> I put a topic/for-asoc branch upstream for you to pull from:

Great, thanks!

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-09-02  9:49 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-31 13:07 [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc Mark Brown
2013-09-01 14:48 ` Samuel Ortiz
2013-09-01 15:08   ` Mark Brown
2013-09-02  8:55     ` Samuel Ortiz
2013-09-02  9:48       ` Mark Brown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).