<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>