> -----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

