[U-Boot] [PATCH v2 0/2] Deprecate Linux bootargs munging with CONFIG_SILENT_CONSOLE.

Wolfgang Denk wd at denx.de
Tue Jan 10 23:48:33 CET 2012


Dear Doug Anderson,

In message <CAD=FV=W+GErKZ2LCeUoYrsWgVFg2QX9EsH3rjJvj7KQ6tjdbVA at mail.gmail.com> you wrote:
> 
> > Please provide a link.
> 
> Sure!  This is the message I'm referring to:
> <http://lists.denx.de/pipermail/u-boot/2011-October/106255.html>.  I

You should do so in the commit mesage (but judging from my current
state of mind you can save the efforts for a v4).

> know it was a long time ago--I got a bit sidetracked for a while.
> Specifically, I offered to provide a very simple version of
> fixup_silent_linux() that just fixed the buffer overrun and you said
> "Please consider it NAKed".

I just replied to one of your earlier patches that did not fix the
overflow, but just shifted it to happen later, in Linux.
> 
> >> "bootm" to mangle the Linux bootargs is not a suggested way to go.
> >
> > So what is "a suggested way to go" to silence the Linux kenrel
> > messages?
> 
> Please see doc/README.silent in part 1 of this patchset for a sample script.

But this does the very same, just in a different way (requiring more
code).  And it does not prevent the overflow at all.

> This change doesn't remove any functionality but instead deprecates it
> and provides a path for boards to move forward to a non-deprecated
> solution.  The non-deprecated solution is documented in
> doc/README.silent.

Your suggestion does not solve anything.

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 shortest unit of time in the multiverse is the News York  Second,
defined  as  the  period  of  time between the traffic lights turning
green and the cab behind you honking.
                                - Terry Pratchett, _Lords and Ladies_


More information about the U-Boot mailing list