Re: [Mesa-dev] [PATCH] intel/genxml: Delete empty groups

2017-11-13 Thread Kenneth Graunke
On Sunday, November 12, 2017 3:43:52 PM PST Jason Ekstrand wrote: > They serve no purpose other than to just fill empty space in the packet > so each dword has something. Just disallowing empty groups is a bit > easier on some of the tools. This does not change the generated packing > headers in

Re: [Mesa-dev] [PATCH] intel/genxml: Delete empty groups

2017-11-13 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 12/11/17 23:43, Jason Ekstrand wrote: They serve no purpose other than to just fill empty space in the packet so each dword has something. Just disallowing empty groups is a bit easier on some of the tools. This does not change the generated packing headers i

[Mesa-dev] [PATCH] intel/genxml: Delete empty groups

2017-11-12 Thread Jason Ekstrand
They serve no purpose other than to just fill empty space in the packet so each dword has something. Just disallowing empty groups is a bit easier on some of the tools. This does not change the generated packing headers in any way. Cc: Lionel Landwerlin --- src/intel/genxml/gen6.xml | 2 -- s