> > We never had a modedb in the old code. The timing is hardcoded in the
> > driver from all I can see. I was under the impression the fbcon code needs
> > to have the modedb entry for some purpose; atafb sure can do without
> > otherwise.
>
> But at least the various old *_encode_bar() routines filled in the values, so
> you can use these for your modedb array.

Well, sort of. The TT driver filled in fixed data, the Falcon driver
calculates them according to the monspecs. I've done a bit of head
scratching and a few educated guesses. Untested modedb patch:

--- drivers/video/atafb.c.geert 2006-12-15 18:31:21.000000000 +0100
+++ drivers/video/atafb.c       2006-12-15 18:32:02.000000000 +0100
@@ -519,15 +519,6 @@

 static int num_atafb_predefined=ARRAY_SIZE(atafb_predefined);

-       /*
-        * Tags used to indicate a specific Pixel Clock
-        *
-        * tag is the shift value to get the timings in 35 ns units
-        */
-
-enum { TAG_SHRES, TAG_HIRES, TAG_LORES };
-
-
 static struct fb_videomode atafb_modedb[] __initdata = {

     /*
@@ -542,27 +533,27 @@

     {
        /* 320x200, 15 kHz, 60 Hz (ST low) */
-       "st-low", 60, 320, 200, TAG_HIRES, 106, 86, 44, 16, 76, 2,
+       "st-low", 60, 320, 200, 32000, 32, 16, 31, 14, 96, 4,
        0, FB_VMODE_NONINTERLACED | FB_VMODE_YWRAP
     }, {
        /* 640x200, 15 kHz, 60 Hz (ST medium) */
-       "st-mid", 60, 640, 200, TAG_HIRES, 106, 86, 88, 33, 76, 4,
+       "st-mid", 60, 640, 200, 32000, 32, 16, 31, 14, 96, 4,
        0, FB_VMODE_NONINTERLACED | FB_VMODE_YWRAP
     }, {
-       /* 640x400, 15 kHz, 60 Hz (ST high) */
-       "st-high", 60, 640, 400, TAG_HIRES, 106, 86, 40, 14, 76, 2,
+       /* 640x400, 30.25 kHz, 63.5 Hz (ST high) */
+       "st-high", 63, 640, 400, 32000, 128, 0, 40, 14, 128, 4,
        0, FB_VMODE_NONINTERLACED | FB_VMODE_YWRAP
     }, {
        /* 320x480, 15 kHz, 60 Hz (TT low) */
-       "tt-low", 60, 320, 480, TAG_HIRES, 106, 86, 80, 29, 76, 4,
+       "tt-low", 60, 320, 480, 31041, 120, 100, 8, 16, 140, 30,
        0, FB_VMODE_NONINTERLACED | FB_VMODE_YWRAP
     }, {
        /* 640x480, 29 kHz, 57 Hz (TT medium) */
-       "tt-mid", 60, 640, 480, TAG_SHRES, 96, 112, 29, 8, 72, 8,
+       "tt-mid", 60, 640, 480, 31041, 120, 100, 8, 16, 140, 30,
        0, FB_VMODE_NONINTERLACED | FB_VMODE_YWRAP
     }, {
        /* 1280x960, 29 kHz, 60 Hz (TT high) */
-       "tt-high", 57, 640, 960, TAG_SHRES, 96, 112, 58, 16, 72, 16,
+       "tt-high", 57, 640, 960, 31041, 120, 100, 8, 16, 140, 30,
        0, FB_VMODE_NONINTERLACED | FB_VMODE_YWRAP
     },

@@ -572,11 +563,11 @@

     {
        /* 640x480, 31 kHz, 60 Hz (VGA) */
-       "vga", 60, 640, 480, TAG_SHRES, 64, 96, 30, 9, 112, 2,
+       "vga", 63.5, 640, 480, 32000, 18, 42, 31, 11, 96, 3,
        0, FB_VMODE_NONINTERLACED | FB_VMODE_YWRAP
     }, {
        /* 640x400, 31 kHz, 70 Hz (VGA) */
-       "vga70", 70, 640, 400, TAG_SHRES, 64, 96, 35, 12, 112, 2,
+       "vga70", 70, 640, 400, 32000, 18, 42, 31, 11, 96, 3,
        FB_SYNC_VERT_HIGH_ACT | FB_SYNC_COMP_HIGH_ACT, FB_VMODE_NONINTERLACED | 
FB_VMODE_YWRAP
     },

@@ -586,7 +577,7 @@

     {
        /* 896x608, 31 kHz, 60 Hz (Falcon High) */
-       "falh", 60, 896, 608, TAG_SHRES, 64, 96, 30, 9, 112, 2,
+       "falh", 60, 896, 608, 32000, 18, 42, 31, 1, 96,3,
        0, FB_VMODE_NONINTERLACED | FB_VMODE_YWRAP
     },

Signed-Off-By: <[EMAIL PROTECTED]>

        Michael

-
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to