On Thu, Jun 21, 2012 at 01:35:56PM +0800, Shawn Guo wrote: > On Thu, Jun 14, 2012 at 03:43:24PM +0200, Sascha Hauer wrote: > > +static const struct of_device_id imx_pd_dt_ids[] = { > > + { .compatible = "fsl,imx-parallel-display", .data = NULL, }, > > Can we use particular soc name to define the compatible string? Just realized that it's actually not representing any hardware block. If that's the case, I feel we should try to get it away from device tree. Regards, Shawn > Also, > the .data initialization seems not needed. > > > + { /* sentinel */ } > > +}; _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel