]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/i915: Drop the extra "plane" from tracepoints
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 18 Dec 2024 17:36:48 +0000 (19:36 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 28 Jan 2025 16:39:25 +0000 (18:39 +0200)
Out plane names already include the "plane" part (or
"primary","sprite","cursor" in some cases). Don't duplicate
that in the tracepoints as that leadst to weird stuff like
"plane plane 1A".

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20241218173650.19782-3-ville.syrjala@linux.intel.com
Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
drivers/gpu/drm/i915/display/intel_display_trace.h

index 54a6e2a46b828b2f04d01a525d668d1f811fd421..989d6f0f3c42a97acb49bc4bad037e182b05bf58 100644 (file)
@@ -397,7 +397,7 @@ TRACE_EVENT(intel_plane_async_flip,
                           __entry->async_flip = async_flip;
                           ),
 
-           TP_printk("dev %s, pipe %c, plane %s, frame=%u, scanline=%u, async_flip=%s",
+           TP_printk("dev %s, pipe %c, %s, frame=%u, scanline=%u, async_flip=%s",
                      __get_str(dev), __entry->pipe_name, __get_str(name),
                      __entry->frame, __entry->scanline, str_yes_no(__entry->async_flip))
 );
@@ -426,7 +426,7 @@ TRACE_EVENT(intel_plane_update_noarm,
                           memcpy(__entry->dst, &plane->base.state->dst, sizeof(__entry->dst));
                           ),
 
-           TP_printk("dev %s, pipe %c, plane %s, frame=%u, scanline=%u, " DRM_RECT_FP_FMT " -> " DRM_RECT_FMT,
+           TP_printk("dev %s, pipe %c, %s, frame=%u, scanline=%u, " DRM_RECT_FP_FMT " -> " DRM_RECT_FMT,
                      __get_str(dev), __entry->pipe_name, __get_str(name),
                      __entry->frame, __entry->scanline,
                      DRM_RECT_FP_ARG((const struct drm_rect *)__entry->src),
@@ -457,7 +457,7 @@ TRACE_EVENT(intel_plane_update_arm,
                           memcpy(__entry->dst, &plane->base.state->dst, sizeof(__entry->dst));
                           ),
 
-           TP_printk("dev %s, pipe %c, plane %s, frame=%u, scanline=%u, " DRM_RECT_FP_FMT " -> " DRM_RECT_FMT,
+           TP_printk("dev %s, pipe %c, %s, frame=%u, scanline=%u, " DRM_RECT_FP_FMT " -> " DRM_RECT_FMT,
                      __get_str(dev), __entry->pipe_name, __get_str(name),
                      __entry->frame, __entry->scanline,
                      DRM_RECT_FP_ARG((const struct drm_rect *)__entry->src),
@@ -484,7 +484,7 @@ TRACE_EVENT(intel_plane_disable_arm,
                           __entry->scanline = intel_get_crtc_scanline(crtc);
                           ),
 
-           TP_printk("dev %s, pipe %c, plane %s, frame=%u, scanline=%u",
+           TP_printk("dev %s, pipe %c, %s, frame=%u, scanline=%u",
                      __get_str(dev), __entry->pipe_name, __get_str(name),
                      __entry->frame, __entry->scanline)
 );
@@ -611,7 +611,7 @@ TRACE_EVENT(intel_fbc_activate,
                           __entry->scanline = intel_get_crtc_scanline(crtc);
                           ),
 
-           TP_printk("dev %s, pipe %c, plane %s, frame=%u, scanline=%u",
+           TP_printk("dev %s, pipe %c, %s, frame=%u, scanline=%u",
                      __get_str(dev), __entry->pipe_name, __get_str(name),
                      __entry->frame, __entry->scanline)
 );
@@ -639,7 +639,7 @@ TRACE_EVENT(intel_fbc_deactivate,
                           __entry->scanline = intel_get_crtc_scanline(crtc);
                           ),
 
-           TP_printk("dev %s, pipe %c, plane %s, frame=%u, scanline=%u",
+           TP_printk("dev %s, pipe %c, %s, frame=%u, scanline=%u",
                      __get_str(dev), __entry->pipe_name, __get_str(name),
                      __entry->frame, __entry->scanline)
 );
@@ -667,7 +667,7 @@ TRACE_EVENT(intel_fbc_nuke,
                           __entry->scanline = intel_get_crtc_scanline(crtc);
                           ),
 
-           TP_printk("dev %s, pipe %c, plane %s, frame=%u, scanline=%u",
+           TP_printk("dev %s, pipe %c, %s, frame=%u, scanline=%u",
                      __get_str(dev), __entry->pipe_name, __get_str(name),
                      __entry->frame, __entry->scanline)
 );