✗ Fi.CI.CHECKPATCH: warning for drm/i915/gen11: Add support for the NV12 format. (rev4)

From: Patchwork <patchwork@emeril.freedesktop.org>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gen11: Add support for the NV12 format. (rev4)
Date: Mon, 22 Oct 2018 15:28:51 -0000	[thread overview]
Message-ID: <20181022152851.28547.77531@emeril.freedesktop.org> (raw)
In-Reply-To: <20181018115134.9061-1-maarten.lankhorst@linux.intel.com>

== Series Details ==

Series: drm/i915/gen11: Add support for the NV12 format. (rev4)
URL   : https://patchwork.freedesktop.org/series/51178/
State : warning

== Summary ==

$ dim checkpatch origin/drm-tip
e63515ae7751 drm/i915: Fix unsigned overflow when calculating total data rate, v2.
-:43: CHECK:CAMELCASE: Avoid CamelCase: <GBps>
#43: FILE: drivers/gpu/drm/i915/intel_pm.c:3806:
+	if (num_active > 1 || total_data_bw >= GBps(12)) {

total: 0 errors, 0 warnings, 1 checks, 130 lines checked
770267f9b2b9 drm/i915/gen11: Enable 6 sprites on gen11
a76a2e2aed7b drm/i915/gen11: Link nv12 Y and UV planes in the atomic state, v5.
-:18: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#18: 
- Put all the state updating login in intel_plane_atomic_check_with_state().

-:213: CHECK:BOOL_MEMBER: Avoid using bool structure members because of possible alignment issues - see: https://lkml.org/lkml/2017/11/21/384
#213: FILE: drivers/gpu/drm/i915/intel_drv.h:560:
+	bool slave;

-:234: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#234: FILE: drivers/gpu/drm/i915/intel_drv.h:1379:
+intel_atomic_get_plane_state(struct intel_atomic_state *state,
+				 struct intel_plane *plane)

total: 0 errors, 1 warnings, 2 checks, 243 lines checked
d03837d789bc drm/i915/gen11: Handle watermarks correctly for separate Y/UV planes, v2.
be5e77cdbaf7 drm/i915/gen11: Program the scalers correctly for planar formats, v3.
6be5b1b2fe9f drm/i915/gen11: Program the chroma upsampler for HDR planes.
-:50: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'pipe' - possible side-effects?
#50: FILE: drivers/gpu/drm/i915/i915_reg.h:6707:
+#define PLANE_CUS_CTL(pipe, plane)   \
+	_MMIO_PLANE(plane, _PLANE_CUS_CTL_1(pipe), _PLANE_CUS_CTL_2(pipe))

total: 0 errors, 0 warnings, 1 checks, 68 lines checked
0df7e71ff240 drm/i915/gen11: Program the Y and UV plane for planar mode correctly, v3.
8ff51f2481a0 drm/i915/gen11: Expose planar format support on gen11, v2.

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