On Tue, Dec 17, 2024 at 05:54:08PM -0800, Patchwork Integration wrote:
> Dear Laurent Pinchart:
>
> Thanks for your patches! Unfortunately media-ci detected some issues:
>
> # Test
> media-patchstyle:./0001-media-nxp-imx8-isi-fix-v4l2-compliance-test-errors.patch
> media style
> WARNING: Commit cf21f328fcafacf4f96e7a30ef9dceede1076378 found in the stable
> tree, but [email protected] not in Cc:
> WARNING: Please use correct Fixes: style 'Fixes: <12 chars of sha1> ("<title
> line>")' - ie: 'Fixes: cf21f328fcaf ("media: nxp: Add i.MX8 ISI driver")'
> #16:
> Fixes: cf21f328fcafac ("media: nxp: Add i.MX8 ISI driver")
I don't think Cc: stable makes sense for this particular commit.
> total: 0 errors, 1 warnings, 15 lines checked
> WARNING: ./0001-media-nxp-imx8-isi-fix-v4l2-compliance-test-errors.patch:
> Fails some mandatory checkpatch tests
>
> # Test
> checkpatch:./0001-media-nxp-imx8-isi-fix-v4l2-compliance-test-errors.patch
> checkpatch
> WARNING: Please use correct Fixes: style 'Fixes: <12 chars of sha1> ("<title
> line>")' - ie: 'Fixes: cf21f328fcaf ("media: nxp: Add i.MX8 ISI driver")'
> #16:
> Fixes: cf21f328fcafac ("media: nxp: Add i.MX8 ISI driver")
>
> total: 0 errors, 1 warnings, 0 checks, 15 lines checked
This I can fix. I'll wait for replies to the above though.
> 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/68354451/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