small radeon frame buffer issues

small radeon frame buffer issues

[Hendrik Sattler]

[-- Attachment #1: Type: text/plain, Size: 1129 bytes --]

Hi,

I write this to you because you are also in the MAINTAINERS file for 
linux-2.6.11.
I have some issues with the radeonfb, rather minor but I want to tell you 
anyways. I am not subscribed to the list, so please CC me when answering.

1.
RADEON FRAMEBUFFER DISPLAY DRIVER
P:      Ani Joshi
M:      ajoshi@shell.unixbox.com
is unreachable. The file should be updated.

2.
I tried with
video=radeonfb:1024x768-24@60
and this resulted in no framebuffer console at all. 24bpp is a common setting 
for X and in my opinion, the driver should just do the right thing with the 
request (e.g. map it 32bpp if necessary).
Or at least you could print a kernel message in the default case that the 
selected bpp value is not supported.

3.
radeonfb is mentioned in no file in Documentation/fb/. This lets one guess on
how to use it. Please add radeonfb to the supported list in 
Documentation/fb/modedb.txt

Thanks.

Hendrik Sattler

-- 
Mein GPG-Key ist auf meiner Homepage verfügbar: http://www.hendrik-sattler.de
        oder über pgp.net

PingoS - Linux-User helfen Schulen: http://www.pingos.org

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

Re: small radeon frame buffer issues

[Benjamin Herrenschmidt]

On Sun, 2005-04-10 at 11:59 +0200, Hendrik Sattler wrote:
> Hi,
> 
> I write this to you because you are also in the MAINTAINERS file for 
> linux-2.6.11.
> I have some issues with the radeonfb, rather minor but I want to tell you 
> anyways. I am not subscribed to the list, so please CC me when answering.
> 
> 1.
> RADEON FRAMEBUFFER DISPLAY DRIVER
> P:      Ani Joshi
> M:      ajoshi@shell.unixbox.com
> is unreachable. The file should be updated.

Indeed, Andrew, please put that in -mm:

This patch does the long overdue updates to MAINTAINERS file for
aty128fb and radeonfb.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>

Index: linux-work/MAINTAINERS
===================================================================
--- linux-work.orig/MAINTAINERS	2005-04-05 14:22:32.000000000 +1000
+++ linux-work/MAINTAINERS	2005-04-11 09:14:58.000000000 +1000
@@ -1860,14 +1860,14 @@
 S:	Maintained
 
 RADEON FRAMEBUFFER DISPLAY DRIVER
-P:	Ani Joshi
-M:	ajoshi@shell.unixbox.com
+P:	Benjamin Herrenschmidt
+M:	benh@kernel.crashing.org
 L:	linux-fbdev-devel@lists.sourceforge.net
 S:	Maintained
 
 RAGE128 FRAMEBUFFER DISPLAY DRIVER
-P:	Ani Joshi
-M:	ajoshi@shell.unixbox.com
+P:	Paul Mackerras
+M:	paulus@samba.org
 L:	linux-fbdev-devel@lists.sourceforge.net
 S:	Maintained
 




-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click

Re: small radeon frame buffer issues

[Benjamin Herrenschmidt]

> I tried with
> video=radeonfb:1024x768-24@60
> and this resulted in no framebuffer console at all. 24bpp is a common setting 
> for X and in my opinion, the driver should just do the right thing with the 
> request (e.g. map it 32bpp if necessary).
> Or at least you could print a kernel message in the default case that the 
> selected bpp value is not supported.

Hrm... Is this the "old" or "new" radeonfb ? Also does it actually work
if you just replace 24 with 32 on the kernel command line ?

> 3.
> radeonfb is mentioned in no file in Documentation/fb/. This lets one guess on
> how to use it. Please add radeonfb to the supported list in 
> Documentation/fb/modedb.txt
> 
> Thanks.
> 
> Hendrik Sattler
> 
-- 
Benjamin Herrenschmidt <benh@kernel.crashing.org>



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click