Re: omap3isp: cropping bug in previewer?

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

 



Hi Florian,

On Friday 06 July 2012 11:55:40 Florian Neuhaus wrote:
> Hi Laurent,
> 
> Laurent Pinchart wrote on 2012-07-05:
> > There's really an issue, which was introduced in v3.5-rc1. Could you
> > please try the following patch instead of yours ?
> > 
> > diff --git a/drivers/media/video/omap3isp/isppreview.c
> > b/drivers/media/video/omap3isp/isppreview.c index 9c6dd44..614752a
> > 100644 --- a/drivers/media/video/omap3isp/isppreview.c +++
> > b/drivers/media/video/omap3isp/isppreview.c @@ -1116,7 +1116,7 @@ static
> > void preview_config_input_size(struct isp_prev_device *prev, u32 active)
> > 
> >  	unsigned int elv = prev->crop.top + prev->crop.height - 1;
> >  	u32 features;
> > 
> > -	if (format->code == V4L2_MBUS_FMT_Y10_1X10) {
> > +	if (format->code != V4L2_MBUS_FMT_Y10_1X10) {
> > 
> >  		sph -= 2;
> >  		eph += 2;
> >  		slv -= 2;
> 
> This patch resolves the issue for me. I get a good picture at both the
> previewer and the resizer output. Thanks for your help!

You're welcome. Can I include your

Tested-by: Florian Neuhaus <florian.neuhaus@xxxxxxxxxxxxxxxxxx>

in the patch ?

-- 
Regards,

Laurent Pinchart

--
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