* [GIT PULL FOR v5.4] Hantro H.264 + finish stateful decoder spec
@ 2019-08-17 9:31 Hans Verkuil
2019-08-17 10:08 ` Jenkins
0 siblings, 1 reply; 2+ messages in thread
From: Hans Verkuil @ 2019-08-17 9:31 UTC (permalink / raw)
To: Linux Media Mailing List
Cc: Ezequiel Garcia, Boris Brezillon, Tomasz Figa, Rasmus Villemoes
Hi Mauro,
This PR takes Ezequiel's series adding H.264 decoding for hantro
(https://patchwork.linuxtv.org/project/linux-media/list/?series=603).
The first patch (lib/sort.c) is Acked by Andrew Morton and is intended to
go in through the media subsystem since this is the first driver that uses
it.
This series substantially improves the H.264 API. Only H.264 slicing support
still requires some more work.
I double-checked that the H.264 control structures have the same layout between
32 and 64 bit architectures and do not contain any holes.
The second part of this PR is core API improvements to help give more
information about the HW capabilities by adding two new format flags.
This second part consists of patches 1-3 and 5-8 of this series:
https://patchwork.linuxtv.org/project/linux-media/list/?series=588
I dropped patch 4 until I have an Ack from Samsung, and patches 9-12
are not ready yet for merging.
Most importantly, the stateful decoder specification is now merged.
The final patch improves pixfmt-compressed.srt. It's still not perfect
and I plan to make more changes there with references to the various
codec standards, so there will be a follow-up patch, but for now this
is still better than what we had.
Note that the v4l2-compliance test fails with vicodec after this PR
is merged. I have a patch for v4l-utils ready to fix this.
Regards,
Hans
The following changes since commit 31d5d15dfc3418a57cfab419a353d8dc5f5698b5:
media: MAINTAINERS: Add entry for the ov5670 driver (2019-08-15 08:17:04 -0300)
are available in the Git repository at:
git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.4l
for you to fetch changes up to bf7ca7e0046e7f4e246876e9cfab5a65ca1ec72a:
pixfmt-compressed.rst: improve H264/HEVC/MPEG1+2/VP8+9 documentation (2019-08-17 10:24:37 +0200)
----------------------------------------------------------------
Tag branch
----------------------------------------------------------------
Boris Brezillon (3):
media: uapi: h264: Add the concept of decoding mode
media: uapi: h264: Get rid of the p0/b0/b1 ref-lists
media: hantro: Move copy_metadata() before doing a decode operation
Ezequiel Garcia (4):
media: uapi: h264: Rename pixel format
media: uapi: h264: Add the concept of start code
media: cedrus: Cleanup control initialization
media: cedrus: Specify H264 startcode and decoding mode
Hans Verkuil (2):
videodev2.h: add V4L2_FMT_FLAG_CONTINUOUS_BYTESTREAM
pixfmt-compressed.rst: improve H264/HEVC/MPEG1+2/VP8+9 documentation
Hertz Wong (3):
media: hantro: Add core bits to support H264 decoding
media: hantro: Add support for H264 decoding on G1
media: hantro: Enable H264 decoding on rk3288
Maxime Jourdan (4):
videodev2.h: add V4L2_FMT_FLAG_DYN_RESOLUTION
media: venus: vdec: flag OUTPUT formats with V4L2_FMT_FLAG_DYN_RESOLUTION
media: mtk-vcodec: flag OUTPUT formats with V4L2_FMT_FLAG_DYN_RESOLUTION
media: vicodec: set flags for vdec/stateful OUTPUT coded formats
Rasmus Villemoes (1):
lib/sort.c: implement sort() variant taking context argument
Tomasz Figa (1):
media: docs-rst: Document memory-to-memory video decoder interface
Documentation/media/uapi/v4l/dev-decoder.rst | 1101 +++++++++++++++++++++++++++++++++++++++++++++++
Documentation/media/uapi/v4l/dev-mem2mem.rst | 8 +-
Documentation/media/uapi/v4l/ext-ctrls-codec.rst | 99 ++++-
Documentation/media/uapi/v4l/pixfmt-compressed.rst | 47 +-
Documentation/media/uapi/v4l/pixfmt-v4l2.rst | 5 +
Documentation/media/uapi/v4l/v4l2.rst | 10 +-
Documentation/media/uapi/v4l/vidioc-decoder-cmd.rst | 41 +-
Documentation/media/uapi/v4l/vidioc-dqevent.rst | 11 +-
Documentation/media/uapi/v4l/vidioc-enum-fmt.rst | 16 +
Documentation/media/videodev2.h.rst.exceptions | 2 +
drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c | 4 +
drivers/media/platform/mtk-vcodec/mtk_vcodec_drv.h | 1 +
drivers/media/platform/qcom/venus/core.h | 1 +
drivers/media/platform/qcom/venus/vdec.c | 11 +
drivers/media/platform/vicodec/vicodec-core.c | 3 +
drivers/media/v4l2-core/v4l2-ctrls.c | 18 +
drivers/media/v4l2-core/v4l2-ioctl.c | 2 +-
drivers/staging/media/hantro/Makefile | 2 +
drivers/staging/media/hantro/hantro.h | 9 +-
drivers/staging/media/hantro/hantro_drv.c | 50 ++-
drivers/staging/media/hantro/hantro_g1_h264_dec.c | 292 +++++++++++++
drivers/staging/media/hantro/hantro_h264.c | 646 +++++++++++++++++++++++++++
drivers/staging/media/hantro/hantro_hw.h | 56 +++
drivers/staging/media/hantro/hantro_v4l2.c | 10 +
drivers/staging/media/hantro/rk3288_vpu_hw.c | 21 +-
drivers/staging/media/sunxi/cedrus/cedrus.c | 63 ++-
drivers/staging/media/sunxi/cedrus/cedrus.h | 3 +-
drivers/staging/media/sunxi/cedrus/cedrus_dec.c | 2 +-
drivers/staging/media/sunxi/cedrus/cedrus_video.c | 6 +-
include/linux/sort.h | 5 +
include/media/h264-ctrls.h | 21 +-
include/uapi/linux/videodev2.h | 6 +-
lib/sort.c | 34 +-
33 files changed, 2523 insertions(+), 83 deletions(-)
create mode 100644 Documentation/media/uapi/v4l/dev-decoder.rst
create mode 100644 drivers/staging/media/hantro/hantro_g1_h264_dec.c
create mode 100644 drivers/staging/media/hantro/hantro_h264.c
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [GIT PULL FOR v5.4] Hantro H.264 + finish stateful decoder spec
2019-08-17 9:31 [GIT PULL FOR v5.4] Hantro H.264 + finish stateful decoder spec Hans Verkuil
@ 2019-08-17 10:08 ` Jenkins
0 siblings, 0 replies; 2+ messages in thread
From: Jenkins @ 2019-08-17 10:08 UTC (permalink / raw)
To: mchehab+samsung, linux-media; +Cc: builder
From: builder@linuxtv.org
Pull request: https://patchwork.linuxtv.org/patch/58252/
Build log: https://builder.linuxtv.org/job/patchwork/10603/
Build time: 00:35:01
Link: https://lore.kernel.org/linux-media/1be8ac17-349b-ef4d-299d-4f38889492ec@xs4all.nl
Summary: 8 patches and/or PDF generation with issues, being 0 at build time
gpg: Signature made Sat 17 Aug 2019 08:26:34 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]
Error/warnings:
patches/0001-lib-sort.c-implement-sort-variant-taking-context-arg.patch:30: WARNING: Non-standard signature: Requested-by:
patches/0001-lib-sort.c-implement-sort-variant-taking-context-arg.patch:114: WARNING: line over 80 characters
patches/0001-lib-sort.c-implement-sort-variant-taking-context-arg.patch:120: WARNING: line over 80 characters
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0001-lib-sort.c-implement-sort-variant-taking-context-arg.patch
patches/0002-media-uapi-h264-Rename-pixel-format.patch:50: WARNING: line over 80 characters
patches/0002-media-uapi-h264-Rename-pixel-format.patch:50: ERROR: trailing statements should be on next line
patches/0002-media-uapi-h264-Rename-pixel-format.patch:107: WARNING: line over 80 characters
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0002-media-uapi-h264-Rename-pixel-format.patch
patches/0003-media-uapi-h264-Add-the-concept-of-decoding-mode.patch:154: WARNING: line over 80 characters
patches/0003-media-uapi-h264-Add-the-concept-of-decoding-mode.patch:174: CHECK: spaces preferred around that '+' (ctx:VxV)
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0003-media-uapi-h264-Add-the-concept-of-decoding-mode.patch
patches/0004-media-uapi-h264-Add-the-concept-of-start-code.patch:118: WARNING: line over 80 characters
patches/0004-media-uapi-h264-Add-the-concept-of-start-code.patch:138: CHECK: spaces preferred around that '+' (ctx:VxV)
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0004-media-uapi-h264-Add-the-concept-of-start-code.patch
patches/0006-media-cedrus-Cleanup-control-initialization.patch:109: WARNING: line over 80 characters
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0006-media-cedrus-Cleanup-control-initialization.patch
patches/0009-media-hantro-Add-core-bits-to-support-H264-decoding.patch:138: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0009-media-hantro-Add-core-bits-to-support-H264-decoding.patch:536: CHECK: Unnecessary parentheses around 'poca < builder->curpoc'
patches/0009-media-hantro-Add-core-bits-to-support-H264-decoding.patch:536: CHECK: Unnecessary parentheses around 'pocb < builder->curpoc'
patches/0009-media-hantro-Add-core-bits-to-support-H264-decoding.patch:577: CHECK: Unnecessary parentheses around 'poca < builder->curpoc'
patches/0009-media-hantro-Add-core-bits-to-support-H264-decoding.patch:577: CHECK: Unnecessary parentheses around 'pocb < builder->curpoc'
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0009-media-hantro-Add-core-bits-to-support-H264-decoding.patch
patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch:32: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch:89: WARNING: line over 80 characters
patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch:95: WARNING: line over 80 characters
patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch:107: WARNING: line over 80 characters
patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch:111: WARNING: line over 80 characters
patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch:125: WARNING: line over 80 characters
patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch:141: WARNING: line over 80 characters
patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch:142: WARNING: line over 80 characters
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0010-media-hantro-Add-support-for-H264-decoding-on-G1.patch
patches/0017-media-docs-rst-Document-memory-to-memory-video-decod.patch:39: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
Error #256 when running ./scripts/checkpatch.pl --terse --mailback --no-summary --strict patches/0017-media-docs-rst-Document-memory-to-memory-video-decod.patch
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-08-17 10:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-17 9:31 [GIT PULL FOR v5.4] Hantro H.264 + finish stateful decoder spec Hans Verkuil
2019-08-17 10:08 ` Jenkins
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.