On Sat, Mar 22, 2014 at 11:42:01AM -0700, Dmitry Torokhov wrote: > On Sat, Mar 22, 2014 at 07:13:36PM +0100, Carlo Caione wrote: > > On Sat, Mar 22, 2014 at 6:33 PM, Mark Brown <broo...@kernel.org> wrote: > > > On Sat, Mar 22, 2014 at 05:51:32PM +0100, Carlo Caione wrote: > > >> On Tue, Mar 18, 2014 at 03:59:19PM +0000, Lee Jones wrote: > > > > > >> > > + of_id = of_match_device(axp20x_of_match, &i2c->dev); > > >> > > + if (!of_id) { > > >> > > + dev_err(&i2c->dev, "Unable to setup AXP20X data\n"); > > >> > > + return -ENODEV; > > >> > > + } > > >> > > + axp20x->variant = (int) of_id->data; > > > > > >> > No need to cast from void *. > > > > > >> My compiler says otherwise :) > > > > > > Are you sure your compiler isn't correctly telling you not to cast away > > > const? If the compiler is complaining about a cast on void then it's > > > spotted a definite bug. > > > > [resend since gmail decided to not put broonie in CC] > > > > drivers/mfd/axp20x.c:172:18: warning: assignment makes integer > > frompointer without a cast [enabled by default] > > You need to cast to long, otherwise you will get warnings when compiling > on 64 bit arches.
Actually no warning also with (int) cast. -- Carlo Caione -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.