From: Devarsh Thakkar <devarsht@ti.com>
To: <mchehab@kernel.org>, <robh+dt@kernel.org>,
<krzysztof.kozlowski+dt@linaro.org>, <conor+dt@kernel.org>,
<hverkuil-cisco@xs4all.nl>, <linux-media@vger.kernel.org>,
<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Cc: <laurent.pinchart@ideasonboard.com>,
<eugen.hristev@collabora.com>, <ezequiel@vanguardiasur.com.ar>,
<u.kleine-koenig@pengutronix.de>, <sakari.ailus@linux.intel.com>,
<praneeth@ti.com>, <nm@ti.com>, <vigneshr@ti.com>,
<a-bhatia1@ti.com>, <j-luthra@ti.com>, <b-brnich@ti.com>,
<detheridge@ti.com>, <p-mantena@ti.com>, <vijayp@ti.com>,
<devarsht@ti.com>
Subject: [PATCH v3 0/4] Add V4L2 M2M Driver for E5010 JPEG Encoder
Date: Wed, 16 Aug 2023 20:52:06 +0530 [thread overview]
Message-ID: <20230816152210.4080779-1-devarsht@ti.com> (raw)
This adds support for V4L2 M2M based driver for E5010 JPEG Encoder
which is a stateful JPEG encoder from Imagination technologies
and is present in TI AM62A SoC.
v4l2-compliance test :
Link: https://gist.github.com/devarsht/17348fc509fe375a630a798196edbb73
E5010 JPEG Encoder Manual tests :
Performance:
Link: https://gist.github.com/devarsht/ef5aba211aae45166681b8b4418d4e65
Functionality:
Link: https://gist.github.com/devarsht/bfef1e3ead8e858b09cbf418aea753e8
Compression Quality:
Link: https://gist.github.com/devarsht/345df9bf7157b4ca094293760e257451
Multi Instance:
Link: https://gist.github.com/devarsht/4b4d734eeb9e0e616837c0836ddbc769
Devarsh Thakkar (4):
dt-bindings: media: Add bindings for Imagination E5010 JPEG Encoder
driver
media: imagination: Add E5010 JPEG Encoder driver
arm64: dts: ti: k3-am62a : Add E5010 JPEG Encoder
arm64: defconfig: Enable E5010 JPEG Encoder
.../bindings/media/img,e5010-jpeg-enc.yaml | 81 +
MAINTAINERS | 7 +
arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 11 +
arch/arm64/boot/dts/ti/k3-am62a.dtsi | 2 +
arch/arm64/configs/defconfig | 117 +-
drivers/media/platform/Kconfig | 1 +
drivers/media/platform/Makefile | 1 +
drivers/media/platform/imagination/Kconfig | 12 +
drivers/media/platform/imagination/Makefile | 3 +
.../platform/imagination/e5010-core-regs.h | 585 ++++++
.../platform/imagination/e5010-jpeg-enc-hw.c | 320 ++++
.../platform/imagination/e5010-jpeg-enc-hw.h | 42 +
.../platform/imagination/e5010-jpeg-enc.c | 1678 +++++++++++++++++
.../platform/imagination/e5010-jpeg-enc.h | 169 ++
.../platform/imagination/e5010-mmu-regs.h | 311 +++
15 files changed, 3265 insertions(+), 75 deletions(-)
create mode 100644 Documentation/devicetree/bindings/media/img,e5010-jpeg-enc.yaml
create mode 100644 drivers/media/platform/imagination/Kconfig
create mode 100644 drivers/media/platform/imagination/Makefile
create mode 100644 drivers/media/platform/imagination/e5010-core-regs.h
create mode 100644 drivers/media/platform/imagination/e5010-jpeg-enc-hw.c
create mode 100644 drivers/media/platform/imagination/e5010-jpeg-enc-hw.h
create mode 100644 drivers/media/platform/imagination/e5010-jpeg-enc.c
create mode 100644 drivers/media/platform/imagination/e5010-jpeg-enc.h
create mode 100644 drivers/media/platform/imagination/e5010-mmu-regs.h
---
Changelog:
V1->V2:
No change (sending dt-binding and driver together)
V2->V3:
- Add DONOTMERGE patches for dts and defconfig
- Update driver with below changes :
- Correct license headers
- Use more generic name core instead of jasper for base registers
- Add Comment for forward declarations
- Simplify quantization table calculations
- Use v4l2_apply_frmsize_constraints for updating framesize and remove unrequired functions
- Place TODO at top of file and in commit message too
- Use dev_err_probe helper in probe function
- Fix return value checking for failure scenarios in probe function
- Use v4l2_err/info/warn helpers instead of dev_err/info/warn helpers
- Fix unexpected indentation
- Correct commit message
- Update dt-bindings with below changes :
- Add vendor specific compatible
- Fix commit title and message
- Update reg names
- Update clocks to 1
- Fix dts example with proper naming
--
2.34.1
next reply other threads:[~2023-08-16 15:23 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-16 15:22 Devarsh Thakkar [this message]
2023-08-16 15:22 ` [PATCH v3 1/4] dt-bindings: media: Add bindings for Imagination E5010 JPEG Encoder driver Devarsh Thakkar
2023-08-19 14:00 ` Krzysztof Kozlowski
2023-08-20 16:46 ` Devarsh Thakkar
2023-08-20 20:25 ` Krzysztof Kozlowski
2023-08-16 15:22 ` [PATCH v3 2/4] media: imagination: Add " Devarsh Thakkar
2023-08-22 10:27 ` Andrzej Pietrasiewicz
2023-08-31 13:24 ` Devarsh Thakkar
2023-09-06 12:38 ` Hans Verkuil
2023-08-16 15:22 ` [DONOTMERGE PATCH v3 3/4] arm64: dts: ti: k3-am62a : Add E5010 JPEG Encoder Devarsh Thakkar
2023-08-16 15:22 ` [DONOTMERGE PATCH v3 4/4] arm64: defconfig: Enable " Devarsh Thakkar
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=20230816152210.4080779-1-devarsht@ti.com \
--to=devarsht@ti.com \
--cc=a-bhatia1@ti.com \
--cc=b-brnich@ti.com \
--cc=conor+dt@kernel.org \
--cc=detheridge@ti.com \
--cc=devicetree@vger.kernel.org \
--cc=eugen.hristev@collabora.com \
--cc=ezequiel@vanguardiasur.com.ar \
--cc=hverkuil-cisco@xs4all.nl \
--cc=j-luthra@ti.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=nm@ti.com \
--cc=p-mantena@ti.com \
--cc=praneeth@ti.com \
--cc=robh+dt@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=u.kleine-koenig@pengutronix.de \
--cc=vigneshr@ti.com \
--cc=vijayp@ti.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).