On Sat, 10 Sep 2016, Dave Gordon <david.s.gordon@xxxxxxxxx> wrote: > Thanks, the other things I was thinking of fixing in the remaining files > were generally things like > > if (INTEL_INFO(dev)->gen < 5 || IS_G33(dev)) Is that a real or a made up example? IS_G33() would be redundant. But I'd like to see the semantic patch to fix that! ;) BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx