On Mon, Jul 20, 2015 at 05:03:13PM +0100, Emil Velikov wrote: > Hi Thomas, > > On 20 July 2015 at 09:37, Thomas Klausner <w...@netbsd.org> wrote: > > From: Matthew Green <m...@netbsd.org> > > > > Signed-off-by: Thomas Klausner <w...@netbsd.org> > > --- > > src/geode.h | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/src/geode.h b/src/geode.h > > index ac69368..244cadf 100644 > > --- a/src/geode.h > > +++ b/src/geode.h > > @@ -28,7 +28,9 @@ > > #define _GEODE_H_ > > > > #include "geode_pcirename.h" > > +#ifdef HAVE_CONFIG_H > > #include "config.h" > > +#endif > > > While you're here, can you move the config.h inclusion before the > geode_pcirename.h one ?
Sure, attached. Thomas
>From 4968210734eca510133f035a7318a6f124d689a2 Mon Sep 17 00:00:00 2001 From: Matthew Green <m...@netbsd.org> Date: Mon, 20 Jul 2015 10:37:16 +0200 Subject: [PATCH:xf86-video-geode] Properly hide config.h inclusion inside HAVE_CONFIG_H. Signed-off-by: Thomas Klausner <w...@netbsd.org> --- src/geode.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/geode.h b/src/geode.h index ac69368..815efc7 100644 --- a/src/geode.h +++ b/src/geode.h @@ -27,8 +27,10 @@ #ifndef _GEODE_H_ #define _GEODE_H_ -#include "geode_pcirename.h" +#ifdef HAVE_CONFIG_H #include "config.h" +#endif +#include "geode_pcirename.h" #ifdef HAVE_XAA_H #include "xaa.h" -- 2.4.6
_______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel