From: Michael Tretter <m.tretter@pengutronix.de>
To: linux-media@vger.kernel.org, devicetree@vger.kernel.org,
Philipp Zabel <p.zabel@pengutronix.de>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Michael Tretter <m.tretter@pengutronix.de>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Fabio Estevam <festevam@gmail.com>,
Alexander Stein <alexander.stein@ew.tq-group.com>,
kernel@pengutronix.de, linux-imx@nxp.com,
linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 00/16] media: imx-pxp: add support for i.MX7D
Date: Fri, 13 Jan 2023 10:54:07 +0100 [thread overview]
Message-ID: <20230112-imx-pxp-v2-0-e2281da1db55@pengutronix.de> (raw)
This is v2 of the series to add support for the PXP found on the i.MX7D to the
imx-pxp driver.
The PXP on the i.MX7D has a few differences compared to the one on the
i.MX6ULL. Especially, it has more processing blocks and slightly different
multiplexers to route the data between the blocks. Therefore, the driver must
configure a different data path depending on the platform.
While the PXP has a version register, the reported version is the same on the
i.MX6ULL and the i.MX7D. Therefore, we cannot use the version register to
change the driver behavior, but have to use the device tree compatible. The
driver still prints the found version to the log to help bringing up the PXP
on further platforms.
The patches are inspired by some earlier patches [0] by Laurent to add PXP
support to the i.MX7d. Compared to the earlier patches, these patches add
different behavior depending on the platform. Furthermore, the patches disable
only the LUT block, but keep the rotator block enabled, as it may now be
configured via the V4L2 rotate control.
In v2, I included Laurent's patch series [1], which was based on this series
anyway and added regmap support.
Patch 1 converts the dt-binding to yaml.
Patches 2 to 5 cleanup and refactor the driver in preparation of handling
different PXP versions.
Patches 6 and 7 add the handling of different platforms and the i.MX7d
specific configuration.
Patch 8 adds the device tree node for the PXP to the i.MX7d device tree.
Patches 9 to 15 are the cleanup and enhancement patches to add media
controller support, implement enum_framesizes, and add pxp_read/pxp_write
helpers.
Patch 16 adds regmap support to the driver.
Michael
[0] https://lore.kernel.org/linux-media/20200510223100.11641-1-laurent.pinchart@ideasonboard.com/
[1] https://lore.kernel.org/linux-media/20230112172507.30579-1-laurent.pinchart@ideasonboard.com
---
Changelog
v2:
- fix device tree binding
- reduce log level of PXP version to debug
- drop fallback for missing pdata
- add cleanup and enhancement patches to series
- convert driver to regmap
Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
To: linux-media@vger.kernel.org
To: devicetree@vger.kernel.org
To: Philipp Zabel <p.zabel@pengutronix.de>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: Alexander Stein <alexander.stein@ew.tq-group.com>
Cc: kernel@pengutronix.de
Cc: linux-imx@nxp.com
Cc: linux-arm-kernel@lists.infradead.org
Cc: Michael Tretter <m.tretter@pengutronix.de>
---
Laurent Pinchart (7):
media: imx-pxp: Sort headers alphabetically
media: imx-pxp: Don't set bus_info manually in .querycap()
media: imx-pxp: Add media controller support
media: imx-pxp: Pass pixel format value to find_format()
media: imx-pxp: Implement frame size enumeration
media: imx-pxp: Introduce pxp_read() and pxp_write() wrappers
media: imx-pxp: Use non-threaded IRQ
Michael Tretter (9):
dt-bindings: media: fsl-pxp: convert to yaml
media: imx-pxp: detect PXP version
media: imx-pxp: extract helper function to setup data path
media: imx-pxp: explicitly disable unused blocks
media: imx-pxp: disable LUT block
media: imx-pxp: make data_path_ctrl0 platform dependent
media: imx-pxp: add support for i.MX7D
ARM: dts: imx7d: add node for PXP
media: imx-pxp: convert to regmap
.../devicetree/bindings/media/fsl,imx6ull-pxp.yaml | 82 +++++
.../devicetree/bindings/media/fsl-pxp.txt | 26 --
arch/arm/boot/dts/imx7d.dtsi | 9 +
drivers/media/platform/nxp/imx-pxp.c | 359 +++++++++++++++------
4 files changed, 355 insertions(+), 121 deletions(-)
---
base-commit: b7bfaa761d760e72a969d116517eaa12e404c262
change-id: 20230112-imx-pxp-073008b3c857
Best regards,
--
Michael Tretter <m.tretter@pengutronix.de>
next reply other threads:[~2023-01-13 9:57 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-13 9:54 Michael Tretter [this message]
2023-01-13 9:54 ` [PATCH v2 01/16] dt-bindings: media: fsl-pxp: convert to yaml Michael Tretter
2023-01-13 10:41 ` Laurent Pinchart
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 11:56 ` Krzysztof Kozlowski
2023-01-13 15:09 ` Michael Tretter
2023-01-13 16:51 ` Krzysztof Kozlowski
2023-01-13 15:26 ` Rob Herring
2023-01-16 11:38 ` [PATCH v2.1 02/17] " Michael Tretter
2023-01-16 11:59 ` Krzysztof Kozlowski
2023-01-16 16:11 ` Laurent Pinchart
2023-01-13 9:54 ` [PATCH v2 02/16] media: imx-pxp: detect PXP version Michael Tretter
2023-01-13 10:44 ` Laurent Pinchart
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 03/16] media: imx-pxp: extract helper function to setup data path Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 04/16] media: imx-pxp: explicitly disable unused blocks Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 05/16] media: imx-pxp: disable LUT block Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 06/16] media: imx-pxp: make data_path_ctrl0 platform dependent Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 12:03 ` Laurent Pinchart
2023-01-13 9:54 ` [PATCH v2 07/16] media: imx-pxp: add support for i.MX7D Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 08/16] ARM: dts: imx7d: add node for PXP Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-16 16:26 ` Laurent Pinchart
2023-01-20 9:06 ` Michael Tretter
2023-01-20 9:20 ` Laurent Pinchart
2023-02-09 14:02 ` Michael Tretter
2023-03-06 1:19 ` Shawn Guo
2023-01-13 9:54 ` [PATCH v2 09/16] media: imx-pxp: Sort headers alphabetically Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 10/16] media: imx-pxp: Don't set bus_info manually in .querycap() Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 11/16] media: imx-pxp: Add media controller support Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 12/16] media: imx-pxp: Pass pixel format value to find_format() Michael Tretter
2023-01-13 11:46 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 13/16] media: imx-pxp: Implement frame size enumeration Michael Tretter
2023-01-13 11:49 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 14/16] media: imx-pxp: Introduce pxp_read() and pxp_write() wrappers Michael Tretter
2023-01-13 11:50 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 15/16] media: imx-pxp: Use non-threaded IRQ Michael Tretter
2023-01-13 11:52 ` Philipp Zabel
2023-01-13 9:54 ` [PATCH v2 16/16] media: imx-pxp: convert to regmap Michael Tretter
2023-01-13 11:56 ` Philipp Zabel
2023-01-14 21:29 ` [PATCH v2 00/16] media: imx-pxp: add support for i.MX7D Laurent Pinchart
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=20230112-imx-pxp-v2-0-e2281da1db55@pengutronix.de \
--to=m.tretter@pengutronix.de \
--cc=alexander.stein@ew.tq-group.com \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=kernel@pengutronix.de \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=p.zabel@pengutronix.de \
--cc=robh+dt@kernel.org \
/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).