<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Thanks, pushed 1-6 for now.<br>
</div>
<div class="moz-cite-prefix"><br>
</div>
<div class="moz-cite-prefix">On 08/11/2018 22:42, Jason Ekstrand
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CAOFGe96w2xJ5k2NL=kTWfEz00AsHGWJTGzzWxgEpFaXuNezhCQ@mail.gmail.com">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<div dir="ltr">
<div>I left a nit on 3. Otherwise, 1-6 are</div>
<div><br>
</div>
<div>Reviewed-by: Jason Ekstrand <<a
href="mailto:jason@jlekstrand.net" moz-do-not-send="true">jason@jlekstrand.net</a>><br>
</div>
</div>
<br>
<div class="gmail_quote">
<div dir="ltr">On Mon, Nov 5, 2018 at 9:36 AM Lionel Landwerlin
<<a href="mailto:lionel.g.landwerlin@intel.com"
moz-do-not-send="true">lionel.g.landwerlin@intel.com</a>>
wrote:<br>
</div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">It's only
used in anv_image.c<br>
<br>
Signed-off-by: Lionel Landwerlin <<a
href="mailto:lionel.g.landwerlin@intel.com" target="_blank"
moz-do-not-send="true">lionel.g.landwerlin@intel.com</a>><br>
---<br>
src/intel/vulkan/anv_image.c | 22 ++++++++++++++++++++++<br>
src/intel/vulkan/anv_private.h | 22 ----------------------<br>
2 files changed, 22 insertions(+), 22 deletions(-)<br>
<br>
diff --git a/src/intel/vulkan/anv_image.c
b/src/intel/vulkan/anv_image.c<br>
index 7a06bde8e9e..4fa44aa8704 100644<br>
--- a/src/intel/vulkan/anv_image.c<br>
+++ b/src/intel/vulkan/anv_image.c<br>
@@ -1247,6 +1247,28 @@ remap_aspect_flags(VkImageAspectFlags
view_aspects)<br>
return view_aspects;<br>
}<br>
<br>
+static uint32_t<br>
+anv_image_aspect_get_planes(VkImageAspectFlags aspect_mask)<br>
+{<br>
+ uint32_t planes = 0;<br>
+<br>
+ if (aspect_mask & (VK_IMAGE_ASPECT_COLOR_BIT |<br>
+ VK_IMAGE_ASPECT_DEPTH_BIT |<br>
+ VK_IMAGE_ASPECT_STENCIL_BIT |<br>
+ VK_IMAGE_ASPECT_PLANE_0_BIT))<br>
+ planes++;<br>
+ if (aspect_mask & VK_IMAGE_ASPECT_PLANE_1_BIT)<br>
+ planes++;<br>
+ if (aspect_mask & VK_IMAGE_ASPECT_PLANE_2_BIT)<br>
+ planes++;<br>
+<br>
+ if ((aspect_mask & VK_IMAGE_ASPECT_DEPTH_BIT) != 0
&&<br>
+ (aspect_mask & VK_IMAGE_ASPECT_STENCIL_BIT) != 0)<br>
+ planes++;<br>
+<br>
+ return planes;<br>
+}<br>
+<br>
VkResult<br>
anv_CreateImageView(VkDevice _device,<br>
const VkImageViewCreateInfo *pCreateInfo,<br>
diff --git a/src/intel/vulkan/anv_private.h
b/src/intel/vulkan/anv_private.h<br>
index f8e7122cf25..8f8cbb4415e 100644<br>
--- a/src/intel/vulkan/anv_private.h<br>
+++ b/src/intel/vulkan/anv_private.h<br>
@@ -2584,28 +2584,6 @@
anv_image_aspect_to_plane(VkImageAspectFlags image_aspects,<br>
}<br>
}<br>
<br>
-static inline uint32_t<br>
-anv_image_aspect_get_planes(VkImageAspectFlags aspect_mask)<br>
-{<br>
- uint32_t planes = 0;<br>
-<br>
- if (aspect_mask & (VK_IMAGE_ASPECT_COLOR_BIT |<br>
- VK_IMAGE_ASPECT_DEPTH_BIT |<br>
- VK_IMAGE_ASPECT_STENCIL_BIT |<br>
- VK_IMAGE_ASPECT_PLANE_0_BIT))<br>
- planes++;<br>
- if (aspect_mask & VK_IMAGE_ASPECT_PLANE_1_BIT)<br>
- planes++;<br>
- if (aspect_mask & VK_IMAGE_ASPECT_PLANE_2_BIT)<br>
- planes++;<br>
-<br>
- if ((aspect_mask & VK_IMAGE_ASPECT_DEPTH_BIT) != 0
&&<br>
- (aspect_mask & VK_IMAGE_ASPECT_STENCIL_BIT) != 0)<br>
- planes++;<br>
-<br>
- return planes;<br>
-}<br>
-<br>
static inline VkImageAspectFlags<br>
anv_plane_to_aspect(VkImageAspectFlags image_aspects,<br>
uint32_t plane)<br>
-- <br>
2.19.1<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org"
target="_blank" moz-do-not-send="true">mesa-dev@lists.freedesktop.org</a><br>
<a
href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev"
rel="noreferrer" target="_blank" moz-do-not-send="true">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote>
</div>
</blockquote>
<p><br>
</p>
</body>
</html>