[PATCH] mips: mt7628: fix ddr_type for MT7688KN

[Stefan Roese]

Hi Weijie,

On 23.02.21 08:12, Weijie Gao wrote:
> The MT7688KN is a multi-chip package with 8MiB DDR1 KGD. So the DDR type
> from bootstrap register must be ignored, and always be assumed as DDR1.
> 
> This patch fixes an issue that mt7628_ddr_pad_ldo_config() may be passed
> with a wrong ddr_type in MT7688KN.
> 
> Signed-off-by: Weijie Gao <weijie.gao at mediatek.com>
> ---
>   arch/mips/mach-mtmips/mt7628/ddr.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/mips/mach-mtmips/mt7628/ddr.c b/arch/mips/mach-mtmips/mt7628/ddr.c
> index b091f2ecff..4e72459906 100644
> --- a/arch/mips/mach-mtmips/mt7628/ddr.c
> +++ b/arch/mips/mach-mtmips/mt7628/ddr.c
> @@ -140,6 +140,9 @@ void mt7628_ddr_init(void)
>   	lspd = readl(sysc + SYSCTL_CLKCFG0_REG) &
>   	       (CPU_PLL_FROM_BBP | CPU_PLL_FROM_XTAL);
>   
> +	if (pkg_type == PKG_ID_KN)
> +		ddr_type = DRAM_DDR1;
> +
>   	mt7628_memc_reset(1);
>   	__udelay(200);
>   
> @@ -152,9 +155,6 @@ void mt7628_ddr_init(void)
>   	param.memsize = 0;
>   	param.bus_width = 0;
>   
> -	if (pkg_type == PKG_ID_KN)
> -		ddr_type = DRAM_DDR1;
> -
>   	if (ddr_type == DRAM_DDR1) {
>   		if (lspd)
>   			param.cfgs = ddr1_cfgs_160mhz;
> 

Reviewed-by: Stefan Roese <sr at denx.de>

Just out of interest. How did you spot this issue?

Thanks,
Stefan