[PATCH] -rc4 : tridentfb compilation fix

From: Jani Monoses (jani@astechnix.cluj.astral.ro)
Date: Fri Feb 22 2002 - 09:44:26 EST


Hi Marcelo

this replaces floating point with fixed point in tridentfb so it compiles
and also contains the strtok -> strsep change made by the janitors.
Please apply..

I only discovered a couple of hours ago while trying rc3 that tridentfb
didn't compile - I didn't know about the FPU issue when submitting.

Thanks
Jani.

--- linux/tridentfb.c Fri Feb 22 13:50:10 2002
+++ linux/drivers/video/tridentfb.c Fri Feb 22 16:28:30 2002
@@ -35,7 +35,7 @@

 #include "tridentfb.h"

-#define VERSION "0.6.8"
+#define VERSION "0.6.9"

 struct tridentfb_par {
         struct fb_var_screeninfo var;
@@ -520,9 +520,8 @@
         write3X4(CRTHiOrd, (read3X4(CRTHiOrd) & 0xF8) | (base & 0xE0000) >> 17);
 }

-
-#error "Floating point maths. This needs fixing before the driver is safe"
-#define calc_freq(n,m,k) ((NTSC * (n+8))/((m+2)*(1<<k)))
+/* Use 20.12 fixed-point for NTSC value and frequency calculation */
+#define calc_freq(n,m,k) ( (unsigned long)0xE517 * (n+8) / (m+2)*(1<<k) )

 /* Set dotclock frequency */
 static void set_vclk(int freq)
@@ -1260,8 +1259,8 @@
         char * opt;
         if (!options || !*options)
                 return 0;
- for(opt = strtok(options,",");opt;opt = strtok(NULL,",")){
- if (!opt) continue;
+ while((opt = strsep(&options,",")) != NULL ) {
+ if (!*opt) continue;
                 if (!strncmp(opt,"noaccel",7))
                         noaccel = 1;
                 else if (!strncmp(opt,"accel",5))

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sat Feb 23 2002 - 21:00:41 EST