* Geert Uytterhoeven | 2008-04-21 11:55:56 [+0200]:

On Mon, 21 Apr 2008, Sebastian Siewior wrote:

quoted

+static int carmine_check_var(struct fb_var_screeninfo *var,
+		struct fb_info *info)
+{
+
+	if (var->xres_virtual < var->xres)
+		var->xres_virtual = var->xres;
+	if (var->yres_virtual < var->yres)
+		var->yres_virtual = var->yres;
+
+	var->bits_per_pixel = 32;
+
+#ifdef __BIG_ENDIAN
+	var->transp.offset = 24;
+	var->red.offset = 0;
+	var->green.offset = 8;
+	var->blue.offset = 16;
+#else
+	var->transp.offset = 24;
+	var->red.offset = 16;
+	var->green.offset = 8;
+	var->blue.offset = 0;
+#endif
+
+	var->red.length = 8;
+	var->green.length = 8;
+	var->blue.length = 8;
+	var->transp.length = 8;
+
+	var->red.msb_right = 0;
+	var->green.msb_right = 0;
+	var->blue.msb_right = 0;
+	var->transp.msb_right = 0;
+
+	return 0;
+}

So carmine_check_var() always returns success, even if the user
requested for a video mode that is not supported?

I though *var points to something that is from my carmine_modedb[].

quoted

+static int carmine_set_par(struct fb_info *info)
+{
+	struct carmine_fb *par = info->par;
+
+	par->new_mode = carmine_find_mode(&info->var);
+
+	if (par->cur_mode != par->new_mode) {
+
+		par->cur_mode = par->new_mode;
+		par->res = &car_modes[par->new_mode];
+
+		carmine_init_display_param(par);
+		set_display_parameters(par);
+	}
+
+	info->fix.line_length = info->var.xres * info->var.bits_per_pixel / 8;
+	return 0;
+}

... in which case you just get the default mode, but info->var still
contains the parameters for the mode that's not supported?

Oh. Okey, so let me change carmine_check_var() and carmine_set_par() to
fail in case of an unsupported mode.

Gr{oetje,eeting}s,

					Geert

Sebastian

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone