From: Patrice CHOTARD <patrice.chotard@foss.st.com>
To: Patrick Delaunay <patrick.delaunay@foss.st.com>, <u-boot@lists.denx.de>
Cc: Bin Meng <bmeng.cn@gmail.com>,
Patrick Delaunay <patrick.delaunay@st.com>,
Simon Glass <sjg@chromium.org>,
U-Boot STM32 <uboot-stm32@st-md-mailman.stormreply.com>
Subject: Re: [PATCH 08/10] stm32mp: stm32prog: change one message level to debug
Date: Fri, 28 May 2021 14:54:29 +0200 [thread overview]
Message-ID: <398944f2-fcf8-ab6c-6d35-c4fb4f9588ae@foss.st.com> (raw)
In-Reply-To: <20210518151206.8.I356522a80eec573678c8a84e06f1dbba4b380f29@changeid>
Hi Patrick
On 5/18/21 3:12 PM, Patrick Delaunay wrote:
> Move the message "Invalid or missing layout file."
> to debug level as it is a normal behavior and not an error
> and add the missing '\n'.
>
> This patch avoids the strange trace :
> Boot over usb0!
> Invalid or missing layout file.DFU alt info setting: done
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
> ---
>
> arch/arm/mach-stm32mp/cmd_stm32prog/cmd_stm32prog.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-stm32mp/cmd_stm32prog/cmd_stm32prog.c b/arch/arm/mach-stm32mp/cmd_stm32prog/cmd_stm32prog.c
> index f3790f900f..feff73c79e 100644
> --- a/arch/arm/mach-stm32mp/cmd_stm32prog/cmd_stm32prog.c
> +++ b/arch/arm/mach-stm32mp/cmd_stm32prog/cmd_stm32prog.c
> @@ -99,7 +99,7 @@ static int do_stm32prog(struct cmd_tbl *cmdtp, int flag, int argc,
>
> ret = stm32prog_init(data, addr, size);
> if (ret)
> - printf("Invalid or missing layout file.");
> + log_debug("Invalid or missing layout file at 0x%lx.\n", addr);
>
> /* prepare DFU for device read/write */
> ret = stm32prog_dfu_init(data);
>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Thanks
Patrice
next prev parent reply other threads:[~2021-05-28 12:54 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-18 13:12 [PATCH 00/10] stm32mp: stm32prog: several features and fixes Patrick Delaunay
2021-05-18 13:12 ` [PATCH 01/10] stm32mp: stm32prog: remove all the header check for UART download Patrick Delaunay
2021-05-28 12:53 ` Patrice CHOTARD
2021-06-18 7:57 ` [Uboot-stm32] " Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 02/10] stm32mp: stm32prog: add timeout in stm32prog_serial_get_buffer Patrick Delaunay
2021-05-28 12:53 ` Patrice CHOTARD
2021-06-18 7:57 ` [Uboot-stm32] " Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 03/10] stm32mp: stm32prog: add support of initrd in flashlayout Patrick Delaunay
2021-05-28 12:53 ` Patrice CHOTARD
2021-06-18 7:57 ` [Uboot-stm32] " Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 04/10] stm32mp: stm32prog: solve compilation with CONFIG_FIT_SIGNATURE Patrick Delaunay
2021-05-28 12:53 ` Patrice CHOTARD
2021-06-18 7:57 ` [Uboot-stm32] " Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 05/10] stm32mp: stm32prog: handle the next phase after USB re-enumeration Patrick Delaunay
2021-05-28 12:53 ` Patrice CHOTARD
2021-06-18 7:57 ` [Uboot-stm32] " Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 06/10] stm32mp: stm32prog: correctly handle DM_PMIC Patrick Delaunay
2021-05-28 12:54 ` Patrice CHOTARD
2021-05-30 21:44 ` Jaehoon Chung
2021-06-18 7:57 ` [Uboot-stm32] " Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 07/10] stm32mp: stm32prog: use get_cpu_dev for GetID command Patrick Delaunay
2021-05-28 12:54 ` Patrice CHOTARD
2021-06-18 7:56 ` [Uboot-stm32] " Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 08/10] stm32mp: stm32prog: change one message level to debug Patrick Delaunay
2021-05-28 12:54 ` Patrice CHOTARD [this message]
2021-06-18 7:56 ` [Uboot-stm32] " Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 09/10] dfu: add error callback Patrick Delaunay
2021-05-28 12:54 ` [Uboot-stm32] " Patrice CHOTARD
2021-06-18 7:56 ` Patrice CHOTARD
2021-05-18 13:12 ` [PATCH 10/10] stm32mp: stm32prog: handle dfu error Patrick Delaunay
2021-05-28 12:54 ` Patrice CHOTARD
2021-06-18 7:56 ` Patrice CHOTARD
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=398944f2-fcf8-ab6c-6d35-c4fb4f9588ae@foss.st.com \
--to=patrice.chotard@foss.st.com \
--cc=bmeng.cn@gmail.com \
--cc=patrick.delaunay@foss.st.com \
--cc=patrick.delaunay@st.com \
--cc=sjg@chromium.org \
--cc=u-boot@lists.denx.de \
--cc=uboot-stm32@st-md-mailman.stormreply.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.