Dear Sakari Ailus:

Thanks for your patches! Unfortunately the Media CI robot detected some
issues:

# Test media-patchstyle:./0003-media-ccs-Avoid-possible-division-by-zero.patch 
media style
ERROR: Commit ccfc97bdb5ae8b8edc55169ac6924e08449836ac NOT found in the stable 
tree, but [email protected] is in Cc:

# Test media-patchstyle:./0013-media-dw9714-Fix-powerup-sequence.patch media 
style
ERROR: Commit 9d00ccabfbb522bb7a23ed329be2bf6aee04fe06 NOT found in the stable 
tree, but [email protected] is in Cc:
WARNING: Reported-by: should be immediately followed by Closes: with a URL to 
the report
#28: 
Reported-by: Naomi Huang <[email protected]>
Cc: [email protected]

total: 0 errors, 1 warnings, 8 lines checked

ERROR: Fails one or more mandatory checkpatch tests

# Test 
media-patchstyle:./0037-media-ccs-Fix-setting-initial-sub-device-state.patch 
media style
ERROR: Commit 5755be5f15d9e651ed433e5dbf7b7b968efb3064 NOT found in the stable 
tree, but [email protected] is in Cc:

# Test 
media-patchstyle:./0056-media-i2c-ov5647-use-our-own-mutex-for-the-ctrl-lock.patch
 media style
ERROR: Commit 4974c2f19fd810ec9a4e534bfc69e176256b7a03 NOT found in the stable 
tree, but [email protected] is in Cc:

# Test checkpatch:./0013-media-dw9714-Fix-powerup-sequence.patch checkpatch
WARNING: Reported-by: should be immediately followed by Closes: with a URL to 
the report
#28: 
Reported-by: Naomi Huang <[email protected]>
Cc: [email protected]

total: 0 errors, 1 warnings, 0 checks, 8 lines checked

# Test 
checkpatch:./0033-media-dt-bindings-i2c-toshiba-et8ek8-Convert-to-DT-s.patch 
checkpatch
WARNING: DT binding docs and includes should be a separate patch. See: 
Documentation/devicetree/bindings/submitting-patches.rst

WARNING: DT binding docs and includes should be a separate patch. See: 
Documentation/devicetree/bindings/submitting-patches.rst

total: 0 errors, 2 warnings, 0 checks, 94 lines checked

# Test 
checkpatch:./0034-media-dt-bindings-ti-omap3isp-Convert-to-DT-schema.patch 
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#19: 
deleted file mode 100644

WARNING: DT binding docs and includes should be a separate patch. See: 
Documentation/devicetree/bindings/submitting-patches.rst

WARNING: DT binding docs and includes should be a separate patch. See: 
Documentation/devicetree/bindings/submitting-patches.rst

total: 0 errors, 3 warnings, 0 checks, 189 lines checked

# Test 
checkpatch:./0036-media-i2c-og0ve1b-make-configurable-vblank-control-o.patch 
checkpatch
CHECK: Alignment should match open parenthesis
#65: FILE: drivers/media/i2c/og0ve1b.c:291:
+ret = __v4l2_ctrl_modify_range(og0ve1b->exposure,
+og0ve1b->exposure->minimum,

total: 0 errors, 0 warnings, 1 checks, 106 lines checked

# Test checkpatch:./0040-media-ccs-Support-frame-descriptors.patch checkpatch
CHECK: Alignment should match open parenthesis
#92: FILE: drivers/media/i2c/ccs/ccs-core.c:2659:
+static int ccs_get_frame_desc(struct v4l2_subdev *subdev, unsigned int pad,
+ struct v4l2_mbus_frame_desc *desc)

total: 0 errors, 0 warnings, 1 checks, 127 lines checked

# Test 
checkpatch:./0050-media-mt9m114-Adjust-IFP-selections-and-source-forma.patch 
checkpatch
CHECK: Unnecessary parentheses around 'format->code == 
MEDIA_BUS_FMT_SGRBG10_1X10'
#90: FILE: drivers/media/i2c/mt9m114.c:1907:
+if ((format->code == MEDIA_BUS_FMT_SGRBG10_1X10) !=
+    (info->code == MEDIA_BUS_FMT_SGRBG10_1X10)) {

CHECK: Unnecessary parentheses around 'info->code == MEDIA_BUS_FMT_SGRBG10_1X10'
#90: FILE: drivers/media/i2c/mt9m114.c:1907:
+if ((format->code == MEDIA_BUS_FMT_SGRBG10_1X10) !=
+    (info->code == MEDIA_BUS_FMT_SGRBG10_1X10)) {

total: 0 errors, 0 warnings, 2 checks, 65 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/90739241/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.

_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to