On 15.07.2021 19:49:42, Dong Aisheng wrote: > > > Then should it be "fsl,imx8mp-flexcan", "fsl,imx8qxp-flexcan" rather than only > > > drop "fsl,imx6q-flexcan"? > > > > No, I will only use " fsl,imx8mp-flexcan" to avoid ECC impact. > > > > Is ECC issue SW or HW compatibility issue? > If SW, then we should keep the backward compatible string as DT is > describing HW. The commit messages describes the needed initialization for devices supporting ECC: https://lore.kernel.org/linux-can/20200929211557.14153-2-qiangqing.zhang@nxp.com/ 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 |