On Thu, May 17, 2018 at 6:58 AM, Peter Mamonov <pmamonov@xxxxxxxxx> wrote: > Signed-off-by: Peter Mamonov <pmamonov@xxxxxxxxx> > --- > drivers/i2c/muxes/i2c-mux-pca954x.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/i2c/muxes/i2c-mux-pca954x.c b/drivers/i2c/muxes/i2c-mux-pca954x.c > index 0d5515b71..395254cdc 100644 > --- a/drivers/i2c/muxes/i2c-mux-pca954x.c > +++ b/drivers/i2c/muxes/i2c-mux-pca954x.c > @@ -180,6 +180,7 @@ static int pca954x_probe(struct device_d *dev) > struct i2c_adapter *adap = to_i2c_adapter(client->dev.parent); > int num, force; > struct pca954x *data; > + uintptr_t tmp; > int ret = -ENODEV; > int gpio; > > @@ -203,7 +204,8 @@ static int pca954x_probe(struct device_d *dev) > goto exit_free; > } > > - ret = dev_get_drvdata(dev, (const void **)&data->type); > + ret = dev_get_drvdata(dev, (const void **)&tmp); > + data->type = tmp; Upstream Linux driver doesn't store "type", but the actual pointer to "chip_desc". I think if you convert the driver to do that instead you can avoid type width problems and not need helper variable minutiae. Thanks, Andrey Smirnov _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox