On Fri, Feb 10, 2012 at 10:19:16PM -0800, Kenneth Graunke wrote: > On my system, sys/fcntl.h contains exactly one line: > > #include <fcntl.h> > > So there's really no need to #ifdef it. Also, intel_mmio.c already > included <fcntl.h>; there's no need to include it twice. > > Signed-off-by: Kenneth Graunke <kenneth at whitecape.org> Thanks for the patch, this looks much nicer. -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48