On Tue, May 02, 2017 at 12:53:16PM +0300, Petri Latvala wrote: > On Wed, Apr 19, 2017 at 01:01:47PM +0200, Arkadiusz Hiler wrote: > > Also igt_chamelium.h included config.h without proper "HAVE_CONFIG_H" > > guard, and the file itself was included unconditionally. > > I see unconditional config.h inclusion in several other places, > is igt_chamelium.h the only file where it causes problems? Yes that we the reason for this ifdef, but that might have been fixed in other way by different patch as I can't reproduce the error anymore. I'll drop that from this patch and later we can have one dedicated to adding ifdefs everywhere. The approach with config_android.h would need that, so that would be good series to do it. -- Cheers, Arek _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx