[Mesa-dev] [PATCH] intel/genxml: remove SLM Enable bit from Gen11

[Emil Velikov]

Hi Lionel,

On 7 September 2018 at 14:43, Lionel Landwerlin
<lionel.g.landwerlin at intel.com> wrote:
> Documentation says this bit is gone and we've indeed set all the slm
> field to 0 in common/gen_l3_config.c
>
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> ---
>  src/intel/genxml/gen11.xml         | 1 -
>  src/intel/vulkan/genX_cmd_buffer.c | 6 ++++--
>  2 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/src/intel/genxml/gen11.xml b/src/intel/genxml/gen11.xml
> index 1b3befbbfc9..f497c2df825 100644
> --- a/src/intel/genxml/gen11.xml
> +++ b/src/intel/genxml/gen11.xml
> @@ -3544,7 +3544,6 @@
>    </register>
>
>    <register name="L3CNTLREG" length="1" num="0x7034">
> -    <field name="SLM Enable" start="0" end="0" type="uint"/>
Trivial fly-by idea:

Worth adding an inline comment, for fellow developers skimming through
X weeks down the line?

HTH
Emil