[U-Boot] Interrupt issue about bootvx command

Wolfgang Denk wd at denx.de
Thu Oct 13 22:20:34 CEST 2011


Dear Peter Pan,

please stop top posting / full quoting.


In message <CAGQXtZFp6oE_BjQGd6SSDAYk7kkg4-dy_1==LNK2Snh5h1Gfmg at mail.gmail.com> you wrote:
> 
> And the following is my updated patch. My git does not have "git
> send-mail" supported, so I have to paste the formatted commit below.
> Sorry for that.

Well, you'd better spend the time installing a full-featured version
of git.

> From 9758bc34e8459c8b906029105e41fe8adf9e79ca Mon Sep 17 00:00:00 2001
> From: Ke Pan <pppeterpppan at gmail.com>
> Date: Thu, 13 Oct 2011 10:34:23 +0800
> Subject: [PATCH] common/cmd_elf : Disable interrupts before boot vxWorks
> 
> The vxWorks needs all interrupts to be disabled before its boot.

Hm... And what exactly was not clear about the following statement:

> > Checkpatch says: total: 2 errors, 3 warnings, 10 lines checked
> >
> > Please see http://www.denx.de/wiki/U-Boot/Patches ; then clean up and
> > resubmit.

- Your commit message is mixed with text that belongs to the comment
  section only.

- Your SoB line is still missing

- There is a typo in "Disableing"; actually you should get rid of this
  printf().



Are there any testers out there?  I don't want to take this untested,
and I don't have any VxWorks anywhere around... (nor any other
proprietary code :-)

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
I'd rather be led to hell than managed to heaven.


More information about the U-Boot mailing list