Hi Patrick
On 4/6/21 9:27 AM, Patrick Delaunay wrote:
> Replace the remaining printf in setup_boot_mode() by log macro
> to handle filtering for log features.
>
> Signed-off-by: Patrick Delaunay <patrick.delau...@foss.st.com>
> ---
>
> arch/arm/mach-stm32mp/cpu.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/mach-stm32mp/cpu.c b/arch/arm/mach-stm32mp/cpu.c
> index 2f05c5e91b..621a6b7c89 100644
> --- a/arch/arm/mach-stm32mp/cpu.c
> +++ b/arch/arm/mach-stm32mp/cpu.c
> @@ -502,8 +502,8 @@ static void setup_boot_mode(void)
> if (IS_ENABLED(CONFIG_CMD_STM32PROG_SERIAL))
> gd->flags &= ~(GD_FLG_SILENT |
> GD_FLG_DISABLE_CONSOLE);
> - printf("uart%d = %s not found in device tree!\n",
> - instance + 1, cmd);
> + log_err("uart%d = %s not found in device tree!\n",
> + instance + 1, cmd);
> break;
> }
> sprintf(cmd, "%d", dev_seq(dev));
> @@ -514,7 +514,7 @@ static void setup_boot_mode(void)
> if (IS_ENABLED(CONFIG_CMD_STM32PROG_SERIAL) &&
> gd->cur_serial_dev != dev) {
> gd->flags &= ~(GD_FLG_SILENT |
> GD_FLG_DISABLE_CONSOLE);
> - printf("serial boot with console enabled!\n");
> + log_info("serial boot with console enabled!\n");
> }
> break;
> case BOOT_SERIAL_USB:
> @@ -546,7 +546,7 @@ static void setup_boot_mode(void)
>
> switch (forced_mode) {
> case BOOT_FASTBOOT:
> - printf("Enter fastboot!\n");
> + log_info("Enter fastboot!\n");
> env_set("preboot", "env set preboot; fastboot 0");
> break;
> case BOOT_STM32PROG:
> @@ -556,7 +556,7 @@ static void setup_boot_mode(void)
> case BOOT_UMS_MMC0:
> case BOOT_UMS_MMC1:
> case BOOT_UMS_MMC2:
> - printf("Enter UMS!\n");
> + log_info("Enter UMS!\n");
> instance = forced_mode - BOOT_UMS_MMC0;
> sprintf(cmd, "env set preboot; ums 0 mmc %d", instance);
> env_set("preboot", cmd);
>
Reviewed-by: Patrice Chotard <patrice.chot...@foss.st.com>
Thanks
Patrice
