Hi Sergei, On Thu, Feb 14, 2019 at 11:52:23AM +0300, Sergei Shtylyov wrote: > > +static int tps6598x_check_mode(struct tps6598x *tps) > > +{ > > + char mode[5] = { }; > > + int ret; > > + > > + ret = tps6598x_read32(tps, TPS_REG_MODE, (void *)mode); > > Casting pointers to 'void *' happens automagically, doesn't i? The third parameter in tps6598x_read32() is 'u32 *'. thanks, -- heikki