Re: [PATCH] matroxfb: set maxvram of vbG200eW to the same as vbG200 to avoid black screen

[Geert Uytterhoeven]

CC Rich (author of commit in Fixes tag)

On Sun, Dec 19, 2021 at 11:31 PM Z. Liu <liuzx@xxxxxxxxxxxx> wrote:
> Start from commit 11be60bd66d54 "matroxfb: add Matrox MGA-G200eW board
> support", when maxvram is 0x800000, monitor become black w/ error message
> said: "The current input timing is not supported by the monitor display.
> Please change your input timing to 1920x1080@60Hz ...".
>
> Fixes: 1be60bd66d54 ("matroxfb: add Matrox MGA-G200eW board support")
> Cc: linux-fbdev@xxxxxxxxxxxxxxx
> Cc: linux-kernel@xxxxxxxxxxxxxxx
> Signed-off-by: Z. Liu <liuzx@xxxxxxxxxxxx>
> ---
>  drivers/video/fbdev/matrox/matroxfb_base.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/video/fbdev/matrox/matroxfb_base.c b/drivers/video/fbdev/matrox/matroxfb_base.c
> index 5c82611e93d9..236521b19daf 100644
> --- a/drivers/video/fbdev/matrox/matroxfb_base.c
> +++ b/drivers/video/fbdev/matrox/matroxfb_base.c
> @@ -1377,7 +1377,7 @@ static struct video_board vbG200 = {
>         .lowlevel = &matrox_G100
>  };
>  static struct video_board vbG200eW = {
> -       .maxvram = 0x800000,
> +       .maxvram = 0x100000,
>         .maxdisplayable = 0x800000,
>         .accelID = FB_ACCEL_MATROX_MGAG200,
>         .lowlevel = &matrox_G100

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds