[PATCH 2/3] ram: rk3399: Mark existing prints via RAM_ROCKCHIP_DEBUG

Kever Yang kever.yang at rock-chips.com
Sat Jul 18 08:15:30 EDT 2020


On 2020/7/14 上午4:06, Jagan Teki wrote:
> Right now all these debug statements are printing on the
> console to make sure proper dram initialization happens.
>
> Mark them into RAM_ROCKCHIP_DEBUG would be more meaningful
> and work like before since the RAM_ROCKCHIP_DEBUG is by
> default yet.
>
> No functionality changes.
>
> Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
Reviewed-by: Kever Yang<kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
>   drivers/ram/rockchip/sdram_rk3399.c | 12 ++++++++----
>   1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/ram/rockchip/sdram_rk3399.c b/drivers/ram/rockchip/sdram_rk3399.c
> index d2f3fde236..670925420d 100644
> --- a/drivers/ram/rockchip/sdram_rk3399.c
> +++ b/drivers/ram/rockchip/sdram_rk3399.c
> @@ -2551,8 +2551,10 @@ static int lpddr4_set_rate(struct dram_info *dram,
>   		lpddr4_set_ctl(dram, params, ctl_fn,
>   			       dfs_cfgs_lpddr4[ctl_fn].base.ddr_freq);
>   
> -		printf("%s: change freq to %d mhz %d, %d\n", __func__,
> -		       dfs_cfgs_lpddr4[ctl_fn].base.ddr_freq, ctl_fn, phy_fn);
> +		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,
> +			       ctl_fn, phy_fn);
>   	}
>   
>   	return 0;
> @@ -2966,8 +2968,10 @@ static int sdram_init(struct dram_info *dram,
>   			params->base.num_channels++;
>   		}
>   
> -		printf("Channel ");
> -		printf(channel ? "1: " : "0: ");
> +		if (IS_ENABLED(CONFIG_RAM_ROCKCHIP_DEBUG)) {
> +			printf("Channel ");
> +			printf(channel ? "1: " : "0: ");
> +		}
>   
>   		if (channel == 0)
>   			set_ddr_stride(dram->pmusgrf, 0x17);





More information about the Linux-rockchip mailing list