✗ Fi.CI.CHECKPATCH: warning for QGV/SAGV related fixes (rev6)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



== Series Details ==

Series: QGV/SAGV related fixes (rev6)
URL   : https://patchwork.freedesktop.org/series/126962/
State : warning

== Summary ==

Error: dim checkpatch failed
e14a10f2859f drm/i915: Add meaningful traces for QGV point info error handling
aa498440ab05 drm/i915: Extract code required to calculate max qgv/psf gv point
e7e9a465da87 drm/i915: Disable SAGV on bw init, to force QGV point recalculation
-:12: WARNING:COMMIT_LOG_LONG_LINE: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
#12: 
(i.e all points allowed), however in reality we might get them all restricted,

-:45: WARNING:LONG_LINE: line length of 110 exceeds 100 columns
#45: FILE: drivers/gpu/drm/i915/display/intel_bw.c:165:
+		drm_err(&dev_priv->drm, "Failed to disable qgv points (%x) points: 0x%x\n", ret, points_mask);

-:87: CHECK:BRACES: braces {} should be used on all arms of this statement
#87: FILE: drivers/gpu/drm/i915/display/intel_bw.c:871:
+		if (max_data_rate > max_bw) {
[...]
+		} else if (max_data_rate == max_bw)
[...]

-:108: CHECK:SPACING: space preferred before that '|' (ctx:VxE)
#108: FILE: drivers/gpu/drm/i915/display/intel_bw.c:892:
+	bw_state->qgv_points_mask = ~(ICL_PCODE_REQ_QGV_PT(qgv_points)|
 	                                                              ^

-:121: CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#121: FILE: drivers/gpu/drm/i915/display/intel_bw.c:905:
+
+}

total: 0 errors, 2 warnings, 3 checks, 98 lines checked





[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux