]> www.infradead.org Git - users/hch/misc.git/commitdiff
drm/{i915,xe}/panic: convert intel_panic_finish() to struct intel_panic
authorJani Nikula <jani.nikula@intel.com>
Tue, 2 Sep 2025 17:51:53 +0000 (20:51 +0300)
committerJani Nikula <jani.nikula@intel.com>
Mon, 8 Sep 2025 11:23:46 +0000 (14:23 +0300)
The intel_panic_finish() function really needs the struct intel_panic
pointer, not struct intel_framebuffer. Make it so.

Cc: Jocelyn Falempe <jfalempe@redhat.com>
Cc: Maarten Lankhorst <dev@lankhorst.se>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Jocelyn Falempe <jfalempe@redhat.com>
Link: https://lore.kernel.org/r/3fdbcbe17e0e90c4a590f2a2486a9ec79a90cf62.1756835342.git.jani.nikula@intel.com
drivers/gpu/drm/i915/display/intel_panic.c
drivers/gpu/drm/i915/display/intel_panic.h
drivers/gpu/drm/i915/display/intel_plane.c
drivers/gpu/drm/i915/gem/i915_gem_object.h
drivers/gpu/drm/i915/gem/i915_gem_pages.c
drivers/gpu/drm/xe/display/xe_panic.c

index 5431bd4d3a7dada4c790a9effd8285234b19b697..c8ac8b5a39f370ac6001f1ae8ebc8170a9ed3661 100644 (file)
@@ -14,7 +14,7 @@ int intel_panic_setup(struct drm_scanout_buffer *sb)
        return i915_gem_object_panic_setup(sb);
 }
 
-void intel_panic_finish(struct intel_framebuffer *fb)
+void intel_panic_finish(struct intel_panic *panic)
 {
-       return i915_gem_object_panic_finish(fb);
+       return i915_gem_object_panic_finish(panic);
 }
index 45ce6104e6fb78b4b4b8c40f55937591d710d2bb..cec193e725a456a88b05ff8ce856d69893ccfabd 100644 (file)
@@ -5,11 +5,10 @@
 #define __INTEL_PANIC_H__
 
 struct drm_scanout_buffer;
-struct intel_framebuffer;
 struct intel_panic;
 
 struct intel_panic *intel_panic_alloc(void);
 int intel_panic_setup(struct drm_scanout_buffer *sb);
-void intel_panic_finish(struct intel_framebuffer *fb);
+void intel_panic_finish(struct intel_panic *panic);
 
 #endif /* __INTEL_PANIC_H__ */
index 890d897bda102e817a910a405efeb8a80ace7eac..6a3f4a7b6dfdcb08e6f26aece90910e49f10f497 100644 (file)
@@ -1326,7 +1326,7 @@ static void intel_panic_flush(struct drm_plane *plane)
        struct drm_framebuffer *fb = plane_state->hw.fb;
        struct intel_framebuffer *intel_fb = to_intel_framebuffer(fb);
 
-       intel_panic_finish(intel_fb);
+       intel_panic_finish(intel_fb->panic);
 
        if (crtc_state->enable_psr2_sel_fetch) {
                /* Force a full update for psr2 */
index 9b3f25cb48db318ffcae10423048f670e59f550b..08da7ec670fe1efa716c764c19e0d6e038f990fd 100644 (file)
@@ -18,7 +18,6 @@
 
 enum intel_region_id;
 struct drm_scanout_buffer;
-struct intel_framebuffer;
 struct intel_panic;
 
 #define obj_to_i915(obj__) to_i915((obj__)->base.dev)
@@ -696,7 +695,7 @@ int i915_gem_object_truncate(struct drm_i915_gem_object *obj);
 
 struct intel_panic *i915_gem_object_alloc_panic(void);
 int i915_gem_object_panic_setup(struct drm_scanout_buffer *sb);
-void i915_gem_object_panic_finish(struct intel_framebuffer *fb);
+void i915_gem_object_panic_finish(struct intel_panic *panic);
 
 /**
  * i915_gem_object_pin_map - return a contiguous mapping of the entire object
index b219474aecc7799272ff5745fbb76910cc302ab8..3505b1842c407f402454c71e375a51d2d82a3ce5 100644 (file)
@@ -475,10 +475,8 @@ int i915_gem_object_panic_setup(struct drm_scanout_buffer *sb)
        return -EOPNOTSUPP;
 }
 
-void i915_gem_object_panic_finish(struct intel_framebuffer *fb)
+void i915_gem_object_panic_finish(struct intel_panic *panic)
 {
-       struct intel_panic *panic = fb->panic;
-
        i915_panic_kunmap(panic);
        panic->page = -1;
        kfree(panic->pages);
index b5a7615708a10e1dc8014bfc1b6694ccb060da31..ea9583473ea3457a9fbb359e1ed28a04c559d9f9 100644 (file)
@@ -76,10 +76,8 @@ int intel_panic_setup(struct drm_scanout_buffer *sb)
        return 0;
 }
 
-void intel_panic_finish(struct intel_framebuffer *fb)
+void intel_panic_finish(struct intel_panic *panic)
 {
-       struct intel_panic *panic = fb->panic;
-
        xe_panic_kunmap(panic);
        panic->page = -1;
 }