Dear Nicolas Dufresne:

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

# Test static:test-sparse
drivers/media/platform/rockchip/rkvdec/rkvdec-cabac.c:25:10: warning: symbol 
'rkvdec_h264_cabac_table' was not declared. Should it be static?
drivers/media/platform/rockchip/rkvdec/rkvdec-cabac.c:514:10: warning: symbol 
'rkvdec_hevc_cabac_table' was not declared. Should it be static?

# Test 
checkpatch:./0002-media-v4l2-ctrls-Add-hevc_ext_sps_-ls-t_rps-controls.patch 
checkpatch
WARNING: Avoid logging continuation uses where feasible
#37: FILE: drivers/media/v4l2-core/v4l2-ctrls-core.c:428:
+pr_cont("HEVC_EXT_SPS_ST_RPS");

WARNING: Avoid logging continuation uses where feasible
#40: FILE: drivers/media/v4l2-core/v4l2-ctrls-core.c:431:
+pr_cont("HEVC_EXT_SPS_LT_RPS");

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

# Test checkpatch:./0003-media-visl-Add-HEVC-short-and-long-term-RPS-sets.patch 
checkpatch
CHECK: Alignment should match open parenthesis
#79: FILE: drivers/media/test-drivers/visl/visl-trace-hevc.h:348:
+DECLARE_EVENT_CLASS(v4l2_ctrl_hevc_ext_sps_lt_rps_tmpl,
+TP_PROTO(const struct v4l2_ctrl_hevc_ext_sps_lt_rps *lt),

CHECK: Alignment should match open parenthesis
#86: FILE: drivers/media/test-drivers/visl/visl-trace-hevc.h:355:
+  __print_flags(__entry->lt.flags, "|",
+  {V4L2_HEVC_EXT_SPS_LT_RPS_FLAG_USED_LT, "USED_LT"}

CHECK: Alignment should match open parenthesis
#93: FILE: drivers/media/test-drivers/visl/visl-trace-hevc.h:362:
+DECLARE_EVENT_CLASS(v4l2_ctrl_hevc_ext_sps_st_rps_tmpl,
+TP_PROTO(const struct v4l2_ctrl_hevc_ext_sps_st_rps *st),

CHECK: Alignment should match open parenthesis
#108: FILE: drivers/media/test-drivers/visl/visl-trace-hevc.h:377:
+  __print_flags(__entry->st.flags, "|",
+  {V4L2_HEVC_EXT_SPS_ST_RPS_FLAG_INTER_REF_PIC_SET_PRED, 
"INTER_REF_PIC_SET_PRED"}

CHECK: Alignment should match open parenthesis
#134: FILE: drivers/media/test-drivers/visl/visl-trace-hevc.h:444:
+DEFINE_EVENT(v4l2_ctrl_hevc_ext_sps_lt_rps_tmpl, v4l2_ctrl_hevc_ext_sps_lt_rps,
+TP_PROTO(const struct v4l2_ctrl_hevc_ext_sps_lt_rps *lt),

CHECK: Alignment should match open parenthesis
#139: FILE: drivers/media/test-drivers/visl/visl-trace-hevc.h:449:
+DEFINE_EVENT(v4l2_ctrl_hevc_ext_sps_st_rps_tmpl, v4l2_ctrl_hevc_ext_sps_st_rps,
+TP_PROTO(const struct v4l2_ctrl_hevc_ext_sps_st_rps *st),

total: 0 errors, 0 warnings, 6 checks, 109 lines checked

# Test 
checkpatch:./0004-media-rkvdec-Switch-to-using-structs-instead-of-writ.patch 
checkpatch
WARNING: Prefer a maximum 75 chars per line (possible unwrapped commit 
description?)
#38: 
[1]: 
https://lore.kernel.org/all/[email protected]/

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

# Test 
checkpatch:./0005-media-rkvdec-Move-cabac-tables-to-their-own-source-f.patch 
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#16: 
 .../{rkvdec-hevc-data.c => rkvdec-cabac.c}    | 506 +++++++++++++++++-

CHECK: Macro argument reuse 'ctxidx' - possible side-effects?
#58: FILE: drivers/media/platform/rockchip/rkvdec/rkvdec-cabac.c:13:
+#define CABAC_ENTRY(ctxidx, idc0_m, idc0_n, idc1_m, idc1_n,\
+    idc2_m, idc2_n, intra_m, intra_n)\
+[0][(ctxidx)] = {idc0_m, idc0_n},\
+[1][(ctxidx)] = {idc1_m, idc1_n},\
+[2][(ctxidx)] = {idc2_m, idc2_n},\
+[3][(ctxidx)] = {intra_m, intra_n}

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

# Test checkpatch:./0007-media-rkvdec-Move-h264-functions-to-common-file.patch 
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#37: 
new file mode 100644

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

# Test checkpatch:./0008-media-rkvdec-Move-hevc-functions-to-common-file.patch 
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#36: 
new file mode 100644

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

# Test checkpatch:./0010-media-rkvdec-Add-RCB-and-SRAM-support.patch checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#49: 
new file mode 100644

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

# Test 
checkpatch:./0014-media-rkvdec-Add-H264-support-for-the-VDPU381-varian.patch 
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#58: 
new file mode 100644

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

# Test 
checkpatch:./0015-media-rkvdec-Add-H264-support-for-the-VDPU383-varian.patch 
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#41: 
new file mode 100644

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

# Test 
checkpatch:./0016-media-rkvdec-Add-HEVC-support-for-the-VDPU381-varian.patch 
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#489: 
new file mode 100644

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

# Test 
checkpatch:./0017-media-rkvdec-Add-HEVC-support-for-the-VDPU383-varian.patch 
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#181: 
new file mode 100644

total: 0 errors, 1 warnings, 0 checks, 969 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/91371036/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