From: Ville Syrjälä <ville.syrj...@linux.intel.com>

Fix the accidentally swapped bpp and depth values passed to
the addfb ioctl when we're testing for dirtyfb presence.
Currently the addfb fails every time so we don't even test
the actual dirtyfb ioctl.

Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 src/sna/kgem.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/sna/kgem.c b/src/sna/kgem.c
index 9c0708a635fb..6a35067c4107 100644
--- a/src/sna/kgem.c
+++ b/src/sna/kgem.c
@@ -1538,8 +1538,8 @@ static bool test_has_dirtyfb(struct kgem *kgem)
        create.width = 32;
        create.height = 32;
        create.pitch = 4*32;
-       create.bpp = 24;
-       create.depth = 32; /* {bpp:24, depth:32} -> x8r8g8b8 */
+       create.bpp = 32;
+       create.depth = 24; /* {bpp:32, depth:24} -> x8r8g8b8 */
        create.handle = gem_create(kgem->fd, 1);
        if (create.handle == 0)
                return false;
-- 
2.23.0

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

Reply via email to