external_mesa3d/src/intel/isl, branch replicant-6.0 external/mesa3d isl: Fix height calculation in isl_msaa_interleaved_scale_px_to_sa 2016-11-24T16:34:40+00:00 Jordan Justen jordan.l.justen@intel.com 2016-11-15T10:18:25+00:00 154cb647218999bdc1b2535ffdc85baf933e718e No known fixed tests, but it looks like a typo from: commit 8ac99eabb6570f0f3c5f7d7da1332a99ce636362 intel/isl: Add a helper for getting the size of an interleaved pixel Cc: "13.0" <mesa-stable@lists.freedesktop.org> Signed-off-by: Jordan Justen <jordan.l.justen@intel.com> Reviewed-by: Jason Ekstrand <jason@jlekstrand.net> (cherry picked from commit 0ac57afa6fbe59e9fd8eef38365cb3da8ec67f95) 
No known fixed tests, but it looks like a typo from:

commit 8ac99eabb6570f0f3c5f7d7da1332a99ce636362

    intel/isl: Add a helper for getting the size of an interleaved pixel

Cc: "13.0" <mesa-stable@lists.freedesktop.org>
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
(cherry picked from commit 0ac57afa6fbe59e9fd8eef38365cb3da8ec67f95)
isl: use ifndef header guards 2016-10-14T10:53:32+00:00 Emil Velikov emil.velikov@collabora.com 2016-10-11T17:26:20+00:00 3a98bffa5924d5e7d078268cfe5a42c9f58f0470 Signed-off-by: Emil Velikov <emil.velikov@collabora.com> Reviewed-by: Chad Versace <chadversary@chromium.org> 
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Chad Versace <chadversary@chromium.org>
isl: make locally used functions static 2016-10-14T10:53:30+00:00 Emil Velikov emil.velikov@collabora.com 2016-10-11T17:26:19+00:00 4c1c9d62a9503d912f0f4c22e0e5a162c7f2e0d3 Signed-off-by: Emil Velikov <emil.velikov@collabra.com> Reviewed-by: Chad Versace <chadversary@chromium.org> 
Signed-off-by: Emil Velikov <emil.velikov@collabra.com>
Reviewed-by: Chad Versace <chadversary@chromium.org>
isl: trivial include-what-you-want cleanups 2016-10-14T10:53:28+00:00 Emil Velikov emil.velikov@collabora.com 2016-10-11T17:26:18+00:00 4fe6e7f2bd35ead2cf833052285474d3de9ebad8 Noticed while skimming through the files. Signed-off-by: Emil Velikov <emil.velikov@collabora.com> Reviewed-by: Chad Versace <chadversary@chromium.org> 
Noticed while skimming through the files.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Chad Versace <chadversary@chromium.org>
isl/gen7: remove unneeded ISL_DEV_GEN check 2016-10-14T10:53:25+00:00 Emil Velikov emil.velikov@collabora.com 2016-10-11T17:26:17+00:00 eac752e54b5ec1b8d75d8c22c730af8c9c2a3007 The function gen7_format_needs_valign2 has two callers - the gen7 only gen7_choose_valign_el() and isl_gen6_filter_tiling(). The latter of which already guarding the invocation appropriately. To be extra cautious add a couple of asserts alongside the removal of the runtime check. Signed-off-by: Emil Velikov <emil.velikov@collabora.com> Reviewed-by: Chad Versace <chadversary@chromium.org> 
The function gen7_format_needs_valign2 has two callers - the gen7 only
gen7_choose_valign_el() and isl_gen6_filter_tiling(). The latter of
which already guarding the invocation appropriately.

To be extra cautious add a couple of asserts alongside the removal of the
runtime check.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Chad Versace <chadversary@chromium.org>
isl: prefix non-static API with isl_ 2016-10-14T10:53:22+00:00 Emil Velikov emil.velikov@collabora.com 2016-10-11T17:26:16+00:00 5b1efb65ce0282071d98bc471c8dc05dcafe08fa The rest of ISL already follows this approach. Be consistent and resolve the final references. Signed-off-by: Emil Velikov <emil.velikov@collabora.com> Reviewed-by: Chad Versace <chadversary@chromium.org> 
The rest of ISL already follows this approach. Be consistent and resolve
the final references.

Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Chad Versace <chadversary@chromium.org>
isl/gen6: correctly check msaa layout samples count 2016-10-14T10:53:15+00:00 Emil Velikov emil.velikov@collabora.com 2016-10-11T17:26:15+00:00 84f9ef1de4680b48d646841965f963ea2b724ab2 Samples == 1 is a valid value, so returning false is plain wrong. Seeming copy/paste typo introduced since day 1. Fixes: afdadec77f5 ("isl: Implement isl_surf_init() for gen4-gen9") Cc: mesa-stable@lists.freedesktop.org Signed-off-by: Emil Velikov <emil.velikov@collabora.com> Reviewed-by: Jason Ekstrand <jason@jlekstrand.net> Reviewed-by: Chad Versace <chadversary@chromium.org> 
Samples == 1 is a valid value, so returning false is plain wrong.
Seeming copy/paste typo introduced since day 1.

Fixes: afdadec77f5 ("isl: Implement isl_surf_init() for gen4-gen9")
Cc: mesa-stable@lists.freedesktop.org
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Chad Versace <chadversary@chromium.org>
isl: Correct a comment in the isl_format enum 2016-10-07T19:54:17+00:00 Nanley Chery nanley.g.chery@intel.com 2016-10-07T19:07:22+00:00 814fa1237935884c3c0ba47a31e3599b975aec1a HiZ is not a color surface, but an auxiliary depth surface. Signed-off-by: Nanley Chery <nanley.g.chery@intel.com> Reviewed-by: Chad Versace <chadversary@chromium.org> Reviewed-by: Jason Ekstrand <jason@jlekstrand.net> 
HiZ is not a color surface, but an auxiliary depth surface.

Signed-off-by: Nanley Chery <nanley.g.chery@intel.com>
Reviewed-by: Chad Versace <chadversary@chromium.org>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
intel/isl: Add an assert to check zero width/height surface 2016-10-04T20:20:34+00:00 Anuj Phogat anuj.phogat@gmail.com 2016-09-20T18:59:01+00:00 2c78b2ec901ca87698b5f5fadb547e0fc727800d Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com> Reviewed-by: Jason Ekstrand <jason@jlekstrand.net> 
Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
intel/isl: Allow non-2D HiZ surfaces 2016-10-03T21:53:01+00:00 Jason Ekstrand jason.ekstrand@intel.com 2016-09-13T00:56:38+00:00 0176c6a69236466a7f4bd1cb41fc09d88a5b4778 Signed-off-by: Jason Ekstrand <jason@jlekstrand.net> Reviewed-by: Nanley Chery <nanley.g.chery@intel.com> 
Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Nanley Chery <nanley.g.chery@intel.com>