On 1/17/23 12:41 AM, Linus Walleij wrote: > The test driver uses the gpiod consumer API so include the right > <linux/gpio/consumer.h> header. This may cause a problem with > struct of_device_id being implcitly pulled in by the legacy > header <linux/gpio.h> so include <linux/mod_devicetable.h> > explicitly as well. > > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > --- > ChangeLog v1->v2: > - Its mod_devicetable not mod_device_table oops. > --- > drivers/hte/hte-tegra194-test.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/hte/hte-tegra194-test.c b/drivers/hte/hte-tegra194-test.c > index 5d776a185bd6..79eb866558d3 100644 > --- a/drivers/hte/hte-tegra194-test.c > +++ b/drivers/hte/hte-tegra194-test.c > @@ -6,10 +6,11 @@ > */ > > #include <linux/err.h> > +#include <linux/mod_devicetable.h> > #include <linux/module.h> > #include <linux/moduleparam.h> > #include <linux/interrupt.h> > -#include <linux/gpio.h> > +#include <linux/gpio/consumer.h> > #include <linux/timer.h> > #include <linux/platform_device.h> > #include <linux/workqueue.h> I am not sure if I have followed up on this patchset. It seems this gives an error mod_devicetable.h not found. Perhaps you meant, linux/mod_devicetable.h instead?