mbox series

[GIT,PULL,FOR,v5.20] v3: hantro/cedrus/vb2 updates

Message ID 7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v5.20] v3: hantro/cedrus/vb2 updates | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20j

Message

Hans Verkuil July 15, 2022, 3:03 p.m. UTC
This PR sits on top of the stateless HEVC uAPI series:

https://patchwork.linuxtv.org/project/linux-media/patch/8409c576-e2f3-489a-23df-dc16386aac13@xs4all.nl/

This series adds hantro 10 bit encoding, various cedrus fixes,
and the series replacing vb2_find_timestamp with vb2_find_buffer.

Changes since v2: dropped "cedrus: Use vb2_find_buffer" and
"videobuf2: Remove vb2_find_timestamp()" since the first cedrus patch introduced
a regression. This can be added later.

Changes since v1: added the last three patches of the cedrus series
(https://patchwork.linuxtv.org/project/linux-media/list/?series=8237) after
Ezequiel reviewed those as well.

Regards,

	Hans

The following changes since commit bb0b9dfa3ba6b403d2f8ce39a024668e6b6e63c7:

  media: uapi: move HEVC stateless controls out of staging (2022-07-15 16:49:13 +0200)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.20j

for you to fetch changes up to 185b34bc5169424c65f3e7aa742cb96f955ebe42:

  rkvdec: Use vb2_find_buffer (2022-07-15 16:49:14 +0200)

----------------------------------------------------------------
Tag branch

----------------------------------------------------------------
Ezequiel Garcia (7):
      media: Add P010 tiled format
      videobuf2: Introduce vb2_find_buffer()
      mediatek: vcodec: Use vb2_find_buffer
      tegra-vde: Use vb2_find_buffer
      vicodec: Use vb2_find_buffer
      hantro: Use vb2_find_buffer
      rkvdec: Use vb2_find_buffer

Jernej Skrabec (13):
      media: hantro: Support format filtering by depth
      media: hantro: postproc: Fix buffer size calculation
      media: hantro: postproc: Fix legacy regs configuration
      media: hantro: postproc: Properly calculate chroma offset
      media: hantro: Store VP9 bit depth in context
      media: hantro: sunxi: Enable 10-bit decoding
      media: cedrus: h265: Fix flag name
      media: cedrus: h265: Fix logic for not low delay flag
      media: cedrus: Improve error messages for controls
      media: cedrus: Add error handling for failed setup
      media: cedrus: h265: Add a couple of error checks
      media: cedrus: Add helper for determining number of elements
      media: cedrus: h265: Implement support for tiles

 Documentation/userspace-api/media/v4l/pixfmt-yuv-planar.rst        |  18 ++++++--
 drivers/media/common/videobuf2/videobuf2-v4l2.c                    |  12 +++++
 drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.c |   7 ++-
 drivers/media/platform/mediatek/vcodec/vdec/vdec_vp8_req_if.c      |   7 ++-
 drivers/media/platform/mediatek/vcodec/vdec/vdec_vp9_req_lat_if.c  |   8 ++--
 drivers/media/platform/nvidia/tegra-vde/h264.c                     |   9 ++--
 drivers/media/test-drivers/vicodec/vicodec-core.c                  |   8 +---
 drivers/media/v4l2-core/v4l2-common.c                              |   1 +
 drivers/media/v4l2-core/v4l2-ioctl.c                               |   1 +
 drivers/staging/media/hantro/hantro.h                              |   4 ++
 drivers/staging/media/hantro/hantro_drv.c                          |  29 ++++++++++--
 drivers/staging/media/hantro/hantro_g2_vp9_dec.c                   |  18 +++-----
 drivers/staging/media/hantro/hantro_postproc.c                     |  38 ++++++++++++----
 drivers/staging/media/hantro/hantro_v4l2.c                         |  50 +++++++++++++++++++--
 drivers/staging/media/hantro/hantro_v4l2.h                         |   3 ++
 drivers/staging/media/hantro/sunxi_vpu_hw.c                        |  27 +++++++++++
 drivers/staging/media/rkvdec/rkvdec-h264.c                         |  41 ++++++-----------
 drivers/staging/media/rkvdec/rkvdec-vp9.c                          |  10 ++---
 drivers/staging/media/sunxi/cedrus/cedrus.c                        |  28 +++++++++++-
 drivers/staging/media/sunxi/cedrus/cedrus.h                        |   7 ++-
 drivers/staging/media/sunxi/cedrus/cedrus_dec.c                    |  27 ++++++++---
 drivers/staging/media/sunxi/cedrus/cedrus_h264.c                   |   5 ++-
 drivers/staging/media/sunxi/cedrus/cedrus_h265.c                   | 154 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----
 drivers/staging/media/sunxi/cedrus/cedrus_mpeg2.c                  |   4 +-
 drivers/staging/media/sunxi/cedrus/cedrus_regs.h                   |   3 +-
 drivers/staging/media/sunxi/cedrus/cedrus_vp8.c                    |   5 ++-
 include/media/videobuf2-v4l2.h                                     |  10 +++++
 include/uapi/linux/videodev2.h                                     |   1 +
 28 files changed, 421 insertions(+), 114 deletions(-)

Comments

Jenkins July 15, 2022, 6 p.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/225090/
Build time: 00:00:00
Link: https://lore.kernel.org/linux-media/7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl

Build aborted due to a fatal error:
FAILED: probably the tree doesn't exist
Jenkins July 17, 2022, 8:37 a.m. UTC | #2
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/225499/
Build time: 00:46:47
Link: https://lore.kernel.org/linux-media/7b7eb926-9600-0116-b983-bc18613e7674@xs4all.nl

gpg: Signature made Fri 15 Jul 2022 02:49:56 PM 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 4/20 patches with issues, being 2 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-media-Add-P010-tiled-format.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:5495 cx23885_dif_setup() parse error: turning off implications after 60 seconds
	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: 3000028Kb sm_state_count = 1725742
	../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/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'
	../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:2884 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

   checkpatch.pl:
	$ cat patches/0001-media-Add-P010-tiled-format.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:73: WARNING: line length of 165 exceeds 100 columns
	-:85: ERROR: trailing statements should be on next line
	-:97: WARNING: line length of 107 exceeds 100 columns

patches/0014-media-cedrus-h265-Implement-support-for-tiles.patch:

   checkpatch.pl:
	$ cat patches/0014-media-cedrus-h265-Implement-support-for-tiles.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:72: CHECK: Alignment should match open parenthesis
	-:74: CHECK: Alignment should match open parenthesis

patches/0015-videobuf2-Introduce-vb2_find_buffer.patch:

    allyesconfig: return code #0:
	../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:2858 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
	../drivers/media/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2548 dvb_register() parse error: OOM: 3000024Kb sm_state_count = 1725741
	../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 52 seconds

patches/0020-rkvdec-Use-vb2_find_buffer.patch:

   checkpatch.pl:
	$ cat patches/0020-rkvdec-Use-vb2_find_buffer.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:57: CHECK: Comparison to NULL could be written "run->ref_buf[ref->index]"


Error #512 when building PDF docs