On Sat, Mar 28, 2020 at 2:12 AM Saravanan Sekar <sravanhome@xxxxxxxxx> wrote: > > mp2629 is a highly-integrated switching-mode battery charge management > device for single-cell Li-ion or Li-polymer battery. > > Add MFD core enables chip access for ADC driver for battery readings, > and a power supply battery-charger driver ... > +#ifndef __MP2629_H__ > +#define __MP2629_H__ > +#include <linux/device.h> > +#include <linux/regmap.h> > +#include <linux/types.h> None of these header is in use here. struct device; struct regmap; would be enough. > +struct mp2629_info { > + struct device *dev; > + struct regmap *regmap; > +}; > + > +#endif -- With Best Regards, Andy Shevchenko