[U-Boot] [PATCH v5 02/13] efi: Init the 'rows' and 'cols' variables
Heinrich Schuchardt
xypron.glpk at gmx.de
Tue Jun 12 05:41:19 UTC 2018
On 06/12/2018 07:26 AM, Simon Glass wrote:
> The current code causes a compiler error on gcc 4.8.4 as used by sandbox
> on Ubuntu 14.04, which is fairly recent. Init these variables to fix the
> problem.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
Is this needed after Alex's
http://git.denx.de/?p=u-boot.git;a=commitdiff;h=80483b2ab62ca7cd200db445b6920ee96d17df88
?
Best regards
Heinrich
> ---
>
> Changes in v5: None
> Changes in v4:
> - Move the fix to query_console_serial()
>
> Changes in v3:
> - Add new patch to init the 'rows' and 'cols' variables
>
> Changes in v2: None
>
> lib/efi_loader/efi_console.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/lib/efi_loader/efi_console.c b/lib/efi_loader/efi_console.c
> index ce66c935ec..bd953a1485 100644
> --- a/lib/efi_loader/efi_console.c
> +++ b/lib/efi_loader/efi_console.c
> @@ -204,8 +204,11 @@ static int query_console_serial(int *rows, int *cols)
> return -1;
>
> /* Read {depth,rows,cols} */
> - if (term_read_reply(n, 3, 't'))
> + if (term_read_reply(n, 3, 't')) {
> + *rows = -1;
> + *cols = -1;
> return -1;
> + }
>
> *cols = n[2];
> *rows = n[1];
>
More information about the U-Boot
mailing list