isl/gen6: correctly check msaa layout samples count
authorEmil Velikov <emil.velikov@collabora.com>
Tue, 11 Oct 2016 17:26:15 +0000 (18:26 +0100)
committerEmil Velikov <emil.l.velikov@gmail.com>
Fri, 14 Oct 2016 10:53:15 +0000 (11:53 +0100)
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>
src/intel/isl/isl_gen6.c

index b30998d044dd696aa559b7823251f65a68eae0a6..07c08f6d31e7c13af9efe59c08855520085c1572 100644 (file)
@@ -35,7 +35,7 @@ gen6_choose_msaa_layout(const struct isl_device *dev,
 
    if (info->samples == 1) {
       *msaa_layout = ISL_MSAA_LAYOUT_NONE;
-      return false;
+      return true;
    }
 
    if (!isl_format_supports_multisampling(dev->info, info->format))