From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Florian Neuhaus <florian.neuhaus@reberinformatik.ch>
Cc: "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>
Subject: Re: AW: omap3-isp-live does not allocate big enough buffers?
Date: Wed, 10 Oct 2012 16:59:17 +0200 [thread overview]
Message-ID: <1417441.VpuOie6hkq@avalon> (raw)
In-Reply-To: <6EE9CD707FBED24483D4CB0162E85467100659AB@AM2PRD0710MB375.eurprd07.prod.outlook.com>
Hi Florian,
On Wednesday 10 October 2012 09:19:37 Florian Neuhaus wrote:
> Laurent Pinchart wrote on 2012-10-08:
> >
> > The OMAP3 ISP resizer can't scale down 1944 pixels (the native sensor
> > height) to exactly 480 pixels as that would exceed the resizer limits.
> > You will thus have to crop the sensor image slightly. Cropping is
> > supported by libomap3isp and by the snapshot application but not by the
> > live application. Ideally the live application or the libomap3isp library
> > should realize that the ISP limits are exceeded and configure cropping on
> > the sensor accordingly. As an interim solution you could add manual crop
> > support to the live application using the snapshot application crop
> > support code as an example.
>
> I have seen, that the resizer "only" supports downscaling by 0.25, so with
> all the cropping, 1944 lines will come down to 482 which is too big for my
> framebuffer. If I apply some cropping in the omap3_isp_viewfinder_setup
> function, the output will work as expected. Now I'm going to crop on the
> sensor (or better on the first entity that supports cropping, as in your
> code) if the ratio "sensor input -> viewfinder output" exceeds 0.25. Are
> you interested in a patch for this?
Sure. I wonder if it wouldn't make more sense to crop on the resizer though,
what do you think ?
--
Regards,
Laurent Pinchart
prev parent reply other threads:[~2012-10-10 14:58 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-08 7:46 omap3-isp-live does not allocate big enough buffers? Florian Neuhaus
2012-10-08 14:06 ` Laurent Pinchart
2012-10-10 9:19 ` AW: " Florian Neuhaus
2012-10-10 14:59 ` Laurent Pinchart [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1417441.VpuOie6hkq@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=florian.neuhaus@reberinformatik.ch \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.