From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Thu, 3 May 2018 16:21:53 -0400 Subject: [U-Boot] [PATCHv2 2/2] stdio_names: Ensure MAX_NAMES is defined before use, don't use 3 directly In-Reply-To: References: <1525353146-21994-1-git-send-email-trini@konsulko.com> <1525353146-21994-2-git-send-email-trini@konsulko.com> Message-ID: <20180503202153.GO9026@bill-the-cat.ec.rr.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, May 03, 2018 at 03:12:47PM +0100, Peter Robinson wrote: > On Thu, May 3, 2018 at 2:12 PM, Tom Rini wrote: > > With tighter build flags the fact that doesn't have a > > reference back to MAX_NAMES causes an error. Include here and > > then in common/console.c use MAX_NAMES rather than 3 when working with > > stdio_names. > > > > Reported-by: Peter Robinson > > Signed-off-by: Tom Rini > > --- > > Changes in v2: > > - New patch > > --- > > common/console.c | 4 ++-- > > include/stdio_dev.h | 1 + > > 2 files changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/common/console.c b/common/console.c > > index 0e0295514b21..f1a5e95c8f39 100644 > > --- a/common/console.c > > +++ b/common/console.c > > @@ -847,7 +847,7 @@ done: > > > > #ifdef CONFIG_SYS_CONSOLE_ENV_OVERWRITE > > /* set the environment variables (will overwrite previous env settings) */ > > - for (i = 0; i < 3; i++) { > > + for (i = 0; i < MAX_NAMES; i++) { > > I think you mean MAX_FILES? With MAX_NAMES it fails, with MAX_FILES it builds. > > With the that change, if it's correct: > Tested-by: Peter Robinson If anyone is wonder if I'll ever learn to not post something I quickly did and was sure it was right, the answer is no. But I don't push those changes out (on purpose) at least! I'll fix it up before merging, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: