On Wed, Jul 20, 2011 at 07:40:38AM +0300, Artem Bityutskiy wrote:
> On Fri, 2011-07-15 at 12:49 +0800, Shawn Guo wrote:
> > > > +static const struct of_device_id dataflash_dt_ids[] = {
> > > > + { .compatible = "atmel,at45xxx", },
> > > > + { .compatible = "atmel,dataflash", },
> > > > + { /* sentinel */ }
> > > > +};
> > > > +
> > >
> > > This should be protected with a #ifdef CONFIG_OF/#else/#endif, and
> > > there should be a MODULE_DEVICE_TABLE().
> > >
> > I personally hate #ifdef stuff. But okay, I can do it since there
> > are people being concerned by this little waste of space.
>
> I guess the question is - will it compile and work if CONFIG_OF is
> unset?
>
Yes, it will compile, as 'struct of_device_id' is defined in
include/linux/mod_devicetable.h unconditionally.
--
Regards,
Shawn
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
