Messages in this thread | | | From | Geert Uytterhoeven <> | Date | Tue, 28 Mar 2017 10:36:43 +0200 | Subject | Re: [PATCH V6 4/7] mfd: da9061: MFD core support |
| |
Hi Lee,
On Tue, Mar 28, 2017 at 10:21 AM, Lee Jones <lee.jones@linaro.org> wrote: >> [auto build test WARNING on ljones-mfd/for-mfd-next] >> [also build test WARNING on v4.11-rc4 next-20170327] >> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] >> >> url: https://github.com/0day-ci/linux/commits/Steve-Twiss/Documentation-devicetree-watchdog-da9062-61-watchdog-timer-binding/20170327-215359 >> base: https://git.kernel.org/pub/scm/linux/kernel/git/lee/mfd.git for-mfd-next >> config: x86_64-randconfig-x009-201713 (attached as .config) >> compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 >> reproduce: >> # save the attached .config to linux build tree >> make ARCH=x86_64 >> >> All warnings (new ones prefixed by >>): >> >> drivers//mfd/da9062-core.c: In function 'da9062_i2c_probe': >> >> drivers//mfd/da9062-core.c:845:21: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] >> chip->chip_type = (int)match->data; >> ^ > > Please use longs or enums.
Enums would still give a warning on 64-bit. The simple fix is change the cast from (int) to (uintptr_t).
Gr{oetje,eeting}s,
Geert
-- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
| |