From: Adam Jackson on
On Tue, 2010-08-10 at 22:35 +0200, Geert Uytterhoeven wrote:

> Now, do we really need <asm/vga.h>? It seems I can make it build and boot using:
>
> --- a/include/linux/vgaarb.h
> +++ b/include/linux/vgaarb.h
> @@ -31,7 +31,9 @@
> #ifndef LINUX_VGA_H
> #define LINUX_VGA_H
>
> +#ifdef CONFIG_VGA_ARB
> #include <asm/vga.h>
> +#endif
>
> /* Legacy VGA regions */
> #define VGA_RSRC_NONE 0x00

That seems fine.

- ajax
From: Dave Airlie on
On Tue, 2010-08-10 at 17:51 -0400, Adam Jackson wrote:
> On Tue, 2010-08-10 at 22:35 +0200, Geert Uytterhoeven wrote:
>
> > Now, do we really need <asm/vga.h>? It seems I can make it build and boot using:
> >

I think we can drop asm/vga.h on all platforms, I'll queue a patch up
for drm-next to see if we hit any problems, it works on x86 at least.

Dave.

> > --- a/include/linux/vgaarb.h
> > +++ b/include/linux/vgaarb.h
> > @@ -31,7 +31,9 @@
> > #ifndef LINUX_VGA_H
> > #define LINUX_VGA_H
> >
> > +#ifdef CONFIG_VGA_ARB
> > #include <asm/vga.h>
> > +#endif
> >
> > /* Legacy VGA regions */
> > #define VGA_RSRC_NONE 0x00
>
> That seems fine.
>
> - ajax
> _______________________________________________
> dri-devel mailing list
> dri-devel(a)lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel


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