On Fri, Aug 5, 2022 at 8:07 PM Colin Foster <colin.foster@xxxxxxxxxxxxxxxx> wrote: > On Fri, Aug 05, 2022 at 07:58:06PM +0200, Andy Shevchenko wrote: > > On Fri, Aug 5, 2022 at 7:44 PM Colin Foster > > <colin.foster@xxxxxxxxxxxxxxxx> wrote: > > > On Tue, Aug 02, 2022 at 10:47:28PM -0700, Colin Foster wrote: ... > > > > +int ocelot_chip_reset(struct device *dev) > > > > > > #include <linux/device.h> > > > > Nope, > > > > struct device; ... > > > > +static int ocelot_spi_initialize(struct device *dev) > > > > > > #include <linux/device.h> > > > > Ditto. > > ocelot-spi.c uses devm_kzalloc, so that should still be included. > > ocelot-core.c uses dev_get_drvdata. > > So I think I still want the includes... but for different reasons. Yes in this case. -- With Best Regards, Andy Shevchenko
