2013-08-31 13:07:53

by Mark Brown

[permalink] [raw]
Subject: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc

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(-)


Attachments:
(No filename) (1.10 kB)
signature.asc (836.00 B)
Digital signature
Download all attachments

2013-09-01 14:48:55

by Samuel Ortiz

[permalink] [raw]
Subject: Re: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc

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/

2013-09-01 15:09:06

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc

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.


Attachments:
(No filename) (327.00 B)
signature.asc (836.00 B)
Digital signature
Download all attachments

2013-09-02 08:55:58

by Samuel Ortiz

[permalink] [raw]
Subject: Re: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc

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/

2013-09-02 09:49:17

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 0/5] Remove ASoC-level I/O functions from cq93vc

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!


Attachments:
(No filename) (142.00 B)
signature.asc (836.00 B)
Digital signature
Download all attachments