[PATCH 1/1] imx: mx7: clock: use correct format strings

Peng Fan peng.fan at nxp.com
Sat Dec 26 01:51:23 CET 2020


> Subject: [PATCH 1/1] imx: mx7: clock: use correct format strings
> 
> Use %u and not %d for unsigned values.
> Print kHz and not khz.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>

Reviewed-by: Peng Fan <peng.fan at nxp.com>

> ---
>  arch/arm/mach-imx/mx7/clock.c | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/arch/arm/mach-imx/mx7/clock.c
> b/arch/arm/mach-imx/mx7/clock.c index aba9461d92..6a1bad855e 100644
> --- a/arch/arm/mach-imx/mx7/clock.c
> +++ b/arch/arm/mach-imx/mx7/clock.c
> @@ -916,7 +916,7 @@ void mxs_set_lcdclk(uint32_t base_addr, uint32_t
> freq)
>  		}
> 
>  		if (5 == i) {
> -			printf("Fail to set rate to %dkhz", freq);
> +			printf("Fail to set rate to %u kHz", freq);
>  			return;
>  		}
>  	}
> @@ -936,7 +936,7 @@ void mxs_set_lcdclk(uint32_t base_addr, uint32_t
> freq)
>  	}
> 
>  	if (best == 0) {
> -		printf("Fail to set rate to %dkhz", freq);
> +		printf("Fail to set rate to %u kHz", freq);
>  		return;
>  	}
> 
> @@ -1115,17 +1115,17 @@ int do_mx7_showclocks(struct cmd_tbl *cmdtp,
> int flag, int argc,
> 
>  	printf("\n");
> 
> -	printf("IPG        %8d kHz\n", mxc_get_clock(MXC_IPG_CLK) / 1000);
> -	printf("UART       %8d kHz\n", mxc_get_clock(MXC_UART_CLK) /
> 1000);
> +	printf("IPG        %8u kHz\n", mxc_get_clock(MXC_IPG_CLK) / 1000);
> +	printf("UART       %8u kHz\n", mxc_get_clock(MXC_UART_CLK) /
> 1000);
>  #ifdef CONFIG_MXC_SPI
> -	printf("CSPI       %8d kHz\n", mxc_get_clock(MXC_CSPI_CLK) /
> 1000);
> +	printf("CSPI       %8u kHz\n", mxc_get_clock(MXC_CSPI_CLK) /
> 1000);
>  #endif
> -	printf("AHB        %8d kHz\n", mxc_get_clock(MXC_AHB_CLK) /
> 1000);
> -	printf("AXI        %8d kHz\n", mxc_get_clock(MXC_AXI_CLK) / 1000);
> -	printf("DDR        %8d kHz\n", mxc_get_clock(MXC_DDR_CLK) /
> 1000);
> -	printf("USDHC1     %8d kHz\n", mxc_get_clock(MXC_ESDHC_CLK) /
> 1000);
> -	printf("USDHC2     %8d kHz\n", mxc_get_clock(MXC_ESDHC2_CLK) /
> 1000);
> -	printf("USDHC3     %8d kHz\n", mxc_get_clock(MXC_ESDHC3_CLK) /
> 1000);
> +	printf("AHB        %8u kHz\n", mxc_get_clock(MXC_AHB_CLK) /
> 1000);
> +	printf("AXI        %8u kHz\n", mxc_get_clock(MXC_AXI_CLK) / 1000);
> +	printf("DDR        %8u kHz\n", mxc_get_clock(MXC_DDR_CLK) /
> 1000);
> +	printf("USDHC1     %8u kHz\n", mxc_get_clock(MXC_ESDHC_CLK) /
> 1000);
> +	printf("USDHC2     %8u kHz\n", mxc_get_clock(MXC_ESDHC2_CLK) /
> 1000);
> +	printf("USDHC3     %8u kHz\n", mxc_get_clock(MXC_ESDHC3_CLK) /
> 1000);
> 
>  	return 0;
>  }
> --
> 2.29.2



More information about the U-Boot mailing list