[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