On Thu, 2 Dec 2010, Baruch Siach wrote:
> Hi Guennadi,
>
> On Wed, Nov 17, 2010 at 08:09:09AM +0100, Guennadi Liakhovetski wrote:
> > On Wed, 10 Nov 2010, Baruch Siach wrote:
> > > Guennadi Liakhovetski <g.liakhovetski <at> gmx.de> writes:
> > > > On Wed, 27 Oct 2010, Baruch Siach wrote:
> > > > > When SOCAM_PCLK_SAMPLE_FALLING, just leave CSICR1_REDGE unset,
> > > > > otherwise we get
> > > > > the inverted behaviour.
> > > > Seems logical to me, that if this is true, then you need the inverse:
> > > >
> > > > if (!(common_flags & SOCAM_PCLK_SAMPLE_FALLING))
> > > > csicr1 |= CSICR1_INV_PCLK;
> > >
> > > No. Doing so you'll get the inverted behaviour of SAMPLE_RISING. When
> > > common_flags have SAMPLE_RISING set and SAMPLE_FALLING unset you get
> > > CSICR1_REDGE set, which triggers on the rising edge, and then also
> > > CSICR1_INV_PCLK set, which invert this. Thus you get the expected
> > > behaviour of SAMPLE_FALLING.
> > >
> > > Currently you get the inverted behaviour only for SAMPLE_FALLING.
> > >
> > > IMO, we should just use CSICR1_REDGE to set the sample timing, and leave
> > > CSICR1_INV_PCLK alone.
> >
> > Ah, right, of course, I've overlooked that CSICR1_REDGE flag. Then yes,
> > your patch makes sense and should go in for 2.6.37.
>
> Thanks. I now see that this patch is queued in the staging/for_v2.6.38 branch
> of media_tree.git. Is there any chance of seeing this fix in .37-rc?
I actually pushed it as a fix for .37, Mauro?
> Once this gets merged I want to forward it to .36 stable.
AFAIK, the maintainer should handle this too.
Thanks
Guennadi
>
> baruch
>
> > > > > if (common_flags & SOCAM_PCLK_SAMPLE_RISING)
> > > > > csicr1 |= CSICR1_REDGE;
> > > > > - if (common_flags & SOCAM_PCLK_SAMPLE_FALLING)
> > > > > - csicr1 |= CSICR1_INV_PCLK;
> > > > > if (common_flags & SOCAM_VSYNC_ACTIVE_HIGH)
> > > > > csicr1 |= CSICR1_SOF_POL;
> > > > > if (common_flags & SOCAM_HSYNC_ACTIVE_HIGH)
>
> --
> ~. .~ Tk Open Systems
> =}------------------------------------------------ooO--U--Ooo------------{=
> - [email protected] - tel: +972.2.679.5364, http://www.tkos.co.il -
>
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
