RE: [PATCH 1/13 v2] viafb: viafb.modes, viafb.txt

From: Geert Uytterhoeven
Date: Fri Aug 08 2008 - 11:25:09 EST


On Fri, 8 Aug 2008, JosephChan@xxxxxxxxxx wrote:
> It should be my carelessness. The check has been added in viafbdev.c already.

OK.

> Do I need to re-send the patch for the documentation? Thanks for your reminding.

Please send an incremental patch. Andrew is good in folding ;-)

> From: Geert Uytterhoeven [mailto:geert@xxxxxxxxxxxxxx]
> Sent: Fri 8/8/2008 7:32 PM
> To: Joseph Chan
> Cc: linux-fbdev-devel@xxxxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; akpm@xxxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/13 v2] viafb: viafb.modes, viafb.txt
>
>
>
> On Fri, 8 Aug 2008, JosephChan@xxxxxxxxxx wrote:
> > + 4. VX800 2D accelerator hasn't been supported in this driver yet. When
> > + using driver on VX800, one should add parameter "viafb_accel=0" to
> > + disable 2D accelerator.
>
> Can't the driver detect this automatically, and disable the 2D
> accelerator when running on a VX800?

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