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
Date: Fri, 19 Feb 2021 22:11:41 -0000 [thread overview]
Message-ID: <161377270156.703.16717686555120810202@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
URL : https://patchwork.freedesktop.org/series/87242/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
ce207403b6ba drm/dp: Rewrap kdocs for struct drm_dp_aux
59ab34a09514 drm/dp: Fixup kernel docs for struct drm_dp_aux
76de8b506bb5 drm/tegra: Don't register DP AUX channels before connectors
4477471e11d3 drm/bridge/tc358767: Don't register DP AUX channel until bridge is attached
466a8724aac1 drm/bridge/cdns-mhdp8546: Register DP aux channel with userspace
bd0cb165e99c drm/bridge/ti-sn65dsi86: (Un)register aux device on bridge attach/detach
b69f1f6665a8 drm/bridge/analogix/anx78xx: Add missing drm_dp_aux_unregister() call
331d6a7a89db drm/bridge/analogix/anx78xx: Setup encoder before registering connector
95ef631558ad drm/bridge/analogix/anx78xx: Cleanup on error in anx78xx_bridge_attach()
a791371e1b1f drm/bridge/analogix/anx6345: Add missing drm_dp_aux_unregister() call
93ea935d7e82 drm/bridge/analogix/anx6345: Don't link encoder until after connector registration
3700184db2e5 drm/bridge/analogix/anx6345: Cleanup on errors in anx6345_bridge_attach()
f6c80edbe8f6 drm/bridge/analogix/dp_core: Unregister DP AUX channel on error in analogix_dp_probe()
011332737c0f drm/nouveau/kms/nv50-: Move AUX adapter reg to connector late register/early unregister
89eb0e9ca1fc drm/dp: Add backpointer to drm_device in drm_dp_aux
886105588723 drm/dp: Rewrap drm_dp_aux_register()/drm_dp_aux_init() kdocs
743613ba9017 drm/dp: Clarify DP AUX registration time
815fe1b4dd16 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
d1b3270dba59 drm/dp: Pass drm_dp_aux to drm_dp_link_train_clock_recovery_delay()
6853812a6368 drm/dp: Pass drm_dp_aux to drm_dp*_link_train_channel_eq_delay()
dec2312b34cc drm/dp: Always print aux channel name in logs
c7df16b9d24e drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_detect()
92e948ca8cba drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_set_tmds_output()
cdcd6958912f drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_max_tmds_clock()
b610ae839ecd drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_get_tmds_output()
6fe319f1b8e8 drm/dp_dual_mode: Pass drm_device to drm_lspcon_(get|set)_mode()
14a0b6c29fc7 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
b215e53e3ce2 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
616d97c8c5fb drm/dp_dual_mode: Convert drm_dp_dual_mode_helper.c to using drm_err/drm_dbg_kms()
1c0133c9bcf0 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
next prev parent reply other threads:[~2021-02-19 22:11 UTC|newest]
Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-02-19 21:52 [Intel-gfx] [PATCH 00/30] drm: Use new DRM printk funcs (like drm_dbg_*()) in DP helpers Lyude Paul
2021-02-19 21:52 ` [Intel-gfx] [PATCH 01/30] drm/dp: Rewrap kdocs for struct drm_dp_aux Lyude Paul
2021-02-19 23:42 ` Randy Dunlap
2021-02-24 18:31 ` Lyude Paul
2021-02-19 21:52 ` [Intel-gfx] [PATCH 02/30] drm/dp: Fixup kernel docs " Lyude Paul
2021-02-19 23:39 ` Randy Dunlap
2021-02-19 21:52 ` [Intel-gfx] [PATCH 03/30] drm/tegra: Don't register DP AUX channels before connectors Lyude Paul
2021-04-14 16:49 ` Thierry Reding
2021-04-14 18:17 ` Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 04/30] drm/bridge/tc358767: Don't register DP AUX channel until bridge is attached Lyude Paul
2021-03-19 14:27 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 05/30] drm/bridge/cdns-mhdp8546: Register DP aux channel with userspace Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 06/30] drm/bridge/ti-sn65dsi86: (Un)register aux device on bridge attach/detach Lyude Paul
2021-02-21 18:11 ` Laurent Pinchart
2021-02-19 21:53 ` [Intel-gfx] [PATCH 07/30] drm/bridge/analogix/anx78xx: Add missing drm_dp_aux_unregister() call Lyude Paul
2021-03-19 14:28 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 08/30] drm/bridge/analogix/anx78xx: Setup encoder before registering connector Lyude Paul
2021-03-19 14:35 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 09/30] drm/bridge/analogix/anx78xx: Cleanup on error in anx78xx_bridge_attach() Lyude Paul
2021-03-19 14:37 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 10/30] drm/bridge/analogix/anx6345: Add missing drm_dp_aux_unregister() call Lyude Paul
2021-03-19 14:39 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 11/30] drm/bridge/analogix/anx6345: Don't link encoder until after connector registration Lyude Paul
2021-03-19 14:43 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 12/30] drm/bridge/analogix/anx6345: Cleanup on errors in anx6345_bridge_attach() Lyude Paul
2021-03-19 14:44 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 13/30] drm/bridge/analogix/dp_core: Unregister DP AUX channel on error in analogix_dp_probe() Lyude Paul
2021-03-19 14:45 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 14/30] drm/nouveau/kms/nv50-: Move AUX adapter reg to connector late register/early unregister Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 15/30] drm/dp: Add backpointer to drm_device in drm_dp_aux Lyude Paul
2021-02-21 18:21 ` Laurent Pinchart
2021-02-24 20:40 ` Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 16/30] drm/dp: Rewrap drm_dp_aux_register()/drm_dp_aux_init() kdocs Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 17/30] drm/dp: Clarify DP AUX registration time Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 18/30] drm/print: Fixup DRM_DEBUG_KMS_RATELIMITED() Lyude Paul
2021-03-19 11:30 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 19/30] drm/dp: Pass drm_dp_aux to drm_dp_link_train_clock_recovery_delay() Lyude Paul
2021-02-21 18:22 ` Laurent Pinchart
2021-02-23 14:49 ` Rodrigo Vivi
2021-02-19 21:53 ` [Intel-gfx] [PATCH 20/30] drm/dp: Pass drm_dp_aux to drm_dp*_link_train_channel_eq_delay() Lyude Paul
2021-02-21 18:23 ` Laurent Pinchart
2021-02-19 21:53 ` [Intel-gfx] [PATCH 21/30] drm/dp: Always print aux channel name in logs Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 22/30] drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_detect() Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 23/30] drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_set_tmds_output() Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 24/30] drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_max_tmds_clock() Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 25/30] drm/dp_dual_mode: Pass drm_device to drm_dp_dual_mode_get_tmds_output() Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 26/30] drm/dp_dual_mode: Pass drm_device to drm_lspcon_(get|set)_mode() Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 27/30] drm/dp_mst: Pass drm_dp_mst_topology_mgr to drm_dp_get_vc_payload_bw() Lyude Paul
2021-03-19 11:35 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 28/30] drm/dp: Convert drm_dp_helper.c to using drm_err/drm_dbg_*() Lyude Paul
2021-03-19 11:39 ` Robert Foss
2021-02-19 21:53 ` [Intel-gfx] [PATCH 29/30] drm/dp_dual_mode: Convert drm_dp_dual_mode_helper.c to using drm_err/drm_dbg_kms() Lyude Paul
2021-02-19 21:53 ` [Intel-gfx] [PATCH 30/30] drm/dp_mst: Convert drm_dp_mst_topology.c to drm_err()/drm_dbg*() Lyude Paul
2021-03-19 11:47 ` Robert Foss
2021-02-19 22:11 ` Patchwork [this message]
2021-02-19 22:14 ` [Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm: Use new DRM printk funcs (like drm_dbg_*()) in DP helpers Patchwork
2021-02-19 22:40 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-02-19 23:56 ` [Intel-gfx] ✗ Fi.CI.IGT: failure " Patchwork
2021-02-20 0:32 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm: Use new DRM printk funcs (like drm_dbg_*()) in DP helpers (rev2) Patchwork
2021-02-20 0:35 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2021-02-20 1:01 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2021-02-20 3:02 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=161377270156.703.16717686555120810202@emeril.freedesktop.org \
--to=patchwork@emeril.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=lyude@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).