[U-Boot] [PATCH v2 REPOST 2/3] image: Implement IH_TYPE_KERNEL_REL

Wolfgang Denk wd at denx.de
Thu Oct 20 20:56:58 CEST 2011


Dear Stephen Warren,

In message <74CDBE0F657A3D45AFBB94109FB122FF173D51C136 at HQMAIL01.nvidia.com> you wrote:
>
> > > > Please give me a bit more context - how much is "a few 10s of bytes",
> > > > and which exact parts of the code are we talkign about?
> > >
> > > How much? However much adding an extra value into an if check compiles to.
> > 
> > As much as needed so I can see what you mean without having to go
> > through the whole thread.
>
> 'How much' above was a quote from where you were asking 'how much is "a
> few bytes"', not 'how much' explanation you want.

You wrote "a few 10s of bytes" - this is obviously more than "adding
an extra value into an if check compiles to", so I guess you ar enot
talking about doing this just once, but more frequently.  So please be
precise.

> OK then, please just disregard the question and review the patch itself
> as I posted it. Just by reviewing and hopefully accepting the patch,
> you'll see the number of ifdefs and whether they're appropriate.

Please provide this information in advance, before I start looking at
the patch.

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
How many NASA managers does it take to screw in a lightbulb?  "That's
a known problem... don't worry about it."


More information about the U-Boot mailing list