✗ Fi.CI.CHECKPATCH: warning for drm/i915/psr: vbt change for psr (rev6)

[Patchwork <patchwork@emeril.freedesktop.org>]

== Series Details ==

Series: drm/i915/psr: vbt change for psr (rev6)
URL   : https://patchwork.freedesktop.org/series/41289/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
824c9eb50c54 drm/i915/psr: vbt change for psr
-:69: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#69: FILE: drivers/gpu/drm/i915/i915_reg.h:4087:
+#define   EDP_PSR2_TP2_TIME_500us	(0<<8)
                                  	  ^

-:70: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#70: FILE: drivers/gpu/drm/i915/i915_reg.h:4088:
+#define   EDP_PSR2_TP2_TIME_100us	(1<<8)
                                  	  ^

-:71: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#71: FILE: drivers/gpu/drm/i915/i915_reg.h:4089:
+#define   EDP_PSR2_TP2_TIME_2500us	(2<<8)
                                   	  ^

-:72: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#72: FILE: drivers/gpu/drm/i915/i915_reg.h:4090:
+#define   EDP_PSR2_TP2_TIME_50us	(3<<8)
                                 	  ^

-:103: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#103: FILE: drivers/gpu/drm/i915/intel_bios.c:707:
+			DRM_DEBUG_KMS("VBT tp1 wakeup time value %d is outside range[0-3], defaulting to max value 2500us\n",
+					psr_table->tp1_wakeup_time);

-:120: WARNING:LINE_CONTINUATIONS: Avoid line continuations in quoted strings
#120: FILE: drivers/gpu/drm/i915/intel_bios.c:724:
+			DRM_DEBUG_KMS("VBT tp2_tp3 wakeup time value %d is outside range[0-3], \

-:121: WARNING:LONG_LINE: line over 100 characters
#121: FILE: drivers/gpu/drm/i915/intel_bios.c:725:
+					defaulting to max value 2500us\n", psr_table->tp2_tp3_wakeup_time);

total: 0 errors, 2 warnings, 5 checks, 133 lines checked

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