[PATCH 2/3] ram: rk3399: Fix faulty frequency change reports

Xavier Drudis Ferran xdrudis at tinet.cat
Mon Jul 4 12:00:15 CEST 2022


El Tue, Jun 21, 2022 at 10:07:28AM +0000, Lee Jones deia:
> Frequency changes to 400MHz are presently reported as:
> 
>   lpddr4_set_rate_0: change freq to 400000000 mhz 0, 1
> 
> This is obviously wrong by 6 orders of magnitude.
> 
> Ensure frequency changes are reported accurately.
>

Not obvious to me what "mhz" means (mHz? MHz? Hz?)
and so not obvious to me by how many orders of magnitude 
it was wrong but in any case now the message is correct and clear.

Tested-by: Xavier Drudis Ferran <xdrudis at tinet.cat>
 
> Signed-off-by: Lee Jones <lee.jones at linaro.org>
> ---
>  drivers/ram/rockchip/sdram_rk3399.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/ram/rockchip/sdram_rk3399.c b/drivers/ram/rockchip/sdram_rk3399.c
> index 0af0fa9e7b..34d6c93f95 100644
> --- a/drivers/ram/rockchip/sdram_rk3399.c
> +++ b/drivers/ram/rockchip/sdram_rk3399.c
> @@ -2552,8 +2552,8 @@ static int lpddr4_set_rate(struct dram_info *dram,
>  			       dfs_cfgs_lpddr4[ctl_fn].base.ddr_freq);
>  
>  		if (IS_ENABLED(CONFIG_RAM_ROCKCHIP_DEBUG))
> -			printf("%s: change freq to %d mhz %d, %d\n", __func__,
> -			       dfs_cfgs_lpddr4[ctl_fn].base.ddr_freq,
> +			printf("%s: change freq to %dMHz %d, %d\n", __func__,
> +			       dfs_cfgs_lpddr4[ctl_fn].base.ddr_freq / MHz,
>  			       ctl_fn, phy_fn);
>  	}
>  
> -- 
> 2.37.0.rc0.104.g0611611a94-goog
> 


More information about the U-Boot mailing list