[PATCH 1/1] sandbox: spi: sandbox_sf_state_name() is required

Simon Glass sjg at chromium.org
Mon Jul 6 03:37:51 CEST 2020


On Sun, 7 Jun 2020 at 01:28, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
>
> Compiling drivers/mtd/spi/sandbox.c fails when compiled with
> CONFIG_LOG=n:
>
> In file included from include/common.h:20,
>                  from drivers/mtd/spi/sandbox.c:13:
> drivers/mtd/spi/sandbox.c:295:15: error: format ‘%s’ expects argument of
> type ‘char *’, but argument 7 has type ‘int’ [-Werror=format=]
>   295 |   log_content(" cmd: transition to %s state\n",
>       |               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/printk.h:37:21: note: in definition of macro ‘pr_fmt’
>    37 | #define pr_fmt(fmt) fmt
>       |                     ^~~
> include/log.h:128:30: note: in expansion of macro ‘log_nop’
>   128 | #define log_content(_fmt...) log_nop(LOG_CATEGORY, \
>       |                              ^~~~~~~
> drivers/mtd/spi/sandbox.c:295:3: note: in expansion of macro
> ‘log_content’
>   295 |   log_content(" cmd: transition to %s state\n",
>       |   ^~~~~~~~~~~
> drivers/mtd/spi/sandbox.c:295:37: note: format string is defined here
>   295 |   log_content(" cmd: transition to %s state\n",
>       |                                    ~^
>       |                                     |
>       |                                     char *
>       |                                    %d
>
> Supply function sandbox_sf_state_name() independent of CONFIG_LOG.
>
> Fixes: c3aed5db591e ("sandbox: spi: Add more logging")
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
>  drivers/mtd/spi/sandbox.c | 2 --
>  1 file changed, 2 deletions(-)
>

Reviewed-by: Simon Glass <sjg at chromium.org>

We probably need to update test/run and CI config with a test that
builds sandbox with this CONFIG_LOG=n

Applied to u-boot-dm/next, thanks!
Applied to u-boot-dm/next, thanks!


More information about the U-Boot mailing list