[alsa-devel] [Uclinux-dist-devel] [PATCH 2/6] ASoC: Blackfin: new machine driver for ADAU1361 codecs

Mark Brown broonie at opensource.wolfsonmicro.com
Sun Aug 8 00:29:24 CEST 2010


On 7 Aug 2010, at 22:14, Mike Frysinger <vapier.adi at gmail.com> wrote:

> On Sat, Aug 7, 2010 at 17:08, Mark Brown wrote:

>> It'd be really helpful if you could do a bit more pre-review on the stuff you're posting, especially where you've got a bunch of drivers that follow similar patterns and are going to generate exactly the same feedback. Posting a large batch of patches at once isn't a problem but if you're sending additional patches later on it'd be helpful to take into account the feedback that has already been given on prior serieses.
> 

> probably, but i'm not familiar with the ASoC stuff to fix/review any
> of this.  i'll open another tracker item for these codecs.

It'd probably be enough to just avoid posting additional batches of patches without incorporating previous review, either by just posting everything in one series or by holding off on later ones after getting the first batch of review. The reason I mentioned this was that I'd already reviewed previous postings (and seen that you'd read my review) but was noticing many of the same issues cropping up in the new postings.


More information about the Alsa-devel mailing list