2010-11-18 20:22:16

by Tim Blechmann

Subject: [PATCH] cx25821: fix build failure

include specific header for bkl functions

drivers/staging/cx25821/cx25821-video.c:815:8: error: implicit declaration of function ‘lock_kernel’
drivers/staging/cx25821/cx25821-video.c:832:9: error: implicit declaration of function ‘unlock_kernel’

Signed-off-by: Tim Blechmann <[email protected]>
---
drivers/staging/cx25821/cx25821-video.h | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/staging/cx25821/cx25821-video.h b/drivers/staging/cx25821/cx25821-video.h
index cc6034b..0df20dc 100644
--- a/drivers/staging/cx25821/cx25821-video.h
+++ b/drivers/staging/cx25821/cx25821-video.h
@@ -34,6 +34,7 @@
#include <linux/interrupt.h>
#include <linux/delay.h>
#include <linux/kthread.h>
+#include <linux/smp_lock.h>
#include <asm/div64.h>

#include "cx25821.h"
--
1.7.1

2010-11-18 20:18:40

by Randy Dunlap

Subject: Re: [PATCH] cx25821: fix build failure

On Thu, 18 Nov 2010 21:15:58 +0100 Tim Blechmann wrote:

> include specific header for bkl functions
>
> drivers/staging/cx25821/cx25821-video.c:815:8: error: implicit declaration of function ‘lock_kernel’
> drivers/staging/cx25821/cx25821-video.c:832:9: error: implicit declaration of function ‘unlock_kernel’
>
> Signed-off-by: Tim Blechmann <[email protected]>

Hi,

This patch has already been posted and is already merged into Linus's git tree.

Thanks.

> ---
> drivers/staging/cx25821/cx25821-video.h | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/staging/cx25821/cx25821-video.h b/drivers/staging/cx25821/cx25821-video.h
> index cc6034b..0df20dc 100644
> --- a/drivers/staging/cx25821/cx25821-video.h
> +++ b/drivers/staging/cx25821/cx25821-video.h
> @@ -34,6 +34,7 @@
> #include <linux/interrupt.h>
> #include <linux/delay.h>
> #include <linux/kthread.h>
> +#include <linux/smp_lock.h>
> #include <asm/div64.h>
>
> #include "cx25821.h"
> --


---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***