mbox series

[GIT,PULL,FOR,v5.16] Venus updates - part2

Message ID 20211021093951.3963813-1-stanimir.varbanov@linaro.org (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,FOR,v5.16] Venus updates - part2 | expand

Pull-request

git://linuxtv.org/svarbanov/media_tree.git tags/venus-for-v5.16-part2

Message

Stanimir Varbanov Oct. 21, 2021, 9:39 a.m. UTC
Hi Mauro,

This second part of Venus updates includes:
  * Add sdm660 DT schema, DT compatible and resources.
  * DPB internal buffers handing as part of DRC.
  * Added pmruntime autosuspend for encoder.
  * An attempt to handle fatal (firmware) errors as per the spec. 

Please pull.

regards,
Stan

The following changes since commit fd2eda71a47b095e81b9170c3f8b7ae82b04e785:

  media: remove myself from dvb media maintainers (2021-10-08 13:56:25 +0200)

are available in the Git repository at:

  git://linuxtv.org/svarbanov/media_tree.git tags/venus-for-v5.16-part2

for you to fetch changes up to 1fc3539ba413a4302efae670fcfae25d75db31e9:

  media: venus: core: Add sdm660 DT compatible and resource struct (2021-10-21 12:20:46 +0300)

----------------------------------------------------------------
Venus updates for v5.16 part2

----------------------------------------------------------------
AngeloGioacchino Del Regno (2):
      dt-bindings: media: venus: Add sdm660 dt schema
      media: venus: core: Add sdm660 DT compatible and resource struct

Mansur Alisha Shaik (1):
      venus: vdec: decoded picture buffer handling during reconfig sequence

Stanimir Varbanov (5):
      venus: venc: Use pmruntime autosuspend
      venus: Make sys_error flag an atomic bitops
      venus: hfi: Check for sys error on session hfi functions
      venus: helpers: Add helper to mark fatal vb2 error
      venus: Handle fatal errors during encoding and decoding

 .../bindings/media/qcom,sdm660-venus.yaml          | 186 +++++++++++++++++++++
 drivers/media/platform/qcom/venus/core.c           |  77 ++++++++-
 drivers/media/platform/qcom/venus/core.h           |   7 +-
 drivers/media/platform/qcom/venus/helpers.c        |  67 +++++++-
 drivers/media/platform/qcom/venus/helpers.h        |   4 +
 drivers/media/platform/qcom/venus/hfi.c            |  48 +++++-
 drivers/media/platform/qcom/venus/vdec.c           |  25 ++-
 drivers/media/platform/qcom/venus/venc.c           | 116 ++++++++++++-
 8 files changed, 511 insertions(+), 19 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/media/qcom,sdm660-venus.yaml

Comments

Jenkins Oct. 21, 2021, 10:16 a.m. UTC | #1
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20211021093951.3963813-1-stanimir.varbanov@linaro.org/
Build log: https://builder.linuxtv.org/job/patchwork/149875/
Build time: 00:21:28
Link: https://lore.kernel.org/linux-media/20211021093951.3963813-1-stanimir.varbanov@linaro.org

gpg: Signature made Thu 21 Oct 2021 09:26:47 AM UTC
gpg:                using RSA key E1558C2497CE3CCC2B5AA30F25B55FC81B7035F2
gpg: Good signature from "Stanimir Varbanov <stanimir.varbanov@linaro.org>" [unknown]
gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: 34CF E039 8A16 AD93 18FD  D5E8 A6D0 26D8 E358 14D4
     Subkey fingerprint: E155 8C24 97CE 3CCC 2B5A  A30F 25B5 5FC8 1B70 35F2

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

Error/warnings:

patches/0001-venus-venc-Use-pmruntime-autosuspend.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/cec/core/cec-adap.c: ../drivers/media/cec/core/cec-adap.c:926 cec_transmit_msg_fh() warn: '&data->list' not removed from list
	../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/intel/ipu3/cio2-bridge.c:242 cio2_bridge_connect_sensor() warn: missing error code 'ret'
	../drivers/media/usb/pvrusb2/pvrusb2-encoder.c: ../drivers/media/usb/pvrusb2/pvrusb2-encoder.c:288 pvr2_encoder_cmd() warn: inconsistent indenting
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:1730 pvr2_hdw_set_streaming() warn: inconsistent indenting
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3461 pvr2_hdw_cpufw_set_enabled() warn: inconsistent indenting
	../drivers/media/usb/pvrusb2/pvrusb2-hdw.c: ../drivers/media/usb/pvrusb2/pvrusb2-hdw.c:3501 pvr2_hdw_cpufw_get() warn: inconsistent indenting
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:627 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 623)
	../drivers/media/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2874 em28xx_v4l2_init() parse error: turning off implications after 60 seconds

patches/0002-venus-Make-sys_error-flag-an-atomic-bitops.patch:

    allyesconfig: return code #0:
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:627 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 623)

patches/0004-venus-helpers-Add-helper-to-mark-fatal-vb2-error.patch:

    allyesconfig: return code #0:
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:627 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 623)

patches/0005-venus-Handle-fatal-errors-during-encoding-and-decodi.patch:

    allyesconfig: return code #0:
	../drivers/media/platform/qcom/venus/helpers.c: ../drivers/media/platform/qcom/venus/helpers.c:627 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 623)

patches/0006-venus-vdec-decoded-picture-buffer-handling-during-re.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)

patches/0007-dt-bindings-media-venus-Add-sdm660-dt-schema.patch:

   checkpatch.pl:
	$ cat patches/0007-dt-bindings-media-venus-Add-sdm660-dt-schema.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:18: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?


Error #512 when building PDF docs