[U-Boot] [PATCH 5/7] JFFS2: Change DEFAULT_EMPTY_SCAN_SIZE to 256 Bytes

Wolfgang Denk wd at denx.de
Sat Apr 30 09:34:16 CEST 2011


Dear Baidu Liu,

In message <BANLkTin7REQN=pv1yTHLFOWrqToj2pafaw at mail.gmail.com> you wrote:
> 
> > A patch description should include _all_ changes.  Either you also put
> > that change into the patch description or you make a separate change.
> > Doing changes not described in the commit log are a good sign to show
> > that one does not care what one does.
> 
> Yes, you are right. Add all the description in the commit log.
> But please firstly make this principle be applied for all guys
> submiting patch. Not just to the people rarely summitint patch.
> Be fair.

We try to be.  We realy strive hard to apply the same, consequent
policy to all submitters. You may be "lucky" and have a poor patch
slip though because nobody foundenough time for a thorough review, but
this is the exception. Or should be.  In any case, when a (new) problem
gets noted, it has to be fixed.

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
The night sky over the planet Krikkit is the least interesting  sight
in the entire Universe.
                 - Douglas Adams _Life, the Universe, and Everything_


More information about the U-Boot mailing list