lkml.org 
[lkml]   [2008]   [May]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch] video: build fix for drivers/media/video/mt9v022.c
On Sat, 3 May 2008, Ingo Molnar wrote:

>
> x86.git testing found the following build bug on latest -git:
>
> CC [M] drivers/media/video/mt9v022.o
> drivers/media/video/mt9v022.c: In function 'bus_switch_request':
> drivers/media/video/mt9v022.c:199: error: implicit declaration of function 'gpio_is_valid'
> drivers/media/video/mt9v022.c:201: error: implicit declaration of function 'gpio_request'
> drivers/media/video/mt9v022.c:207: error: implicit declaration of function 'gpio_direction_output'
> drivers/media/video/mt9v022.c:211: error: implicit declaration of function 'gpio_free'
> drivers/media/video/mt9v022.c: In function 'bus_switch_act':
> drivers/media/video/mt9v022.c:237: error: implicit declaration of function 'gpio_set_value_cansleep'
> make[2]: *** [drivers/media/video] Error 2
> make[1]: *** [drivers/media] Error 2
> make[1]: *** Waiting for unfinished jobs....
> make: *** [drivers] Error 2
>
> with this config:
>
> http://redhat.com/~mingo/misc/config-Sat_May__3_16_08_39_CEST_2008.bad
>
> the bug was that the driver uses GPIO functionality but only includes
> the GPIO interface definitions for the CONFIG_MT9M001_PCA9536_SWITCH
> case, which was not set in this config.

Ok, once again a good catch and a wrong fix, sorry:-) The bug is that not
CONFIG_MT9M001_PCA9536_SWITCH but CONFIG_MT9V022_PCA9536_SWITCH shall be
checked for, of course. Copy-paste:-( I'll prepare a correct patch and
submit it. As for the "cleanup" side - don't know. Would it be better to
unconditionally include it? It won't hurt of course, looks better, but is
unneeded when the GPIO is not used. And, although grep reports most
drivers including asm/gpio.h, including linux/gpio.h seems indeed better.

Thanks
Guennadi
---
Guennadi Liakhovetski


\
 
 \ /
  Last update: 2008-05-03 19:05    [W:0.075 / U:0.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site