[alsa-devel] [PATCH 11/20] ASoC: SAMSUNG: I2S: Add bit definitions

Mark Brown broonie at opensource.wolfsonmicro.com
Wed Mar 10 12:00:21 CET 2010


On Wed, Mar 10, 2010 at 04:49:00PM +0900, Jassi Brar wrote:
> Define more bit definitions in the order of mainline support for the
> SoC.

For changes like this it'd be better if the changelog said something
like "Add register bit definitions for S5PC1xx" - this is all mainline
code so it's much clearer to say that this is adding support for a new
SoC.

One other thing I'd suggest is that when constructing a patch series
it'd be better to put the more invasive or controversial changes (like
moving the headers) last.  This makes it easer to apply bits of the
series if there is any controversy.


More information about the Alsa-devel mailing list