panfrost: Inline pan_fbd_has_zs_crc_ext

It has a single user -- in a section of code that only runs for MFBD GPUs and
that has already decided whether to use CRCs -- so inlining it simplifies its
definition greatly and may avoid redeciding the CRC setting.

[Note for mesa-stable maintainers: This is not a bug fix but is marked for
backport so the next patch applies cleanly.]

Signed-off-by: Alyssa Rosenzweig <alyssa@collabora.com>
Cc: mesa-stable
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16983>
This commit is contained in:
Alyssa Rosenzweig
2022-06-10 11:43:53 -04:00
committed by Marge Bot
parent 11bbcc423c
commit cac0578ee5
2 changed files with 1 additions and 9 deletions
+1 -1
View File
@@ -694,7 +694,7 @@ GENX(pan_emit_fbd)(const struct panfrost_device *dev,
unsigned tile_size;
unsigned internal_cbuf_size = pan_internal_cbuf_size(fb, &tile_size);
int crc_rt = GENX(pan_select_crc_rt)(fb);
bool has_zs_crc_ext = pan_fbd_has_zs_crc_ext(fb);
bool has_zs_crc_ext = (fb->zs.view.zs || fb->zs.view.s || crc_rt >= 0);
pan_section_pack(fbd, FRAMEBUFFER, PARAMETERS, cfg) {
#if PAN_ARCH >= 6
-8
View File
@@ -154,14 +154,6 @@ GENX(pan_emit_tls)(const struct pan_tls_info *info,
int
GENX(pan_select_crc_rt)(const struct pan_fb_info *fb);
static inline bool
pan_fbd_has_zs_crc_ext(const struct pan_fb_info *fb)
{
return PAN_ARCH >= 5 &&
(fb->zs.view.zs || fb->zs.view.s ||
GENX(pan_select_crc_rt)(fb) >= 0);
}
unsigned
GENX(pan_emit_fbd)(const struct panfrost_device *dev,
const struct pan_fb_info *fb,