Re: [PATCH 1/3 v5] sound/soc/codecs: add LAPIS Semiconductor ML26124

From: Mark Brown
Date: Thu Dec 22 2011 - 05:50:15 EST


On Thu, Dec 22, 2011 at 05:31:08PM +0900, Tomoya MORINAGA wrote:
> 2011/12/20 Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>:

> > How have you tested this code?

> No, I haven't tested at all. (Of course, I tested before ASoC.)
> I'll test the new these drivers after you say "Roughly, how to use
> ASoC framework is OK"

Don't do this unless you really have to, and if for some reason it's
needed or useful then indicate that this is what you're doing. People
shouldn't be finding issues on review that mean that the code will never
run. If the testing is very light that's fine but you need to make some
effort to verify that things will work.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/