> -----Original Message----- > From: Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> > Sent: 2020年9月28日 16:21 > To: Joakim Zhang <qiangqing.zhang@xxxxxxx>; linux-can@xxxxxxxxxxxxxxx; > Pankaj Bansal <pankaj.bansal@xxxxxxx> > Cc: dl-linux-imx <linux-imx@xxxxxxx>; netdev@xxxxxxxxxxxxxxx > Subject: Re: [PATCH linux-can-next/flexcan 1/4] can: flexcan: initialize all flexcan > memory for ECC function > > On 9/28/20 9:58 AM, Joakim Zhang wrote: > >> Can someone check the vf610, too? > > > > I check the VF610 RM just now, indeed it has ECC feature, there is also a > NOTE in "12.1.4.13 Detection and Correction of Memory Errors" section: > > > > All FlexCAN memory must be initialized before starting its operation > > in order to have the parity bits in memory properly updated. The > > WRMFRZ bit in Control 2 Register (CTRL2) grants write access to all > > memory positions from 0x080 to 0xADF. > > Sounds good. Could I send out a V3 to review firstly, then wait Pankaj have time to do the test? Best Regards, Joakim Zhang > Marc > > -- > Pengutronix e.K. | Marc Kleine-Budde | > Embedded Linux | https://www.pengutronix.de | > Vertretung West/Dortmund | Phone: +49-231-2826-924 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
