| Message ID | 1455009512-26806-1-git-send-email-daniel.wagner@bmw-carit.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 09/02/16 11:18, Daniel Wagner wrote: > As the function name already indicates that get_opt_bool() parses > for a bool. It is not a surprise that compiler is complaining > about it when -Werror=incompatible-pointer-types is used: > > drivers/video/fbdev/intelfb/intelfbdrv.c: In function ‘intelfb_setup’: > drivers/video/fbdev/intelfb/intelfbdrv.c:353:39: error: passing argument 3 of ‘get_opt_bool’ from incompatible pointer type [-Werror=incompatible-pointer-types] > if (get_opt_bool(this_opt, "accel", &accel)) > > Signed-off-by: Daniel Wagner <daniel.wagner@bmw-carit.de> > Reported-by: Fengguang Wu <fengguang.wu@intel.com> > --- > [resend because Maik's emial address bounced last try] > > Hi, > > In the 'simple wait queue support' series is a patch > which turns on -Werror=incompatible-pointer-types which will > result in a compile error for intelfb. > > https://lkml.org/lkml/2016/1/28/462 > > Even if that patch wont make it, this one makes sense (at least > for me :)) > > I'll prepend this patch to the next version of the series in order > to see if I got rid of all incompatible pointer types errors caught > by the kbuild test robot. The patch looks good to me, but I didn't quite catch the above. So do you want me to apply this to fbdev tree, or do you need to take this via some other tree? If the latter, you have my ack. Tomi
On 02/16/2016 02:14 PM, Tomi Valkeinen wrote: > > On 09/02/16 11:18, Daniel Wagner wrote: >> As the function name already indicates that get_opt_bool() parses >> for a bool. It is not a surprise that compiler is complaining >> about it when -Werror=incompatible-pointer-types is used: >> >> drivers/video/fbdev/intelfb/intelfbdrv.c: In function ‘intelfb_setup’: >> drivers/video/fbdev/intelfb/intelfbdrv.c:353:39: error: passing argument 3 of ‘get_opt_bool’ from incompatible pointer type [-Werror=incompatible-pointer-types] >> if (get_opt_bool(this_opt, "accel", &accel)) >> >> Signed-off-by: Daniel Wagner <daniel.wagner@bmw-carit.de> >> Reported-by: Fengguang Wu <fengguang.wu@intel.com> >> --- >> [resend because Maik's emial address bounced last try] >> >> Hi, >> >> In the 'simple wait queue support' series is a patch >> which turns on -Werror=incompatible-pointer-types which will >> result in a compile error for intelfb. >> >> https://lkml.org/lkml/2016/1/28/462 >> >> Even if that patch wont make it, this one makes sense (at least >> for me :)) >> >> I'll prepend this patch to the next version of the series in order >> to see if I got rid of all incompatible pointer types errors caught >> by the kbuild test robot. > > The patch looks good to me, but I didn't quite catch the above. So do > you want me to apply this to fbdev tree, or do you need to take this via > some other tree? If the latter, you have my ack. I don't have any preference on the routing. So if you take it via fbdev that would be nice. One thing less to care about :) thanks, daniel -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/video/fbdev/intelfb/intelfbdrv.c b/drivers/video/fbdev/intelfb/intelfbdrv.c index bbec737..bf20744 100644 --- a/drivers/video/fbdev/intelfb/intelfbdrv.c +++ b/drivers/video/fbdev/intelfb/intelfbdrv.c @@ -302,7 +302,7 @@ static __inline__ int get_opt_int(const char *this_opt, const char *name, } static __inline__ int get_opt_bool(const char *this_opt, const char *name, - int *ret) + bool *ret) { if (!ret) return 0;
As the function name already indicates that get_opt_bool() parses for a bool. It is not a surprise that compiler is complaining about it when -Werror=incompatible-pointer-types is used: drivers/video/fbdev/intelfb/intelfbdrv.c: In function ‘intelfb_setup’: drivers/video/fbdev/intelfb/intelfbdrv.c:353:39: error: passing argument 3 of ‘get_opt_bool’ from incompatible pointer type [-Werror=incompatible-pointer-types] if (get_opt_bool(this_opt, "accel", &accel)) Signed-off-by: Daniel Wagner <daniel.wagner@bmw-carit.de> Reported-by: Fengguang Wu <fengguang.wu@intel.com> --- [resend because Maik's emial address bounced last try] Hi, In the 'simple wait queue support' series is a patch which turns on -Werror=incompatible-pointer-types which will result in a compile error for intelfb. https://lkml.org/lkml/2016/1/28/462 Even if that patch wont make it, this one makes sense (at least for me :)) I'll prepend this patch to the next version of the series in order to see if I got rid of all incompatible pointer types errors caught by the kbuild test robot. cheers, daniel drivers/video/fbdev/intelfb/intelfbdrv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)