public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Peter Chubb <Peter.Chubb@data61.csiro.au>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] ARM: tegra: increase console buffer size and sys args num
Date: Tue, 23 Aug 2016 10:00:22 +1000	[thread overview]
Message-ID: <844m6cwe09.wl-Peter.Chubb@data61.csiro.au> (raw)
In-Reply-To: <80e858d4-a14c-2d08-0934-f26a2a2a7cd8@wwwdotorg.org>


The Linux-for-Tegra kernel uses a very long command line.

The default value of CONFIG_SYS_CBSIZE is too small to print out the
long command line and causes a message like:
  bootarg overflow 602+0+0+1 > 512
on the console, and the board refuses to boot.

The default value of CONFIG_SYS_MAXARGS is too small to add a long
long command line, and the kernel won't boot without the complete
bootargs.

Increasing these two config options solves this problem.

(patch cherry-picked from the NVIDIA u-boot source; original author
Bryan Wu)

Signed-off-by: Bryan Wu <pengw@nvidia.com>
Signed-off-by: Peter Chubb <Peter.Chubb@data61.csiro.au>
---
 include/configs/tegra-common.h | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/include/configs/tegra-common.h b/include/configs/tegra-common.h
index 7b0940a..6bfd7b2 100644
--- a/include/configs/tegra-common.h
+++ b/include/configs/tegra-common.h
@@ -76,11 +76,12 @@
  * Increasing the size of the IO buffer as default nfsargs size is more
  *  than 256 and so it is not possible to edit it
  */
-#define CONFIG_SYS_CBSIZE		(256 * 2) /* Console I/O Buffer Size */
+#define CONFIG_SYS_CBSIZE		(1024 * 2) /* Console I/O Buffer Size */
 /* Print Buffer Size */
 #define CONFIG_SYS_PBSIZE		(CONFIG_SYS_CBSIZE + \
 					sizeof(CONFIG_SYS_PROMPT) + 16)
-#define CONFIG_SYS_MAXARGS		32	/* max number of command args */
+#define CONFIG_SYS_MAXARGS		64	/* max number of command args */
+
 /* Boot Argument Buffer Size */
 #define CONFIG_SYS_BARGSIZE		(CONFIG_SYS_CBSIZE)
 
-- 
2.9.3

-- 
Dr Peter Chubb	       Tel: +61 2 9490 5852      http://www.data61.csiro.au
http://ts.data61.csiro.au      Software Systems Research Group/NICTA/Data61

  reply	other threads:[~2016-08-23  0:00 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-22 22:46 [U-Boot] ARM: tegra: jetson-tk1: Increase console buffer size Peter Chubb
2016-08-22 23:10 ` Stephen Warren
2016-08-23  0:00   ` Peter Chubb [this message]
2016-08-23 16:04     ` [U-Boot] [PATCH] ARM: tegra: increase console buffer size and sys args num Stephen Warren
2016-08-23 22:21       ` Peter Chubb
2016-08-23 22:35         ` Stephen Warren
2016-09-01 22:42   ` [U-Boot] ARM: tegra: jetson-tk1: Increase console buffer size Tom Warren
2016-09-01 22:53     ` Peter.Chubb at data61.csiro.au
2016-09-01 23:19       ` Tom Warren
2016-09-01 23:25       ` Stephen Warren
2016-09-01 23:47         ` Peter.Chubb at data61.csiro.au

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=844m6cwe09.wl-Peter.Chubb@data61.csiro.au \
    --to=peter.chubb@data61.csiro.au \
    --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