| Message ID | 000d2cfe-c0b0-da23-025c-1dc863a95e81@xs4all.nl (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,v5.18] v2: amphion video decoder/encoder driver | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/000d2cfe-c0b0-da23-025c-1dc863a95e81@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/185632/
Build time: 00:31:48
Link: https://lore.kernel.org/linux-media/000d2cfe-c0b0-da23-025c-1dc863a95e81@xs4all.nl
gpg: Signature made Thu 24 Feb 2022 07:23:49 AM UTC
gpg: using EDDSA key 52ADCAAE8A4F70B99ACD8D726B425DF79B1C1E76
gpg: Can't check signature: No public key
Summary: got 10/12 patches with issues, being 2 at build time, plus one error when buinding PDF document
Error/warnings:
patches/0001-dt-bindings-media-amphion-add-amphion-video-codec-bi.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:2900 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
checkpatch.pl:
$ cat patches/0001-dt-bindings-media-amphion-add-amphion-video-codec-bi.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:20: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0002-media-add-nv12m_8l128-and-nv12m_10be_8l128-video-for.patch:
allyesconfig: return code #0:
../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:2888 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
checkpatch.pl:
$ cat patches/0002-media-add-nv12m_8l128-and-nv12m_10be_8l128-video-for.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:86: ERROR: trailing statements should be on next line
-:87: ERROR: trailing statements should be on next line
-:100: WARNING: line length of 110 exceeds 100 columns
patches/0003-media-amphion-add-amphion-vpu-device-driver.patch:
checkpatch.pl:
$ cat patches/0003-media-amphion-add-amphion-vpu-device-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:38: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
-:348: CHECK: Macro argument reuse 'inst' - possible side-effects?
-:348: CHECK: Macro argument reuse 'op' - possible side-effects?
-:348: CHECK: Macro argument 'op' may be better as '(op)' to avoid precedence issues
-:351: CHECK: Macro argument reuse 'inst' - possible side-effects?
-:351: CHECK: Macro argument reuse 'op' - possible side-effects?
-:351: CHECK: Macro argument 'op' may be better as '(op)' to avoid precedence issues
patches/0004-media-amphion-add-vpu-core-driver.patch:
checkpatch.pl:
$ cat patches/0004-media-amphion-add-vpu-core-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:33: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0005-media-amphion-implement-vpu-core-communication-based.patch:
checkpatch.pl:
$ cat patches/0005-media-amphion-implement-vpu-core-communication-based.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:35: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0006-media-amphion-add-vpu-v4l2-m2m-support.patch:
checkpatch.pl:
$ cat patches/0006-media-amphion-add-vpu-v4l2-m2m-support.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:31: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0007-media-amphion-add-v4l2-m2m-vpu-encoder-stateful-driv.patch:
checkpatch.pl:
$ cat patches/0007-media-amphion-add-v4l2-m2m-vpu-encoder-stateful-driv.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:19: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0008-media-amphion-add-v4l2-m2m-vpu-decoder-stateful-driv.patch:
checkpatch.pl:
$ cat patches/0008-media-amphion-add-v4l2-m2m-vpu-decoder-stateful-driv.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:19: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0009-media-amphion-implement-windsor-encoder-rpc-interfac.patch:
checkpatch.pl:
$ cat patches/0009-media-amphion-implement-windsor-encoder-rpc-interfac.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:22: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0010-media-amphion-implement-malone-decoder-rpc-interface.patch:
checkpatch.pl:
$ cat patches/0010-media-amphion-implement-malone-decoder-rpc-interface.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
Ming Qian, Thank you very much for your hard work on this! Very nice to see this go in. Regards, Hans Changes since v1: - Added the Makefile/Kconfig last after all the other driver sources are in place to avoid bisect problems. 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.18r for you to fetch changes up to c40c0f562b42cad3bc97bc29c273aaa5776a67a5: media: amphion: add amphion vpu entry in Kconfig and Makefile (2022-02-24 08:22:49 +0100) ---------------------------------------------------------------- Tag branch ---------------------------------------------------------------- Ming Qian (12): dt-bindings: media: amphion: add amphion video codec bindings media: add nv12m_8l128 and nv12m_10be_8l128 video format. media: amphion: add amphion vpu device driver media: amphion: add vpu core driver media: amphion: implement vpu core communication based on mailbox media: amphion: add vpu v4l2 m2m support media: amphion: add v4l2 m2m vpu encoder stateful driver media: amphion: add v4l2 m2m vpu decoder stateful driver media: amphion: implement windsor encoder rpc interface media: amphion: implement malone decoder rpc interface MAINTAINERS: add AMPHION VPU CODEC V4L2 driver entry media: amphion: add amphion vpu entry in Kconfig and Makefile Documentation/devicetree/bindings/media/amphion,vpu.yaml | 180 ++++++ Documentation/userspace-api/media/v4l/pixfmt-yuv-planar.rst | 28 +- MAINTAINERS | 9 + drivers/media/platform/Kconfig | 19 + drivers/media/platform/Makefile | 2 + drivers/media/platform/amphion/Makefile | 20 + drivers/media/platform/amphion/vdec.c | 1691 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/media/platform/amphion/venc.c | 1358 +++++++++++++++++++++++++++++++++++++++++++++ drivers/media/platform/amphion/vpu.h | 362 ++++++++++++ drivers/media/platform/amphion/vpu_cmds.c | 433 +++++++++++++++ drivers/media/platform/amphion/vpu_cmds.h | 25 + drivers/media/platform/amphion/vpu_codec.h | 68 +++ drivers/media/platform/amphion/vpu_color.c | 183 ++++++ drivers/media/platform/amphion/vpu_core.c | 871 +++++++++++++++++++++++++++++ drivers/media/platform/amphion/vpu_core.h | 15 + drivers/media/platform/amphion/vpu_dbg.c | 494 +++++++++++++++++ drivers/media/platform/amphion/vpu_defs.h | 187 +++++++ drivers/media/platform/amphion/vpu_drv.c | 260 +++++++++ drivers/media/platform/amphion/vpu_helpers.c | 413 ++++++++++++++ drivers/media/platform/amphion/vpu_helpers.h | 74 +++ drivers/media/platform/amphion/vpu_imx8q.c | 271 +++++++++ drivers/media/platform/amphion/vpu_imx8q.h | 115 ++++ drivers/media/platform/amphion/vpu_malone.c | 1625 +++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/media/platform/amphion/vpu_malone.h | 44 ++ drivers/media/platform/amphion/vpu_mbox.c | 118 ++++ drivers/media/platform/amphion/vpu_mbox.h | 16 + drivers/media/platform/amphion/vpu_msgs.c | 385 +++++++++++++ drivers/media/platform/amphion/vpu_msgs.h | 14 + drivers/media/platform/amphion/vpu_rpc.c | 257 +++++++++ drivers/media/platform/amphion/vpu_rpc.h | 456 +++++++++++++++ drivers/media/platform/amphion/vpu_v4l2.c | 712 ++++++++++++++++++++++++ drivers/media/platform/amphion/vpu_v4l2.h | 55 ++ drivers/media/platform/amphion/vpu_windsor.c | 1169 +++++++++++++++++++++++++++++++++++++++ drivers/media/platform/amphion/vpu_windsor.h | 37 ++ drivers/media/v4l2-core/v4l2-ioctl.c | 2 + include/uapi/linux/videodev2.h | 2 + 36 files changed, 11967 insertions(+), 3 deletions(-) create mode 100644 Documentation/devicetree/bindings/media/amphion,vpu.yaml create mode 100644 drivers/media/platform/amphion/Makefile create mode 100644 drivers/media/platform/amphion/vdec.c create mode 100644 drivers/media/platform/amphion/venc.c create mode 100644 drivers/media/platform/amphion/vpu.h create mode 100644 drivers/media/platform/amphion/vpu_cmds.c create mode 100644 drivers/media/platform/amphion/vpu_cmds.h create mode 100644 drivers/media/platform/amphion/vpu_codec.h create mode 100644 drivers/media/platform/amphion/vpu_color.c create mode 100644 drivers/media/platform/amphion/vpu_core.c create mode 100644 drivers/media/platform/amphion/vpu_core.h create mode 100644 drivers/media/platform/amphion/vpu_dbg.c create mode 100644 drivers/media/platform/amphion/vpu_defs.h create mode 100644 drivers/media/platform/amphion/vpu_drv.c create mode 100644 drivers/media/platform/amphion/vpu_helpers.c create mode 100644 drivers/media/platform/amphion/vpu_helpers.h create mode 100644 drivers/media/platform/amphion/vpu_imx8q.c create mode 100644 drivers/media/platform/amphion/vpu_imx8q.h create mode 100644 drivers/media/platform/amphion/vpu_malone.c create mode 100644 drivers/media/platform/amphion/vpu_malone.h create mode 100644 drivers/media/platform/amphion/vpu_mbox.c create mode 100644 drivers/media/platform/amphion/vpu_mbox.h create mode 100644 drivers/media/platform/amphion/vpu_msgs.c create mode 100644 drivers/media/platform/amphion/vpu_msgs.h create mode 100644 drivers/media/platform/amphion/vpu_rpc.c create mode 100644 drivers/media/platform/amphion/vpu_rpc.h create mode 100644 drivers/media/platform/amphion/vpu_v4l2.c create mode 100644 drivers/media/platform/amphion/vpu_v4l2.h create mode 100644 drivers/media/platform/amphion/vpu_windsor.c create mode 100644 drivers/media/platform/amphion/vpu_windsor.h