✗ CI.checkpatch: warning for drm/xe/xe_gt_idle: Drop redundant newline in name (rev3)

[Lucas De Marchi]

On Tue, Feb 06, 2024 at 11:37:43AM -0800, Ashutosh Dixit wrote:
>On Tue, 06 Feb 2024 11:31:00 -0800, Patchwork wrote:
>>
>> == Series Details ==
>>
>> Series: drm/xe/xe_gt_idle: Drop redundant newline in name (rev3)
>> URL   : https://patchwork.freedesktop.org/series/129495/
>> 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
>> 35591fb8b4d5305b37ce31483f85ac0956eaa536
>> + cd /kernel
>> + git config --global --add safe.directory /kernel
>> + git log -n1
>> commit 37e8820af8f2d82261e27714f2c1d6aca2043e91
>> Author: Ashutosh Dixit <ashutosh.dixit at intel.com>
>> Date:   Tue Feb 6 11:27:31 2024 -0800
>>
>>     drm/xe/xe_gt_idle: Drop redundant newline in name
>>
>>     Newline in name is redunant and produces an unnecessary empty line during
>>     'cat name'. Newline is added during sysfs_emit. See '27a1a1e2e47d ("drm/xe:
>>     stringify the argument to avoid potential vulnerability")'.
>>
>>     v2: Add Fixes tag (Riana)
>>
>>     Fixes: 7b076d14f21a ("drm/xe/mtl: Add support to get C6 residency/status of MTL")
>>     Reviewed-by: Riana Tauro <riana.tauro at intel.com>
>>     Signed-off-by: Ashutosh Dixit <ashutosh.dixit at intel.com>
>> + /mt/dim checkpatch 8a42fafc783505db4f1d6a70005a5f103069c9d9 drm-intel
>> 37e8820af drm/xe/xe_gt_idle: Drop redundant newline in name
>> -:12: WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '7b076d14f21a', maybe rebased or not pulled?
>
>Warning is incorrect, 7b076d14f21a is present.

I reported this issue some time ago... CI does a checkout of drm-tip
--depth=1, so when it runs checkpatch it doesn't know about past
commits :(

Lucas De Marchi

>
>> #12:
>> Fixes: 7b076d14f21a ("drm/xe/mtl: Add support to get C6 residency/status of MTL")
>>
>> total: 0 errors, 1 warnings, 0 checks, 12 lines checked
>>
>>