> -----Original Message-----
> From: york sun
> Sent: Friday, August 26, 2016 11:00 PM
> To: Qianyu Gong <[email protected]>; [email protected]
> Cc: Prabhakar Kushwaha <[email protected]>; Mingkai Hu
> <[email protected]>; Shaohui Xie <[email protected]>; Zhiqiang Hou
> <[email protected]>; Wenbin Song <[email protected]>
> Subject: Re: [PATCH 1/8] drivers/ddr/fsl: add DEBUG_38
> 
> On 08/26/2016 04:40 AM, Gong Qianyu wrote:
> > From: Mingkai Hu <[email protected]>
> >
> > DEBUG_38 is needed for rev2 DDR controller.
> >
> > Signed-off-by: Mingkai Hu <[email protected]>
> > Signed-off-by: Gong Qianyu <[email protected]>
> > ---
> >  drivers/ddr/fsl/ctrl_regs.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/ddr/fsl/ctrl_regs.c b/drivers/ddr/fsl/ctrl_regs.c
> > index 24fd366..4ae8b80 100644
> > --- a/drivers/ddr/fsl/ctrl_regs.c
> > +++ b/drivers/ddr/fsl/ctrl_regs.c
> > @@ -2526,5 +2526,7 @@ compute_fsl_memctl_config_regs(const
> unsigned int ctrl_num,
> >             ddr->debug[2] |= 0x00000200;    /* set bit 22 */
> >  #endif
> >
> > +   ddr->debug[37] = 0x80000000;
> > +
> 
> NAK. You can't simply add a new register without checking if it exists on 
> older
> controllers.
> 
> York

Thanks, York. This is not needed and will remove it in next version.

Thanks,
Mingkai
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to