== Series Details ==
Series: Send a hotplug when edid changes (rev2)
URL : https://patchwork.freedesktop.org/series/62816/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
4a0b026f1681 drm: Add helper to compare edids.
-:32: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "edid1"
#32: FILE: drivers/gpu/drm/drm_edid.c:1375:
+ bool edid1_present = edid1 != NULL;
-:33: CHECK:COMPARISON_TO_NULL: Comparison to NULL could be written "edid2"
#33: FILE: drivers/gpu/drm/drm_edid.c:1376:
+ bool edid2_present = edid2 != NULL;
-:39: CHECK:BRACES: Blank lines aren't necessary after an open brace '{'
#39: FILE: drivers/gpu/drm/drm_edid.c:1382:
+ if (edid1) {
+
-:54: CHECK:LINE_SPACING: Please don't use multiple blank lines
#54: FILE: drivers/gpu/drm/drm_edid.c:1397:
+
+
total: 0 errors, 0 warnings, 4 checks, 54 lines checked
0910fd34e6ab drm: Introduce change counter to drm_connector
-:42: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u64' over 'uint64_t'
#42: FILE: drivers/gpu/drm/drm_probe_helper.c:779:
+ uint64_t old_epoch_counter;
-:69: WARNING:BRACES: braces {} are not necessary for single statement blocks
#69: FILE: drivers/gpu/drm/drm_probe_helper.c:811:
+ if (old_status != connector->status) {
changed = true;
+ }
-:102: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u64' over 'uint64_t'
#102: FILE: include/drm/drm_connector.h:1285:
+ uint64_t epoch_counter;
total: 0 errors, 1 warnings, 2 checks, 69 lines checked
7d356e798c65 drm/i915: Send hotplug event if edid had changed.
-:8: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description
(prefer a maximum 75 chars per line)
#8:
to calling layer using drm_connector->change_counter(proposed by Daniel Vetter).
-:45: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#45: FILE: drivers/gpu/drm/i915/display/intel_dp.c:5522:
+ DRM_DEBUG_KMS("[CONNECTOR:%d:%s] Edid was changed! Updating
blob property.\n",
+ connector->base.id, connector->name);
-:82: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#82: FILE: drivers/gpu/drm/i915/display/intel_hdmi.c:2535:
+ DRM_DEBUG_KMS("[CONNECTOR:%d:%s] Edid was changed! Updating
blob property.\n",
+ connector->base.id, connector->name);
-:132: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#132: FILE: drivers/gpu/drm/i915/display/intel_hotplug.c:293:
+ DRM_DEBUG_KMS("[CONNECTOR:%d:%s] status updated from %s to
%s(change counter %llu)\n",
connector->base.base.id,
total: 0 errors, 1 warnings, 3 checks, 101 lines checked
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
