[alsa-devel] [PATCH v2 2/6] ASoC: ux500: Do not clear state if already idle

Mark Brown broonie at kernel.org
Thu May 9 11:28:25 CEST 2013


On Wed, May 08, 2013 at 03:06:18PM +0100, Lee Jones wrote:

> Bringing this back on subject, whilst trying not to drag this out for
> longer than we have to. I think replying to one's first patch with a
> subsequent version has its merits. And as long as the thread hasn't
> become too overgrown I see it as a useful tool to obtain an Ack or
> further comment easily and without churn overhead. This also aids in
> your quest to save maintainers from reviewing the same code
> repeatedly, as once the Ack is in place it's easy to see which patches
> are in need of further review and which ones can be skipped.

If you want to do that what often works better is to post an incremental
diff rather than the full patch to show exactly what's been done in the
fix.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20130509/b9e26acc/attachment.sig>


More information about the Alsa-devel mailing list