[PATCH 3/7] xyz-modem: Put xyzModem_stream_close debug diagnostic message into ZM_DEBUG()

Heinrich Schuchardt xypron.glpk at gmx.de
Wed Aug 4 11:30:25 CEST 2021



On 03.08.21 16:28, Pali Rohár wrote:
> This is how all other debug / diagnostic messages are handled.

This commit message is misleading. What you do is hide an output.

>
> Signed-off-by: Pali Rohár <pali at kernel.org>
> ---
>   common/xyzModem.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/common/xyzModem.c b/common/xyzModem.c
> index 631c44e11adf..c200c9ff9177 100644
> --- a/common/xyzModem.c
> +++ b/common/xyzModem.c
> @@ -608,10 +608,10 @@ xyzModem_stream_read (char *buf, int size, int *err)
>   void
>   xyzModem_stream_close (int *err)
>   {
> -  diag_printf
> +  ZM_DEBUG (zm_dprintf
>       ("xyzModem - %s mode, %d(SOH)/%d(STX)/%d(CAN) packets, %d retries\n",
>        xyz.crc_mode ? "CRC" : "Cksum", xyz.total_SOH, xyz.total_STX,
> -     xyz.total_CAN, xyz.total_retries);
> +     xyz.total_CAN, xyz.total_retries));
>     ZM_DEBUG (zm_flush ());

zm_dprintf prints into the debug buffer which seems not to be overflow
protected.

diag_printf prints to the console.

Why do you want to hide this message?
Isn't the CRC what the user will want to check?
And if many retries occur doesn't this alert the user that the transfer
may be corrupted?

Best regards

Heinrich

>   }
>
>


More information about the U-Boot mailing list