On Wed, 19 Oct 2016 07:50:08 +0200, Kuninori Morimoto wrote:
Hi Mark, Lars-Peter
These are part of "replace snd_soc_read/write to snd_soc_component_read/write" patch-set. Full-patch-set will be over 100 patches.
[0/x] - [x-1/x] : ASoC: use snd_soc_component_read/write on xxxx [x/x] : ASoC: remove snd_soc_component_read/write
[0/x] - [x-1/x] are almost same patches. so, I pickuped few of them for reviewing to avoid patch flood on ML.
If these review were OK, I will post full-patch-set, or send git-pull-request.
Main purpose of these patches are replace current snd_soc_read/write to snd_soc_component_read/write to remove codec related function.
I really don't see any big merits by these changes. What's wrong with keeping as is? The driver is accessing the codec register, after all.
For keeping consistency, replacing a few exceptions would be OK. But replacing hundreds of callers needs a proper justification to do it so.
thanks,
Takashi