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.

Reply via email to