From: Luca Ceresoli <luca.ceresoli@bootlin.com>
To: linux-tegra@vger.kernel.org
Cc: Luca Ceresoli <luca.ceresoli@bootlin.com>,
Thierry Reding <thierry.reding@gmail.com>,
Jonathan Hunter <jonathanh@nvidia.com>,
Sowjanya Komatineni <skomatineni@nvidia.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-staging@lists.linux.dev,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Hans Verkuil <hverkuil-cisco@xs4all.nl>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
Paul Kocialkowski <paul.kocialkowski@bootlin.com>,
Richard Leitner <richard.leitner@skidata.com>
Subject: [PATCH v6 00/20] Add Tegra20 parallel video input capture
Date: Tue, 18 Apr 2023 10:00:34 +0200 [thread overview]
Message-ID: <20230418080054.452955-1-luca.ceresoli@bootlin.com> (raw)
New in v6: a oneliner fix to patch 14 for an unlock imbalance in MIPI CSI
calibration (Tegra210 only). Many thanks to Hans for testing and spotting
this!
Full details follow.
Tegra20 and other Tegra SoCs have a video input (VI) peripheral that can
receive from either MIPI CSI-2 or parallel video (called respectively "CSI"
and "VIP" in the documentation). The kernel currently has a staging driver
for Tegra210 CSI capture. This patch set adds support for Tegra20 VIP
capture.
Unfortunately I had no real documentation available to base this work on.
I only had a working downstream 3.1 kernel, so I started with the driver
found there and heavily reworked it to fit into the mainline tegra-video
driver structure. The existing code appears written with the intent of
being modular and allow adding new input mechanisms and new SoCs while
keeping a unique VI core module. However its modularity and extensibility
was not enough to add Tegra20 VIP support, so I added some hooks to turn
hard-coded behaviour into per-SoC or per-bus customizable code. There are
also a fix, some generic cleanups and DT bindings.
Quick tour of the patches:
* Device tree bindings
01. dt-bindings: display: tegra: add Tegra20 VIP
02. dt-bindings: display: tegra: vi: add 'vip' property and example
* Minor improvements to logging, comments, cleanups
03. staging: media: tegra-video: improve documentation of tegra_video_format fields
04. staging: media: tegra-video: document tegra_channel_get_remote_source_subdev
05. staging: media: tegra-video: fix typos in comment
06. staging: media: tegra-video: improve error messages
07. staging: media: tegra-video: slightly simplify cleanup on errors
08. staging: media: tegra-video: move private struct declaration to C file
09. staging: media: tegra-video: move tegra210_csi_soc to C file
10. staging: media: tegra-video: remove unneeded include
* Preparation to make the VI module generic enough to host Tegra20 and VIP
11. staging: media: tegra-video: Kconfig: allow TPG only on Tegra210
12. staging: media: tegra-video: move tegra_channel_fmt_align to a per-soc op
13. staging: media: tegra-video: move default format to soc-specific data
14. staging: media: tegra-video: move MIPI calibration calls from VI to CSI
15. staging: media: tegra-video: add a per-soc enable/disable op
16. staging: media: tegra-video: move syncpt init/free to a per-soc op
17. staging: media: tegra-video: add syncpts for Tegra20 to struct tegra_vi
18. staging: media: tegra-video: add hooks for planar YUV and H/V flip
19. staging: media: tegra-video: add H/V flip controls
* Implementation of VIP and Tegra20
20. staging: media: tegra-video: add support for Tegra20 parallel input
Enjoy!
Changed in v5:
- patch 14: fixed unlock imbalance on Tegra210 in MIPI CSI calibration
Changed in v5:
- removed patch 3 as requested by Hans Verkuil; now the driver is kept
video-node-centric and the enum_format logic is unchanged
- rebased on top of that
- trivial fixes (typos)
Changed in RESEND,v4:
- add Rob's review tag on patch 2
Changed in v4:
- fixed the leftovers after the removal of 'channel@0' in DT
- added review tags by Dimtry
Changed in v3:
- removed the 'channel@0' node from the device tree representation of vip
- squashed the last two patches (VIP + T20) into one
- small cleanups
- rebase on v6.2-rc1
Changed in v2:
- improved dt-bindings patches based on reviews
- removed patches 3 and 4 adding DT labels without a mainline user
- two small fixes to the last patch
[v5] https://lore.kernel.org/linux-tegra/20230407133852.2850145-1-luca.ceresoli@bootlin.com/
[v4,resend] https://lore.kernel.org/linux-tegra/20230309144320.2937553-1-luca.ceresoli@bootlin.com/
[v4] https://lore.kernel.org/linux-tegra/20230130141603.323221-1-luca.ceresoli@bootlin.com/
[v3] https://lore.kernel.org/linux-media/20221229133205.981397-1-luca.ceresoli@bootlin.com/
[v2] https://lore.kernel.org/linux-tegra/20221222100328.6e341874@booty/T/#t
[v1] https://lore.kernel.org/linux-tegra/20221124155634.5bc2a423@booty/T/#t
Luca
Luca Ceresoli (20):
dt-bindings: display: tegra: add Tegra20 VIP
dt-bindings: display: tegra: vi: add 'vip' property and example
staging: media: tegra-video: improve documentation of
tegra_video_format fields
staging: media: tegra-video: document
tegra_channel_get_remote_source_subdev
staging: media: tegra-video: fix typos in comment
staging: media: tegra-video: improve error messages
staging: media: tegra-video: slightly simplify cleanup on errors
staging: media: tegra-video: move private struct declaration to C file
staging: media: tegra-video: move tegra210_csi_soc to C file
staging: media: tegra-video: remove unneeded include
staging: media: tegra-video: Kconfig: allow TPG only on Tegra210
staging: media: tegra-video: move tegra_channel_fmt_align to a per-soc
op
staging: media: tegra-video: move default format to soc-specific data
staging: media: tegra-video: move MIPI calibration calls from VI to
CSI
staging: media: tegra-video: add a per-soc enable/disable op
staging: media: tegra-video: move syncpt init/free to a per-soc op
staging: media: tegra-video: add syncpts for Tegra20 to struct
tegra_vi
staging: media: tegra-video: add hooks for planar YUV and H/V flip
staging: media: tegra-video: add H/V flip controls
staging: media: tegra-video: add support for Tegra20 parallel input
.../display/tegra/nvidia,tegra20-vi.yaml | 59 ++
.../display/tegra/nvidia,tegra20-vip.yaml | 41 ++
MAINTAINERS | 3 +
drivers/staging/media/tegra-video/Kconfig | 1 +
drivers/staging/media/tegra-video/Makefile | 2 +
drivers/staging/media/tegra-video/csi.c | 48 ++
drivers/staging/media/tegra-video/csi.h | 4 -
drivers/staging/media/tegra-video/tegra20.c | 661 ++++++++++++++++++
drivers/staging/media/tegra-video/tegra210.c | 90 +++
drivers/staging/media/tegra-video/vi.c | 222 ++----
drivers/staging/media/tegra-video/vi.h | 71 +-
drivers/staging/media/tegra-video/video.c | 5 +
drivers/staging/media/tegra-video/video.h | 2 +-
drivers/staging/media/tegra-video/vip.c | 290 ++++++++
drivers/staging/media/tegra-video/vip.h | 68 ++
15 files changed, 1380 insertions(+), 187 deletions(-)
create mode 100644 Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-vip.yaml
create mode 100644 drivers/staging/media/tegra-video/tegra20.c
create mode 100644 drivers/staging/media/tegra-video/vip.c
create mode 100644 drivers/staging/media/tegra-video/vip.h
--
2.34.1
next reply other threads:[~2023-04-18 8:01 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-18 8:00 Luca Ceresoli [this message]
2023-04-18 8:00 ` [PATCH v6 01/20] dt-bindings: display: tegra: add Tegra20 VIP Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 02/20] dt-bindings: display: tegra: vi: add 'vip' property and example Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 03/20] staging: media: tegra-video: improve documentation of tegra_video_format fields Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 04/20] staging: media: tegra-video: document tegra_channel_get_remote_source_subdev Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 05/20] staging: media: tegra-video: fix typos in comment Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 06/20] staging: media: tegra-video: improve error messages Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 07/20] staging: media: tegra-video: slightly simplify cleanup on errors Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 08/20] staging: media: tegra-video: move private struct declaration to C file Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 09/20] staging: media: tegra-video: move tegra210_csi_soc " Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 10/20] staging: media: tegra-video: remove unneeded include Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 11/20] staging: media: tegra-video: Kconfig: allow TPG only on Tegra210 Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 12/20] staging: media: tegra-video: move tegra_channel_fmt_align to a per-soc op Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 13/20] staging: media: tegra-video: move default format to soc-specific data Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 14/20] staging: media: tegra-video: move MIPI calibration calls from VI to CSI Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 15/20] staging: media: tegra-video: add a per-soc enable/disable op Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 16/20] staging: media: tegra-video: move syncpt init/free to a per-soc op Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 17/20] staging: media: tegra-video: add syncpts for Tegra20 to struct tegra_vi Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 18/20] staging: media: tegra-video: add hooks for planar YUV and H/V flip Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 19/20] staging: media: tegra-video: add H/V flip controls Luca Ceresoli
2023-04-18 8:00 ` [PATCH v6 20/20] staging: media: tegra-video: add support for Tegra20 parallel input Luca Ceresoli
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=20230418080054.452955-1-luca.ceresoli@bootlin.com \
--to=luca.ceresoli@bootlin.com \
--cc=airlied@gmail.com \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=gregkh@linuxfoundation.org \
--cc=hverkuil-cisco@xs4all.nl \
--cc=jonathanh@nvidia.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=linux-staging@lists.linux.dev \
--cc=linux-tegra@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=paul.kocialkowski@bootlin.com \
--cc=richard.leitner@skidata.com \
--cc=robh+dt@kernel.org \
--cc=skomatineni@nvidia.com \
--cc=thierry.reding@gmail.com \
--cc=thomas.petazzoni@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®