Hi Ahmad, On Mon, May 31, 2021 at 09:38:04AM +0200, Ahmad Fatoum wrote: > +#include <common.h> > +#include <driver.h> > +#include <errno.h> > +#include <i2c/i2c.h> > +#include <linux/mfd/core.h> > +#include <init.h> > +#include <malloc.h> > +#include <of.h> > +#include <regmap.h> > +#include <xfuncs.h> > + > +#include <linux/mfd/tps65086.h> > + > +static const struct mfd_cell tps65086_cells[] = { > + { .name = "tps65086-regulator", }, > + { .name = "tps65086-gpio", }, > +}; Ok to have cells here which do not have a driver yet, > +static struct driver_d tps65086_restart_driver = { > + .name = "tps65086-restart", > + .probe = tps65086_restart_probe, > +}; but you should register the cells for the existing driver. Sascha -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox