Re: [Patch] v4l cx88 hue offset fix
On Tue, Aug 02, 2005 at 08:41:27AM -0400, Michael Krufky wrote: > Horms wrote: > > >Hi Marcelo, > > > >Another fix from 2.6.12.3 that seems approprite for 2.4. > >Should apply cleanly to your tree. > > > > > Horms- > > Thank you for the effort of trying to get this into 2.4, but I don't > think there is any support at all for cx88 cards in kernel 2.4 v4l. I > wasn't involved in v4l back then, but I took a look at Marcelo's > linux-2.4.git/tree , and /drivers/media/video/cx88* just doesn't exist. > There is no cx88-video.c file present for this patch to be applied to! > > This one-line fix does make a very big difference, but unfortunately, it > looks like 2.4 doesn't have the cx88 driver for the fixing... > > Good to see that debian has backported it to their 2.6.8 kernel, > though. That really makes me smile. :-) Sorry about the noise. Dann Frazier and I ran through a bunch of patches from 2.6.12-stable to see which onese were appropriate for Debian's 2.4.27 and 2.6.8. This patch did indeed get backported to 2.6.8. And in the course of juggling all the patches I made a mistake and added it our 2.4.27 patchset and subsequently sent it here. cx88 indeed is not in 2.4 and the patch is completely invalid for that tree. Please ignore :) I do have few other patches from 2.6.12-stable that do seem appropriate for 2.4, but after this mess up I am waiting for all the builds to finish before forwarding stuff on. -- Horms - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [Patch] v4l cx88 hue offset fix
On Tue, Aug 02, 2005 at 08:41:27AM -0400, Michael Krufky wrote: Horms wrote: Hi Marcelo, Another fix from 2.6.12.3 that seems approprite for 2.4. Should apply cleanly to your tree. Horms- Thank you for the effort of trying to get this into 2.4, but I don't think there is any support at all for cx88 cards in kernel 2.4 v4l. I wasn't involved in v4l back then, but I took a look at Marcelo's linux-2.4.git/tree , and /drivers/media/video/cx88* just doesn't exist. There is no cx88-video.c file present for this patch to be applied to! This one-line fix does make a very big difference, but unfortunately, it looks like 2.4 doesn't have the cx88 driver for the fixing... Good to see that debian has backported it to their 2.6.8 kernel, though. That really makes me smile. :-) Sorry about the noise. Dann Frazier and I ran through a bunch of patches from 2.6.12-stable to see which onese were appropriate for Debian's 2.4.27 and 2.6.8. This patch did indeed get backported to 2.6.8. And in the course of juggling all the patches I made a mistake and added it our 2.4.27 patchset and subsequently sent it here. cx88 indeed is not in 2.4 and the patch is completely invalid for that tree. Please ignore :) I do have few other patches from 2.6.12-stable that do seem appropriate for 2.4, but after this mess up I am waiting for all the builds to finish before forwarding stuff on. -- Horms - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [Patch] v4l cx88 hue offset fix
On Tue, Aug 02, 2005 at 04:20:00PM +0900, Horms wrote: > Hi Marcelo, > > Another fix from 2.6.12.3 that seems approprite for 2.4. > Should apply cleanly to your tree. Please ignore this. I sent the wrong patch. I don't think thie cx88 driver is in 2.4 at all. > Signed-off-by: Horms <[EMAIL PROTECTED]> > > From: Michael Krufky <[EMAIL PROTECTED]> > Date: Thu, 30 Jun 2005 20:06:41 + (-0400) > Subject: [PATCH] v4l cx88 hue offset fix > X-Git-Tag: v2.6.12.3 > X-Git-Url: > http://www.kernel.org/git/?p=linux/kernel/git/gregkh/linux-2.6.12.y.git;a=commitdiff;h=aebaaf4060dd0db163694da8e4ab7ba86add57b9 > > [PATCH] v4l cx88 hue offset fix > > 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]> > Signed-off-by: Chris Wright <[EMAIL PROTECTED]> > Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]> > --- > > Backported to Debian's kernel-source-2.6.8 by dann frazier <[EMAIL PROTECTED]> > > --- a/drivers/media/video/cx88/cx88-video.c > +++ b/drivers/media/video/cx88/cx88-video.c > @@ -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, - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [Patch] v4l cx88 hue offset fix
Horms wrote: Hi Marcelo, Another fix from 2.6.12.3 that seems approprite for 2.4. Should apply cleanly to your tree. Horms- Thank you for the effort of trying to get this into 2.4, but I don't think there is any support at all for cx88 cards in kernel 2.4 v4l. I wasn't involved in v4l back then, but I took a look at Marcelo's linux-2.4.git/tree , and /drivers/media/video/cx88* just doesn't exist. There is no cx88-video.c file present for this patch to be applied to! This one-line fix does make a very big difference, but unfortunately, it looks like 2.4 doesn't have the cx88 driver for the fixing... Good to see that debian has backported it to their 2.6.8 kernel, though. That really makes me smile. :-) Signed-off-by: Horms <[EMAIL PROTECTED]> From: Michael Krufky <[EMAIL PROTECTED]> Date: Thu, 30 Jun 2005 20:06:41 + (-0400) Subject: [PATCH] v4l cx88 hue offset fix X-Git-Tag: v2.6.12.3 X-Git-Url: http://www.kernel.org/git/?p=linux/kernel/git/gregkh/linux-2.6.12.y.git;a=commitdiff;h=aebaaf4060dd0db163694da8e4ab7ba86add57b9 [PATCH] v4l cx88 hue offset fix 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]> Signed-off-by: Chris Wright <[EMAIL PROTECTED]> Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]> --- Backported to Debian's kernel-source-2.6.8 by dann frazier <[EMAIL PROTECTED]> --- a/drivers/media/video/cx88/cx88-video.c +++ b/drivers/media/video/cx88/cx88-video.c @@ -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, -- Michael Krufky - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [Patch] v4l cx88 hue offset fix
Horms wrote: Hi Marcelo, Another fix from 2.6.12.3 that seems approprite for 2.4. Should apply cleanly to your tree. Horms- Thank you for the effort of trying to get this into 2.4, but I don't think there is any support at all for cx88 cards in kernel 2.4 v4l. I wasn't involved in v4l back then, but I took a look at Marcelo's linux-2.4.git/tree , and /drivers/media/video/cx88* just doesn't exist. There is no cx88-video.c file present for this patch to be applied to! This one-line fix does make a very big difference, but unfortunately, it looks like 2.4 doesn't have the cx88 driver for the fixing... Good to see that debian has backported it to their 2.6.8 kernel, though. That really makes me smile. :-) Signed-off-by: Horms [EMAIL PROTECTED] From: Michael Krufky [EMAIL PROTECTED] Date: Thu, 30 Jun 2005 20:06:41 + (-0400) Subject: [PATCH] v4l cx88 hue offset fix X-Git-Tag: v2.6.12.3 X-Git-Url: http://www.kernel.org/git/?p=linux/kernel/git/gregkh/linux-2.6.12.y.git;a=commitdiff;h=aebaaf4060dd0db163694da8e4ab7ba86add57b9 [PATCH] v4l cx88 hue offset fix 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] Signed-off-by: Chris Wright [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- Backported to Debian's kernel-source-2.6.8 by dann frazier [EMAIL PROTECTED] --- a/drivers/media/video/cx88/cx88-video.c +++ b/drivers/media/video/cx88/cx88-video.c @@ -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, -- Michael Krufky - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [Patch] v4l cx88 hue offset fix
On Tue, Aug 02, 2005 at 04:20:00PM +0900, Horms wrote: Hi Marcelo, Another fix from 2.6.12.3 that seems approprite for 2.4. Should apply cleanly to your tree. Please ignore this. I sent the wrong patch. I don't think thie cx88 driver is in 2.4 at all. Signed-off-by: Horms [EMAIL PROTECTED] From: Michael Krufky [EMAIL PROTECTED] Date: Thu, 30 Jun 2005 20:06:41 + (-0400) Subject: [PATCH] v4l cx88 hue offset fix X-Git-Tag: v2.6.12.3 X-Git-Url: http://www.kernel.org/git/?p=linux/kernel/git/gregkh/linux-2.6.12.y.git;a=commitdiff;h=aebaaf4060dd0db163694da8e4ab7ba86add57b9 [PATCH] v4l cx88 hue offset fix 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] Signed-off-by: Chris Wright [EMAIL PROTECTED] Signed-off-by: Greg Kroah-Hartman [EMAIL PROTECTED] --- Backported to Debian's kernel-source-2.6.8 by dann frazier [EMAIL PROTECTED] --- a/drivers/media/video/cx88/cx88-video.c +++ b/drivers/media/video/cx88/cx88-video.c @@ -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, - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
