[alsa-devel] [PATCH 1/3] new ad1938 codec driver based on asoc

Mark Brown broonie at opensource.wolfsonmicro.com
Thu Jul 16 12:09:27 CEST 2009


On Thu, Jul 16, 2009 at 04:00:05PM +0800, Barry Song wrote:
> Signed-off-by: Barry Song <21cnbao at gmail.com>

This is fine from a code point of view but there are a *lot* of
tab/space problems in the code.  I'd suggest running checkpatch over
your patches before sending them to make sure you catch issues like
that.  Also, the patch was against the ASoC dev branch rather than ALSA
git - the most current mailine ALSA code is in the topic/asoc branch of:

  git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git

Please submit patches againt that or my current queue which will be in:

  git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound-2.6.git

Since the code is all good I've applied the patch as-is but a followup
fixing the tab/space stuff would be appreciated.


More information about the Alsa-devel mailing list