From: Sven Van Asbroeck <thesve...@gmail.com> Sent: Wednesday, June 24, 2020 10:56 AM > Hi Andy, > > On Tue, Jun 23, 2020 at 9:40 PM Andy Duan <fugang.d...@nxp.com> wrote: > > > > The patch looks good. > > Reviewed-by: Fugang Duan <fugang.d...@nxp.com> > > Thank you ! > > To check we're on a plus, the patch uses: > cpu_is_imx6q() && imx_get_soc_revision() >= IMX_CHIP_REVISION_2_0 > > Fabio Estevam suggested that perhaps checking > of_machine_is_compatible("fsl,imx6qp") > might be preferable. What is your opinion on this?
Yes, I also think of_machine_is_compatible() is much better. Thanks.