From: Hans de Goede <hdegoede@redhat.com>
To: "DongSoo(Nathaniel) Kim" <dongsoo.kim@gmail.com>
Cc: Adam Baker <linux@baker-net.org.uk>,
linux-media@vger.kernel.org,
Jean-Francois Moine <moinejf@free.fr>,
Hans Verkuil <hverkuil@xs4all.nl>,
kilgota@banach.math.auburn.edu,
Olivier Lorin <o.lorin@laposte.net>,
Mauro Carvalho Chehab <mchehab@infradead.org>,
Trent Piepho <xyzzy@speakeasy.org>,
linux-omap@vger.kernel.org
Subject: Re: [RFC] How to pass camera Orientation to userspace
Date: Wed, 18 Feb 2009 15:36:28 +0100 [thread overview]
Message-ID: <499C1CEC.5030705@redhat.com> (raw)
In-Reply-To: <5e9665e10902171810v45d0f454ucad4c1c10deca8c4@mail.gmail.com>
DongSoo(Nathaniel) Kim wrote:
> Hello Adam,
>
> I've been thinking exactly the same issue not usb but SoC based camera.
> I have no idea about how usb cameras work but I am quite curious about
> is it really possible to make proper orientation with only querying
> camera driver.
> Because in case of SoC based camera device, many of camera ISPs are
> supporting VFLIP, HFLIP register on their own, and we can read current
> orientation by reading those registers.
>
> But the problem is ISP's registers are set as not flipped at all but
> it physically mounted upside down, because the H/W vendor has packed
> the camera module upside down. (it sounds ridiculous but happens
> sometimes)
That happens a lot with webcams too. Given that these SoC systems will come
with some board specific config anyways, all that is needed is to pass some
boardconfig in to the camera driver (through platform data for example) which
tells the camera driver that on this board the sensor is mounted upside down.
> So in that case when we query orientation of camera, it returns not
> flipped vertically or horizontally at all but actually it turns out to
> be upside down. Actually we are setting camera device to be flipped
> for default in that case.
Ack, but the right thing to do is not to set the vflip and hflip video4linux2
controls on by default, but to invert their meaning, so when the sensor is
upside down, the hflip and vflip controls as seen by the application through
the v4l2 API will report not flipping, but the hwcontrols will actually be set
to flipping, and when an app enables flipping at the v4l2 API level it will
actually gets disables at the HW level, this way the upside downness is 100%
hidden from userspace. So your problem does not need any of the new API we are
working on. The new API is for when the hardware cannot flip and we need to
tell userspace to correct for this in software.
Regards,
Hans
next prev parent reply other threads:[~2009-02-18 14:38 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-18 0:30 [RFC] How to pass camera Orientation to userspace Adam Baker
2009-02-18 2:10 ` DongSoo(Nathaniel) Kim
2009-02-18 14:36 ` Hans de Goede [this message]
2009-02-18 20:45 ` Dongsoo Kim
2009-02-21 11:53 ` Hans Verkuil
2009-02-22 11:17 ` Hans de Goede
2009-02-22 11:53 ` Hans Verkuil
2009-02-22 12:21 ` Hans de Goede
2009-02-22 18:42 ` kilgota
2009-02-22 18:58 ` Hans de Goede
2009-02-22 20:01 ` kilgota
2009-02-22 21:57 ` Hans de Goede
2009-02-22 22:47 ` kilgota
2009-02-22 22:51 ` Trent Piepho
2009-02-22 22:54 ` Hans de Goede
2009-02-22 23:12 ` Trent Piepho
2009-02-22 23:27 ` Hans de Goede
2009-02-23 0:19 ` Trent Piepho
2009-02-23 8:23 ` Hans de Goede
2009-02-22 23:24 ` Hans Verkuil
2009-02-22 23:56 ` Trent Piepho
2009-02-23 7:34 ` Hans Verkuil
2009-02-23 11:30 ` Mauro Carvalho Chehab
2009-02-22 21:46 ` Adam Baker
2009-02-23 11:07 ` Mauro Carvalho Chehab
2009-02-23 22:37 ` Adam Baker
2009-02-24 0:51 ` kilgota
2009-02-24 20:23 ` Mauro Carvalho Chehab
2009-02-25 0:38 ` kilgota
2009-02-25 0:53 ` Mauro Carvalho Chehab
2009-02-25 2:12 ` kilgota
2009-02-25 3:16 ` Mauro Carvalho Chehab
2009-02-25 6:27 ` kilgota
2009-02-25 3:03 ` Thomas Kaiser
2009-02-25 6:19 ` kilgota
2009-02-25 13:11 ` Thomas Kaiser
2009-02-25 7:40 ` Hans de Goede
2009-03-01 23:45 ` [PATCH] libv4lconvert support for SQ905C decompression kilgota
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=499C1CEC.5030705@redhat.com \
--to=hdegoede@redhat.com \
--cc=dongsoo.kim@gmail.com \
--cc=hverkuil@xs4all.nl \
--cc=kilgota@banach.math.auburn.edu \
--cc=linux-media@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux@baker-net.org.uk \
--cc=mchehab@infradead.org \
--cc=moinejf@free.fr \
--cc=o.lorin@laposte.net \
--cc=xyzzy@speakeasy.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox