Re: Zooming with V4L2

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

 



Hi Hans,

On Monday 22 November 2010 12:23:49 Hans de Goede wrote:
> On 11/22/2010 11:59 AM, Laurent Pinchart wrote:
> > On Sunday 21 November 2010 00:50:59 Shuzhen Wang wrote:
> >> Hello, Laurent,
> >> 
> >> Thank you for the reply.
> >> 
> >> In our case, most of the time the sensor outputs bigger image size than
> >> the output size, so the ISP hardware does downscaling.
> >> When zooming in, we can do cropping, and less downscaling to achieve the
> >> same output size. All these happen under of the hood of ISP driver.
> >> That's why I said it was like optical zoom to the application.
> >> 
> >> So if "digital zoom == cropping and upscaling", then I don't think my
> >> case fits in digital zoom category.
> > 
> > Digital zoom is cropping and scaling. Whether it's downscaling or
> > upscaling (or even no scaling it all for a specific resolution) is not
> > really relevant here. It depends on the output resolution but it's still
> > digital zoom that should be implemented using the crop API.
> 
> Not sure I agree with this, given that AFAIK no application actually uses
> the crop API, and that almost no mere human seems to understand the crop
> API, I would like to advocate for exporting digital zoom as a separate
> control (for those cases were the hardware adds something to merely doing
> this in software).

I disagree with that :-)

The purpose of V4L drivers is to let userspace control the hardware. If we 
added a digital zoom control drivers would then need to implement policies 
(for instance how do you configure crop settings to get a linear zoom value ?) 
and that doesn't belong to kernelspace.

If the hardware can perform cropping and scaling, let's expose that to 
applications as cropping and scaling. Whether they use it to implement digital 
zoom, or for an entirely different purpose, is up to them. I don't want to see 
drivers exposing cropping and scaling as zoom, pan and tilt CIDs.

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