From: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
To: igt-dev@lists.freedesktop.org
Cc: Petri Latvala <petri.latvala@intel.com>,
Eben Upton <eben@raspberrypi.org>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Subject: [igt-dev] [PATCH i-g-t 5/9] lib/igt_fb: Add support for allocating T-tiled VC4 buffers
Date: Thu, 6 Dec 2018 15:11:28 +0100 [thread overview]
Message-ID: <20181206141132.23349-6-paul.kocialkowski@bootlin.com> (raw)
In-Reply-To: <20181206141132.23349-1-paul.kocialkowski@bootlin.com>
This introduces the required bits for allocating buffers with a T-tiled
disposition, that is specific to the VC4. It includes calculating the
top-tile width and creating a buffer object with the VC4-specific
helper. The tiling flag is set to the buffer object so that this can
be reused for GPU tests if needed later.
Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
---
lib/igt_fb.c | 25 +++++++++++++++++++++++++
1 file changed, 25 insertions(+)
diff --git a/lib/igt_fb.c b/lib/igt_fb.c
index ccdf07d0..61250b0e 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -37,6 +37,7 @@
#include "igt_fb.h"
#include "igt_kms.h"
#include "igt_matrix.h"
+#include "igt_vc4.h"
#include "igt_x86.h"
#include "ioctl_wrappers.h"
#include "intel_batchbuffer.h"
@@ -179,6 +180,9 @@ static const struct format_desc_struct *lookup_drm_format(uint32_t drm_format)
void igt_get_fb_tile_size(int fd, uint64_t tiling, int fb_bpp,
unsigned *width_ret, unsigned *height_ret)
{
+ if (is_vc4_device(fd))
+ tiling = fourcc_mod_broadcom_mod(tiling);
+
switch (tiling) {
case LOCAL_DRM_FORMAT_MOD_NONE:
*width_ret = 64;
@@ -228,6 +232,17 @@ void igt_get_fb_tile_size(int fd, uint64_t tiling, int fb_bpp,
igt_assert(false);
}
break;
+ case DRM_FORMAT_MOD_BROADCOM_VC4_T_TILED:
+ igt_require_vc4(fd);
+ switch (fb_bpp) {
+ case 32:
+ *width_ret = 32 * fb_bpp / 8;
+ *height_ret = 32;
+ break;
+ default:
+ igt_assert(false);
+ }
+ break;
default:
igt_assert(false);
}
@@ -568,6 +583,13 @@ static int create_bo_for_fb(struct igt_fb *fb)
if (is_i915_device(fd)) {
return i915_create_gem_for_fb(fb);
+ } else if (is_vc4_device(fd)) {
+ fb->gem_handle = igt_vc4_create_bo(fd, fb->size);
+
+ if (fb->tiling == DRM_FORMAT_MOD_BROADCOM_VC4_T_TILED)
+ igt_vc4_set_tiling(fd, fb->gem_handle, fb->tiling);
+
+ return fb->gem_handle;
} else {
bool driver_has_gem_api = false;
@@ -1480,6 +1502,9 @@ static void *map_bo(int fd, struct igt_fb *fb)
else if (is_i915_device(fd))
ptr = gem_mmap__gtt(fd, fb->gem_handle, fb->size,
PROT_READ | PROT_WRITE);
+ else if (is_vc4_device(fd))
+ ptr = igt_vc4_mmap_bo(fd, fb->gem_handle, fb->size,
+ PROT_READ | PROT_WRITE);
else
igt_assert(ptr != MAP_FAILED);
--
2.19.2
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
next prev parent reply other threads:[~2018-12-06 14:13 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-06 14:11 [igt-dev] [PATCH i-g-t 0/9] Chamelium VC4 plane testing, with T-tiled mode Paul Kocialkowski
2018-12-06 14:11 ` [igt-dev] [PATCH i-g-t 1/9] chamelium: Pass dimensions instead of mode to pattern generation helper Paul Kocialkowski
2018-12-06 21:43 ` Lyude Paul
2018-12-07 7:37 ` Paul Kocialkowski
2018-12-07 22:41 ` Lyude Paul
2018-12-06 14:11 ` [igt-dev] [PATCH i-g-t 2/9] chamelium: Pass the pattern block size as argument to helpers Paul Kocialkowski
2018-12-06 22:54 ` Lyude Paul
2018-12-06 14:11 ` [igt-dev] [PATCH i-g-t 3/9] lib: drmtest: Add helpers to check and require the VC4 driver Paul Kocialkowski
2018-12-06 22:55 ` Lyude Paul
2018-12-06 14:11 ` [igt-dev] [PATCH i-g-t 4/9] lib/igt_fb: Add checks on i915 for i915-specific tiled buffer allocation Paul Kocialkowski
2018-12-06 14:30 ` Maxime Ripard
2018-12-10 16:30 ` Ville Syrjälä
2018-12-13 13:29 ` Paul Kocialkowski
2018-12-06 14:11 ` Paul Kocialkowski [this message]
2018-12-06 14:11 ` [igt-dev] [PATCH i-g-t 6/9] lib/igt_vc4: Add helpers for converting linear to T-tiled XRGB buffers Paul Kocialkowski
2018-12-06 21:51 ` Lyude Paul
2018-12-07 7:38 ` Paul Kocialkowski
2018-12-06 14:11 ` [igt-dev] [PATCH i-g-t 7/9] lib/igt_fb: Add a stride-provisioned fashion of igt_fb_convert Paul Kocialkowski
2018-12-06 14:32 ` Maxime Ripard
2018-12-06 22:57 ` Lyude Paul
2018-12-06 14:11 ` [igt-dev] [PATCH i-g-t 8/9] lib/igt_fb: Add a helper to retreive the plane bpp for a given format Paul Kocialkowski
2018-12-06 22:58 ` Lyude Paul
2018-12-06 14:11 ` [igt-dev] [PATCH i-g-t 9/9] chamelium: Add a CRC-based display test for randomized planes Paul Kocialkowski
2018-12-06 14:49 ` Maxime Ripard
2018-12-06 22:52 ` Lyude Paul
2018-12-13 14:39 ` Paul Kocialkowski
2018-12-06 14:19 ` [igt-dev] ✗ Fi.CI.BAT: failure for Chamelium VC4 plane testing, with T-tiled mode Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181206141132.23349-6-paul.kocialkowski@bootlin.com \
--to=paul.kocialkowski@bootlin.com \
--cc=eben@raspberrypi.org \
--cc=igt-dev@lists.freedesktop.org \
--cc=petri.latvala@intel.com \
--cc=thomas.petazzoni@bootlin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox