Re: [PATCH 01/10] fbdev: Clean up of sparc FB options

["Antonino A. Daplas" <adaplas@gmail.com>]

On Wed, 2007-05-16 at 14:47 -0700, David Miller wrote:
> From: "Antonino A. Daplas" <adaplas@gmail.com>
> Date: Thu, 17 May 2007 05:17:34 +0800
> 
> > From: Krzysztof Helt <krzysztof.h1@wp.pl>
> > 
> > This patch puts all SBUS/UPA selection under one option "SBUS/UPA
> > framebuffers" and moves all sparc specific drivers next to them
> > in one group.
> > 
> > Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
> > Signed-off-by: Antonino Daplas <adaplas@gmail.com>
> 
> Thanks for cleaning this up.
> 
> Signed-off-by: David S. Miller <davem@davemloft.net>
> 
> On a related note I've been trying to figure out how to solve a
> particular issue on sparc, perhaps you can help?
> 
> The firmware has properties that we could use to determine exactly
> which framebuffer device is being used as the console in multi-head
> situations.
> 
> Currently there is no easy way to make use of that so the user just
> gets whatever the framebuffer device probing order gives them which
> isn't nice and is often wrong.
> 

This is a potential problem for the x86 too, actually.

> One idea I have is to provide a way to set some flag in the
> fbinfo.  The flag would indicate that this framebuffer should
> be used as the primary console by default.  Command line options
> et al. could of course override this.

A flag may be the easiest. But we can also define an arch-specific
function (ie, fb_get_primary_device()) in asm/fb.h. With the x86, this
can be easily done by checking the IORESOURCE_ROM_SHADOW bit flag.  

I'll add this for the x86, and then perhaps others can follow suit.

Tony  



-------------------------------------------------------------------------
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/