diff options
| author | Daniel Wagner <daniel.wagner@bmw-carit.de> | 2016-02-09 04:18:32 -0500 |
|---|---|---|
| committer | Tomi Valkeinen <tomi.valkeinen@ti.com> | 2016-02-16 08:09:20 -0500 |
| commit | 206fc20598157ce15597822cf01b94377e30075b (patch) | |
| tree | 4352f1b8d1b90965c449fe9768e05052b33fcd02 /drivers/video/fbdev/intelfb | |
| parent | 17f2e8e1dbe2cae66e99fd1a684db10bde792570 (diff) | |
video: Use bool instead int pointer for get_opt_bool() argument
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>
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Diffstat (limited to 'drivers/video/fbdev/intelfb')
| -rw-r--r-- | drivers/video/fbdev/intelfb/intelfbdrv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/video/fbdev/intelfb/intelfbdrv.c b/drivers/video/fbdev/intelfb/intelfbdrv.c index bbec737eef30..bf207444ba0c 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, | |||
| 302 | } | 302 | } |
| 303 | 303 | ||
| 304 | static __inline__ int get_opt_bool(const char *this_opt, const char *name, | 304 | static __inline__ int get_opt_bool(const char *this_opt, const char *name, |
| 305 | int *ret) | 305 | bool *ret) |
| 306 | { | 306 | { |
| 307 | if (!ret) | 307 | if (!ret) |
| 308 | return 0; | 308 | return 0; |
