[PATCH:xf86-video-geode] Properly hide config.h inclusion inside HAVE_CONFIG_H.

[Thomas Klausner]

On Wed, Jul 22, 2015 at 03:05:28PM +0100, Emil Velikov wrote:
> On 21 July 2015 at 22:12, Thomas Klausner <wiz at netbsd.org> wrote:
> > On Mon, Jul 20, 2015 at 05:03:13PM +0100, Emil Velikov wrote:
> >> Hi Thomas,
> >>
> >> On 20 July 2015 at 09:37, Thomas Klausner <wiz at netbsd.org> wrote:
> >> > From: Matthew Green <mrg at NetBSD.org>
> >> >
> >> > Signed-off-by: Thomas Klausner <wiz at 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.
> Personally, I'd mention something like "v2: make config.h inclusion
> the first one". But either way
> 
> Reviewed-by: Emil Velikov <emil.l.velikov at gmail.com>

Sure, I've amended the commit message in the suggested way and pushed
the change.

Thanks for the review,
 Thomas