From 99d17fb771b77af5c693ce86aae6a61a05c05843 Mon Sep 17 00:00:00 2001 From: Alyssa Rosenzweig Date: Tue, 11 Aug 2020 21:04:01 -0400 Subject: [PATCH] panfrost: XMLify MSAA writeout mode Signed-off-by: Alyssa Rosenzweig Reviewed-by: Tomeu Vizoso Part-of: --- src/panfrost/include/panfrost-job.h | 16 +--------------- src/panfrost/lib/decode.c | 18 +----------------- src/panfrost/lib/midgard.xml | 10 ++++++++++ 3 files changed, 12 insertions(+), 32 deletions(-) diff --git a/src/panfrost/include/panfrost-job.h b/src/panfrost/include/panfrost-job.h index 48379cce905..c4481de5101 100644 --- a/src/panfrost/include/panfrost-job.h +++ b/src/panfrost/include/panfrost-job.h @@ -1391,20 +1391,6 @@ struct mali_single_framebuffer { } __attribute__((packed)); -/* SINGLE to disable multisampling, AVERAGE for - * EXT_multisampled_render_to_texture operation where multiple tilebuffer - * samples are implicitly resolved before writeout, MULTIPLE to write multiple - * samples inline, and LAYERED for ES3-style multisampling with each sample in - * a different buffer. - */ - -enum mali_msaa_mode { - MALI_MSAA_SINGLE = 0, - MALI_MSAA_AVERAGE = 1, - MALI_MSAA_MULTIPLE = 2, - MALI_MSAA_LAYERED = 3, -}; - #define MALI_MFBD_FORMAT_SRGB (1 << 0) struct mali_rt_format { @@ -1416,7 +1402,7 @@ struct mali_rt_format { unsigned unk3 : 4; unsigned unk4 : 1; enum mali_block_format block : 2; - enum mali_msaa_mode msaa : 2; + enum mali_msaa msaa : 2; unsigned flags : 2; unsigned swizzle : 12; diff --git a/src/panfrost/lib/decode.c b/src/panfrost/lib/decode.c index c47b8f52b6c..6538b47f8a3 100644 --- a/src/panfrost/lib/decode.c +++ b/src/panfrost/lib/decode.c @@ -453,22 +453,6 @@ static char *pandecode_format(enum mali_format format) #undef DEFINE_CASE -#define DEFINE_CASE(name) case MALI_MSAA_ ## name: return "MALI_MSAA_" #name -static char * -pandecode_msaa_mode(enum mali_msaa_mode mode) -{ - switch (mode) { - DEFINE_CASE(SINGLE); - DEFINE_CASE(AVERAGE); - DEFINE_CASE(MULTIPLE); - DEFINE_CASE(LAYERED); - default: - unreachable("Impossible"); - return ""; - } -} -#undef DEFINE_CASE - static char *pandecode_attr_mode_short(enum mali_attr_mode mode) { switch(mode) { @@ -1027,7 +1011,7 @@ pandecode_rt_format(struct mali_rt_format format) pandecode_log_decoded_flags(mfbd_fmt_flag_info, format.flags); pandecode_log_cont(",\n"); - pandecode_prop("msaa = %s", pandecode_msaa_mode(format.msaa)); + pandecode_prop("msaa = %s", mali_msaa_as_str(format.msaa)); /* In theory, the no_preload bit can be cleared to enable MFBD preload, * which is a faster hardware-based alternative to the wallpaper method diff --git a/src/panfrost/lib/midgard.xml b/src/panfrost/lib/midgard.xml index 57de518972a..37aa98987de 100644 --- a/src/panfrost/lib/midgard.xml +++ b/src/panfrost/lib/midgard.xml @@ -51,6 +51,16 @@ + + + + + + + + + + -- 2.30.2