[TRIVIAL 2.6.12 / 2.6.13 PATCH] v4l cx88 hue offset fix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Mauro Carvalho Chehab wrote:

Acked-by: Mauro Carvalho Chehab <[email protected]>

This small patch fixes top complaint about CX88 cards, which had a
different behavior than other V4L cards for hue setting.

It can also be applied also at 2.6.13 mainstream.

I must add: Not only does this apply to 2.6.13, but it also applies to 2.6.12 ... Can we have this put into the patch queue for 2.6.12.3 ? This IS a bug fix, and makes a HUGE improvement for cx88 boards.

I'd really like to see this in the mainstream kernel before the livecd distros make their next releases.

I would send this to Greg Kroah-Hartman / Chris Wright myself, but I don't know if that is proper protocol for doing this. Andrew, please send this to the correct person.

Thank you.

Attached is a diff against 2.6.12.2

--
Michael Krufky

Changed hue offset to 128 to correct behavior in cx88 cards.  Previously, 
setting 0% or 100% hue was required to avoid blue/green people on screen.  
Now, 50% Hue means no offset, just like bt878 stuff.

Signed-off-by: Michael Krufky <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 cx88-video.c |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)

diff -upr linux-2.6.12.2/drivers/media/video/cx88/cx88-video.c linux/drivers/media/video/cx88/cx88-video.c
--- linux-2.6.12.2/drivers/media/video/cx88/cx88-video.c	2005-06-17 15:48:29.000000000 -0400
+++ linux/drivers/media/video/cx88/cx88-video.c	2005-06-30 11:47:49.000000000 -0400
@@ -261,7 +261,7 @@ static struct cx88_ctrl cx8800_ctls[] = 
 			.default_value = 0,
 			.type          = V4L2_CTRL_TYPE_INTEGER,
 		},
-		.off                   = 0,
+		.off                   = 128,
 		.reg                   = MO_HUE,
 		.mask                  = 0x00ff,
 		.shift                 = 0,

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux