linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bryan Stillwell <bryan@bokeoa.com>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: linux-fbdev-devel@lists.sourceforge.net
Subject: Re: radeonfb and 16bit color problem
Date: Tue, 25 Jun 2002 10:41:17 -0600	[thread overview]
Message-ID: <20020625104117.A31689@bokeoa.com> (raw)
In-Reply-To: <20020624032412.27294@192.168.4.1>; from benh@kernel.crashing.org on Mon, Jun 24, 2002 at 05:24:11AM +0200

On Mon, Jun 24, 2002 at 05:24:11AM +0200, Benjamin Herrenschmidt wrote:
>>Any ideas would be helpful.  I tried looking at the code myself, but I'm
>>still quite new to the framebuffer code.  However, this part looked kind
>>of strange to me:
>>
>>   /* For 565, the green component is mixed one order below */
>>   if (rinfo->depth == 16) {
>>      OUTREG(PALETTE_INDEX, pindex>>1);
>>      OUTREG(PALETTE_DATA, (rinfo->palette[regno>>1].red << 16) |
>>             (green << 8) | (rinfo->palette[regno>>1].blue));
>>      green = rinfo->palette[regno<<1].green;
>>   }
>>
>>
>>BTW, I have a Radeon 8500 (QL).
>
>I wrote that part and it works just fine on my powerbook... at least
>with the radeonfb version that is in my tree, but I don't think it
>differs from the official one in this regard. You can try making sure
>it's properly setting 32 palette entries for green when using that
>mode.

I thought that piece of code looked weird because it's setting green
equal to rinfo->palette[regno<<1].green, but doesn't seem to ever use
green during the rest of the function...

BTW, don't you mean 64 palette entries for green (being that it's 6
bits)?  Anyways, I took some time this weekend to look over the code
some more, and I really doubt my problems originates in that function.
As I understand it, the palette (colormap?) will only be used when
displaying colors in text mode.

My problem seems to affect many more colors (practically all 65536 of
them), but I've yet to track it down.  I put in a request on ATI's
website to get access to the specs (hoping that'll help me), but I
haven't heard anything back from them yet.

Thanks,
Bryan

PS I'll try checking out your tree the next time I get a chance and see
if I still see the problem.


-------------------------------------------------------
This sf.net email is sponsored by: Jabber Inc.
Don't miss the IM event of the season | Special offer for OSDN members! 
JabConf 2002, Aug. 20-22, Keystone, CO http://www.jabberconf.com/osdn

  reply	other threads:[~2002-06-25 16:41 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-21  3:29 radeonfb and 16bit color problem Bryan Stillwell
2002-06-24  3:24 ` Benjamin Herrenschmidt
2002-06-25 16:41   ` Bryan Stillwell [this message]
2002-06-24  5:41     ` Benjamin Herrenschmidt

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=20020625104117.A31689@bokeoa.com \
    --to=bryan@bokeoa.com \
    --cc=benh@kernel.crashing.org \
    --cc=linux-fbdev-devel@lists.sourceforge.net \
    /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).