public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Kim Phillips <kim.phillips@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] powerpc/85xx: corenet_ds: increase console buffer size to 1024
Date: Mon, 26 Sep 2011 11:27:56 -0500	[thread overview]
Message-ID: <20110926112756.bb93d41b.kim.phillips@freescale.com> (raw)
In-Reply-To: <20110925201134.0975D1407995@gemini.denx.de>

On Sun, 25 Sep 2011 22:11:34 +0200
Wolfgang Denk <wd@denx.de> wrote:

> Dear Kim Phillips,
> 
> In message <20110923140020.1eac241b.kim.phillips@freescale.com> you wrote:
> >
> > > Can you explain this in a bit more detail.
> > 
> > if, for whatever reason, the nfsboot env var has been
> > modified/deleted, one can't restore it via the command line
> > because it doesn't fit in the buffer.  Currently command-line typing
> > stops 2 chars short of its full definition, i.e,. at the 2nd 'd' in
> > 'fdtaddr':
> > 
> > => setenv nfsboot 'setenv bootargs root=/dev/nfs rw nfsroot=$serverip:$rootpath ip=$ipaddr:$serverip:$gatewayip:$netmask:$hostname:$netdev:off console=$consoledev,$baudrate $othbootargs;tftp $loadaddr $bootfile;tftp $fdtaddr $fdtfile;bootm $loadaddr - $
> > fdtadd
> 
> Instead of extending buffer sizes just to be able to enter some
> incomrephensible looooong variable setting you should try and figure

this variable (which I have been comprehending for years now), is
part of the default environment configuration for that board, and
many others - it's defined in the board config file.

> out how to structure environment settings, so they become easier to
> understand and to manage to the end user.

with the amount of boardfarm users we have (recall the 'my_boot' and
'my_dtb' env vars in the grepenv rationale?), it's unrealistic to
expect all to agree on how they want to set up and use their
environment.  

We need to enable reverting an env var to its original default
definition.

> The manual holds some pretty edicative examples.

Perhaps over time the nfsboot norm setting should be migrated to
something more modular in the board config files, but right now,
users are complaining about simply expecting to being able to type
two more characters on the command line.

Kim

  reply	other threads:[~2011-09-26 16:27 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-22 16:38 [U-Boot] [PATCH] powerpc/85xx: corenet_ds: increase console buffer size to 1024 Kim Phillips
2011-09-23 18:22 ` Kumar Gala
2011-09-23 19:00   ` Kim Phillips
2011-09-25 20:11     ` Wolfgang Denk
2011-09-26 16:27       ` Kim Phillips [this message]
2011-09-26 18:09         ` Wolfgang Denk
2011-09-26 21:11           ` Scott Wood
2011-09-26 22:15             ` Brian Grayson
2011-09-26 22:55             ` Kim Phillips
2011-09-27  9:45             ` Wolfgang Denk
2011-09-27 18:26               ` Scott Wood
2011-09-28 20:57                 ` Wolfgang Denk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110926112756.bb93d41b.kim.phillips@freescale.com \
    --to=kim.phillips@freescale.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox