Re: [PULL] http://linuxtv.org/hg/~eandren/v4l-dvb/

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

 



Em Sun, 05 Jul 2009 21:25:09 +0200
Erik Andrén <erik.andren@xxxxxxxxx> escreveu:

> 
> 
> Mauro Carvalho Chehab wrote:
> > Em Sat, 4 Jul 2009 21:58:31 +0200
> > Erik Andrén <erik.andren@xxxxxxxxx> escreveu:
> > 
> >> 08/24: gspca - m5602-ov7660: Create blue gain control
> >> http://linuxtv.org/hg/~eandren/v4l-dvb?cmd=changeset;node=5675978999c5
> >>
> >> 09/24: gspca - m5602-ov7660: Add red gain control
> >> http://linuxtv.org/hg/~eandren/v4l-dvb?cmd=changeset;node=802e9a025e93
> > 
> > +#define RED_BALANCE_IDX 3
> > +       {
> > +               {
> > +                       .id             = V4L2_CID_RED_BALANCE,
> > +                       .type           = V4L2_CTRL_TYPE_INTEGER,
> > +                       .name           = "red balance",
> > +                       .minimum        = 0x00,
> > +                       .maximum        = 0x7f,
> > +                       .step           = 0x1,
> > +                       .default_value  = OV7660_DEFAULT_RED_GAIN,
> > +                       .flags          = V4L2_CTRL_FLAG_SLIDER
> > +               },
> > +               .set = ov7660_set_red_gain,
> > +               .get = ov7660_get_red_gain
> > +       },
> >  };
> > x
> > Hmm... as far as I understand, "Red Balance" and "Red gain" are different ways
> > to see the same measure. Unfortunately, the V4L2 API is not clear about those
> > controls. 
> > 
> > According with the spec, we have:
> > 
> > V4L2_CID_CONTRAST	integer	Picture contrast or luma gain.
> > V4L2_CID_SATURATION	integer	Picture color saturation or chroma gain.
> > V4L2_CID_HUE		integer	Hue or color balance.
> > V4L2_CID_RED_BALANCE	integer	Red chroma balance.
> > V4L2_CID_BLUE_BALANCE	integer	Blue chroma balance.
> > V4L2_CID_GAIN		integer	Gain control.
> > 
> > From what I'm understanding from the term "balance", it should be a shift over
> > the gain control (so, 0 means normal colors, like on HUE balance).
> > 
> > So, in order to convert from a RED GAIN into a RED BALANCE, we need to
> > calculate it as a function of the V4L2_CID_GAIN or V4L2_CID_SATURATION.
> > Positive values should mean more gain than the other colors, while negative
> > values would mean the opposite.
> 
> Is such a function defined somewhere? Do you have an example
> implementing it?

No, I don't have. We should better analize it, since drivers are doing
different things with the gains.

> I've done the same thing in the m5602 ov9650 sensor cod



Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux