On Sunday, October 17, 2010 21:28:29 Mauro Carvalho Chehab wrote:
> This is an automatic generated email to let you know that the following patch 
> were queued at the 
> http://git.linuxtv.org/media_tree.git tree:
> 
> Subject: [media] Add driver for Siliconfile SR030PC30 VGA camera
> Author:  Sylwester Nawrocki <s.nawro...@samsung.com>
> Date:    Mon Oct 11 13:33:57 2010 -0300
> 
> Add an I2C/v4l2-subdev driver for Siliconfile SR030PC30 VGA
> camera sensor with Image Signal Processor. SR030PC30 is
> the low resolution camera sensor on Samsung Aquila boards.
> 
> Signed-off-by: Sylwester Nawrocki <s.nawro...@samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.p...@samsung.com>
> Signed-off-by: Mauro Carvalho Chehab <mche...@redhat.com>
> 
>  drivers/media/video/Kconfig     |    6 +
>  drivers/media/video/Makefile    |    1 +
>  drivers/media/video/sr030pc30.c |  893 
> +++++++++++++++++++++++++++++++++++++++
>  include/media/sr030pc30.h       |   21 +
>  4 files changed, 921 insertions(+), 0 deletions(-)

It fails to compile with this error:

drivers/media/video/sr030pc30.c: In function ‘sr030pc30_probe’:
drivers/media/video/sr030pc30.c:834: error: implicit declaration of function 
‘kzalloc’
drivers/media/video/sr030pc30.c:834: warning: assignment makes pointer from 
integer without a cast
drivers/media/video/sr030pc30.c: In function ‘sr030pc30_remove’:
drivers/media/video/sr030pc30.c:858: error: implicit declaration of function 
‘kfree’

Here is the patch to fix this:

diff --git a/drivers/media/video/sr030pc30.c b/drivers/media/video/sr030pc30.c
index f82e1f3..ec8d875 100644
--- a/drivers/media/video/sr030pc30.c
+++ b/drivers/media/video/sr030pc30.c
@@ -18,6 +18,7 @@
 
 #include <linux/i2c.h>
 #include <linux/delay.h>
+#include <linux/slab.h>
 #include <media/v4l2-device.h>
 #include <media/v4l2-subdev.h>
 #include <media/v4l2-mediabus.h>


Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

-- 
Hans Verkuil - video4linux developer - sponsored by TANDBERG, part of Cisco
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to