Sent V8 for below two patches, fixing 2 alignment problems.
Please note: Ignored following 2 warnings in patch 7, to maintain the existing driver style for bit macros (also mentioned in the commit message).
-:64: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#64: FILE: drivers/gpu/drm/i915/i915_reg.h:8725:
+#define TRANS_MSA_SAMPLING_444 (2<<1)
^
-:65: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#65: FILE: drivers/gpu/drm/i915/i915_reg.h:8726:
+#define TRANS_MSA_CLRSP_YCBCR (2<<3)
Regards
Shashank
-----Original Message-----
From: Patchwork [mailto:patchwork@xxxxxxxxxxxxxxxxxxxxxx]
Sent: Tuesday, April 3, 2018 3:10 PM
To: Sharma, Shashank <shashank.sharma@xxxxxxxxx>
Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
Subject: ✗ Fi.CI.CHECKPATCH: warning for YCBCR 4:2:0/4:4:4 output support for LSPCON (rev5)
== Series Details ==
Series: YCBCR 4:2:0/4:4:4 output support for LSPCON (rev5)
URL : https://patchwork.freedesktop.org/series/36068/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
093f7ee765f1 drm/i915: Introduce CRTC output format
-:87: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#87: FILE: drivers/gpu/drm/i915/intel_display.c:10690:
+ DRM_DEBUG_KMS("output format: %s\n",
+ output_formats(pipe_config->output_format));
total: 0 errors, 0 warnings, 1 checks, 139 lines checked
5aad7ff8f693 drm/i915: Add CRTC output format YCBCR 4:2:0 ba253735184f drm/i915: Add CRTC output format YCBCR 4:4:4
544b43272930 drm/i915: Check LSPCON vendor OUI
7602c66d2d56 drm/i915: Add AVI infoframe support for LSPCON
d1459992db55 drm/i915: Write AVI infoframes for MCA LSPCON
a03dd8af9824 drm/i915: Write AVI infoframes for Parade LSPCON
-:87: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis
#87: FILE: drivers/gpu/drm/i915/intel_lspcon.c:287:
+ DRM_DEBUG_KMS("LSPCON FW not ready, block %d\n",
+ block_count);
total: 0 errors, 0 warnings, 1 checks, 141 lines checked cbb6eaa686ed drm/i915: Add YCBCR 4:2:0/4:4:4 support for LSPCON
-:64: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#64: FILE: drivers/gpu/drm/i915/i915_reg.h:8725:
+#define TRANS_MSA_SAMPLING_444 (2<<1)
^
-:65: CHECK:SPACING: spaces preferred around that '<<' (ctx:VxV)
#65: FILE: drivers/gpu/drm/i915/i915_reg.h:8726:
+#define TRANS_MSA_CLRSP_YCBCR (2<<3)
^
total: 0 errors, 0 warnings, 2 checks, 128 lines checked
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
