[U-Boot-Users] [PATCH 1/11] Fix miscellaneous typos
Wolfgang Denk
wd at denx.de
Sat May 6 00:49:03 CEST 2006
Dear Keith,
I'm a bit unhappy about your patches.
First, I don't know how to make heads or tails out of it.
In this subject you write "PATCH 1/11" which looks as if this were a
couple of related patches coming in 11 parts, but actually all of
them seem to be completely independent?
Then, I received patches 5/11, 8/11, 2/11, 11/11, 6/11, 1/11, 9/11,
and 4/11. What about parts 3, 5, 7, 9, and 10?
In message <OFBEDA0037.AED1556F-ON07257165.005DED58-07257165.005E506A at mck.us.ray.com> you wrote:
> This patch fixes a few typos. No CHANGELOG entry due to trivial nature of
> patch
Where in the README does it say that a CHANGELOG entry is not
necessary for simple patches? I need this for each and every patch.
Then, you are violating the Coding Style requirements in a couple of
places (I didn't check carefully as I was not able to apply the
patches [see below], but I've seen at least indentation with spaces
instead of the required TABs, indentation not by multiples of 8, and
trailing white space.
Then you add nasty editor commands to your sources, like this one:
/* vim: set ts=4 tw=80 sw=4 cindent fo=tcroq: */
All of this is bad, but this one is fatal and forces me to reject all
your patches as they are not usable: your patches are line-wrapped
which makes it impossible to apply them; for example, in [PATCH 8/11]
Add support for environment storage in SystemACE:
...
+- CFG_ENV_IS_IN_SYSACE:
+
+ Define this if you have a Compact Flash (CF) card connected to a
+ Xilinx SystemACE controller in which you wish to store the
environment.
+
+ - CFG_SYSTEMACE_ENV_DEV:
+ The device (instance) number of the SystemACE device you wich to
use.
+ This is normally zero for a system with a single SystemACE.
+
+ - CFG_ENV_SIZE:
+ The size of the environment in bytes. Note that presently this
must
...
Please clean up and resubmit, but please the rules as documetned in
the README. And make sure to configure your mailer correctly so that
it doesn't corrupt your postings.
All patches rejected. Sorry.
Best regards,
Wolfgang Denk
--
Software Engineering: Embedded and Realtime Systems, Embedded Linux
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
In C we had to code our own bugs, in C++ we can inherit them.
More information about the U-Boot
mailing list