[U-Boot] [PATCH 01/39] DEBUG: Fix debug macros

Marek Vasut marek.vasut at gmail.com
Tue Oct 25 10:27:10 CEST 2011


> Dear Marek Vasut,
> 
> In message <201110250211.07058.marek.vasut at gmail.com> you wrote:
> > I know this very well, I just need to feedback and the patchset is really
> > huge.
> 
> What you attempt will not work.
> 
> You may get some feedback on patches posted on the list.
> 
> I will_never_ send any feedback to any stuff in some repository
> anywhere. I will never even try to read it.  And I m pretty sure many
> others work like that, too.

I expect that, all right.
> 
> If the patch set is huge, then do yourself a favour and break it down
> into small, managable pieces.   Don't try to do everything at once.

Well I'm not quite sure how you'd like to break a set of patches like this one 
into smaller chunks ... ?
> 
> Send a short sub-set first - maybe some 4...8 patches or so.  Maybe
> even 10.  Then let people digest these, and only when they are
> scheduled for merging start and post the next of your batches.

Well ... it's insane. Can you start picking patches from this one series which 
you feel are ok? I'll rebase the rest on that and submit it. The patches are 
pretty much independent anyway.
> 
> _Nobody_ will review a 50 patch series.  And nobdy will apply it
> either.
> 
> > I'd wait for you to merge some of the stuff from this patchset (you can
> > merge what you feel fitting but the 0001 patch ...) and then rebase and
> > resubmit the rest on top of that.
> 
> I'd rather you split the series in smaller, digestable batches.

Well see above. Can you cherrypick something from this series already which you 
feel fitting ? There's no dependency anyway.

> 
> Best regards,
> 
> Wolfgang Denk

Cheers


More information about the U-Boot mailing list