Ricardo, should I drop those patches and wait for a new version, or is
everybody happy ignoring those warnings ? Some of the warnings look
reasonable to me, were they ignored, or was the last version of the
series run through CI ?
On Sat, Mar 01, 2025 at 04:10:14PM -0800, Patchwork Integration wrote:
> Dear Laurent Pinchart:
>
> Thanks for your patches! Unfortunately the Media CI robot detected some
> issues:
>
> # Test checkpatch:./0001-media-v4l2_ctrl-Add-V4L2_CTRL_TYPE_RECT.patch checkpatch
> WARNING: Avoid logging continuation uses where feasible
> #83: FILE: drivers/media/v4l2-core/v4l2-ctrls-core.c:374:
> +pr_cont("%ux%u@%dx%d",
>
> total: 0 errors, 1 warnings, 0 checks, 98 lines checked
>
> # Test checkpatch:./0002-media-v4l2-ctrls-add-support-for-V4L2_CTRL_WHICH_MIN.patch checkpatch
> CHECK: Alignment should match open parenthesis
> #286: FILE: drivers/media/v4l2-core/v4l2-ctrls-core.c:186:
> +static void std_min_compound(const struct v4l2_ctrl *ctrl, u32 idx,
> + union v4l2_ctrl_ptr ptr)
>
> CHECK: Alignment should match open parenthesis
> #297: FILE: drivers/media/v4l2-core/v4l2-ctrls-core.c:197:
> +static void std_max_compound(const struct v4l2_ctrl *ctrl, u32 idx,
> union v4l2_ctrl_ptr ptr)
>
> total: 0 errors, 0 warnings, 2 checks, 598 lines checked
>
> # Test checkpatch:./0008-media-uvcvideo-Support-any-size-for-mapping-get-set.patch checkpatch
> CHECK: Alignment should match open parenthesis
> #282: FILE: drivers/media/usb/uvc/uvc_ctrl.c:1406:
> +v4l2_ctrl->default_value = uvc_mapping_get_s32(mapping,
> +UVC_GET_DEF, uvc_ctrl_data(ctrl, UVC_CTRL_DATA_DEF));
>
> CHECK: Alignment should match open parenthesis
> #293: FILE: drivers/media/usb/uvc/uvc_ctrl.c:1440:
> +v4l2_ctrl->minimum = uvc_mapping_get_s32(mapping, UVC_GET_MIN,
> +uvc_ctrl_data(ctrl, UVC_CTRL_DATA_MIN));
>
> CHECK: Alignment should match open parenthesis
> #299: FILE: drivers/media/usb/uvc/uvc_ctrl.c:1444:
> +v4l2_ctrl->maximum = uvc_mapping_get_s32(mapping, UVC_GET_MAX,
> +uvc_ctrl_data(ctrl, UVC_CTRL_DATA_MAX));
>
> CHECK: Alignment should match open parenthesis
> #305: FILE: drivers/media/usb/uvc/uvc_ctrl.c:1448:
> +v4l2_ctrl->step = uvc_mapping_get_s32(mapping, UVC_GET_RES,
> +uvc_ctrl_data(ctrl, UVC_CTRL_DATA_RES));
>
> total: 0 errors, 0 warnings, 4 checks, 332 lines checked
>
> # Test checkpatch:./0017-media-uvcvideo-document-UVC-v1.5-ROI.patch checkpatch
> WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit description?)
> #10:
> https://chromium.googlesource.com/chromiumos/platform2/+/refs/heads/release-R121-15699.B/camera/hal/usb/
>
> total: 0 errors, 1 warnings, 0 checks, 73 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/72096058/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.
--
Regards,
Laurent Pinchart
