Currently, all but one uses of generic_handle_irq_safe() report error messages using .*_err_ratelimited() error reporting helper functions. These helper functions declare their error messages in the following form:
"error handling [COMPONENT NAME] irq: %d\n" .*_err_ratelimited() already logs "error" as a part of the error message, so declaring the error is redundant. Reword it. The last usage of generic_handle_irq_safe() is in xe_i2c_irq_present(), and it currently lacks the same error reporting as in the other cases. This is not intentional, so put some logging there. While we're here, we should also update the XE cases to use xe_err_ratelimited() instead of directly calling drm_err_ratelimited(). However, mixing drm error reporting functions with xe error reporting functions in the same file looks unprofessional, so update all drm error reporting functions in xe_heci_gsc.c and xe_i2c.c to use their xe counterparts. This includes updating drm_warn_once() to use xe_warn_once() instead. Unfortunately, xe_warn_once() does not exist, so create it. We should also update some of the error messages on XE and I915 to better report the error codes while we're touching them. v2: - Split XE and I915 refactors, squashing the XE refactors earlier (Wajdeczko) - Use different error message for generic_handle_irq_safe() (Wajdeczko) - Refactor other error reporting functions on XE (Wajdeczko) Jonathan Cavitt (5): drm/xe/printk: Add xe_warn_once() drm/xe/heci: Use xe print functions in xe_heci_gsc.c drm/xe/i2c: Use xe print functions in xe_i2c.c drm/i915: Refactor generic_handle_irq_safe() error messages drm/xe/i2c: Report i2c irq handler issue .../gpu/drm/i915/display/intel_lpe_audio.c | 2 +- drivers/gpu/drm/i915/gt/intel_gsc.c | 2 +- drivers/gpu/drm/xe/xe_heci_gsc.c | 21 +++++++++---------- drivers/gpu/drm/xe/xe_i2c.c | 11 ++++++---- drivers/gpu/drm/xe/xe_printk.h | 3 +++ 5 files changed, 22 insertions(+), 17 deletions(-) -- 2.53.0
