From: Sven Van Asbroeck <[email protected]> Sent: Wednesday, June 24, 2020 10:56 AM > Hi Andy, > > On Tue, Jun 23, 2020 at 9:40 PM Andy Duan <[email protected]> wrote: > > > > The patch looks good. > > Reviewed-by: Fugang Duan <[email protected]> > > 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.
