On Mon, 11 Mar 2019, "Shankar, Uma" <uma.shan...@intel.com> wrote: >>-----Original Message----- >>From: Jani Nikula [mailto:jani.nik...@linux.intel.com] >>Sent: Monday, March 11, 2019 7:25 PM >>To: Patchwork <patchw...@emeril.freedesktop.org>; Shankar, Uma >><uma.shan...@intel.com> >>Cc: intel-gfx@lists.freedesktop.org >>Subject: Re: [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for Add HDR Metadata >>Parsing >>and handling in DRM layer (rev5) >> >>On Mon, 11 Mar 2019, Patchwork <patchw...@emeril.freedesktop.org> wrote: >>> == Series Details == >>> >>> Series: Add HDR Metadata Parsing and handling in DRM layer (rev5) >>> URL : https://patchwork.freedesktop.org/series/25091/ >>> State : warning >>> >>> == Summary == >>> >>> $ dim checkpatch origin/drm-tip >>> b2d03105049d drm: Add HDR source metadata property >>> fe440823e535 drm: Parse HDR metadata info from EDID >>> -:59: CHECK:BRACES: Blank lines aren't necessary after an open brace '{' >>> #59: FILE: drivers/gpu/drm/drm_edid.c:3839: >>> +{ >>> + >>> >>> -:65: CHECK:BRACES: Blank lines aren't necessary before a close brace '}' >>> #65: FILE: drivers/gpu/drm/drm_edid.c:3845: >>> + >>> +} >>> >>> -:69: CHECK:BRACES: Blank lines aren't necessary after an open brace '{' >>> #69: FILE: drivers/gpu/drm/drm_edid.c:3849: >>> +{ >>> + >>> >>> -:77: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #77: FILE: drivers/gpu/drm/drm_edid.c:3857: >>> + uint16_t len; >>> >>> total: 0 errors, 0 warnings, 4 checks, 76 lines checked >>> 5ef8cb0e93e9 drm: Parse Colorimetry data block from EDID >>> -:51: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #51: FILE: drivers/gpu/drm/drm_edid.c:3842: >>> + uint16_t len; >>> >>> -:58: CHECK:LINE_SPACING: Please don't use multiple blank lines >>> #58: FILE: drivers/gpu/drm/drm_edid.c:3849: >>> + >>> + >>> >>> total: 0 errors, 0 warnings, 2 checks, 53 lines checked aa78bb330fcc >>> drm/i915: Attach HDR metadata property to connector >>> -:24: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open >>> parenthesis >>> #24: FILE: drivers/gpu/drm/i915/intel_hdmi.c:2723: >>> + drm_object_attach_property(&connector->base, >>> + connector->dev->mode_config.hdr_output_metadata_property, 0); >>> >>> total: 0 errors, 0 warnings, 1 checks, 8 lines checked 98de918105db >>> drm: Implement HDR output metadata set and get property handling >>> -:52: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open >>> parenthesis >>> #52: FILE: drivers/gpu/drm/drm_atomic_uapi.c:741: >>> + ret = drm_atomic_replace_property_blob_from_id(dev, >>> + &state->hdr_output_metadata_blob_ptr, >>> >>> total: 0 errors, 0 warnings, 1 checks, 39 lines checked >>> 149de61e2c39 drm: Enable HDR infoframe support >>> -:65: CHECK:LINE_SPACING: Please don't use multiple blank lines >>> #65: FILE: drivers/gpu/drm/drm_edid.c:4944: >>> + >>> + >>> >>> -:94: CHECK:LINE_SPACING: Please don't use multiple blank lines >>> #94: FILE: drivers/gpu/drm/drm_edid.c:4973: >>> + >>> + >>> >>> -:232: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open >>> parenthesis >>> #232: FILE: drivers/video/hdmi.c:1392: >>> + hdmi_infoframe_log_header(level, dev, >>> + (struct hdmi_any_infoframe *)frame); >>> >>> -:245: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open >>> parenthesis >>> #245: FILE: drivers/video/hdmi.c:1405: >>> + hdmi_log("max_mastering_display_luminance: %d\n", >>> + frame->max_mastering_display_luminance); >>> >>> -:247: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open >>> parenthesis >>> #247: FILE: drivers/video/hdmi.c:1407: >>> + hdmi_log("min_mastering_display_luminance: %d\n", >>> + frame->min_mastering_display_luminance); >>> >>> -:304: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #304: FILE: include/linux/hdmi.h:196: >>> + uint16_t x, y; >>> >>> -:307: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #307: FILE: include/linux/hdmi.h:199: >>> + uint16_t x, y; >>> >>> -:309: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #309: FILE: include/linux/hdmi.h:201: >>> + uint16_t max_mastering_display_luminance; >>> >>> -:310: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #310: FILE: include/linux/hdmi.h:202: >>> + uint16_t min_mastering_display_luminance; >>> >>> -:311: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #311: FILE: include/linux/hdmi.h:203: >>> + uint16_t max_fall; >>> >>> -:312: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #312: FILE: include/linux/hdmi.h:204: >>> + uint16_t max_cll; >>> >>> -:313: CHECK:PREFER_KERNEL_TYPES: Prefer kernel type 'u16' over 'uint16_t' >>> #313: FILE: include/linux/hdmi.h:205: >>> + uint16_t min_cll; >>> >>> total: 0 errors, 0 warnings, 12 checks, 279 lines checked >>> c65e9f436e86 drm/i915: Write HDR infoframe and send to panel >>> -:51: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open >>> parenthesis >>> #51: FILE: drivers/gpu/drm/i915/intel_hdmi.c:783: >>> +intel_hdmi_compute_drm_infoframe(struct intel_encoder *encoder, >>> + struct intel_crtc_state *crtc_state, >>> >>> total: 0 errors, 0 warnings, 1 checks, 64 lines checked dd6bc359799a >>> drm/i915: [DO NOT MERGE] hack for glk board outputs 96858c0b173a >>> drm/i915: Add HLG EOTF >>> 2d2578db31c3 drm/i915: Enable infoframes on GLK+ for HDR >>> -:49: WARNING:LONG_LINE: line over 100 characters >>> #49: FILE: drivers/gpu/drm/i915/i915_reg.h:8155: >>> +#define GLK_TVIDEO_DIP_DRM_DATA(trans, i) _MMIO_TRANS2(trans, >>_GLK_VIDEO_DIP_DRM_DATA_A + (i) * 4) >> >>I think all of the warnings except this one are valid. > > Yes Jani, The local patchwork (which comes default with scripts in kernel) > didn't had the issues. So local > checkpatch scan looked clean.
You'll need to pass in the --strict option to checkpatch.pl for some of the checks. BR, Jani. > > Will fix these in next version. > > Thanks & Regards, > Uma Shankar > >>BR, >>Jani. >> >>> >>> total: 0 errors, 1 warnings, 0 checks, 76 lines checked b8f04b68ec8f >>> drm/i915:Enabled Modeset when HDR Infoframe changes >>> -:57: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open >>> parenthesis >>> #57: FILE: drivers/gpu/drm/i915/intel_hdmi.c:796: >>> + if (!conn_state->hdr_output_metadata_blob_ptr || >>> + conn_state->hdr_output_metadata_blob_ptr->length == 0) >>> >>> total: 0 errors, 0 warnings, 1 checks, 37 lines checked 3d5cb1bb27ca >>> drm/i915: Set Infoframe for non modeset case for HDR >>> -:35: CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open >>> parenthesis >>> #35: FILE: drivers/gpu/drm/i915/intel_ddi.c:3586: >>> + if ((INTEL_GEN(dev_priv) >= 10 || IS_GEMINILAKE(dev_priv)) && >>> + conn_state->hdr_metadata_changed) >>> >>> total: 0 errors, 0 warnings, 1 checks, 25 lines checked >>> 2c84b2fe5c71 video/hdmi: Add const variants for drm infoframe >>> >>> _______________________________________________ >>> Intel-gfx mailing list >>> Intel-gfx@lists.freedesktop.org >>> https://lists.freedesktop.org/mailman/listinfo/intel-gfx >> >>-- >>Jani Nikula, Intel Open Source Graphics Center -- Jani Nikula, Intel Open Source Graphics Center _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
