Mesa (main): pvr: Fix incorrect samples to cr_isp_aa_mode conversion.

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Jun 8 08:43:01 UTC 2022


Module: Mesa
Branch: main
Commit: 721fb18a386c855b3403b3a4d07b45cab6c518e6
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=721fb18a386c855b3403b3a4d07b45cab6c518e6

Author: Karmjit Mahil <Karmjit.Mahil at imgtec.com>
Date:   Mon May 30 11:30:07 2022 +0100

pvr: Fix incorrect samples to cr_isp_aa_mode conversion.

This commit fixes the use of 'case 3' where 'case 4' should have
been.

Reported-by: Matt Coster <matt.coster at imgtec.com>
Signed-off-by: Karmjit Mahil <Karmjit.Mahil at imgtec.com>
Reviewed-by: Frank Binns <frank.binns at imgtec.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/16883>

---

 src/imagination/vulkan/pvr_csb_enum_helpers.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/imagination/vulkan/pvr_csb_enum_helpers.h b/src/imagination/vulkan/pvr_csb_enum_helpers.h
index 6ab7b6052c9..725a90107d5 100644
--- a/src/imagination/vulkan/pvr_csb_enum_helpers.h
+++ b/src/imagination/vulkan/pvr_csb_enum_helpers.h
@@ -35,6 +35,7 @@
    CR
  ******************************************************************************/
 
+/* TODO: Use VkSampleCountFlagBits as param type? */
 /* clang-format off */
 static inline enum PVRX(CR_ISP_AA_MODE_TYPE)
 pvr_cr_isp_aa_mode_type(uint32_t samples)
@@ -45,7 +46,7 @@ pvr_cr_isp_aa_mode_type(uint32_t samples)
       return PVRX(CR_ISP_AA_MODE_TYPE_AA_NONE);
    case 2:
       return PVRX(CR_ISP_AA_MODE_TYPE_AA_2X);
-   case 3:
+   case 4:
       return PVRX(CR_ISP_AA_MODE_TYPE_AA_4X);
    case 8:
       return PVRX(CR_ISP_AA_MODE_TYPE_AA_8X);



More information about the mesa-commit mailing list