Dear Nicolas Dufresne:
Thanks for your patches! Unfortunately the Media CI robot detected some
issues:
# Test
checkpatch:./0003-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:./0004-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:./0005-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:./0006-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?
#61: 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, 1078 lines checked
# Test checkpatch:./0008-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, 716 lines checked
# Test checkpatch:./0009-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, 525 lines checked
# Test checkpatch:./0011-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:./0015-media-rkvdec-Add-H264-support-for-the-VDPU381-varian.patch
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#44:
new file mode 100644
total: 0 errors, 1 warnings, 0 checks, 1176 lines checked
# Test
checkpatch:./0016-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, 1016 lines checked
# Test
checkpatch:./0017-media-rkvdec-Add-HEVC-support-for-the-VDPU381-varian.patch
checkpatch
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#487:
new file mode 100644
total: 0 errors, 1 warnings, 0 checks, 1193 lines checked
# Test
checkpatch:./0018-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
WARNING: line length of 105 exceeds 100 columns
#385: FILE: drivers/media/platform/rockchip/rkvdec/rkvdec-vdpu383-hevc.c:200:
+static void set_pps_ref_pic_poc(struct rkvdec_hevc_sps_pps *hw_ps, const
struct v4l2_hevc_dpb_entry *dpb)
WARNING: Missing a blank line after declarations
#563: FILE: drivers/media/platform/rockchip/rkvdec/rkvdec-vdpu383-hevc.c:378:
+u32 valid = !!(dec_params->num_active_dpb_entries > i);
+hw_ps->ref_is_valid |= valid << i;
total: 0 errors, 3 warnings, 0 checks, 911 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/91554911/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]
