Dear Laurent Pinchart: Thanks for your patches! Unfortunately the Media CI robot detected some issues:
# Test media-patchstyle:./0001-media-uvcvideo-Fix-assignment-in-if-condition.patch media style ERROR: ./0001-media-uvcvideo-Fix-assignment-in-if-condition.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0003-media-uvcvideo-Fix-comments-in-uvc_meta_detect_msxu.patch media style ERROR: Commit 3b964863b4650f1583ee04826266f7f4ab716e45 needs to provide a reason to avoid the nomination to stable # Test media-patchstyle:./0004-media-uvcvideo-Drop-unneeded-memset-in-meta-device-i.patch media style ERROR: ./0004-media-uvcvideo-Drop-unneeded-memset-in-meta-device-i.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0005-media-uvcvideo-Add-missing-curly-braces.patch media style ERROR: ./0005-media-uvcvideo-Add-missing-curly-braces.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0006-media-uvcvideo-Move-MSXU_CONTROL_METADATA-definition.patch media style ERROR: ./0006-media-uvcvideo-Move-MSXU_CONTROL_METADATA-definition.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0007-media-uvcvideo-Drop-stream-mutex.patch media style ERROR: ./0007-media-uvcvideo-Drop-stream-mutex.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0008-media-uvcvideo-Move-video_device-under-video_queue.patch media style ERROR: ./0008-media-uvcvideo-Move-video_device-under-video_queue.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0009-media-uvcvideo-Use-intf-instead-of-udev-for-printks.patch media style ERROR: ./0009-media-uvcvideo-Use-intf-instead-of-udev-for-printks.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0010-media-uvcvideo-Do-not-re-reference-dev-udev.patch media style ERROR: ./0010-media-uvcvideo-Do-not-re-reference-dev-udev.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0011-media-uvcvideo-Shorten-the-transfer-size-non-complia.patch media style ERROR: ./0011-media-uvcvideo-Shorten-the-transfer-size-non-complia.patch: Committer Laurent Pinchart <[email protected]> is not the last SoB: Hans de Goede <[email protected]> # Test media-patchstyle:./0014-media-uvcvideo-Mark-invalid-entities-with-id-UVC_INV.patch media style WARNING: Reported-by: should be immediately followed by Closes: with a URL to the report #128: Reported-by: Youngjun Lee <[email protected]> Fixes: a3fbc2e6bb05 ("media: mc-entity.c: use WARN_ON, validate link pads") total: 0 errors, 1 warnings, 147 lines checked ERROR: Fails one or more mandatory checkpatch tests # Test checkpatch:./0006-media-uvcvideo-Move-MSXU_CONTROL_METADATA-definition.patch checkpatch WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit description?) #13: https://learn.microsoft.com/en-us/windows-hardware/drivers/stream/uvc-extensions-1-5#222-extension-unit-controls. total: 0 errors, 1 warnings, 0 checks, 65 lines checked # Test checkpatch:./0009-media-uvcvideo-Use-intf-instead-of-udev-for-printks.patch checkpatch WARNING: Prefer dev_dbg(... to dev_printk(KERN_DEBUG, ... #245: FILE: drivers/media/usb/uvc/uvcvideo.h:668: +dev_printk(KERN_DEBUG, &(_dev)->intf->dev, fmt,\ total: 0 errors, 1 warnings, 0 checks, 187 lines checked # Test checkpatch:./0014-media-uvcvideo-Mark-invalid-entities-with-id-UVC_INV.patch checkpatch WARNING: Reported-by: should be immediately followed by Closes: with a URL to the report #128: Reported-by: Youngjun Lee <[email protected]> Fixes: a3fbc2e6bb05 ("media: mc-entity.c: use WARN_ON, validate link pads") total: 0 errors, 1 warnings, 0 checks, 147 lines checked Please fix your series, and upload a new version. If you have a patchwork account, do not forget to mark the current series as Superseded. For more details, check the full report at: https://linux-media.pages.freedesktop.org/-/users/patchwork/-/jobs/84434377/artifacts/report.htm . Best regards, and Happy Hacking! Media CI robot on behalf of the linux-media community. --- Check the latest rules for contributing your patches at: https://docs.kernel.org/driver-api/media/maintainer-entry-profile.html If you believe that the CI is wrong, kindly open an issue at https://gitlab.freedesktop.org/linux-media/media-ci/-/issues or reply-all to this message.
