On 15 March 2018 at 14:24, Lionel Landwerlin <lionel.g.landwer...@intel.com> wrote: > Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> > --- > src/intel/genxml/gen10.xml | 6 ++++++ > src/intel/genxml/gen11.xml | 6 ++++++ > src/intel/genxml/gen6.xml | 7 +++++++ > src/intel/genxml/gen7.xml | 7 +++++++ > src/intel/genxml/gen75.xml | 7 +++++++ > src/intel/genxml/gen8.xml | 7 +++++++ > src/intel/genxml/gen9.xml | 6 ++++++ > 7 files changed, 46 insertions(+) > AFAICT patch 3/4 adds the needed definitions to brw_defines.h, right? Guessing that you were planning to roll patch 5/5, but ran out of time?
Might be a good idea to add a "TODO: use the xml definitions" beside the brw_defines.h macros. HTH Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
