2 * Copyright © 2014-2015 Broadcom
4 * Permission is hereby granted, free of charge, to any person obtaining a
5 * copy of this software and associated documentation files (the "Software"),
6 * to deal in the Software without restriction, including without limitation
7 * the rights to use, copy, modify, merge, publish, distribute, sublicense,
8 * and/or sell copies of the Software, and to permit persons to whom the
9 * Software is furnished to do so, subject to the following conditions:
11 * The above copyright notice and this permission notice (including the next
12 * paragraph) shall be included in all copies or substantial portions of the
15 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
16 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
17 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
18 * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
19 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
20 * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
25 * DOC: Render command list generation
27 * In the V3D hardware, render command lists are what load and store
28 * tiles of a framebuffer and optionally call out to binner-generated
29 * command lists to do the 3D drawing for that tile.
31 * In the VC4 driver, render command list generation is performed by the
32 * kernel instead of userspace. We do this because validating a
33 * user-submitted command list is hard to get right and has high CPU overhead,
34 * while the number of valid configurations for render command lists is
35 * actually fairly low.
38 #include "uapi/drm/vc4_drm.h"
40 #include "vc4_packet.h"
42 struct vc4_rcl_setup
{
43 struct drm_gem_cma_object
*color_read
;
44 struct drm_gem_cma_object
*color_write
;
45 struct drm_gem_cma_object
*zs_read
;
46 struct drm_gem_cma_object
*zs_write
;
47 struct drm_gem_cma_object
*msaa_color_write
;
48 struct drm_gem_cma_object
*msaa_zs_write
;
50 struct drm_gem_cma_object
*rcl
;
53 u32 next_write_bo_index
;
56 static inline void rcl_u8(struct vc4_rcl_setup
*setup
, u8 val
)
58 *(u8
*)(setup
->rcl
->vaddr
+ setup
->next_offset
) = val
;
59 setup
->next_offset
+= 1;
62 static inline void rcl_u16(struct vc4_rcl_setup
*setup
, u16 val
)
64 *(u16
*)(setup
->rcl
->vaddr
+ setup
->next_offset
) = val
;
65 setup
->next_offset
+= 2;
68 static inline void rcl_u32(struct vc4_rcl_setup
*setup
, u32 val
)
70 *(u32
*)(setup
->rcl
->vaddr
+ setup
->next_offset
) = val
;
71 setup
->next_offset
+= 4;
75 * Emits a no-op STORE_TILE_BUFFER_GENERAL.
77 * If we emit a PACKET_TILE_COORDINATES, it must be followed by a store of
78 * some sort before another load is triggered.
80 static void vc4_store_before_load(struct vc4_rcl_setup
*setup
)
82 rcl_u8(setup
, VC4_PACKET_STORE_TILE_BUFFER_GENERAL
);
84 VC4_SET_FIELD(VC4_LOADSTORE_TILE_BUFFER_NONE
,
85 VC4_LOADSTORE_TILE_BUFFER_BUFFER
) |
86 VC4_STORE_TILE_BUFFER_DISABLE_COLOR_CLEAR
|
87 VC4_STORE_TILE_BUFFER_DISABLE_ZS_CLEAR
|
88 VC4_STORE_TILE_BUFFER_DISABLE_VG_MASK_CLEAR
);
89 rcl_u32(setup
, 0); /* no address, since we're in None mode */
93 * Calculates the physical address of the start of a tile in a RCL surface.
95 * Unlike the other load/store packets,
96 * VC4_PACKET_LOAD/STORE_FULL_RES_TILE_BUFFER don't look at the tile
97 * coordinates packet, and instead just store to the address given.
99 static uint32_t vc4_full_res_offset(struct vc4_exec_info
*exec
,
100 struct drm_gem_cma_object
*bo
,
101 struct drm_vc4_submit_rcl_surface
*surf
,
102 uint8_t x
, uint8_t y
)
104 return bo
->paddr
+ surf
->offset
+ VC4_TILE_BUFFER_SIZE
*
105 (DIV_ROUND_UP(exec
->args
->width
, 32) * y
+ x
);
109 * Emits a PACKET_TILE_COORDINATES if one isn't already pending.
111 * The tile coordinates packet triggers a pending load if there is one, are
112 * used for clipping during rendering, and determine where loads/stores happen
113 * relative to their base address.
115 static void vc4_tile_coordinates(struct vc4_rcl_setup
*setup
,
116 uint32_t x
, uint32_t y
)
118 rcl_u8(setup
, VC4_PACKET_TILE_COORDINATES
);
123 static void emit_tile(struct vc4_exec_info
*exec
,
124 struct vc4_rcl_setup
*setup
,
125 uint8_t x
, uint8_t y
, bool first
, bool last
)
127 struct drm_vc4_submit_cl
*args
= exec
->args
;
128 bool has_bin
= args
->bin_cl_size
!= 0;
130 /* Note that the load doesn't actually occur until the
131 * tile coords packet is processed, and only one load
132 * may be outstanding at a time.
134 if (setup
->color_read
) {
135 if (args
->color_read
.flags
&
136 VC4_SUBMIT_RCL_SURFACE_READ_IS_FULL_RES
) {
137 rcl_u8(setup
, VC4_PACKET_LOAD_FULL_RES_TILE_BUFFER
);
139 vc4_full_res_offset(exec
, setup
->color_read
,
140 &args
->color_read
, x
, y
) |
141 VC4_LOADSTORE_FULL_RES_DISABLE_ZS
);
143 rcl_u8(setup
, VC4_PACKET_LOAD_TILE_BUFFER_GENERAL
);
144 rcl_u16(setup
, args
->color_read
.bits
);
145 rcl_u32(setup
, setup
->color_read
->paddr
+
146 args
->color_read
.offset
);
150 if (setup
->zs_read
) {
151 if (args
->zs_read
.flags
&
152 VC4_SUBMIT_RCL_SURFACE_READ_IS_FULL_RES
) {
153 rcl_u8(setup
, VC4_PACKET_LOAD_FULL_RES_TILE_BUFFER
);
155 vc4_full_res_offset(exec
, setup
->zs_read
,
156 &args
->zs_read
, x
, y
) |
157 VC4_LOADSTORE_FULL_RES_DISABLE_COLOR
);
159 if (setup
->color_read
) {
160 /* Exec previous load. */
161 vc4_tile_coordinates(setup
, x
, y
);
162 vc4_store_before_load(setup
);
165 rcl_u8(setup
, VC4_PACKET_LOAD_TILE_BUFFER_GENERAL
);
166 rcl_u16(setup
, args
->zs_read
.bits
);
167 rcl_u32(setup
, setup
->zs_read
->paddr
+
168 args
->zs_read
.offset
);
172 /* Clipping depends on tile coordinates having been
173 * emitted, so we always need one here.
175 vc4_tile_coordinates(setup
, x
, y
);
177 /* Wait for the binner before jumping to the first
180 if (first
&& has_bin
)
181 rcl_u8(setup
, VC4_PACKET_WAIT_ON_SEMAPHORE
);
184 rcl_u8(setup
, VC4_PACKET_BRANCH_TO_SUB_LIST
);
185 rcl_u32(setup
, (exec
->tile_alloc_offset
+
186 (y
* exec
->bin_tiles_x
+ x
) * 32));
189 if (setup
->msaa_color_write
) {
190 bool last_tile_write
= (!setup
->msaa_zs_write
&&
192 !setup
->color_write
);
193 uint32_t bits
= VC4_LOADSTORE_FULL_RES_DISABLE_ZS
;
195 if (!last_tile_write
)
196 bits
|= VC4_LOADSTORE_FULL_RES_DISABLE_CLEAR_ALL
;
198 bits
|= VC4_LOADSTORE_FULL_RES_EOF
;
199 rcl_u8(setup
, VC4_PACKET_STORE_FULL_RES_TILE_BUFFER
);
201 vc4_full_res_offset(exec
, setup
->msaa_color_write
,
202 &args
->msaa_color_write
, x
, y
) |
206 if (setup
->msaa_zs_write
) {
207 bool last_tile_write
= (!setup
->zs_write
&&
208 !setup
->color_write
);
209 uint32_t bits
= VC4_LOADSTORE_FULL_RES_DISABLE_COLOR
;
211 if (setup
->msaa_color_write
)
212 vc4_tile_coordinates(setup
, x
, y
);
213 if (!last_tile_write
)
214 bits
|= VC4_LOADSTORE_FULL_RES_DISABLE_CLEAR_ALL
;
216 bits
|= VC4_LOADSTORE_FULL_RES_EOF
;
217 rcl_u8(setup
, VC4_PACKET_STORE_FULL_RES_TILE_BUFFER
);
219 vc4_full_res_offset(exec
, setup
->msaa_zs_write
,
220 &args
->msaa_zs_write
, x
, y
) |
224 if (setup
->zs_write
) {
225 bool last_tile_write
= !setup
->color_write
;
227 if (setup
->msaa_color_write
|| setup
->msaa_zs_write
)
228 vc4_tile_coordinates(setup
, x
, y
);
230 rcl_u8(setup
, VC4_PACKET_STORE_TILE_BUFFER_GENERAL
);
231 rcl_u16(setup
, args
->zs_write
.bits
|
233 0 : VC4_STORE_TILE_BUFFER_DISABLE_COLOR_CLEAR
));
235 (setup
->zs_write
->paddr
+ args
->zs_write
.offset
) |
236 ((last
&& last_tile_write
) ?
237 VC4_LOADSTORE_TILE_BUFFER_EOF
: 0));
240 if (setup
->color_write
) {
241 if (setup
->msaa_color_write
|| setup
->msaa_zs_write
||
243 vc4_tile_coordinates(setup
, x
, y
);
247 rcl_u8(setup
, VC4_PACKET_STORE_MS_TILE_BUFFER_AND_EOF
);
249 rcl_u8(setup
, VC4_PACKET_STORE_MS_TILE_BUFFER
);
253 static int vc4_create_rcl_bo(struct drm_device
*dev
, struct vc4_exec_info
*exec
,
254 struct vc4_rcl_setup
*setup
)
256 struct drm_vc4_submit_cl
*args
= exec
->args
;
257 bool has_bin
= args
->bin_cl_size
!= 0;
258 uint8_t min_x_tile
= args
->min_x_tile
;
259 uint8_t min_y_tile
= args
->min_y_tile
;
260 uint8_t max_x_tile
= args
->max_x_tile
;
261 uint8_t max_y_tile
= args
->max_y_tile
;
262 uint8_t xtiles
= max_x_tile
- min_x_tile
+ 1;
263 uint8_t ytiles
= max_y_tile
- min_y_tile
+ 1;
265 uint32_t size
, loop_body_size
;
266 bool positive_x
= true;
267 bool positive_y
= true;
269 if (args
->flags
& VC4_SUBMIT_CL_FIXED_RCL_ORDER
) {
270 if (!(args
->flags
& VC4_SUBMIT_CL_RCL_ORDER_INCREASING_X
))
272 if (!(args
->flags
& VC4_SUBMIT_CL_RCL_ORDER_INCREASING_Y
))
276 size
= VC4_PACKET_TILE_RENDERING_MODE_CONFIG_SIZE
;
277 loop_body_size
= VC4_PACKET_TILE_COORDINATES_SIZE
;
279 if (args
->flags
& VC4_SUBMIT_CL_USE_CLEAR_COLOR
) {
280 size
+= VC4_PACKET_CLEAR_COLORS_SIZE
+
281 VC4_PACKET_TILE_COORDINATES_SIZE
+
282 VC4_PACKET_STORE_TILE_BUFFER_GENERAL_SIZE
;
285 if (setup
->color_read
) {
286 if (args
->color_read
.flags
&
287 VC4_SUBMIT_RCL_SURFACE_READ_IS_FULL_RES
) {
288 loop_body_size
+= VC4_PACKET_LOAD_FULL_RES_TILE_BUFFER_SIZE
;
290 loop_body_size
+= VC4_PACKET_LOAD_TILE_BUFFER_GENERAL_SIZE
;
293 if (setup
->zs_read
) {
294 if (args
->zs_read
.flags
&
295 VC4_SUBMIT_RCL_SURFACE_READ_IS_FULL_RES
) {
296 loop_body_size
+= VC4_PACKET_LOAD_FULL_RES_TILE_BUFFER_SIZE
;
298 if (setup
->color_read
&&
299 !(args
->color_read
.flags
&
300 VC4_SUBMIT_RCL_SURFACE_READ_IS_FULL_RES
)) {
301 loop_body_size
+= VC4_PACKET_TILE_COORDINATES_SIZE
;
302 loop_body_size
+= VC4_PACKET_STORE_TILE_BUFFER_GENERAL_SIZE
;
304 loop_body_size
+= VC4_PACKET_LOAD_TILE_BUFFER_GENERAL_SIZE
;
309 size
+= VC4_PACKET_WAIT_ON_SEMAPHORE_SIZE
;
310 loop_body_size
+= VC4_PACKET_BRANCH_TO_SUB_LIST_SIZE
;
313 if (setup
->msaa_color_write
)
314 loop_body_size
+= VC4_PACKET_STORE_FULL_RES_TILE_BUFFER_SIZE
;
315 if (setup
->msaa_zs_write
)
316 loop_body_size
+= VC4_PACKET_STORE_FULL_RES_TILE_BUFFER_SIZE
;
319 loop_body_size
+= VC4_PACKET_STORE_TILE_BUFFER_GENERAL_SIZE
;
320 if (setup
->color_write
)
321 loop_body_size
+= VC4_PACKET_STORE_MS_TILE_BUFFER_SIZE
;
323 /* We need a VC4_PACKET_TILE_COORDINATES in between each store. */
324 loop_body_size
+= VC4_PACKET_TILE_COORDINATES_SIZE
*
325 ((setup
->msaa_color_write
!= NULL
) +
326 (setup
->msaa_zs_write
!= NULL
) +
327 (setup
->color_write
!= NULL
) +
328 (setup
->zs_write
!= NULL
) - 1);
330 size
+= xtiles
* ytiles
* loop_body_size
;
332 setup
->rcl
= &vc4_bo_create(dev
, size
, true, VC4_BO_TYPE_RCL
)->base
;
333 if (IS_ERR(setup
->rcl
))
334 return PTR_ERR(setup
->rcl
);
335 list_add_tail(&to_vc4_bo(&setup
->rcl
->base
)->unref_head
,
338 /* The tile buffer gets cleared when the previous tile is stored. If
339 * the clear values changed between frames, then the tile buffer has
340 * stale clear values in it, so we have to do a store in None mode (no
341 * writes) so that we trigger the tile buffer clear.
343 if (args
->flags
& VC4_SUBMIT_CL_USE_CLEAR_COLOR
) {
344 rcl_u8(setup
, VC4_PACKET_CLEAR_COLORS
);
345 rcl_u32(setup
, args
->clear_color
[0]);
346 rcl_u32(setup
, args
->clear_color
[1]);
347 rcl_u32(setup
, args
->clear_z
);
348 rcl_u8(setup
, args
->clear_s
);
350 vc4_tile_coordinates(setup
, 0, 0);
352 rcl_u8(setup
, VC4_PACKET_STORE_TILE_BUFFER_GENERAL
);
353 rcl_u16(setup
, VC4_LOADSTORE_TILE_BUFFER_NONE
);
354 rcl_u32(setup
, 0); /* no address, since we're in None mode */
357 rcl_u8(setup
, VC4_PACKET_TILE_RENDERING_MODE_CONFIG
);
359 (setup
->color_write
? (setup
->color_write
->paddr
+
360 args
->color_write
.offset
) :
362 rcl_u16(setup
, args
->width
);
363 rcl_u16(setup
, args
->height
);
364 rcl_u16(setup
, args
->color_write
.bits
);
366 for (yi
= 0; yi
< ytiles
; yi
++) {
367 int y
= positive_y
? min_y_tile
+ yi
: max_y_tile
- yi
;
368 for (xi
= 0; xi
< xtiles
; xi
++) {
369 int x
= positive_x
? min_x_tile
+ xi
: max_x_tile
- xi
;
370 bool first
= (xi
== 0 && yi
== 0);
371 bool last
= (xi
== xtiles
- 1 && yi
== ytiles
- 1);
373 emit_tile(exec
, setup
, x
, y
, first
, last
);
377 BUG_ON(setup
->next_offset
!= size
);
378 exec
->ct1ca
= setup
->rcl
->paddr
;
379 exec
->ct1ea
= setup
->rcl
->paddr
+ setup
->next_offset
;
384 static int vc4_full_res_bounds_check(struct vc4_exec_info
*exec
,
385 struct drm_gem_cma_object
*obj
,
386 struct drm_vc4_submit_rcl_surface
*surf
)
388 struct drm_vc4_submit_cl
*args
= exec
->args
;
389 u32 render_tiles_stride
= DIV_ROUND_UP(exec
->args
->width
, 32);
391 if (surf
->offset
> obj
->base
.size
) {
392 DRM_DEBUG("surface offset %d > BO size %zd\n",
393 surf
->offset
, obj
->base
.size
);
397 if ((obj
->base
.size
- surf
->offset
) / VC4_TILE_BUFFER_SIZE
<
398 render_tiles_stride
* args
->max_y_tile
+ args
->max_x_tile
) {
399 DRM_DEBUG("MSAA tile %d, %d out of bounds "
400 "(bo size %zd, offset %d).\n",
401 args
->max_x_tile
, args
->max_y_tile
,
410 static int vc4_rcl_msaa_surface_setup(struct vc4_exec_info
*exec
,
411 struct drm_gem_cma_object
**obj
,
412 struct drm_vc4_submit_rcl_surface
*surf
)
414 if (surf
->flags
!= 0 || surf
->bits
!= 0) {
415 DRM_DEBUG("MSAA surface had nonzero flags/bits\n");
419 if (surf
->hindex
== ~0)
422 *obj
= vc4_use_bo(exec
, surf
->hindex
);
426 exec
->rcl_write_bo
[exec
->rcl_write_bo_count
++] = *obj
;
428 if (surf
->offset
& 0xf) {
429 DRM_DEBUG("MSAA write must be 16b aligned.\n");
433 return vc4_full_res_bounds_check(exec
, *obj
, surf
);
436 static int vc4_rcl_surface_setup(struct vc4_exec_info
*exec
,
437 struct drm_gem_cma_object
**obj
,
438 struct drm_vc4_submit_rcl_surface
*surf
,
441 uint8_t tiling
= VC4_GET_FIELD(surf
->bits
,
442 VC4_LOADSTORE_TILE_BUFFER_TILING
);
443 uint8_t buffer
= VC4_GET_FIELD(surf
->bits
,
444 VC4_LOADSTORE_TILE_BUFFER_BUFFER
);
445 uint8_t format
= VC4_GET_FIELD(surf
->bits
,
446 VC4_LOADSTORE_TILE_BUFFER_FORMAT
);
450 if (surf
->flags
& ~VC4_SUBMIT_RCL_SURFACE_READ_IS_FULL_RES
) {
451 DRM_DEBUG("Extra flags set\n");
455 if (surf
->hindex
== ~0)
458 *obj
= vc4_use_bo(exec
, surf
->hindex
);
463 exec
->rcl_write_bo
[exec
->rcl_write_bo_count
++] = *obj
;
465 if (surf
->flags
& VC4_SUBMIT_RCL_SURFACE_READ_IS_FULL_RES
) {
466 if (surf
== &exec
->args
->zs_write
) {
467 DRM_DEBUG("general zs write may not be a full-res.\n");
471 if (surf
->bits
!= 0) {
472 DRM_DEBUG("load/store general bits set with "
473 "full res load/store.\n");
477 ret
= vc4_full_res_bounds_check(exec
, *obj
, surf
);
484 if (surf
->bits
& ~(VC4_LOADSTORE_TILE_BUFFER_TILING_MASK
|
485 VC4_LOADSTORE_TILE_BUFFER_BUFFER_MASK
|
486 VC4_LOADSTORE_TILE_BUFFER_FORMAT_MASK
)) {
487 DRM_DEBUG("Unknown bits in load/store: 0x%04x\n",
492 if (tiling
> VC4_TILING_FORMAT_LT
) {
493 DRM_DEBUG("Bad tiling format\n");
497 if (buffer
== VC4_LOADSTORE_TILE_BUFFER_ZS
) {
499 DRM_DEBUG("No color format should be set for ZS\n");
503 } else if (buffer
== VC4_LOADSTORE_TILE_BUFFER_COLOR
) {
505 case VC4_LOADSTORE_TILE_BUFFER_BGR565
:
506 case VC4_LOADSTORE_TILE_BUFFER_BGR565_DITHER
:
509 case VC4_LOADSTORE_TILE_BUFFER_RGBA8888
:
513 DRM_DEBUG("Bad tile buffer format\n");
517 DRM_DEBUG("Bad load/store buffer %d.\n", buffer
);
521 if (surf
->offset
& 0xf) {
522 DRM_DEBUG("load/store buffer must be 16b aligned.\n");
526 if (!vc4_check_tex_size(exec
, *obj
, surf
->offset
, tiling
,
527 exec
->args
->width
, exec
->args
->height
, cpp
)) {
535 vc4_rcl_render_config_surface_setup(struct vc4_exec_info
*exec
,
536 struct vc4_rcl_setup
*setup
,
537 struct drm_gem_cma_object
**obj
,
538 struct drm_vc4_submit_rcl_surface
*surf
)
540 uint8_t tiling
= VC4_GET_FIELD(surf
->bits
,
541 VC4_RENDER_CONFIG_MEMORY_FORMAT
);
542 uint8_t format
= VC4_GET_FIELD(surf
->bits
,
543 VC4_RENDER_CONFIG_FORMAT
);
546 if (surf
->flags
!= 0) {
547 DRM_DEBUG("No flags supported on render config.\n");
551 if (surf
->bits
& ~(VC4_RENDER_CONFIG_MEMORY_FORMAT_MASK
|
552 VC4_RENDER_CONFIG_FORMAT_MASK
|
553 VC4_RENDER_CONFIG_MS_MODE_4X
|
554 VC4_RENDER_CONFIG_DECIMATE_MODE_4X
)) {
555 DRM_DEBUG("Unknown bits in render config: 0x%04x\n",
560 if (surf
->hindex
== ~0)
563 *obj
= vc4_use_bo(exec
, surf
->hindex
);
567 exec
->rcl_write_bo
[exec
->rcl_write_bo_count
++] = *obj
;
569 if (tiling
> VC4_TILING_FORMAT_LT
) {
570 DRM_DEBUG("Bad tiling format\n");
575 case VC4_RENDER_CONFIG_FORMAT_BGR565_DITHERED
:
576 case VC4_RENDER_CONFIG_FORMAT_BGR565
:
579 case VC4_RENDER_CONFIG_FORMAT_RGBA8888
:
583 DRM_DEBUG("Bad tile buffer format\n");
587 if (!vc4_check_tex_size(exec
, *obj
, surf
->offset
, tiling
,
588 exec
->args
->width
, exec
->args
->height
, cpp
)) {
595 int vc4_get_rcl(struct drm_device
*dev
, struct vc4_exec_info
*exec
)
597 struct vc4_rcl_setup setup
= {0};
598 struct drm_vc4_submit_cl
*args
= exec
->args
;
599 bool has_bin
= args
->bin_cl_size
!= 0;
602 if (args
->min_x_tile
> args
->max_x_tile
||
603 args
->min_y_tile
> args
->max_y_tile
) {
604 DRM_DEBUG("Bad render tile set (%d,%d)-(%d,%d)\n",
605 args
->min_x_tile
, args
->min_y_tile
,
606 args
->max_x_tile
, args
->max_y_tile
);
611 (args
->max_x_tile
> exec
->bin_tiles_x
||
612 args
->max_y_tile
> exec
->bin_tiles_y
)) {
613 DRM_DEBUG("Render tiles (%d,%d) outside of bin config "
615 args
->max_x_tile
, args
->max_y_tile
,
616 exec
->bin_tiles_x
, exec
->bin_tiles_y
);
620 ret
= vc4_rcl_render_config_surface_setup(exec
, &setup
,
626 ret
= vc4_rcl_surface_setup(exec
, &setup
.color_read
, &args
->color_read
,
631 ret
= vc4_rcl_surface_setup(exec
, &setup
.zs_read
, &args
->zs_read
,
636 ret
= vc4_rcl_surface_setup(exec
, &setup
.zs_write
, &args
->zs_write
,
641 ret
= vc4_rcl_msaa_surface_setup(exec
, &setup
.msaa_color_write
,
642 &args
->msaa_color_write
);
646 ret
= vc4_rcl_msaa_surface_setup(exec
, &setup
.msaa_zs_write
,
647 &args
->msaa_zs_write
);
651 /* We shouldn't even have the job submitted to us if there's no
652 * surface to write out.
654 if (!setup
.color_write
&& !setup
.zs_write
&&
655 !setup
.msaa_color_write
&& !setup
.msaa_zs_write
) {
656 DRM_DEBUG("RCL requires color or Z/S write\n");
660 return vc4_create_rcl_bo(dev
, exec
, &setup
);