linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Paulo Marques <pmarques@grupopie.com>
To: "Hans-Jürgen Koch" <hjk@linutronix.de>
Cc: Thomas Gleixner <tglx@linutronix.de>,
	linux-fbdev-devel@lists.sourceforge.net,
	linux-arm-kernel@lists.arm.linux.org.uk
Subject: Re: [PATCH] pxafb: Add support for other palette formats
Date: Tue, 03 Jul 2007 16:24:18 +0100	[thread overview]
Message-ID: <468A6A22.90008@grupopie.com> (raw)
In-Reply-To: <200707031703.53102.hjk@linutronix.de>

Hans-Jürgen Koch wrote:
> Am Dienstag 03 Juli 2007 16:46 schrieb Paulo Marques:
>> [...]
>> I've been working on getting the non-palletized 18bpp modes working and 
>> had a preliminar patch that just didn't support packed mode yet (using 
>> 24 bits per pixel).
> 
> Can userspace applications work with such a weired format?
> If yes, I'm looking forward to your patch.

It is not really that weird: rgb is packed 6 bits per component into 18 
bits. Those 18 bits will then occupy the lower bits of a 24 bit value or 
a 32 bit value depending on using packed or unpacked mode.

I'm doing this for the OpenEZX project to handle some motorola phones 
that have 18bpp support.

Motorola uses this mode in their software, so I suppose Qt already 
supports it.

Nevertheless, it seems a really big waste to have such a colorful 
display and then just use a palletized frame-buffer :P

So, if there are some userspace applications not supporting this mode, 
we should address them, but the support has to be in the kernel first... :(

>> We can either apply this patch as is and I'll just merge my work later, 
>> or I can try to merge both patches and send a single patch upstream. 
>> Either way is fine by me.
> 
> It's probably cleaner to merge two small patches than a big one.

Ok. It is also less stressful for me, too ;)

-- 
Paulo Marques - www.grupopie.com

"All I ask is a chance to prove that money can't make me happy."

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/

  reply	other threads:[~2007-07-03 15:24 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-03 14:07 [PATCH] pxafb: Add support for other palette formats Hans-Jürgen Koch
2007-07-03 14:46 ` Paulo Marques
2007-07-03 15:03   ` Hans-Jürgen Koch
2007-07-03 15:24     ` Paulo Marques [this message]
2007-07-03 15:50     ` Lothar Wassmann
2007-07-03 15:58       ` Geert Uytterhoeven
2007-07-12  7:56         ` Antonino A. Daplas
2007-07-12  8:01           ` Antonino A. Daplas
2007-07-03 16:03       ` Paulo Marques
2007-07-03 16:45         ` Hans-Jürgen Koch
2007-07-03 17:21           ` Paulo Marques
2007-07-03 17:44             ` Hans-Jürgen Koch
2007-07-03 18:18               ` Paulo Marques
2007-07-04  7:17                 ` Lothar Wassmann
2007-07-04 11:00                   ` Paulo Marques
2007-07-04  6:56             ` Lothar Wassmann
2007-07-03 16:39       ` Hans-Jürgen Koch
2007-07-03 17:14         ` Lothar Wassmann
2007-07-03 17:33           ` Hans-Jürgen Koch
2007-07-13 12:20     ` Antonino A. Daplas
2007-07-13 12:39       ` Hans-Jürgen Koch
2007-07-03 19:01 ` Clemens Koller
2007-07-03 19:11   ` Hans-Jürgen Koch
2007-07-03 20:36 ` Hans-Jürgen Koch
2007-07-04  6:38   ` eric miao
2007-07-04  7:51   ` Lothar Wassmann

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=468A6A22.90008@grupopie.com \
    --to=pmarques@grupopie.com \
    --cc=hjk@linutronix.de \
    --cc=linux-arm-kernel@lists.arm.linux.org.uk \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    --cc=tglx@linutronix.de \
    /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;
as well as URLs for NNTP newsgroup(s).