On Tue, 3 Mar 2009, Kuninori Morimoto wrote: > > Signed-off-by: Kuninori Morimoto <[email protected]> > --- > drivers/media/video/sh_mobile_ceu_camera.c | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/drivers/media/video/sh_mobile_ceu_camera.c > b/drivers/media/video/sh_mobile_ceu_camera.c > index 96cf857..bb10899 100644 > --- a/drivers/media/video/sh_mobile_ceu_camera.c > +++ b/drivers/media/video/sh_mobile_ceu_camera.c > @@ -107,6 +107,8 @@ static unsigned long make_bus_param(struct > sh_mobile_ceu_dev *pcdev) > > flags = SOCAM_MASTER | > SOCAM_PCLK_SAMPLE_RISING | > + SOCAM_DATA_ACTIVE_HIGH | > + SOCAM_DATA_ACTIVE_LOW | > SOCAM_HSYNC_ACTIVE_HIGH | > SOCAM_HSYNC_ACTIVE_LOW | > SOCAM_VSYNC_ACTIVE_HIGH | > -- > 1.5.6.3
Please, have a look at the current V4L/DVB next tree at git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-next.git I already added the SOCAM_DATA_ACTIVE_HIGH flag to sh-mobile-ceu flags, because otherwise it would be a regression. Sorry, I should have told you, that I had to extend your patch a bit. As for supporting SOCAM_DATA_ACTIVE_LOW too - does, for instance, sh7722 really support that? I remember I had a short look at it at that time and didn't find any way to switch. If it does, and if you want to support it, you have to also add support for switching between the two polarities. For now a NAK. Thanks Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer -- 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
