[alsa-devel] [PATCH 11/20] ASoC: SAMSUNG: I2S: Add bit definitions
broonie at opensource.wolfsonmicro.com
Wed Mar 10 13:38:11 CET 2010
On Wed, Mar 10, 2010 at 09:24:45PM +0900, jassi brar wrote:
> On Wed, Mar 10, 2010 at 8:00 PM, Mark Brown
> > 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.
> I thought I already made that sure.
> In my opinion, the only controversial patch was 'header-copying' and I couldn't
> move that any further down the series or other changes without that.
I agree that this is the only really controversial change, but it seemed
like pretty much all the changes that added stuff to the headers or used
them could've been done pre-move so that they didn't depend on it.
More information about the Alsa-devel