| Message ID | 9792587d-b4f3-27bc-add4-287126ed8eae@xs4all.nl (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,v5.16] v2: Allegro fixes/enhancements | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/9792587d-b4f3-27bc-add4-287126ed8eae@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/149014/
Build time: 00:32:12
Link: https://lore.kernel.org/linux-media/9792587d-b4f3-27bc-add4-287126ed8eae@xs4all.nl
gpg: Signature made Mon 18 Oct 2021 09:15:08 AM UTC
gpg: using RSA key AAA7FFBA4D2D77EF4CAEA1421326E0CD23ABDCE5
gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown]
gpg: aka "Hans Verkuil <hverkuil@xs4all.nl>" [unknown]
gpg: Note: This key has expired!
Primary key fingerprint: 052C DE7B C215 053B 689F 1BCA BD2D 6148 6614 3B4C
Subkey fingerprint: AAA7 FFBA 4D2D 77EF 4CAE A142 1326 E0CD 23AB DCE5
Summary: got 2/13 patches with issues, being 2 at build time, plus one error when buinding PDF document
Error/warnings:
patches/0001-media-allegro-ignore-interrupt-if-mailbox-is-not-ini.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
SPARSE:../drivers/media/rc/ir_toy.c ../drivers/media/rc/ir_toy.c:321:20: warning: incorrect type in assignment (different base types)
SPARSE:../drivers/media/rc/ir_toy.c ../drivers/media/rc/ir_toy.c:321:20: expected restricted __be16 [usertype]
SPARSE:../drivers/media/rc/ir_toy.c ../drivers/media/rc/ir_toy.c:321:20: got int
../drivers/media/pci/intel/ipu3/cio2-bridge.c:242 cio2_bridge_connect_sensor() 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/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:627 venus_helper_get_bufreq() error: we previously assumed 'req' could be null (see line 623)
../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/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2900 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
patches/0006-media-allegro-add-control-to-disable-encoder-buffer.patch:
allyesconfig: return code #0:
../drivers/media/pci/intel/ipu3/cio2-bridge.c:242 cio2_bridge_connect_sensor() warn: missing error code 'ret'
../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/pvrusb2/pvrusb2-encoder.c: ../drivers/media/usb/pvrusb2/pvrusb2-encoder.c:288 pvr2_encoder_cmd() warn: inconsistent indenting
../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/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/usb/em28xx/em28xx-video.c: ../drivers/media/usb/em28xx/em28xx-video.c:2894 em28xx_v4l2_init() parse error: turning off implications after 60 seconds
checkpatch.pl:
$ cat patches/0006-media-allegro-add-control-to-disable-encoder-buffer.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:115: CHECK: Alignment should match open parenthesis
Error #512 when building PDF docs
Updated version of the previous 'Allegro fixes/enhancements' PR. Changes: moved the #include <linux/videodev2.h> additions for nal-h264.h and nal-hevc.h from patches 11/13 and 12/13 to 10/13, where they are needed. The end result is the same as for v1 of this PR, but there are now no warnings when compiling after applying patch 10 and not the following patches. Regards, Hans 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/hverkuil/media_tree.git tags/br-v5.16i for you to fetch changes up to b9e9237724e078efe33aa63a5fb0f219097f8d35: media: allegro: write vui parameters for HEVC (2021-10-18 11:13:01 +0200) ---------------------------------------------------------------- Tag branch ---------------------------------------------------------------- Michael Tretter (13): media: allegro: ignore interrupt if mailbox is not initialized media: allegro: fix module removal if initialization failed media: allegro: lookup VCU settings media: allegro: add pm_runtime support media: allegro: add encoder buffer support media: allegro: add control to disable encoder buffer media: allegro: fix row and column in response message media: allegro: remove external QP table media: allegro: correctly scale the bit rate in SPS media: allegro: extract nal value lookup functions to header media: allegro: write correct colorspace into SPS media: allegro: nal-hevc: implement generator for vui media: allegro: write vui parameters for HEVC drivers/media/platform/allegro-dvt/allegro-core.c | 311 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++------ drivers/media/platform/allegro-dvt/allegro-mail.c | 23 ++--- drivers/media/platform/allegro-dvt/allegro-mail.h | 10 +- drivers/media/platform/allegro-dvt/nal-h264.c | 74 --------------- drivers/media/platform/allegro-dvt/nal-h264.h | 200 +++++++++++++++++++++++++++++++++++++++- drivers/media/platform/allegro-dvt/nal-hevc.c | 202 ++++++++++++++++++++++++++-------------- drivers/media/platform/allegro-dvt/nal-hevc.h | 189 ++++++++++++++++++++++++++++++++++++-- include/uapi/linux/v4l2-controls.h | 5 + 8 files changed, 811 insertions(+), 203 deletions(-)