[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.58.0906121220520.32713@shell2.speakeasy.net>
Date: Fri, 12 Jun 2009 12:31:29 -0700 (PDT)
From: Trent Piepho <xyzzy@...akeasy.org>
To: Randy Dunlap <randy.dunlap@...cle.com>
cc: Stephen Rothwell <sfr@...b.auug.org.au>,
linux-next@...r.kernel.org, LKML <linux-kernel@...r.kernel.org>,
Mauro Carvalho Chehab <mchehab@...radead.org>,
linux-media@...r.kernel.org
Subject: Re: [PATCH -next] v4l: expose function outside of ifdef/endif block
On Fri, 12 Jun 2009, Randy Dunlap wrote:
> From: Randy Dunlap <randy.dunlap@...cle.com>
>
> Move v4l_bound_align_image() outside of an #ifdef CONFIG_I2C block
> so that it is always built. Fixes a build error:
clamp_align() should be moved as well, since it's only used by
v4l_bound_align_image(). I'm attaching an alternate version that fixes
this. Labeled the endif too.
> drivers/media/video/v4l2-common.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> --- linux-next-20090612.orig/drivers/media/video/v4l2-common.c
> +++ linux-next-20090612/drivers/media/video/v4l2-common.c
> @@ -915,6 +915,7 @@ const unsigned short *v4l2_i2c_tuner_add
> return NULL;
> }
> EXPORT_SYMBOL_GPL(v4l2_i2c_tuner_addrs);
> +#endif
>
> /* Clamp x to be between min and max, aligned to a multiple of 2^align. min
> * and max don't have to be aligned, but there must be at least one valid
> @@ -986,5 +987,3 @@ void v4l_bound_align_image(u32 *w, unsig
> }
> }
> EXPORT_SYMBOL_GPL(v4l_bound_align_image);
> -
> -#endif
View attachment "tmp.patch" of type "TEXT/PLAIN" (1347 bytes)
Powered by blists - more mailing lists