mbox series

[GIT,PULL,FOR,v5.18] Move tegra-vde out of staging

Message ID 19a007e4-e287-145a-1891-ecde68aaa95a@xs4all.nl (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v5.18] Move tegra-vde out of staging | expand

Pull-request

git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.18p

Message

Hans Verkuil Feb. 22, 2022, 8:27 a.m. UTC
This series converts the driver to the standard V4L2 stateless decoder API
and removes the legacy API. This allows us to finally move the driver out
of staging.

Thank you very much, Dmitry, for working on this!

Regards,

	Hans


The following changes since commit 2b891d3980f6c255459d0e1c29ce2152ec7cf678:

  media: xilinx: csi2rxss: Use mipi-csi2.h (2022-02-17 10:32:26 +0100)

are available in the Git repository at:

  git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.18p

for you to fetch changes up to 1988a49f6baeb8292953682d219e6a15d28b5495:

  media: staging: tegra-vde: De-stage driver (2022-02-22 09:11:16 +0100)

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

----------------------------------------------------------------
Dmitry Osipenko (6):
      media: v4l2-ctrls: Add new V4L2_H264_DECODE_PARAM_FLAG_P/BFRAME flags
      media: staging: tegra-vde: Factor out H.264 code
      media: staging: tegra-vde: Support V4L stateless video decoder API
      media: staging: tegra-vde: Remove legacy UAPI support
      media: staging: tegra-vde: Bump BSEV DMA timeout
      media: staging: tegra-vde: De-stage driver

 Documentation/userspace-api/media/v4l/ext-ctrls-codec-stateless.rst          |    6 +
 MAINTAINERS                                                                  |    2 +-
 drivers/media/platform/Kconfig                                               |   17 +
 drivers/media/platform/Makefile                                              |    2 +
 drivers/media/platform/tegra/vde/Makefile                                    |    3 +
 drivers/{staging/media/tegra-vde => media/platform/tegra/vde}/dmabuf-cache.c |    2 +-
 drivers/media/platform/tegra/vde/h264.c                                      |  946 +++++++++++++++++++++++++++++++++++++++
 drivers/{staging/media/tegra-vde => media/platform/tegra/vde}/iommu.c        |    2 +-
 drivers/{staging/media/tegra-vde => media/platform/tegra/vde}/trace.h        |    2 +-
 drivers/media/platform/tegra/vde/v4l2.c                                      | 1018 ++++++++++++++++++++++++++++++++++++++++++
 drivers/media/platform/tegra/vde/vde.c                                       |  551 +++++++++++++++++++++++
 drivers/media/platform/tegra/vde/vde.h                                       |  242 ++++++++++
 drivers/staging/media/Kconfig                                                |    2 -
 drivers/staging/media/Makefile                                               |    1 -
 drivers/staging/media/tegra-vde/Kconfig                                      |   10 -
 drivers/staging/media/tegra-vde/Makefile                                     |    3 -
 drivers/staging/media/tegra-vde/TODO                                         |    4 -
 drivers/staging/media/tegra-vde/uapi.h                                       |   73 ---
 drivers/staging/media/tegra-vde/vde.c                                        | 1358 --------------------------------------------------------
 drivers/staging/media/tegra-vde/vde.h                                        |  125 ------
 include/uapi/linux/v4l2-controls.h                                           |    2 +
 21 files changed, 2791 insertions(+), 1580 deletions(-)
 create mode 100644 drivers/media/platform/tegra/vde/Makefile
 rename drivers/{staging/media/tegra-vde => media/platform/tegra/vde}/dmabuf-cache.c (99%)
 create mode 100644 drivers/media/platform/tegra/vde/h264.c
 rename drivers/{staging/media/tegra-vde => media/platform/tegra/vde}/iommu.c (98%)
 rename drivers/{staging/media/tegra-vde => media/platform/tegra/vde}/trace.h (97%)
 create mode 100644 drivers/media/platform/tegra/vde/v4l2.c
 create mode 100644 drivers/media/platform/tegra/vde/vde.c
 create mode 100644 drivers/media/platform/tegra/vde/vde.h
 delete mode 100644 drivers/staging/media/tegra-vde/Kconfig
 delete mode 100644 drivers/staging/media/tegra-vde/Makefile
 delete mode 100644 drivers/staging/media/tegra-vde/TODO
 delete mode 100644 drivers/staging/media/tegra-vde/uapi.h
 delete mode 100644 drivers/staging/media/tegra-vde/vde.c
 delete mode 100644 drivers/staging/media/tegra-vde/vde.h

Comments

Jenkins Feb. 22, 2022, 8:58 a.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/19a007e4-e287-145a-1891-ecde68aaa95a@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/185096/
Build time: 00:25:30
Link: https://lore.kernel.org/linux-media/19a007e4-e287-145a-1891-ecde68aaa95a@xs4all.nl

gpg: Signature made Tue 22 Feb 2022 08:22:30 AM UTC
gpg:                using EDDSA key 52ADCAAE8A4F70B99ACD8D726B425DF79B1C1E76
gpg: Can't check signature: No public key

Summary: got 5/6 patches with issues, being 1 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-media-v4l2-ctrls-Add-new-V4L2_H264_DECODE_PARAM_FLAG.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/rc/meson-ir-tx.c:22: warning: expecting prototype for meson(). Prototype was for DEVICE_NAME() instead
	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/pci/cx23885/cx23885-dvb.c: ../drivers/media/pci/cx23885/cx23885-dvb.c:2625 dvb_register() parse error: turning off implications after 60 seconds
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:658 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 654)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2894 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0002-media-staging-tegra-vde-Factor-out-H.264-code.patch:

   checkpatch.pl:
	$ cat patches/0002-media-staging-tegra-vde-Factor-out-H.264-code.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:29: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:643: CHECK: Lines should not end with a '('

patches/0003-media-staging-tegra-vde-Support-V4L-stateless-video-.patch:

   checkpatch.pl:
	$ cat patches/0003-media-staging-tegra-vde-Support-V4L-stateless-video-.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:381: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:1609: CHECK: struct mutex definition without comment

patches/0004-media-staging-tegra-vde-Remove-legacy-UAPI-support.patch:

   checkpatch.pl:
	$ cat patches/0004-media-staging-tegra-vde-Remove-legacy-UAPI-support.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:138: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0006-media-staging-tegra-vde-De-stage-driver.patch:

   checkpatch.pl:
	$ cat patches/0006-media-staging-tegra-vde-De-stage-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:60: WARNING: please write a help paragraph that fully describes the config symbol
	-:126: CHECK: spaces preferred around that '/' (ctx:VxV)
	-:126: CHECK: spaces preferred around that '/' (ctx:VxV)
	-:126: CHECK: spaces preferred around that '/' (ctx:VxV)
	-:126: CHECK: spaces preferred around that '/' (ctx:VxV)
	-:126: CHECK: spaces preferred around that '/' (ctx:VxV)
	-:126: CHECK: spaces preferred around that '/' (ctx:VxV)


Error #512 when building PDF docs