Re: ✗ CI.checkpatch: warning for drm/xe: display support should not depend on EXPERT

[Jani Nikula <jani.nikula@intel.com>]

On Thu, 11 Jan 2024, Patchwork <patchwork@emeril.freedesktop.org> wrote:
> == Series Details ==
>
> Series: drm/xe: display support should not depend on EXPERT
> URL   : https://patchwork.freedesktop.org/series/128648/
> State : warning
>
> == Summary ==
>
> + KERNEL=/kernel
> + git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
> Cloning into 'mt'...
> warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
> + git -C mt rev-list -n1 origin/master
> faa12d3432d7ef7793d934cd9338c555e95a5aad
> + cd /kernel
> + git config --global --add safe.directory /kernel
> + git log -n1
> commit 9b662af5259f4ddaac4101a7ce4ecdfe606f08b7
> Author: Jani Nikula <jani.nikula@intel.com>
> Date:   Thu Jan 11 12:47:16 2024 +0200
>
>     drm/xe: display support should not depend on EXPERT
>     
>     Remove the DRM_XE_DISPLAY config dependency on EXPERT. I can only
>     presume the idea was only experts should be able to disable it, but the
>     effect is the opposite.
>     
>     Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>     Reported-by: Eero Tamminen <eero.t.tamminen@intel.com>
>     Reviewed-by: Francois Dugast <francois.dugast@intel.com>
> + /mt/dim checkpatch 79184e72263e91528195db01783148435c7e4fad drm-intel
> 9b662af52 drm/xe: display support should not depend on EXPERT
> -:11: WARNING:BAD_REPORTED_BY_LINK: Reported-by: should be immediately followed by Closes: with a URL to the report

Kind of hard when it was in private. :p

BR,
Jani.


> #11: 
> Reported-by: Eero Tamminen <eero.t.tamminen@intel.com>
> Reviewed-by: Francois Dugast <francois.dugast@intel.com>
>
> total: 0 errors, 1 warnings, 0 checks, 8 lines checked
>
>

-- 
Jani Nikula, Intel