Re: [PATCH 09/13] drm/i915/debugfs: pass struct drm_printer to intel_{dp, dp_mst, hdmi}_crtc_info()

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] drm/i915/debugfs: commit subject names *_crtc_info() instead of *_info()
--

commit cbb2b1bf881fc1a6e876fdc89f4b11f179ca6fee
Author: Jani Nikula <jani.nikula@intel.com>

>     drm/i915/debugfs: pass struct drm_printer to intel_{dp, dp_mst, hdmi}_crtc_info()
>     
>     Pass struct drm_printer to intel_dp_info(), intel_dp_mst_info(), and
>     intel_hdmi_info().
>     
>     Prefer struct drm_printer * over struct seq_file * to unify and allow
>     use of drm_printer helpers.
>     
>     Signed-off-by: Jani Nikula <jani.nikula@intel.com>

[Severity: Low]
This isn't a bug, but the commit subject references
intel_{dp, dp_mst, hdmi}_crtc_info() whereas the functions modified in
drivers/gpu/drm/i915/display/intel_display_debugfs.c (called from
intel_connector_info()) are intel_dp_info(), intel_dp_mst_info(), and
intel_hdmi_info(). Should the subject line refer to
intel_{dp, dp_mst, hdmi}_info() instead?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/cover.1793361738.git.jani.nikula@intel.com?part=9