Mauro
Since you still haven't pulled my fixes branch, and I have one more fix,
that I've posted a while ago but didn't commit to linuxtv.org. Can I now
push it to the same 2.6.37-rc3-fixes branch for you to pull from?
Thanks
Guennadi
On Thu, 2 Dec 2010, Guennadi Liakhovetski wrote:
> 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------------{=
> > - bar...@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -
> >
>
> ---
> Guennadi Liakhovetski, Ph.D.
> Freelance Open-Source Software Developer
> http://www.open-technology.de/
>
---
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 majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
