== Series Details ==

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

== Summary ==

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

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

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

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

-:132: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#132: FILE: drivers/gpu/drm/i915/intel_bios.c:714:
+               if (psr_table->tp1_wakeup_time < 0 ||
+                       psr_table->tp1_wakeup_time > 3) {

-:134: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#134: FILE: drivers/gpu/drm/i915/intel_bios.c:716:
+                       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);

-:137: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#137: FILE: drivers/gpu/drm/i915/intel_bios.c:719:
+               if (psr_table->tp2_tp3_wakeup_time < 0 ||
+                       psr_table->tp2_tp3_wakeup_time > 3) {

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

-:143: WARNING:LONG_LINE: line over 100 characters
#143: FILE: drivers/gpu/drm/i915/intel_bios.c:725:
+       dev_priv->vbt.psr.tp1_wakeup_time_us = 
vbt_psr_to_us(is_waketime_options, psr_table->tp1_wakeup_time);

-:144: WARNING:LONG_LINE: line over 100 characters
#144: FILE: drivers/gpu/drm/i915/intel_bios.c:726:
+       dev_priv->vbt.psr.tp2_tp3_wakeup_time_us = 
vbt_psr_to_us(is_waketime_options, psr_table->tp2_tp3_wakeup_time);

total: 0 errors, 2 warnings, 8 checks, 137 lines checked

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

Reply via email to