All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: James Simmons <jsimmons@infradead.org>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
	Linux Fbdev development list
	<linux-fbdev-devel@lists.sourceforge.net>
Subject: Re: set_par/check-var vs. fbcon
Date: Thu, 30 Oct 2003 12:16:45 +1100	[thread overview]
Message-ID: <1067476604.27090.37.camel@gaston> (raw)
In-Reply-To: <Pine.LNX.4.44.0310292220340.21040-100000@phoenix.infradead.org>


>    We have the ability to create a var based on height, width and another 
> variable. When we lack EDID data we use the GTF formulas. The function for
> this is fb_get_mode in fbmon.c. Now when we do have EDID info we use the function. 
> For this we have fb_get_monitor_limits. 
>    The monitor info is very important. It is used to validate or invalidate a mode.  
> The next set of functions that are important are:
> 
> fb_validate_mode		-> Used in all cases. Validates a var.
> fb_create_modedb
> fb_destroy_modedb

Well, I know we have those functions, but that doesn't help me
about how we shall use them. Specifically in radeonfb where I
do have the modedb based on EDID at hand (provided I could probe
it, if not, I want to default to the std VESA modes, not some
GTF formula).

The point I'm trying to make is that we are on one side called with
only width/height valid basically (by fbcon) or with a full mode with
timings (from userland ioctl). How do we choose what to do ? Also
userland may want to use our ability to "calculate" a mode based
only on H/W, or may want us to pick one with H/W and a freq, or may
provide us with a full mode with all timings in there. There is
currently no way to make a difference from within the driver on what
data in the passed in "var" are actually to be used to make the
actual mode.

Ben.




-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?   SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/

  reply	other threads:[~2003-10-30  1:17 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-10-28 22:19 set_par/check-var vs. fbcon Benjamin Herrenschmidt
2003-10-29  9:37 ` Geert Uytterhoeven
2003-10-29 22:09   ` Benjamin Herrenschmidt
2003-10-29 22:39     ` James Simmons
2003-10-30  1:16       ` Benjamin Herrenschmidt [this message]
2003-10-30  9:16         ` Geert Uytterhoeven

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=1067476604.27090.37.camel@gaston \
    --to=benh@kernel.crashing.org \
    --cc=geert@linux-m68k.org \
    --cc=jsimmons@infradead.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.