[PATCH] mips: mt7628: fix ddr_type for MT7688KN

[Weijie Gao <weijie.gao@mediatek.com>]

On Fri, 2021-02-26 at 12:23 +0100, Stefan Roese wrote:
> 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@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@denx.de>
> 
> Just out of interest. How did you spot this issue?

I was told by another developer.

> 
> Thanks,
> Stefan