From: Ezequiel Garcia <ezequiel@collabora.com>
To: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: kernel@collabora.com, Jonas Karlman <jonas@kwiboo.se>,
Hans Verkuil <hverkuil@xs4all.nl>,
Nicolas Dufresne <nicolas.dufresne@collabora.com>,
Philipp Zabel <p.zabel@pengutronix.de>,
Maxime Ripard <mripard@kernel.org>,
Paul Kocialkowski <paul.kocialkowski@bootlin.com>,
Jernej Skrabec <jernej.skrabec@siol.net>,
Ezequiel Garcia <ezequiel@collabora.com>
Subject: [PATCH v2 0/3] MPEG-2 stateless API cleanup
Date: Thu, 5 Nov 2020 17:51:15 -0300 [thread overview]
Message-ID: <20201105205118.178458-1-ezequiel@collabora.com> (raw)
Hello everyone,
Following the H.264 API cleanup, and with all the lessons
learned from it, here's a revisit on the MPEG-2 API.
The biggest changes are:
* Drop slice size and offset parameters, given the API doesn't
seem to be designed to work per-slice.
* Convert all the 1-bit syntax elements from structure fields
to bits in a bit field flags field, following the other codec APIs.
* Split the sequence, picture and slice headers to separate controls.
This is build tested only, I haven't yet modified the existing
Ffmpeg-based implementation, as I'd like to have some consensus first.
Reviews and feedback are appreciated!
v2:
* Fixed bad use of boolean negation in a flag, which
was fortunately reported by 0day bot.
Thanks,
Ezequiel
Ezequiel Garcia (3):
media: uapi: mpeg2: Cleanup flags
media: uapi: mpeg2: Remove unused slice size and offset
media: uapi: mpeg2: Split sequence and picture parameters
.../media/v4l/ext-ctrls-codec.rst | 159 +++++++++++-------
.../media/v4l/pixfmt-compressed.rst | 5 +-
drivers/media/v4l2-core/v4l2-ctrls.c | 59 ++++---
drivers/staging/media/hantro/hantro_drv.c | 10 ++
.../media/hantro/hantro_g1_mpeg2_dec.c | 82 +++++----
.../media/hantro/rk3399_vpu_hw_mpeg2_dec.c | 82 +++++----
drivers/staging/media/sunxi/cedrus/cedrus.c | 14 ++
drivers/staging/media/sunxi/cedrus/cedrus.h | 2 +
.../staging/media/sunxi/cedrus/cedrus_mpeg2.c | 49 +++---
include/media/mpeg2-ctrls.h | 153 +++++++++++++----
include/media/v4l2-ctrls.h | 4 +
11 files changed, 395 insertions(+), 224 deletions(-)
--
2.27.0
next reply other threads:[~2020-11-05 20:51 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-05 20:51 Ezequiel Garcia [this message]
2020-11-05 20:51 ` [PATCH v2 1/3] media: uapi: mpeg2: Cleanup flags Ezequiel Garcia
2020-11-05 20:51 ` [PATCH v2 2/3] media: uapi: mpeg2: Remove unused slice size and offset Ezequiel Garcia
2020-11-05 20:51 ` [PATCH v2 3/3] media: uapi: mpeg2: Split sequence and picture parameters Ezequiel Garcia
2020-11-10 18:56 ` Dan Carpenter
2020-11-10 18:56 ` Dan Carpenter
2020-11-10 18:56 ` Dan Carpenter
2020-11-29 22:59 ` Jonas Karlman
2020-11-30 19:12 ` Ezequiel Garcia
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201105205118.178458-1-ezequiel@collabora.com \
--to=ezequiel@collabora.com \
--cc=hverkuil@xs4all.nl \
--cc=jernej.skrabec@siol.net \
--cc=jonas@kwiboo.se \
--cc=kernel@collabora.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mripard@kernel.org \
--cc=nicolas.dufresne@collabora.com \
--cc=p.zabel@pengutronix.de \
--cc=paul.kocialkowski@bootlin.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.