| Message ID | fe899b13-8e1c-4550-4b1a-a2b125e19cc9@xs4all.nl (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,v5.20] cedrus fix, hantro fixes and destaging | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/fe899b13-8e1c-4550-4b1a-a2b125e19cc9@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/226064/
Build time: 00:19:55
Link: https://lore.kernel.org/linux-media/fe899b13-8e1c-4550-4b1a-a2b125e19cc9@xs4all.nl
gpg: Signature made Tue 19 Jul 2022 09:30:09 AM UTC
gpg: using EDDSA key 52ADCAAE8A4F70B99ACD8D726B425DF79B1C1E76
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg: aka "Hans Verkuil <hverkuil@xs4all.nl>" [full]
Summary: got 2/4 patches with issues, being 1 at build time, plus one error when buinding PDF document
Error/warnings:
patches/0001-media-cedrus-hevc-Add-check-for-invalid-timestamp.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
allyesconfig: return code #0:
../drivers/media/i2c/cx25840/cx25840-core.c: ../drivers/media/i2c/cx25840/cx25840-core.c:5473 cx23885_dif_setup() parse error: turning off implications after 60 seconds
../drivers/media/platform/sunxi/sun6i-mipi-csi2/sun6i_mipi_csi2.c:193 sun6i_mipi_csi2_s_stream() warn: missing error code 'ret'
../drivers/media/platform/sunxi/sun8i-a83t-mipi-csi2/sun8i_a83t_mipi_csi2.c:225 sun8i_a83t_mipi_csi2_s_stream() warn: missing error code 'ret'
SMATCH:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: :warning: array of flexible structures
SPARSE:../drivers/media/usb/siano/smsusb.c ../drivers/media/usb/siano/smsusb.c:53:38: warning: array of flexible structures
../drivers/media/dvb-frontends/sp887x.c: ../drivers/media/dvb-frontends/sp887x.c:178 sp887x_initial_setup() error: __memcpy() '&buf[2]' too small (30 vs 16384)
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000008Kb sm_state_count = 1725655
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() warn: Function too hairy. No more merges.
../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: __split_smt: function too hairy. Giving up after 50 seconds
../drivers/media/test-drivers/vimc/vimc-core.c: ../drivers/media/test-drivers/vimc/vimc-core.c:214 vimc_create_links() warn: passing a valid pointer to 'PTR_ERR'
../drivers/media/usb/uvc/uvc_ctrl.c: ../drivers/media/usb/uvc/uvc_ctrl.c:2472 uvc_ctrl_init_ctrl() error: we previously assumed 'mapping' could be null (see line 2458)
../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2878 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
patches/0003-hantro-Remove-dedicated-control-documentation.patch:
checkpatch.pl:
$ cat patches/0003-hantro-Remove-dedicated-control-documentation.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:21: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
Error #512 when building PDF docs
Em Tue, 19 Jul 2022 11:34:55 +0200 Hans Verkuil <hverkuil@xs4all.nl> escreveu: > Hi Mauro, > > This PR has three cedrus/hantro fixes and destages hantro in the final patch. > That last patch just moves code into the new directory and has no other changes. > > Ezequiel, I decided to postpone the v3 of the vb2_find_buffer series to 5.21. > It's just an improvement, and the cedrus changes are a bit larger than I'd like > so close to the code freeze. I'd like to have those tested first. It sounds too late to merge it for the upcoming merge window. So, I applied only patch 1/4: media: cedrus: hevc: Add check for invalid timestamp Let's merge the remaining ones after -rc1. Regards, Mauro > > Regards, > > Hans > > The following changes since commit 8bd1dbf8d580c425605fb8936309a4e9745a7a95: > > media: rkisp1: debug: Add dump file in debugfs for MI main path registers (2022-07-17 12:34:43 +0100) > > are available in the Git repository at: > > git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20m > > for you to fetch changes up to 10b516fc3544cc9057495d981e43df3953bd264d: > > hantro: Remove incorrect HEVC SPS validation (2022-07-19 11:14:41 +0200) > > ---------------------------------------------------------------- > Tag branch > > ---------------------------------------------------------------- > Ezequiel Garcia (3): > media: destage Hantro VPU driver > hantro: Remove dedicated control documentation > hantro: Remove incorrect HEVC SPS validation > > Jernej Skrabec (1): > media: cedrus: hevc: Add check for invalid timestamp > > Documentation/userspace-api/media/drivers/hantro.rst | 19 ------------------- > Documentation/userspace-api/media/drivers/index.rst | 1 - > MAINTAINERS | 2 +- > drivers/media/platform/Kconfig | 1 + > drivers/media/platform/Makefile | 1 + > drivers/{staging/media/hantro => media/platform/verisilicon}/Kconfig | 6 +++++- > drivers/{staging/media/hantro => media/platform/verisilicon}/Makefile | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro.h | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_drv.c | 12 ++++++------ > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_h264_dec.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_mpeg2_dec.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_regs.h | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_vp8_dec.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_hevc_dec.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_regs.h | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_vp9_dec.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_jpeg_enc.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_regs.h | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h264.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hevc.c | 9 +-------- > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hw.h | 1 - > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.h | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_mpeg2.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_postproc.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.h | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp8.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.h | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/imx8m_vpu_hw.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_h264_dec.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_jpeg_enc.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_mpeg2_dec.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_vp8_dec.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_regs.h | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu_hw.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/sama5d4_vdec_hw.c | 0 > drivers/{staging/media/hantro => media/platform/verisilicon}/sunxi_vpu_hw.c | 0 > drivers/staging/media/Kconfig | 2 -- > drivers/staging/media/Makefile | 1 - > drivers/staging/media/hantro/TODO | 2 -- > drivers/staging/media/sunxi/cedrus/cedrus_h265.c | 3 +++ > 45 files changed, 18 insertions(+), 42 deletions(-) > delete mode 100644 Documentation/userspace-api/media/drivers/hantro.rst > rename drivers/{staging/media/hantro => media/platform/verisilicon}/Kconfig (91%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/Makefile (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro.h (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_drv.c (99%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_h264_dec.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_mpeg2_dec.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_regs.h (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_vp8_dec.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_hevc_dec.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_regs.h (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_vp9_dec.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_jpeg_enc.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_regs.h (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h264.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hevc.c (96%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hw.h (99%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.h (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_mpeg2.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_postproc.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.h (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp8.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.h (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/imx8m_vpu_hw.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_h264_dec.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_jpeg_enc.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_mpeg2_dec.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_vp8_dec.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_regs.h (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu_hw.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/sama5d4_vdec_hw.c (100%) > rename drivers/{staging/media/hantro => media/platform/verisilicon}/sunxi_vpu_hw.c (100%) > delete mode 100644 drivers/staging/media/hantro/TODO Thanks, Mauro
Em Wed, 27 Jul 2022 20:35:51 +0200 Mauro Carvalho Chehab <mchehab@kernel.org> escreveu: > Em Tue, 19 Jul 2022 11:34:55 +0200 > Hans Verkuil <hverkuil@xs4all.nl> escreveu: > > > Hi Mauro, > > > > This PR has three cedrus/hantro fixes and destages hantro in the final patch. > > That last patch just moves code into the new directory and has no other changes. > > > > Ezequiel, I decided to postpone the v3 of the vb2_find_buffer series to 5.21. > > It's just an improvement, and the cedrus changes are a bit larger than I'd like > > so close to the code freeze. I'd like to have those tested first. > > It sounds too late to merge it for the upcoming merge window. > > So, I applied only patch 1/4: > > media: cedrus: hevc: Add check for invalid timestamp > > Let's merge the remaining ones after -rc1. Applied patch 4/4 as well, as this is actually a fix: hantro: Remove incorrect HEVC SPS validation Regards, Mauro
Hi Mauro, This PR has three cedrus/hantro fixes and destages hantro in the final patch. That last patch just moves code into the new directory and has no other changes. Ezequiel, I decided to postpone the v3 of the vb2_find_buffer series to 5.21. It's just an improvement, and the cedrus changes are a bit larger than I'd like so close to the code freeze. I'd like to have those tested first. Regards, Hans The following changes since commit 8bd1dbf8d580c425605fb8936309a4e9745a7a95: media: rkisp1: debug: Add dump file in debugfs for MI main path registers (2022-07-17 12:34:43 +0100) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20m for you to fetch changes up to 10b516fc3544cc9057495d981e43df3953bd264d: hantro: Remove incorrect HEVC SPS validation (2022-07-19 11:14:41 +0200) ---------------------------------------------------------------- Tag branch ---------------------------------------------------------------- Ezequiel Garcia (3): media: destage Hantro VPU driver hantro: Remove dedicated control documentation hantro: Remove incorrect HEVC SPS validation Jernej Skrabec (1): media: cedrus: hevc: Add check for invalid timestamp Documentation/userspace-api/media/drivers/hantro.rst | 19 ------------------- Documentation/userspace-api/media/drivers/index.rst | 1 - MAINTAINERS | 2 +- drivers/media/platform/Kconfig | 1 + drivers/media/platform/Makefile | 1 + drivers/{staging/media/hantro => media/platform/verisilicon}/Kconfig | 6 +++++- drivers/{staging/media/hantro => media/platform/verisilicon}/Makefile | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro.h | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_drv.c | 12 ++++++------ drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_h264_dec.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_mpeg2_dec.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_regs.h | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_vp8_dec.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_hevc_dec.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_regs.h | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_vp9_dec.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_jpeg_enc.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_regs.h | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h264.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hevc.c | 9 +-------- drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hw.h | 1 - drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.h | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_mpeg2.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_postproc.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.h | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp8.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.h | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/imx8m_vpu_hw.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_h264_dec.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_jpeg_enc.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_mpeg2_dec.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_vp8_dec.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_regs.h | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu_hw.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/sama5d4_vdec_hw.c | 0 drivers/{staging/media/hantro => media/platform/verisilicon}/sunxi_vpu_hw.c | 0 drivers/staging/media/Kconfig | 2 -- drivers/staging/media/Makefile | 1 - drivers/staging/media/hantro/TODO | 2 -- drivers/staging/media/sunxi/cedrus/cedrus_h265.c | 3 +++ 45 files changed, 18 insertions(+), 42 deletions(-) delete mode 100644 Documentation/userspace-api/media/drivers/hantro.rst rename drivers/{staging/media/hantro => media/platform/verisilicon}/Kconfig (91%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/Makefile (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro.h (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_drv.c (99%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_h264_dec.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_mpeg2_dec.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_regs.h (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g1_vp8_dec.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_hevc_dec.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_regs.h (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_g2_vp9_dec.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_jpeg_enc.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h1_regs.h (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_h264.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hevc.c (96%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_hw.h (99%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_jpeg.h (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_mpeg2.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_postproc.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_v4l2.h (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp8.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/hantro_vp9.h (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/imx8m_vpu_hw.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_h264_dec.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_jpeg_enc.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_mpeg2_dec.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_hw_vp8_dec.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu2_regs.h (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/rockchip_vpu_hw.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/sama5d4_vdec_hw.c (100%) rename drivers/{staging/media/hantro => media/platform/verisilicon}/sunxi_vpu_hw.c (100%) delete mode 100644 drivers/staging/media/hantro/TODO