igt_aux.h already provides the optimal igt_fls(), so use that in
preference to open coding the brute force version.

Reported-by: Stuart Summers <stuart.summ...@intel.com>
Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
Cc: Stuart Summers <stuart.summ...@intel.com>
---
 tests/i915/gem_blits.c | 12 +-----------
 1 file changed, 1 insertion(+), 11 deletions(-)

diff --git a/tests/i915/gem_blits.c b/tests/i915/gem_blits.c
index e0346a7c7..f9cb12bb8 100644
--- a/tests/i915/gem_blits.c
+++ b/tests/i915/gem_blits.c
@@ -57,16 +57,6 @@ enum mode {
        WC,
 };
 
-static int fls(uint64_t x)
-{
-       int t;
-
-       for (t = 0; x >> t; t++)
-               ;
-
-       return t;
-}
-
 static unsigned int
 get_tiling_stride(const struct device *device,
                  unsigned int width, unsigned int tiling)
@@ -81,7 +71,7 @@ get_tiling_stride(const struct device *device,
                else
                        stride = ALIGN(stride, 128);
                if (device->gen < 4)
-                       stride = 1 << fls(stride - 1);
+                       stride = 1 << igt_fls(stride - 1);
        } else {
                if (device->gen >= 8)
                        stride = ALIGN(stride, 64);
-- 
2.24.0

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to