On Wed, Nov 4, 2020 at 2:23 AM AlexChen <alex.c...@huawei.com> wrote:
>
> We should use printf format specifier "%u" instead of "%d" for
> argument of type "unsigned int".
>
> Reported-by: Euler Robot <euler.ro...@huawei.com>
> Signed-off-by: Alex Chen <alex.c...@huawei.com>
Reviewed-by: Alistair Francis <alistair.fran...@wdc.com>
Alistair
> ---
> hw/ssi/imx_spi.c | 2 +-
> hw/ssi/xilinx_spi.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/ssi/imx_spi.c b/hw/ssi/imx_spi.c
> index 7f703d8328..d8885ae454 100644
> --- a/hw/ssi/imx_spi.c
> +++ b/hw/ssi/imx_spi.c
> @@ -53,7 +53,7 @@ static const char *imx_spi_reg_name(uint32_t reg)
> case ECSPI_MSGDATA:
> return "ECSPI_MSGDATA";
> default:
> - sprintf(unknown, "%d ?", reg);
> + sprintf(unknown, "%u ?", reg);
> return unknown;
> }
> }
> diff --git a/hw/ssi/xilinx_spi.c b/hw/ssi/xilinx_spi.c
> index fec8817d94..49ff275593 100644
> --- a/hw/ssi/xilinx_spi.c
> +++ b/hw/ssi/xilinx_spi.c
> @@ -142,7 +142,7 @@ static void xlx_spi_update_irq(XilinxSPI *s)
> irq chain unless things really changed. */
> if (pending != s->irqline) {
> s->irqline = pending;
> - DB_PRINT("irq_change of state %d ISR:%x IER:%X\n",
> + DB_PRINT("irq_change of state %u ISR:%x IER:%X\n",
> pending, s->regs[R_IPISR], s->regs[R_IPIER]);
> qemu_set_irq(s->irq, pending);
> }
> --
> 2.19.1
>
