drm/imagination: Switch reset_reason fields from enum to u32

Update the reset_reason fwif structure fields from enum to u32 to remove
any ambiguity from the interface (enum is not a fixed size thus is unfit
for the purpose of the data type).

Fixes: a26f067feac1f ("drm/imagination: Add FWIF headers")
Signed-off-by: Alexandru Dadu <alexandru.dadu@imgtec.com>
Reviewed-by: Matt Coster <matt.coster@imgtec.com>
Link: https://patch.msgid.link/20260323-b4-firmware-context-reset-notification-handling-v3-2-1a66049a9a65@imgtec.com
Signed-off-by: Matt Coster <matt.coster@imgtec.com>

diff --git a/drivers/gpu/drm/imagination/pvr_rogue_fwif.h b/drivers/gpu/drm/imagination/pvr_rogue_fwif.h
index 172886be4c820a32ae1b08d62bf0814250f2a9bd..5d590c4c25663e431601dd26944465b62c31526b 100644 (file)
--- a/drivers/gpu/drm/imagination/pvr_rogue_fwif.h
+++ b/drivers/gpu/drm/imagination/pvr_rogue_fwif.h
@@ -1347,8 +1347,12 @@ struct rogue_fwif_fwccb_cmd_freelists_reconstruction_data {
 struct rogue_fwif_fwccb_cmd_context_reset_data {
        /* Context affected by the reset */
        u32 server_common_context_id;
-       /* Reason for reset */
-       enum rogue_context_reset_reason reset_reason;
+       /*
+        * Reason for reset
+        * The valid values for reset_reason are the ones from
+        * enum rogue_context_reset_reason
+        */
+       u32 reset_reason;
        /* Data Master affected by the reset */
        u32 dm;
        /* Job ref running at the time of reset */

diff --git a/drivers/gpu/drm/imagination/pvr_rogue_fwif_shared.h b/drivers/gpu/drm/imagination/pvr_rogue_fwif_shared.h
index f622553cdc110ca276f563cbfbccc7318d6e5c38..869d904e3649f99bb4d38e18560bcc9ab14c5221 100644 (file)
--- a/drivers/gpu/drm/imagination/pvr_rogue_fwif_shared.h
+++ b/drivers/gpu/drm/imagination/pvr_rogue_fwif_shared.h
@@ -261,7 +261,11 @@ enum rogue_context_reset_reason {
 };
 
 struct rogue_context_reset_reason_data {
-       enum rogue_context_reset_reason reset_reason;
+       /*
+        * The valid values for reset_reason are the ones from
+        * enum rogue_context_reset_reason
+        */
+       u32 reset_reason;
        u32 reset_ext_job_ref;
 };