[U-Boot] [PATCH] spl: avoid printf when we might be doing Ymodem with CONFIG_SPL_FIT_SIGNATURE

Marek Vasut marex at denx.de
Thu Mar 21 00:45:43 UTC 2019


On 3/20/19 8:52 PM, Alex Kiernan wrote:
> If CONFIG_SPL_FIT_SIGNATURE is enabled and U-Boot is being loaded from UART
> over Ymodem then we can't emit messages using printf() without causing
> errors like:
> 
>   Sending: u-boot-dtb.img
>   Ymodem sectors/kbytes sent: 3009/376kRetry 0: Got 23 for sector ACK
>   Retry 0: NAK on sector
>   Retry 0: Got 68 for sector ACK
>   Retry 0: NAK on sector
>   Ymodem sectors/kbytes sent: 3273/409kRetry 0: Got 23 for sector ACK
>   Retry 0: NAK on sector
>   Retry 0: Got 68 for sector ACK
>   Retry 0: NAK on sector
> 
> Use debug() rather than printf() to avoid sending messages on the serial
> port.
> 
> Signed-off-by: Alex Kiernan <alex.kiernan at gmail.com>
> ---
> 
>  common/spl/spl_fit.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/common/spl/spl_fit.c b/common/spl/spl_fit.c
> index db436268cbcd..08faf2c1b058 100644
> --- a/common/spl/spl_fit.c
> +++ b/common/spl/spl_fit.c
> @@ -240,12 +240,12 @@ static int spl_load_fit_image(struct spl_load_info *info, ulong sector,
>  	}
>  
>  #ifdef CONFIG_SPL_FIT_SIGNATURE
> -	printf("## Checking hash(es) for Image %s ... ",
> -	       fit_get_name(fit, node, NULL));
> +	debug("## Checking hash(es) for Image %s ... ",
> +	      fit_get_name(fit, node, NULL));
>  	if (!fit_image_verify_with_data(fit, node,
>  					 src, length))
>  		return -EPERM;
> -	puts("OK\n");
> +	debug("OK\n");
>  #endif

Checking printf("Signature failed") to debug() is probably not what you
want to do. It'd be hiding some pretty important output.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list