From mboxrd@z Thu Jan 1 00:00:00 1970 From: Josh Wu Date: Mon, 19 May 2014 14:18:58 +0800 Subject: [U-Boot] [PATCH] mx6sabre_common: Avoid console hang due to small CONFIG_SYS_PBSIZE In-Reply-To: <1400464963-5578-1-git-send-email-festevam@gmail.com> References: <1400464963-5578-1-git-send-email-festevam@gmail.com> Message-ID: <5379A252.4080502@atmel.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi, Dear Fabio On 5/19/2014 10:02 AM, Fabio Estevam wrote: > From: Fabio Estevam > > Entering the maximum number of characters defined by CONFIG_SYS_CBSIZE into > the console and hitting enter afterwards, causes a hang in the system because > CONFIG_SYS_PBSIZE is not capable of storing the characters of the error message: > "Unknown command '' - try 'help'" > > Increase the size of CONFIG_SYS_PBSIZE so that it can store the error message > and allows the error message to be printed correctly with no hang. > > Signed-off-by: Fabio Estevam > --- > This problem affects many boards. This hang up can be reproduced in at91sam9m10g45ek board. > > Maybe a global fix is to add a CONFIG_SYS_PBSIZE entry into config_fallbacks > and remove its definition from the board config files when it is defined as: I think this way is a really fix for the issue. So it's better to generate such a patch to fix it for many boards in U-Boot. Best Regards, Josh Wu > #define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE + sizeof(CONFIG_SYS_PROMPT) + 16) > > or even when it is equal to CONFIG_SYS_CBSIZE. > > include/configs/mx6sabre_common.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/configs/mx6sabre_common.h b/include/configs/mx6sabre_common.h > index e59a3b4..6017a20 100644 > --- a/include/configs/mx6sabre_common.h > +++ b/include/configs/mx6sabre_common.h > @@ -217,7 +217,7 @@ > #define CONFIG_SYS_CBSIZE 256 > > /* Print Buffer Size */ > -#define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE + sizeof(CONFIG_SYS_PROMPT) + 16) > +#define CONFIG_SYS_PBSIZE 384 > #define CONFIG_SYS_MAXARGS 16 > #define CONFIG_SYS_BARGSIZE CONFIG_SYS_CBSIZE >