Problem with xorg-server and s3c2410fb driver
michel at daenzer.net
Wed Aug 4 02:56:25 PDT 2010
On Mon, 2010-08-02 at 13:56 -0400, Adam Jackson wrote:
> On Mon, 2010-08-02 at 18:24 +0200, Michel Dänzer wrote:
> > AFAICT this will only work when using the 'built-in' mode, otherwise
> > mode->PrivFlags will never be set? One possible solution for that could
> > be
> > - var->pixclock = mode->Clock ? 1000000000/mode->Clock : 0;
> > + var->pixclock = mode->PrivFlags ? mode->PrivFlags :
> > (mode->Clock ? 1000000000/mode->Clock : 0);
> > in xfree2fbdev_timing().
> Right you are.
> So I guess the question is what we're trying to accomplish with the
> fbdev_modes_equal() check at all.
From commit f6815cb68b0f6698497348fc6e4214dacef33b95 which added it:
The fbdev API allows the driver to 'accept' modes it doesn't really support by
modifying it to the nearest supported mode. Without this check, e.g. vesafb
would appear to accept all modes, even though it actually can't set any modes
other than the bootup mode at all.
Earthling Michel Dänzer | http://www.vmware.com
Libre software enthusiast | Debian, X and DRI developer
More information about the xorg