| Message ID | 1555444645-15156-2-git-send-email-info@metux.net (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/4] mod_devicetable: helper macro for declaring oftree module device table | expand |
Hi Enrico, On Tue, Apr 16, 2019 at 09:57:22PM +0200, Enrico Weigelt, metux IT consult wrote: > Little helper macro that declares an oftree module device table, > if CONFIG_OF is enabled. Otherwise it's just noop. > > This is also helpful if some drivers can be built w/ or w/o > oftree support. This should go to OF folks, please. > > Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> > --- > include/linux/mod_devicetable.h | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/include/linux/mod_devicetable.h b/include/linux/mod_devicetable.h > index 448621c..853e301 100644 > --- a/include/linux/mod_devicetable.h > +++ b/include/linux/mod_devicetable.h > @@ -245,6 +245,15 @@ struct of_device_id { > const void *data; > }; > > +/* > + * macro for adding the of module device table only if CONFIG_OF enabled > + */ > +#ifdef CONFIG_OF > +#define MODULE_DEVICE_TABLE_OF(name) MODULE_DEVICE_TABLE(of,name) > +#else > +#define MODULE_DEVICE_TABLE_OF(name) > +#endif /* CONFIG_OF */ > + > /* VIO */ > struct vio_device_id { > char type[32]; > -- > 1.9.1 >
On 19.04.19 09:40, Dmitry Torokhov wrote: > Hi Enrico, > > On Tue, Apr 16, 2019 at 09:57:22PM +0200, Enrico Weigelt, metux IT consult wrote: >> Little helper macro that declares an oftree module device table, >> if CONFIG_OF is enabled. Otherwise it's just noop. >> >> This is also helpful if some drivers can be built w/ or w/o >> oftree support. > > This should go to OF folks, please. hmm, they should be CCed, if my script works right. This one is only needed for the 4th patch (skip oftree...). --mtx
On 4/24/19 3:48 AM, Enrico Weigelt, metux IT consult wrote: > On 19.04.19 09:40, Dmitry Torokhov wrote: >> Hi Enrico, >> >> On Tue, Apr 16, 2019 at 09:57:22PM +0200, Enrico Weigelt, metux IT consult wrote: >>> Little helper macro that declares an oftree module device table, >>> if CONFIG_OF is enabled. Otherwise it's just noop. >>> >>> This is also helpful if some drivers can be built w/ or w/o >>> oftree support. >> >> This should go to OF folks, please. > > hmm, they should be CCed, if my script works right. > This one is only needed for the 4th patch (skip oftree...). Your script did not work (BTDT). I just happened to notice this on lkml. -Frank > > > --mtx >
diff --git a/include/linux/mod_devicetable.h b/include/linux/mod_devicetable.h index 448621c..853e301 100644 --- a/include/linux/mod_devicetable.h +++ b/include/linux/mod_devicetable.h @@ -245,6 +245,15 @@ struct of_device_id { const void *data; }; +/* + * macro for adding the of module device table only if CONFIG_OF enabled + */ +#ifdef CONFIG_OF +#define MODULE_DEVICE_TABLE_OF(name) MODULE_DEVICE_TABLE(of,name) +#else +#define MODULE_DEVICE_TABLE_OF(name) +#endif /* CONFIG_OF */ + /* VIO */ struct vio_device_id { char type[32];
Little helper macro that declares an oftree module device table, if CONFIG_OF is enabled. Otherwise it's just noop. This is also helpful if some drivers can be built w/ or w/o oftree support. Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net> --- include/linux/mod_devicetable.h | 9 +++++++++ 1 file changed, 9 insertions(+)