linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Re: fb_imageblit semantic
@ 2003-03-17 14:24 Antonino Daplas
  2003-03-18 21:46 ` Detecting if the mode can be changed? Kendall Bennett
  0 siblings, 1 reply; 2+ messages in thread
From: Antonino Daplas @ 2003-03-17 14:24 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Petr Vandrovec, James Simmons,
	Linux Frame Buffer Device Development

On Mon, 2003-03-17 at 21:47, Geert Uytterhoeven wrote:
>
> IMHO always using an image depth of 8 is fine. That's a nice trade off between
> genericity and easy access of image data.
> 
> Except for monochrome, there I prefer a depth of 1, since doing 1->8 and 8->1
> in a row is a bit too wasteful.
> 

An exception to the rule :-)  Well, I guess if it will significantly
benefit monochrome drivers...

> BTW, I just realized there's no need to distinguish between monochrome copy and
> monochrome-to-color expansion. The monochrome logo code just has to write the
> correct values to fb_image.[fb]gcolor. I.e. all zeroes or all ones (number of
> bits = var.bits_per_pixel, so it works for 8-bit monochrome, too).
> 

I remember mentioning this to you before, and you said that there might
be rare cases that fgcolor can be equal to bgcolor.  However, using -1
instead for bg_color/fg_color may work, at least for the current setup
and only for monochrome, (except perhaps 32-bit monochrome, if there is
such a thing)

I still prefer splitting fb_imageblit() into two though, and still keep
image->depth for logo drawing only, to allow for future expansion.
. 
> > Do we still use the LUT?  
> 
> Yes. An alternative is to enlarge the pseudo palette to 256 entries (if there

Yes.  Which also means logo drawing will also work for quirky drivers,
and the upper layer will be finally totally independent of the low level
drivers.

> are enough number of colors). But since imageblit is done for the logo only,
> doing the transform from LUT index to pixel value in imageblit is OK, I think.
> 

Yes, I also prefer referring to the LUT whatever the format even for
character drawing.  I don't expect any noticeable performance penalty
except for logo drawing.  The way it's currently done is because,
looking at all drivers, not one fills up the pseudo_palette when bpp <=
8.

Tony



-------------------------------------------------------
This SF.net email is sponsored by:Crypto Challenge is now open! 
Get cracking and register here for some mind boggling fun and 
the chance of winning an Apple iPod:
http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2003-03-19 19:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <3E7759E5.8300.528C610@localhost>
     [not found] ` <20030319020755.45526.qmail@web14904.mail.yahoo.com>
2003-03-19 19:42   ` Enumerating available display modes? Kendall Bennett
2003-03-17 14:24 fb_imageblit semantic Antonino Daplas
2003-03-18 21:46 ` Detecting if the mode can be changed? Kendall Bennett
2003-03-18 23:50   ` Enumerating available display modes? Kendall Bennett

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