2010-08-10 20:35:34

by Geert Uytterhoeven

[permalink] [raw]
Subject: asm/vga.h (was: Re: drm: Add support for platform devices to register as DRM devices)

On Fri, Aug 6, 2010 at 01:59, Linux Kernel Mailing List
<[email protected]> wrote:
> Gitweb:     http://git.kernel.org/linus/dcdb167402cbdca1d021bdfa5f63995ee0a79317
> Commit:     dcdb167402cbdca1d021bdfa5f63995ee0a79317
> Parent:     01d73a6967f12fe6c4bbde1834a9fe662264a2eb
> Author:     Jordan Crouse <[email protected]>
> AuthorDate: Thu May 27 13:40:25 2010 -0600
> Committer:  Dave Airlie <[email protected]>
> CommitDate: Tue Jun 1 10:07:39 2010 +1000
>
>    drm: Add support for platform devices to register as DRM devices

> --- a/drivers/gpu/drm/Kconfig
> +++ b/drivers/gpu/drm/Kconfig
> @@ -6,7 +6,7 @@
>  #
>  menuconfig DRM
>        tristate "Direct Rendering Manager (XFree86 4.1.0 and higher DRI support)"
> -       depends on (AGP || AGP=n) && PCI && !EMULATED_CMPXCHG && MMU
> +       depends on (AGP || AGP=n) && !EMULATED_CMPXCHG && MMU

Since this change, I can enable DRM on m68k, but it fails with:

| In file included from drivers/gpu/drm/drm_irq.c:42:
| include/linux/vgaarb.h:34:21: error: asm/vga.h: No such file or directory

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

Alternatively, I can provide an empty <asm/vga.h> (cfr. frv, mn10300,
parisc, sh)
or a simple (probably incorrect) one that includes <asm-generic/vga.h>
(cfr. microblaze).
But so far we never needed <asm/vga.h> on m68k.

For the record, this "problem" has been visible in linux-next (see
m68k/allmodconfig)
since July 9...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

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


2010-08-10 21:51:22

by Adam Jackson

[permalink] [raw]
Subject: Re: asm/vga.h (was: Re: drm: Add support for platform devices to register as DRM devices)

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


Attachments:
signature.asc (198.00 B)
This is a digitally signed message part

2010-08-11 23:18:16

by David Airlie

[permalink] [raw]
Subject: Re: asm/vga.h (was: Re: drm: Add support for platform devices to register as DRM devices)

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
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/dri-devel