AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

v3d: use job's nr_cbufs field

Browse Source 
Instead of computing the number of color buffers in the start binning
function, just use the job's nr_cbufs, which already contains that
value.

Reviewed-by: Alejandro Piñeiro <apinheiro@igalia.com>
Reviewed-by: Jose Maria Casanova Crespo <jmcasanova@igalia.com>
Signed-off-by: Juan A. Suarez Romero <jasuarez@igalia.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8006>
This commit is contained in:
Juan A. Suarez Romero
2020-12-09 14:41:32 +01:00
committed by Marge Bot
parent e95465ae0e
commit 1a535722d3
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/gallium/drivers/v3d/v3dx_draw.c
+2 -6
View File
@@ -81,10 +81,6 @@ v3d_start_binning(struct v3d_context *v3d, struct v3d_job *job)
tsda_per_tile_size,
"TSDA");
ubyte nr_cbufs = V3D_MAX_DRAW_BUFFERS;
while (nr_cbufs > 0 && !job->cbufs[nr_cbufs - 1])
nr_cbufs--;
#if V3D_VERSION >= 41
/* This must go before the binning mode configuration. It is
* required for layered framebuffers to work.
@@ -101,7 +97,7 @@ v3d_start_binning(struct v3d_context *v3d, struct v3d_job *job)
config.width_in_pixels = job->draw_width;
config.height_in_pixels = job->draw_height;
config.number_of_render_targets =
MAX2(nr_cbufs, 1);
MAX2(job->nr_cbufs, 1);
config.multisample_mode_4x = job->msaa;
@@ -127,7 +123,7 @@ v3d_start_binning(struct v3d_context *v3d, struct v3d_job *job)
config.height_in_tiles = job->draw_tiles_y;
/* Must be >= 1 */
config.number_of_render_targets =
MAX2(nr_cbufs, 1);
MAX2(job->nr_cbufs, 1);
config.multisample_mode_4x = job->msaa;