Kindly ping.
On 2020/11/2 17:55, AlexChen wrote:
> We should use printf format specifier "%u" instead of "%i" for
> argument of type "unsigned int".
>
> Reported-by: Euler Robot <euler.ro...@huawei.com>
> Signed-off-by: Alex Chen <alex.c...@huawei.com>
> ---
> hw/arm/pxa2xx.c | 2 +-
> hw/arm/spitz.c | 2 +-
> hw/arm/tosa.c | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/hw/arm/pxa2xx.c b/hw/arm/pxa2xx.c
> index 591776ba88..1a98f3bd5c 100644
> --- a/hw/arm/pxa2xx.c
> +++ b/hw/arm/pxa2xx.c
> @@ -675,7 +675,7 @@ static void pxa2xx_ssp_write(void *opaque, hwaddr addr,
> if (value & SSCR0_MOD)
> printf("%s: Attempt to use network mode\n", __func__);
> if (s->enable && SSCR0_DSS(value) < 4)
> - printf("%s: Wrong data size: %i bits\n", __func__,
> + printf("%s: Wrong data size: %u bits\n", __func__,
> SSCR0_DSS(value));
> if (!(value & SSCR0_SSE)) {
> s->sssr = 0;
> diff --git a/hw/arm/spitz.c b/hw/arm/spitz.c
> index 32bdeacfd3..772662f149 100644
> --- a/hw/arm/spitz.c
> +++ b/hw/arm/spitz.c
> @@ -586,7 +586,7 @@ struct SpitzLCDTG {
> static void spitz_bl_update(SpitzLCDTG *s)
> {
> if (s->bl_power && s->bl_intensity)
> - zaurus_printf("LCD Backlight now at %i/63\n", s->bl_intensity);
> + zaurus_printf("LCD Backlight now at %u/63\n", s->bl_intensity);
> else
> zaurus_printf("LCD Backlight now off\n");
> }
> diff --git a/hw/arm/tosa.c b/hw/arm/tosa.c
> index fe88ed89fe..66b244aeff 100644
> --- a/hw/arm/tosa.c
> +++ b/hw/arm/tosa.c
> @@ -150,7 +150,7 @@ static void tosa_gpio_setup(PXA2xxState *cpu,
>
> static uint32_t tosa_ssp_tansfer(SSISlave *dev, uint32_t value)
> {
> - fprintf(stderr, "TG: %d %02x\n", value >> 5, value & 0x1f);
> + fprintf(stderr, "TG: %u %02x\n", value >> 5, value & 0x1f);
> return 0;
> }
>
