[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm: Use new DRM printk funcs (like drm_dbg_*()) in DP helpers (rev2)

From: Patchwork <patchwork@emeril.freedesktop.org>
To: "Lyude Paul" <lyude@redhat.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm: Use new DRM printk funcs (like drm_dbg_*()) in DP helpers (rev2)
Date: Sat, 20 Feb 2021 00:32:55 -0000	[thread overview]
Message-ID: <161378117575.24213.6964914363181904081@emeril.freedesktop.org> (raw)
In-Reply-To: <20210219215326.2227596-1-lyude@redhat.com>

== Series Details ==

Series: drm: Use new DRM printk funcs (like drm_dbg_*()) in DP helpers (rev2)
URL   : https://patchwork.freedesktop.org/series/87242/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
428923af2258 drm/dp: Rewrap kdocs for struct drm_dp_aux
2c3cb32ffa61 drm/dp: Fixup kernel docs for struct drm_dp_aux
33e9a50b217b drm/tegra: Don't register DP AUX channels before connectors
dd60e381b29d drm/bridge/tc358767: Don't register DP AUX channel until bridge is attached
677d32a7cff2 drm/bridge/cdns-mhdp8546: Register DP aux channel with userspace
3101c201d6a8 drm/bridge/ti-sn65dsi86: (Un)register aux device on bridge attach/detach
0cb48941ab95 drm/bridge/analogix/anx78xx: Add missing drm_dp_aux_unregister() call
7cd6e3c0d3b4 drm/bridge/analogix/anx78xx: Setup encoder before registering connector
4ccade9381d1 drm/bridge/analogix/anx78xx: Cleanup on error in anx78xx_bridge_attach()
10a994e19166 drm/bridge/analogix/anx6345: Add missing drm_dp_aux_unregister() call
c16e486032a9 drm/bridge/analogix/anx6345: Don't link encoder until after connector registration
956e9e041e4f drm/bridge/analogix/anx6345: Cleanup on errors in anx6345_bridge_attach()
97ea41708a5c drm/bridge/analogix/dp_core: Unregister DP AUX channel on error in analogix_dp_probe()
afe30b2db23b drm/nouveau/kms/nv50-: Move AUX adapter reg to connector late register/early unregister
b4f490e26caf drm/dp: Add backpointer to drm_device in drm_dp_aux
4abb0f9466cf drm/dp: Rewrap drm_dp_aux_register()/drm_dp_aux_init() kdocs
929d221431fe drm/dp: Clarify DP AUX registration time
792d8a99cee0 drm/print: Fixup DRM_DEBUG_KMS_RATELIMITED()
-:45: ERROR:CODE_INDENT: code indent should use tabs where possible
#45: FILE: include/drm/drm_print.h:531:
+                                                                                                 \$

-:45: WARNING:LEADING_SPACE: please, no spaces at the start of a line
#45: FILE: include/drm/drm_print.h:531:
+                                                                                                 \$

total: 1 errors, 1 warnings, 0 checks, 28 lines checked
974c1ef66f86 drm/dp: Pass drm_dp_aux to drm_dp_link_train_clock_recovery_delay()
4f02b339d731 drm/dp: Pass drm_dp_aux to drm_dp*_link_train_channel_eq_delay()
336c34cfa9bc drm/dp: Always print aux channel name in logs
6c6c14e703d2 drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_detect()
4f788c3e7ee9 drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_set_tmds_output()
5d7b1f94cad4 drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_max_tmds_clock()
b9dc820145fe drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_get_tmds_output()
47d3136d3a5b drm/dp_dual_mode: Pass drm_device to drm_lspcon_(get|set)_mode()
de6f1d8c3401 drm/dp_mst: Pass drm_dp_mst_topology_mgr to drm_dp_get_vc_payload_bw()
-:56: WARNING:LONG_LINE: line length of 102 exceeds 100 columns
#56: FILE: drivers/gpu/drm/i915/display/intel_dp_mst.c:74:
+									       crtc_state->port_clock,

total: 0 errors, 1 warnings, 0 checks, 43 lines checked
194a493c5b7f drm/dp: Convert drm_dp_helper.c to using drm_err/drm_dbg_*()
-:72: ERROR:CODE_INDENT: code indent should use tabs where possible
#72: FILE: drivers/gpu/drm/drm_dp_helper.c:224:
+^I^I           aux->name, offset, arrow, ret, min(ret, 20),$

-:73: ERROR:CODE_INDENT: code indent should use tabs where possible
#73: FILE: drivers/gpu/drm/drm_dp_helper.c:225:
+^I^I           buffer);$

-:78: ERROR:CODE_INDENT: code indent should use tabs where possible
#78: FILE: drivers/gpu/drm/drm_dp_helper.c:228:
+^I^I           aux->name, offset, arrow, ret);$

total: 3 errors, 0 warnings, 0 checks, 277 lines checked
b2be68664eda drm/dp_dual_mode: Convert drm_dp_dual_mode_helper.c to using drm_err/drm_dbg_kms()
222c03341b34 drm/dp_mst: Convert drm_dp_mst_topology.c to drm_err()/drm_dbg*()
-:689: WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
#689: FILE: drivers/gpu/drm/drm_dp_mst_topology.c:4113:
 	if (!up_req) {
+		drm_err(mgr->dev, "Not enough memory to process MST up req\n");

total: 0 errors, 1 warnings, 0 checks, 906 lines checked


_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx