| Message ID | 79eb0d83-1967-8caf-e205-b29beec6fa23@xs4all.nl (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [GIT,PULL,FOR,v5.14] MPEG-2 stateless API cleanup and destaging | expand |
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/79eb0d83-1967-8caf-e205-b29beec6fa23@xs4all.nl/
Build log: https://builder.linuxtv.org/job/patchwork/102371/
Build time: 00:29:01
Link: https://lore.kernel.org/linux-media/79eb0d83-1967-8caf-e205-b29beec6fa23@xs4all.nl
gpg: Signature made Fri 30 Apr 2021 06:03:41 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>" [full]
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 6/10 patches with issues, being 0 at build time, plus one error when buinding PDF document
Error/warnings:
patches/0001-media-uapi-mpeg2-Rename-quantization-to-quantisation.patch:
checkpatch.pl:
$ cat patches/0001-media-uapi-mpeg2-Rename-quantization-to-quantisation.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:153: WARNING: line length of 102 exceeds 100 columns
-:336: CHECK: Alignment should match open parenthesis
-:423: CHECK: spaces preferred around that '+' (ctx:VxV)
patches/0002-media-uapi-mpeg2-rework-quantisation-matrices-semant.patch:
checkpatch.pl:
$ cat patches/0002-media-uapi-mpeg2-rework-quantisation-matrices-semant.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:36: WARNING: Duplicate signature
patches/0003-media-uapi-mpeg2-Cleanup-flags.patch:
checkpatch.pl:
$ cat patches/0003-media-uapi-mpeg2-Cleanup-flags.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:542: WARNING: line length of 101 exceeds 100 columns
-:543: WARNING: line length of 105 exceeds 100 columns
-:544: WARNING: line length of 111 exceeds 100 columns
patches/0004-media-uapi-mpeg2-Split-sequence-and-picture-paramete.patch:
checkpatch.pl:
$ cat patches/0004-media-uapi-mpeg2-Split-sequence-and-picture-paramete.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:511: CHECK: Alignment should match open parenthesis
-:513: CHECK: Alignment should match open parenthesis
-:551: CHECK: spaces preferred around that '+' (ctx:VxV)
-:552: CHECK: spaces preferred around that '+' (ctx:VxV)
patches/0009-media-controls-Log-MPEG-2-stateless-control-in-.std_.patch:
checkpatch.pl:
$ cat patches/0009-media-controls-Log-MPEG-2-stateless-control-in-.std_.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:26: WARNING: Avoid logging continuation uses where feasible
-:29: WARNING: Avoid logging continuation uses where feasible
-:32: WARNING: Avoid logging continuation uses where feasible
patches/0010-media-uapi-move-MPEG-2-stateless-controls-out-of-sta.patch:
checkpatch.pl:
$ cat patches/0010-media-uapi-move-MPEG-2-stateless-controls-out-of-sta.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
-:551: WARNING: line length of 102 exceeds 100 columns
-:694: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
-:849: CHECK: spaces preferred around that '+' (ctx:VxV)
-:897: CHECK: spaces preferred around that '+' (ctx:VxV)
-:928: CHECK: spaces preferred around that '+' (ctx:VxV)
Error #512 when building PDF docs
Hi Mauro, This series cleans up and destages the MPEG-2 stateless decoder API. After this the only remaining staging codec API is that for HEVC. That will require more work and also requires support for dynamic array controls, for which I posted a patch series recently. Many thanks to Ezequiel et al for working on this! Regards, Hans The following changes since commit 0b276e470a4d43e1365d3eb53c608a3d208cabd4: media: coda: fix macroblocks count control usage (2021-04-15 13:23:26 +0200) are available in the Git repository at: git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.14a for you to fetch changes up to 210c886393c28ae158a32ec0208409d49a10414e: media: uapi: move MPEG-2 stateless controls out of staging (2021-04-30 07:58:16 +0200) ---------------------------------------------------------------- Tag branch ---------------------------------------------------------------- Ezequiel Garcia (10): media: uapi: mpeg2: Rename "quantization" to "quantisation" media: uapi: mpeg2: rework quantisation matrices semantics media: uapi: mpeg2: Cleanup flags media: uapi: mpeg2: Split sequence and picture parameters media: uapi: mpeg2: Move reference buffer fields media: hantro/cedrus: Remove unneeded slice size and slice offset media: uapi: mpeg2: Remove V4L2_CID_MPEG_VIDEO_MPEG2_SLICE_PARAMS media: uapi: Move the MPEG-2 stateless control type out of staging media: controls: Log MPEG-2 stateless control in .std_log media: uapi: move MPEG-2 stateless controls out of staging Documentation/userspace-api/media/v4l/ext-ctrls-codec-stateless.rst | 214 +++++++++++++++++++++++++++++++ Documentation/userspace-api/media/v4l/ext-ctrls-codec.rst | 217 -------------------------------- Documentation/userspace-api/media/v4l/pixfmt-compressed.rst | 11 +- Documentation/userspace-api/media/v4l/vidioc-g-ext-ctrls.rst | 12 ++ Documentation/userspace-api/media/v4l/vidioc-queryctrl.rst | 18 ++- Documentation/userspace-api/media/videodev2.h.rst.exceptions | 5 +- drivers/media/v4l2-core/v4l2-ctrls.c | 122 ++++++++++++------ drivers/staging/media/hantro/hantro_drv.c | 9 +- drivers/staging/media/hantro/hantro_g1_mpeg2_dec.c | 110 ++++++++-------- drivers/staging/media/hantro/hantro_hw.h | 2 +- drivers/staging/media/hantro/hantro_mpeg2.c | 2 +- drivers/staging/media/hantro/rk3399_vpu_hw_mpeg2_dec.c | 106 +++++++--------- drivers/staging/media/sunxi/cedrus/cedrus.c | 10 +- drivers/staging/media/sunxi/cedrus/cedrus.h | 5 +- drivers/staging/media/sunxi/cedrus/cedrus_dec.c | 10 +- drivers/staging/media/sunxi/cedrus/cedrus_mpeg2.c | 97 +++++--------- include/media/mpeg2-ctrls.h | 82 ------------ include/media/v4l2-ctrls.h | 11 +- include/uapi/linux/v4l2-controls.h | 112 +++++++++++++++++ include/uapi/linux/videodev2.h | 7 ++ 20 files changed, 610 insertions(+), 552 deletions(-) delete mode 100644 include/media/mpeg2-ctrls.h