From: Marc Deslauriers <marc.deslauriers@ubuntu.com>
To: Hans de Goede <hdegoede@redhat.com>
Cc: linux-media@vger.kernel.org
Subject: Re: [PATCH] v4l-utils: libv4l1: When asked for RGB, return RGB and not BGR
Date: Mon, 18 Oct 2010 10:42:56 -0400 [thread overview]
Message-ID: <1287412976.6471.49.camel@mdlinux> (raw)
In-Reply-To: <4CBC5AC0.0@redhat.com>
On Mon, 2010-10-18 at 16:33 +0200, Hans de Goede wrote:
> Hi,
>
> NACK
>
> The byte ordering in v4l1's VIDEO_PALETTE_RGB24 was never really
> clear, but the kernel v4l1 compatibility ioctl handling has
> been mapping VIDEO_PALETTE_RGB24 <-> V4L2_PIX_FMT_BGR24
> for ever and many v4l1 apps actually expect VIDEO_PALETTE_RGB24
> to be BGR24. The only one I know of to get this wrong is camorama
> and the solution there is to:
> 1) not use camorama
> 2) if you use camorama anyway, fix it, there is a list of patches
> fixing various issues available here:
> http://pkgs.fedoraproject.org/gitweb/?p=camorama.git;a=tree
>
> Regards,
>
I guess you can add sane to the list of apps that actually expect RGB24
to be RGB24. I was trying to debug why colors were inverted in the sane
v4l backend when I did this.
Marc.
prev parent reply other threads:[~2010-10-18 14:43 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-18 12:44 [PATCH] v4l-utils: libv4l1: When asked for RGB, return RGB and not BGR Marc Deslauriers
2010-10-18 14:33 ` Hans de Goede
2010-10-18 14:42 ` Marc Deslauriers [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=1287412976.6471.49.camel@mdlinux \
--to=marc.deslauriers@ubuntu.com \
--cc=hdegoede@redhat.com \
--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.