Re: [Linux-fbdev-devel] [PATCH] fbdev: don't allow to set a video mode via vga= if FB doesn't support it

From: Michal Januszewski
Date: Mon Sep 22 2008 - 16:32:52 EST


On Mon, Sep 22, 2008 at 22:00, Bruno PrÃmont <bonbons@xxxxxxxxxxxxxxxxx> wrote:

> This looks pretty much the same as what I requested some time ago:
>
> http://bugzilla.kernel.org/show_bug.cgi?id=8062
>
> Would need to check -mm archives for the actual patch.
>
> Don't remember why it did not make it to mainline...

Well, the bug and the mm fix [1] are about a similar, yet slightly different
issue than the one I'm trying to fix with my patch (note that the issue
reported in the bug is currently fixed in the kernel, but the actual fix
is differrent than the one in -mm due to x86 boot code refactoring
that took place since then).

My patch fixes the case where the framebuffer is enabled, but none of
the drivers support boot-time video mode setting, whereas the old
patch fixed the case where it was possible to set a video mode even
though the kernel didn't contain the framebuffer layer.

[1] http://www.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.21-rc2/2.6.21-rc2-mm1/broken-out/fbdev-ignore-vesa-modes-if-framebuffer-is-disabled.patch

Best regards,
--
Michal Januszewski, Gentoo Linux Developer
http://people.gentoo.org/spock
¢éì®&Þ~º&¶¬–+-±éÝ¥Šw®žË±Êâmébžìdz¹Þ)í…æèw*jg¬±¨¶‰šŽŠÝj/êäz¹ÞŠà2ŠÞ¨è­Ú&¢)ß«a¶Úþø®G«éh®æj:+v‰¨Šwè†Ù>Wš±êÞiÛaxPjØm¶Ÿÿà -»+ƒùdš_