On Tue, Jan 12, 2021 at 08:36:49PM +0100, Andreas Kemnade wrote: > On Sat, 9 Jan 2021 19:02:16 +0100 > Jonathan Neuschäfer <[email protected]> wrote: [...] > > +static const struct of_device_id of_ntxec_match_table[] = { > > + { .compatible = "netronix,ntxec", }, > > + {} > > +}; > > + > MODULE_DEVICE_TABLE?
Yes, good idea.
> > +static struct i2c_driver ntxec_driver = {
> > + .driver = {
> > + .name = "ntxec",
> > + .of_match_table = of_ntxec_match_table,
> > + },
> > + .probe_new = ntxec_probe,
> > + .remove = ntxec_remove,
> > +};
> > +module_i2c_driver(ntxec_driver);
>
> MODULE_LICENSE()?
>
> modpost moans about that here.
I forgot those. Thanks for noticing.
I'll add the MODULE_ lines in v8.
Thanks,
Jonathan
signature.asc
Description: PGP signature
