From: builder@xxxxxxxxxxx Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/970c5f07-f18b-cb0d-0f5d-49c69b2e070e@xxxxxxxxx/ Build log: https://builder.linuxtv.org/job/patchwork/327365/ Build time: 00:15:55 Link: https://lore.kernel.org/linux-media/970c5f07-f18b-cb0d-0f5d-49c69b2e070e@xxxxxxxxx gpg: Signature made Wed 02 Aug 2023 03:39:13 PM UTC gpg: using EDDSA key 52ADCAAE8A4F70B99ACD8D726B425DF79B1C1E76 gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xxxxxxxxx>" [unknown] gpg: aka "Hans Verkuil <hverkuil@xxxxxxxxx>" [full] Summary: got 13/16 patches with issues, being 10 at build time Error/warnings: patches/0001-media-mediatek-vcodec-remove-unused-parameter.patch: allyesconfig: return code #0: ../scripts/genksyms/parse.y: warning: 9 shift/reduce conflicts [-Wconflicts-sr] ../scripts/genksyms/parse.y: warning: 5 reduce/reduce conflicts [-Wconflicts-rr] ../scripts/genksyms/parse.y: note: rerun with option '-Wcounterexamples' to generate conflict counterexamples ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c: ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:476 ov2680_init_cfg() error: we previously assumed 'sd_state' could be null (see line 468) ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:524 ov2680_s_stream() warn: missing error code 'ret' ../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:447 gc0310_s_stream() warn: missing error code 'ret' ../drivers/staging/media/atomisp/pci/atomisp_fops.c: ../drivers/staging/media/atomisp/pci/atomisp_fops.c:517 atomisp_open() warn: missing unwind goto? ../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2801 atomisp_cp_dvs_6axis_config() warn: missing unwind goto? ../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2900 atomisp_cp_morph_table() warn: missing unwind goto? allyesconfig: return code #0: ../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto? ../drivers/media/usb/dvb-usb-v2/af9035.c: ../drivers/media/usb/dvb-usb-v2/af9035.c:467 af9035_i2c_master_xfer() warn: inconsistent returns '&d->i2c_mutex'. Locked on : 326,387 Unlocked on: 465,467 ../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v1'. ../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v2'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_local_data'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_input_ctrl'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_pin_sts'. ../drivers/media/i2c/ds90ub960.c: ../drivers/media/i2c/ds90ub960.c:1780 ub960_init_tx_ports() error: uninitialized symbol 'pll_div'. SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2772 mxc_jpeg_probe() warn: missing unwind goto? ../drivers/media/i2c/ov5645.c: ../drivers/media/i2c/ov5645.c:687 ov5645_set_power_on() warn: 'ov5645->xclk' from clk_prepare_enable() not released on lines: 687. ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: OOM: 3000004Kb sm_state_count = 1971478 ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() warn: Function too hairy. No more merges. ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: __split_smt: function too hairy. Giving up after 54 seconds ../drivers/media/usb/uvc/uvc_v4l2.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../include/linux/rcuwait.h, ...): SPARSE:../drivers/media/usb/uvc/uvc_v4l2.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression ../drivers/media/pci/ivtv/ivtvfb.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../drivers/media/pci/ivtv/ivtv-driver.h): SPARSE:../drivers/media/pci/ivtv/ivtvfb.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting ../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2831 em28xx_v4l2_init() parse error: turning off implications after 60 seconds patches/0005-media-mediatek-vcodec-remove-the-dependency-of-vcode.patch: checkpatch.pl: $ cat patches/0005-media-mediatek-vcodec-remove-the-dependency-of-vcode.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:77: CHECK: Macro argument reuse 'ctx' - possible side-effects? -:80: CHECK: Macro argument reuse 'ctx' - possible side-effects? -:83: CHECK: Macro argument reuse 'ctx' - possible side-effects? -:86: CHECK: Macro argument reuse 'ctx' - possible side-effects? patches/0006-media-mediatek-vcodec-replace-pr_-with-dev_-for-v4l2.patch: allyesconfig: return code #0: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c:94 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 93) patches/0007-media-mediatek-vcodec-separate-struct-mtk_vcodec_ctx.patch: allyesconfig: return code #0: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c:94 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 93) checkpatch.pl: $ cat patches/0007-media-mediatek-vcodec-separate-struct-mtk_vcodec_ctx.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:66: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? -:702: CHECK: struct mutex definition without comment -:1930: CHECK: struct mutex definition without comment patches/0008-media-mediatek-vcodec-separate-struct-mtk_vcodec_dev.patch: allyesconfig: return code #0: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c:93 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 92) checkpatch.pl: $ cat patches/0008-media-mediatek-vcodec-separate-struct-mtk_vcodec_dev.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:607: CHECK: struct mutex definition without comment -:610: CHECK: spinlock_t definition without comment -:798: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? -:1316: CHECK: struct mutex definition without comment -:1320: CHECK: spinlock_t definition without comment patches/0009-media-mediatek-vcodec-fix-unreasonable-parameter-def.patch: allyesconfig: return code #0: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c:93 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 92) patches/0010-media-mediatek-vcodec-remove-unused-include-header.patch: allyesconfig: return code #0: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c:92 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 91) patches/0011-media-mediatek-vcodec-separate-decoder-and-encoder.patch: allyesconfig: return code #0: ../drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c:92 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 91) checkpatch.pl: $ cat patches/0011-media-mediatek-vcodec-separate-decoder-and-encoder.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:19: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? -:483: CHECK: Macro argument reuse 'ctx' - possible side-effects? -:486: CHECK: Macro argument reuse 'ctx' - possible side-effects? -:782: CHECK: Macro argument reuse 'ctx' - possible side-effects? -:785: CHECK: Macro argument reuse 'ctx' - possible side-effects? patches/0012-media-mediatek-vcodec-Add-capture-format-to-support-.patch: allyesconfig: return code #0: ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c: ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:476 ov2680_init_cfg() error: we previously assumed 'sd_state' could be null (see line 468) ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:524 ov2680_s_stream() warn: missing error code 'ret' ../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:447 gc0310_s_stream() warn: missing error code 'ret' ../drivers/staging/media/atomisp/pci/atomisp_fops.c: ../drivers/staging/media/atomisp/pci/atomisp_fops.c:517 atomisp_open() warn: missing unwind goto? ../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2801 atomisp_cp_dvs_6axis_config() warn: missing unwind goto? ../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2900 atomisp_cp_morph_table() warn: missing unwind goto? allyesconfig: return code #0: ../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto? ../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v1'. ../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v2'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_local_data'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_input_ctrl'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_pin_sts'. ../drivers/media/i2c/ds90ub960.c: ../drivers/media/i2c/ds90ub960.c:1780 ub960_init_tx_ports() error: uninitialized symbol 'pll_div'. ../drivers/media/usb/dvb-usb-v2/af9035.c: ../drivers/media/usb/dvb-usb-v2/af9035.c:467 af9035_i2c_master_xfer() warn: inconsistent returns '&d->i2c_mutex'. Locked on : 326,387 Unlocked on: 465,467 ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2772 mxc_jpeg_probe() warn: missing unwind goto? ../drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c:92 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 91) ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: OOM: 3000004Kb sm_state_count = 1971044 ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() warn: Function too hairy. No more merges. ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: __split_smt: function too hairy. Giving up after 55 seconds ../drivers/media/i2c/ov5645.c: ../drivers/media/i2c/ov5645.c:687 ov5645_set_power_on() warn: 'ov5645->xclk' from clk_prepare_enable() not released on lines: 687. ../drivers/media/pci/ivtv/ivtvfb.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../drivers/media/pci/ivtv/ivtv-driver.h): SPARSE:../drivers/media/pci/ivtv/ivtvfb.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression ../drivers/media/usb/uvc/uvc_v4l2.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../include/linux/rcuwait.h, ...): SPARSE:../drivers/media/usb/uvc/uvc_v4l2.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting ../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2890 em28xx_v4l2_init() parse error: turning off implications after 60 seconds checkpatch.pl: $ cat patches/0012-media-mediatek-vcodec-Add-capture-format-to-support-.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:46: WARNING: line length of 190 exceeds 100 columns -:59: ERROR: trailing statements should be on next line patches/0013-media-mediatek-vcodec-Add-capture-format-to-support-.patch: allyesconfig: return code #0: ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c: ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:476 ov2680_init_cfg() error: we previously assumed 'sd_state' could be null (see line 468) ../drivers/staging/media/atomisp/i2c/atomisp-ov2680.c:524 ov2680_s_stream() warn: missing error code 'ret' ../drivers/staging/media/atomisp/i2c/atomisp-gc0310.c:447 gc0310_s_stream() warn: missing error code 'ret' ../drivers/staging/media/atomisp/pci/atomisp_fops.c: ../drivers/staging/media/atomisp/pci/atomisp_fops.c:517 atomisp_open() warn: missing unwind goto? ../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2801 atomisp_cp_dvs_6axis_config() warn: missing unwind goto? ../drivers/staging/media/atomisp/pci/atomisp_cmd.c: ../drivers/staging/media/atomisp/pci/atomisp_cmd.c:2900 atomisp_cp_morph_table() warn: missing unwind goto? allyesconfig: return code #0: ../drivers/media/i2c/adp1653.c: ../drivers/media/i2c/adp1653.c:444 adp1653_of_init() warn: missing unwind goto? ../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v1'. ../drivers/media/i2c/ds90ub913.c: ../drivers/media/i2c/ds90ub913.c:479 ub913_log_status() error: uninitialized symbol 'v2'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_local_data'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_input_ctrl'. ../drivers/media/i2c/ds90ub953.c: ../drivers/media/i2c/ds90ub953.c:642 ub953_log_status() error: uninitialized symbol 'gpio_pin_sts'. ../drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c:92 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 91) ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c: ../drivers/media/platform/nxp/imx-jpeg/mxc-jpeg.c:2772 mxc_jpeg_probe() warn: missing unwind goto? ../drivers/media/i2c/ds90ub960.c: ../drivers/media/i2c/ds90ub960.c:1780 ub960_init_tx_ports() error: uninitialized symbol 'pll_div'. ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: OOM: 3000004Kb sm_state_count = 1971566 ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() warn: Function too hairy. No more merges. ../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2558 dvb_register() parse error: __split_smt: function too hairy. Giving up after 54 seconds ../drivers/media/pci/ivtv/ivtvfb.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../drivers/media/pci/ivtv/ivtv-driver.h): SPARSE:../drivers/media/pci/ivtv/ivtvfb.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression ../drivers/media/i2c/ov5645.c: ../drivers/media/i2c/ov5645.c:687 ov5645_set_power_on() warn: 'ov5645->xclk' from clk_prepare_enable() not released on lines: 687. ../drivers/media/usb/dvb-usb-v2/af9035.c: ../drivers/media/usb/dvb-usb-v2/af9035.c:467 af9035_i2c_master_xfer() warn: inconsistent returns '&d->i2c_mutex'. Locked on : 326,387 Unlocked on: 465,467 ../drivers/media/usb/uvc/uvc_v4l2.c: note: in included file (through ../arch/x86/include/asm/uaccess.h, ../include/linux/uaccess.h, ../include/linux/sched/task.h, ../include/linux/sched/signal.h, ../include/linux/rcuwait.h, ...): SPARSE:../drivers/media/usb/uvc/uvc_v4l2.c ../arch/x86/include/asm/uaccess_64.h:88:24: warning: cast removes address space '__user' of expression ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3293 pvr2_hdw_get_tuner_status() warn: inconsistent indenting ../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2890 em28xx_v4l2_init() parse error: turning off implications after 60 seconds checkpatch.pl: $ cat patches/0013-media-mediatek-vcodec-Add-capture-format-to-support-.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:45: WARNING: line length of 190 exceeds 100 columns -:58: ERROR: trailing statements should be on next line -:70: WARNING: line length of 101 exceeds 100 columns patches/0014-media-mediatek-vcodec-Add-driver-to-support-10bit.patch: allyesconfig: return code #0: ../drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c: ../drivers/media/platform/mediatek/vcodec/decoder/vdec_vpu_if.c:92 vpu_dec_ipi_handler() error: we previously assumed 'vpu' could be null (see line 91) patches/0015-media-mediatek-vcodec-Fix-possible-invalid-memory-ac.patch: checkpatch.pl: $ cat patches/0015-media-mediatek-vcodec-Fix-possible-invalid-memory-ac.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:12: WARNING: Reported-by: should be immediately followed by Closes: with a URL to the report patches/0016-media-mediatek-vcodec-Fix-possible-invalid-memory-ac.patch: checkpatch.pl: $ cat patches/0016-media-mediatek-vcodec-Fix-possible-invalid-memory-ac.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:12: WARNING: Reported-by: should be immediately followed by Closes: with a URL to the report
