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

Wolfgang Denk wd at denx.de
Tue Oct 25 09:37:08 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.

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.

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.

_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.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Severe culture shock results when experts from another protocol suite
[...] try to read OSI documents. The term "osified" is used to  refer
to  such  documents. [...] Any relationship to the word "ossified" is
purely intentional.                                - Marshall T. Rose


More information about the U-Boot mailing list