--- v6
+++ v5
@@ -9,37 +9,22 @@
Patches 1-5 fix a few compliance issues with the decoder and encoder, most
notably by adding support for the START and STOP command for the latter. These
-patches were last up until v4 but have been moved to the beginning so they can
-be applied sooner.
+patches were last in the previous series but have been moved to the beginning so
+they can be applied sooner.
Patches 6-9 separates the "stateful" part of the driver into its own file and
add support for the new firmware and pixel format used by MT8183.
Patches 10-14 add support for H.264 stateless decoding and MT8183.
-Note that a few checkpatch issues have been left unadressed on purpose:
-* Conversion from e.g. uint32_t to u32 can't be done without breaking
- consistency. This should be done by a driver-wide patch.
-* Some macro warning suggesting parentheses for parameters expanded as struct
- members, which is obviously not applicable here.
-* Warnings about adding new files without an update the MAINTAINERS, which is
- irrelevant as the new files are already covered by the existing wildcards.
-
-Changes since v5:
-* Rebased against latest media tree.
-* Applied most suggestions of `checkpatch.pl --strict`. Some proposed fixes were
- not applied because they would require a larger refactoring (i.e. large-scale
- type changes) of the code.
-* Applied Reviewed-by and fix suggestions from Tzung-bi.
-* Check for ABORT state in vidioc_encoder_cmd.
-
Changes since v4:
-* Reorganized fixup patches first.
+* Moved compliance fix patches to the head of the series.
* Select MEDIA_CONTROLLER_REQUEST_API.
* Properly capitalize MM21's format description string.
* Reorganize stateless code as suggested by Hans.
* Fix compilation errors when DEBUG is defined.
-* Merge double-free fixup patch into the patch that introduced the issue.
+* Merge double-free fixup patch into the patch that introduced the issue (was
+ a separate patch coming right after the one introducing the issue).
Changes since v3:
* Stop checking that controls are set for every request.
@@ -77,15 +62,15 @@
.../media/v4l/pixfmt-reserved.rst | 7 +
drivers/media/platform/Kconfig | 3 +
drivers/media/platform/mtk-vcodec/Makefile | 3 +
- .../platform/mtk-vcodec/mtk_vcodec_dec.c | 820 +++---------------
- .../platform/mtk-vcodec/mtk_vcodec_dec.h | 25 +-
- .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 65 +-
- .../mtk-vcodec/mtk_vcodec_dec_stateful.c | 626 +++++++++++++
- .../mtk-vcodec/mtk_vcodec_dec_stateless.c | 360 ++++++++
+ .../platform/mtk-vcodec/mtk_vcodec_dec.c | 817 +++---------------
+ .../platform/mtk-vcodec/mtk_vcodec_dec.h | 27 +-
+ .../platform/mtk-vcodec/mtk_vcodec_dec_drv.c | 66 +-
+ .../mtk-vcodec/mtk_vcodec_dec_stateful.c | 668 ++++++++++++++
+ .../mtk-vcodec/mtk_vcodec_dec_stateless.c | 366 ++++++++
.../platform/mtk-vcodec/mtk_vcodec_drv.h | 58 +-
- .../platform/mtk-vcodec/mtk_vcodec_enc.c | 140 ++-
- .../platform/mtk-vcodec/mtk_vcodec_enc_drv.c | 6 +-
- .../mtk-vcodec/vdec/vdec_h264_req_if.c | 771 ++++++++++++++++
+ .../platform/mtk-vcodec/mtk_vcodec_enc.c | 135 ++-
+ .../platform/mtk-vcodec/mtk_vcodec_enc_drv.c | 4 +
+ .../mtk-vcodec/vdec/vdec_h264_req_if.c | 780 +++++++++++++++++
.../media/platform/mtk-vcodec/vdec_drv_if.c | 3 +
.../media/platform/mtk-vcodec/vdec_drv_if.h | 1 +
.../media/platform/mtk-vcodec/vdec_ipi_msg.h | 23 +-
@@ -93,13 +78,13 @@
.../media/platform/mtk-vcodec/vdec_vpu_if.h | 5 +
drivers/media/v4l2-core/v4l2-ioctl.c | 1 +
include/uapi/linux/videodev2.h | 1 +
- 20 files changed, 2235 insertions(+), 727 deletions(-)
+ 20 files changed, 2290 insertions(+), 722 deletions(-)
create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_stateful.c
create mode 100644 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec_stateless.c
create mode 100644 drivers/media/platform/mtk-vcodec/vdec/vdec_h264_req_if.c
--
-2.32.0.93.g670b81a890-goog
+2.31.1.751.gd2f1c929bd-goog
_______________________________________________