✗ Fi.CI.CHECKPATCH: warning for series starting with [1/3] drm/i915: Implement Selective Write workaround for Haswell GT1 (rev2)

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

 



== Series Details ==

Series: series starting with [1/3] drm/i915: Implement Selective Write workaround for Haswell GT1 (rev2)
URL   : https://patchwork.freedesktop.org/series/54518/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
fdb7ca9dc0c5 drm/i915: Implement Selective Write workaround for Haswell GT1
-:7: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#7: 
The docs specify this needs to be set on HSW GT1 parts. I've implemented it as

-:19: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 2c550183476d ("drm/i915: Disable PSMI sleep messages on all rings around context switches")'
#19: 
commit 2c550183476dfa25641309ae9a28d30feed14379

-:28: ERROR:GIT_COMMIT_ID: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 2c550183476d ("drm/i915: Disable PSMI sleep messages on all rings around context switches")'
#28: 
References: 2c550183476d ("drm/i915: Disable PSMI sleep messages on all rings around context switches")

-:42: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'dev_priv' - possible side-effects?
#42: FILE: drivers/gpu/drm/i915/i915_drv.h:2279:
+#define IS_HSW_GT1(dev_priv)	(IS_HASWELL(dev_priv) && \
+				 (dev_priv)->info.gt == 1)

total: 2 errors, 1 warnings, 1 checks, 120 lines checked
d3d2ac8365f6 drm/i915: Remove HW semaphores for gen7 inter-engine synchronisation
ba2afc3c9641 drm/i915: Drop debugfs/i915_next_seqno

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




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

  Powered by Linux