From: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
To: linux-media@vger.kernel.org,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Hans Verkuil <hverkuil@xs4all.nl>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>
Cc: "Tomi Valkeinen" <tomi.valkeinen@ideasonboard.com>,
"Andrzej Hajda" <a.hajda@samsung.com>,
"Sylwester Nawrocki" <sylvester.nawrocki@gmail.com>,
"Lars-Peter Clausen" <lars@metafoo.de>,
"Kieran Bingham" <kieran.bingham@ideasonboard.com>,
"Hans Verkuil" <hverkuil-cisco@xs4all.nl>,
"Pavel Machek" <pavel@ucw.cz>, "Shawn Tu" <shawnx.tu@intel.com>,
"Ricardo Ribalda" <ribalda@kernel.org>,
"Dave Stevenson" <dave.stevenson@raspberrypi.com>,
"Leon Luo" <leonl@leopardimaging.com>,
"Manivannan Sadhasivam" <mani@kernel.org>,
"Bingbu Cao" <bingbu.cao@intel.com>,
"Paul J. Murphy" <paul.j.murphy@intel.com>,
"Daniele Alessandrelli" <daniele.alessandrelli@intel.com>,
"Tianshu Qiu" <tian.shu.qiu@intel.com>,
"Kyungmin Park" <kyungmin.park@samsung.com>,
"Heungjun Kim" <riverful.kim@samsung.com>,
"Jacopo Mondi" <jacopo+renesas@jmondi.org>,
"Laurent Pinchart" <laurent.pinchart+renesas@ideasonboard.com>,
"Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>,
"Dongchun Zhu" <dongchun.zhu@mediatek.com>,
"Lad, Prabhakar" <prabhakar.csengg@gmail.com>,
"Rui Miguel Silva" <rmfrfs@gmail.com>,
"Shunqian Zheng" <zhengsq@rock-chips.com>,
"Steve Longerbeam" <slongerbeam@gmail.com>,
"Chiranjeevi Rapolu" <chiranjeevi.rapolu@intel.com>,
"Hyungwoo Yang" <hyungwoo.yang@intel.com>,
"Wenyou Yang" <wenyou.yang@microchip.com>,
"Petr Cvek" <petrcvekcz@gmail.com>,
"Akinobu Mita" <akinobu.mita@gmail.com>,
"Sylwester Nawrocki" <s.nawrocki@samsung.com>,
"Mickael Guene" <mickael.guene@st.com>,
"Mats Randgaard" <matrandg@cisco.com>,
"Tim Harvey" <tharvey@gateworks.com>,
"Andy Walls" <awalls@md.metrocast.net>,
"Yong Zhi" <yong.zhi@intel.com>,
"Dan Scally" <djrscally@gmail.com>,
"Eugen Hristev" <eugen.hristev@microchip.com>,
"Nicolas Ferre" <nicolas.ferre@microchip.com>,
"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
"Ludovic Desroches" <ludovic.desroches@microchip.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Krzysztof Kozlowski" <krzysztof.kozlowski@canonical.com>,
"Robert Foss" <robert.foss@linaro.org>,
"Todor Tomov" <todor.too@gmail.com>,
"Andy Gross" <agross@kernel.org>,
"Bjorn Andersson" <bjorn.andersson@linaro.org>,
"Niklas Söderlund" <niklas.soderlund@ragnatech.se>,
"Jacopo Mondi" <jacopo@jmondi.org>,
"Helen Koike" <helen.koike@collabora.com>,
"Dafna Hirschfeld" <dafna.hirschfeld@collabora.com>,
"Heiko Stuebner" <heiko@sntech.de>,
"Hugues Fruchet" <hugues.fruchet@foss.st.com>,
"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
"Chen-Yu Tsai" <wens@csie.org>,
"Jernej Skrabec" <jernej.skrabec@gmail.com>,
"Benoit Parrot" <bparrot@ti.com>,
"Philipp Zabel" <p.zabel@pengutronix.de>,
"Kieran Bingham" <kieran.bingham+renesas@ideasonboard.com>,
"Hyun Kwon" <hyun.kwon@xilinx.com>,
"Michal Simek" <michal.simek@xilinx.com>,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, linux-arm-msm@vger.kernel.org,
linux-renesas-soc@vger.kernel.org,
linux-rockchip@lists.infradead.org,
linux-stm32@st-md-mailman.stormreply.com,
linux-sunxi@lists.linux.dev,
"Shuah Khan" <skhan@linuxfoundation.org>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Shawn Guo" <shawnguo@kernel.org>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Fabio Estevam" <festevam@gmail.com>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Thierry Reding" <thierry.reding@gmail.com>,
"Jonathan Hunter" <jonathanh@nvidia.com>,
"Sowjanya Komatineni" <skomatineni@nvidia.com>
Subject: [PATCH v5 0/9] media: v4l2-subdev: add subdev-wide state struct
Date: Thu, 10 Jun 2021 17:55:57 +0300 [thread overview]
Message-ID: <20210610145606.3468235-1-tomi.valkeinen@ideasonboard.com> (raw)
Hi,
v5 of the series. No content changes compared to v4, but the series is
split into smaller parts to enable reviews.
The split is artificial, and all the patches need to be squashed into
one before merging.
The point of the series is explained in "media: v4l2-subdev: add
subdev-wide state struct", but for easier reviews I add it partially
here:
We have 'struct v4l2_subdev_pad_config' which contains configuration for
a single pad used for the TRY functionality, and an array of those
structs is passed to various v4l2_subdev_pad_ops.
I was working on subdev internal routing between pads, and realized that
there's no way to add TRY functionality for routes, which is not pad
specific configuration. Adding a separate struct for try-route config
wouldn't work either, as e.g. set-fmt needs to know the try-route
configuration to propagate the settings.
This patch adds a new struct, 'struct v4l2_subdev_state' (which at the
moment only contains the v4l2_subdev_pad_config array) and the new
struct is used in most of the places where v4l2_subdev_pad_config was
used. All v4l2_subdev_pad_ops functions taking v4l2_subdev_pad_config
are changed to instead take v4l2_subdev_state.
Tomi
Tomi Valkeinen (9):
media: v4l2-subdev: add subdev-wide state struct
media: s5k5baf: changes for subdev-wide state struct
media: s3c-camif: changes for subdev-wide state struct
media: i2c: changes for subdev-wide state struct
media: pci: changes for subdev-wide state struct
media: platform: changes for subdev-wide state struct
media: test-drivers: changes for subdev-wide state struct
media: usb: changes for subdev-wide state struct
media: staging: changes for subdev-wide state struct
drivers/media/i2c/adv7170.c | 6 +-
drivers/media/i2c/adv7175.c | 6 +-
drivers/media/i2c/adv7180.c | 18 +--
drivers/media/i2c/adv7183.c | 8 +-
drivers/media/i2c/adv748x/adv748x-afe.c | 13 +-
drivers/media/i2c/adv748x/adv748x-csi2.c | 14 +-
drivers/media/i2c/adv748x/adv748x-hdmi.c | 13 +-
drivers/media/i2c/adv7511-v4l2.c | 10 +-
drivers/media/i2c/adv7604.c | 12 +-
drivers/media/i2c/adv7842.c | 12 +-
drivers/media/i2c/ak881x.c | 6 +-
drivers/media/i2c/ccs/ccs-core.c | 84 ++++++-----
drivers/media/i2c/cx25840/cx25840-core.c | 2 +-
drivers/media/i2c/et8ek8/et8ek8_driver.c | 23 +--
drivers/media/i2c/hi556.c | 15 +-
drivers/media/i2c/imx208.c | 19 +--
drivers/media/i2c/imx214.c | 37 ++---
drivers/media/i2c/imx219.c | 30 ++--
drivers/media/i2c/imx258.c | 19 +--
drivers/media/i2c/imx274.c | 38 ++---
drivers/media/i2c/imx290.c | 20 +--
drivers/media/i2c/imx319.c | 18 +--
drivers/media/i2c/imx334.c | 28 ++--
drivers/media/i2c/imx355.c | 18 +--
drivers/media/i2c/m5mols/m5mols_core.c | 21 ++-
drivers/media/i2c/max9286.c | 17 ++-
drivers/media/i2c/ml86v7667.c | 4 +-
drivers/media/i2c/mt9m001.c | 18 +--
drivers/media/i2c/mt9m032.c | 38 ++---
drivers/media/i2c/mt9m111.c | 18 +--
drivers/media/i2c/mt9p031.c | 45 +++---
drivers/media/i2c/mt9t001.c | 44 +++---
drivers/media/i2c/mt9t112.c | 14 +-
drivers/media/i2c/mt9v011.c | 6 +-
drivers/media/i2c/mt9v032.c | 44 +++---
drivers/media/i2c/mt9v111.c | 23 +--
drivers/media/i2c/noon010pc30.c | 19 ++-
drivers/media/i2c/ov02a10.c | 17 ++-
drivers/media/i2c/ov13858.c | 18 +--
drivers/media/i2c/ov2640.c | 16 +-
drivers/media/i2c/ov2659.c | 14 +-
drivers/media/i2c/ov2680.c | 23 +--
drivers/media/i2c/ov2685.c | 10 +-
drivers/media/i2c/ov2740.c | 15 +-
drivers/media/i2c/ov5640.c | 14 +-
drivers/media/i2c/ov5645.c | 38 ++---
drivers/media/i2c/ov5647.c | 26 ++--
drivers/media/i2c/ov5648.c | 14 +-
drivers/media/i2c/ov5670.c | 19 +--
drivers/media/i2c/ov5675.c | 15 +-
drivers/media/i2c/ov5695.c | 15 +-
drivers/media/i2c/ov6650.c | 28 ++--
drivers/media/i2c/ov7251.c | 39 ++---
drivers/media/i2c/ov7670.c | 17 ++-
drivers/media/i2c/ov772x.c | 12 +-
drivers/media/i2c/ov7740.c | 17 ++-
drivers/media/i2c/ov8856.c | 15 +-
drivers/media/i2c/ov8865.c | 14 +-
drivers/media/i2c/ov9640.c | 8 +-
drivers/media/i2c/ov9650.c | 17 ++-
drivers/media/i2c/ov9734.c | 15 +-
drivers/media/i2c/rdacm20.c | 4 +-
drivers/media/i2c/rdacm21.c | 4 +-
drivers/media/i2c/rj54n1cb0c.c | 12 +-
drivers/media/i2c/s5c73m3/s5c73m3-core.c | 55 +++----
drivers/media/i2c/s5k4ecgx.c | 22 +--
drivers/media/i2c/s5k5baf.c | 49 +++---
drivers/media/i2c/s5k6a3.c | 19 ++-
drivers/media/i2c/s5k6aa.c | 39 ++---
drivers/media/i2c/saa6752hs.c | 6 +-
drivers/media/i2c/saa7115.c | 2 +-
drivers/media/i2c/saa717x.c | 2 +-
drivers/media/i2c/sr030pc30.c | 8 +-
drivers/media/i2c/st-mipid02.c | 21 +--
drivers/media/i2c/tc358743.c | 8 +-
drivers/media/i2c/tda1997x.c | 14 +-
drivers/media/i2c/tvp514x.c | 12 +-
drivers/media/i2c/tvp5150.c | 20 +--
drivers/media/i2c/tvp7002.c | 11 +-
drivers/media/i2c/tw9910.c | 10 +-
drivers/media/i2c/vs6624.c | 8 +-
drivers/media/pci/cx18/cx18-av-core.c | 2 +-
drivers/media/pci/intel/ipu3/ipu3-cio2-main.c | 17 ++-
drivers/media/pci/saa7134/saa7134-empress.c | 5 +-
drivers/media/platform/atmel/atmel-isc-base.c | 19 ++-
drivers/media/platform/atmel/atmel-isi.c | 19 ++-
drivers/media/platform/cadence/cdns-csi2tx.c | 14 +-
.../media/platform/exynos4-is/fimc-capture.c | 22 +--
drivers/media/platform/exynos4-is/fimc-isp.c | 37 +++--
drivers/media/platform/exynos4-is/fimc-lite.c | 39 ++---
drivers/media/platform/exynos4-is/mipi-csis.c | 17 ++-
.../media/platform/marvell-ccic/mcam-core.c | 5 +-
drivers/media/platform/omap3isp/ispccdc.c | 85 ++++++-----
drivers/media/platform/omap3isp/ispccp2.c | 49 +++---
drivers/media/platform/omap3isp/ispcsi2.c | 41 ++---
drivers/media/platform/omap3isp/isppreview.c | 69 +++++----
drivers/media/platform/omap3isp/ispresizer.c | 70 +++++----
drivers/media/platform/pxa_camera.c | 5 +-
.../media/platform/qcom/camss/camss-csid.c | 35 ++---
.../media/platform/qcom/camss/camss-csiphy.c | 40 ++---
.../media/platform/qcom/camss/camss-ispif.c | 36 ++---
drivers/media/platform/qcom/camss/camss-vfe.c | 84 ++++++-----
drivers/media/platform/rcar-vin/rcar-csi2.c | 8 +-
drivers/media/platform/rcar-vin/rcar-v4l2.c | 10 +-
drivers/media/platform/renesas-ceu.c | 7 +-
.../platform/rockchip/rkisp1/rkisp1-isp.c | 112 ++++++++------
.../platform/rockchip/rkisp1/rkisp1-resizer.c | 95 +++++++-----
.../media/platform/s3c-camif/camif-capture.c | 18 +--
drivers/media/platform/stm32/stm32-dcmi.c | 14 +-
.../platform/sunxi/sun4i-csi/sun4i_v4l2.c | 16 +-
drivers/media/platform/ti-vpe/cal-camerarx.c | 35 +++--
drivers/media/platform/via-camera.c | 5 +-
drivers/media/platform/video-mux.c | 22 +--
drivers/media/platform/vsp1/vsp1_brx.c | 34 +++--
drivers/media/platform/vsp1/vsp1_clu.c | 13 +-
drivers/media/platform/vsp1/vsp1_entity.c | 59 ++++----
drivers/media/platform/vsp1/vsp1_entity.h | 20 +--
drivers/media/platform/vsp1/vsp1_histo.c | 51 ++++---
drivers/media/platform/vsp1/vsp1_hsit.c | 14 +-
drivers/media/platform/vsp1/vsp1_lif.c | 13 +-
drivers/media/platform/vsp1/vsp1_lut.c | 13 +-
drivers/media/platform/vsp1/vsp1_rwpf.c | 32 ++--
drivers/media/platform/vsp1/vsp1_rwpf.h | 2 +-
drivers/media/platform/vsp1/vsp1_sru.c | 22 +--
drivers/media/platform/vsp1/vsp1_uds.c | 22 +--
drivers/media/platform/vsp1/vsp1_uif.c | 27 ++--
.../media/platform/xilinx/xilinx-csi2rxss.c | 26 ++--
drivers/media/platform/xilinx/xilinx-tpg.c | 25 +--
drivers/media/platform/xilinx/xilinx-vip.c | 12 +-
drivers/media/platform/xilinx/xilinx-vip.h | 4 +-
.../media/test-drivers/vimc/vimc-debayer.c | 20 +--
drivers/media/test-drivers/vimc/vimc-scaler.c | 36 ++---
drivers/media/test-drivers/vimc/vimc-sensor.c | 16 +-
drivers/media/usb/go7007/s2250-board.c | 2 +-
drivers/media/v4l2-core/v4l2-subdev.c | 142 ++++++++++--------
.../media/atomisp/i2c/atomisp-gc0310.c | 10 +-
.../media/atomisp/i2c/atomisp-gc2235.c | 10 +-
.../media/atomisp/i2c/atomisp-mt9m114.c | 12 +-
.../media/atomisp/i2c/atomisp-ov2680.c | 10 +-
.../media/atomisp/i2c/atomisp-ov2722.c | 10 +-
.../media/atomisp/i2c/ov5693/atomisp-ov5693.c | 10 +-
.../staging/media/atomisp/pci/atomisp_cmd.c | 33 ++--
.../staging/media/atomisp/pci/atomisp_csi2.c | 28 ++--
.../staging/media/atomisp/pci/atomisp_csi2.h | 2 +-
.../staging/media/atomisp/pci/atomisp_file.c | 14 +-
.../staging/media/atomisp/pci/atomisp_fops.c | 6 +-
.../media/atomisp/pci/atomisp_subdev.c | 64 ++++----
.../media/atomisp/pci/atomisp_subdev.h | 9 +-
.../staging/media/atomisp/pci/atomisp_tpg.c | 12 +-
drivers/staging/media/imx/imx-ic-prp.c | 19 +--
drivers/staging/media/imx/imx-ic-prpencvf.c | 31 ++--
drivers/staging/media/imx/imx-media-csi.c | 82 +++++-----
drivers/staging/media/imx/imx-media-utils.c | 4 +-
drivers/staging/media/imx/imx-media-vdic.c | 24 +--
drivers/staging/media/imx/imx-media.h | 2 +-
drivers/staging/media/imx/imx6-mipi-csi2.c | 12 +-
drivers/staging/media/imx/imx7-media-csi.c | 33 ++--
drivers/staging/media/imx/imx7-mipi-csis.c | 34 +++--
drivers/staging/media/ipu3/ipu3-v4l2.c | 26 ++--
drivers/staging/media/omap4iss/iss_csi2.c | 37 ++---
drivers/staging/media/omap4iss/iss_ipipe.c | 37 ++---
drivers/staging/media/omap4iss/iss_ipipeif.c | 47 +++---
drivers/staging/media/omap4iss/iss_resizer.c | 39 ++---
drivers/staging/media/tegra-video/csi.c | 10 +-
drivers/staging/media/tegra-video/vi.c | 24 +--
include/media/v4l2-subdev.h | 74 +++++----
166 files changed, 2162 insertions(+), 1802 deletions(-)
--
2.25.1
next reply other threads:[~2021-06-10 15:05 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-10 14:55 Tomi Valkeinen [this message]
2021-06-10 14:56 ` [PATCH v5 6/9] media: platform: changes for subdev-wide state struct Tomi Valkeinen
2021-06-11 6:14 ` [PATCH v5 0/9] media: v4l2-subdev: add " Sakari Ailus
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=20210610145606.3468235-1-tomi.valkeinen@ideasonboard.com \
--to=tomi.valkeinen@ideasonboard.com \
--cc=a.hajda@samsung.com \
--cc=agross@kernel.org \
--cc=akinobu.mita@gmail.com \
--cc=alexandre.belloni@bootlin.com \
--cc=alexandre.torgue@foss.st.com \
--cc=awalls@md.metrocast.net \
--cc=bingbu.cao@intel.com \
--cc=bjorn.andersson@linaro.org \
--cc=bparrot@ti.com \
--cc=chiranjeevi.rapolu@intel.com \
--cc=dafna.hirschfeld@collabora.com \
--cc=daniele.alessandrelli@intel.com \
--cc=dave.stevenson@raspberrypi.com \
--cc=djrscally@gmail.com \
--cc=dongchun.zhu@mediatek.com \
--cc=eugen.hristev@microchip.com \
--cc=festevam@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=heiko@sntech.de \
--cc=helen.koike@collabora.com \
--cc=hugues.fruchet@foss.st.com \
--cc=hverkuil-cisco@xs4all.nl \
--cc=hverkuil@xs4all.nl \
--cc=hyun.kwon@xilinx.com \
--cc=hyungwoo.yang@intel.com \
--cc=jacopo+renesas@jmondi.org \
--cc=jacopo@jmondi.org \
--cc=jernej.skrabec@gmail.com \
--cc=jonathanh@nvidia.com \
--cc=kernel@pengutronix.de \
--cc=kieran.bingham+renesas@ideasonboard.com \
--cc=kieran.bingham@ideasonboard.com \
--cc=krzysztof.kozlowski@canonical.com \
--cc=kyungmin.park@samsung.com \
--cc=lars@metafoo.de \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=leonl@leopardimaging.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-imx@nxp.com \
--cc=linux-media@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux-stm32@st-md-mailman.stormreply.com \
--cc=linux-sunxi@lists.linux.dev \
--cc=ludovic.desroches@microchip.com \
--cc=mani@kernel.org \
--cc=matrandg@cisco.com \
--cc=mchehab@kernel.org \
--cc=mcoquelin.stm32@gmail.com \
--cc=michal.simek@xilinx.com \
--cc=mickael.guene@st.com \
--cc=mripard@kernel.org \
--cc=nicolas.ferre@microchip.com \
--cc=niklas.soderlund+renesas@ragnatech.se \
--cc=niklas.soderlund@ragnatech.se \
--cc=p.zabel@pengutronix.de \
--cc=paul.j.murphy@intel.com \
--cc=pavel@ucw.cz \
--cc=petrcvekcz@gmail.com \
--cc=prabhakar.csengg@gmail.com \
--cc=ribalda@kernel.org \
--cc=riverful.kim@samsung.com \
--cc=rmfrfs@gmail.com \
--cc=robert.foss@linaro.org \
--cc=s.hauer@pengutronix.de \
--cc=s.nawrocki@samsung.com \
--cc=sakari.ailus@linux.intel.com \
--cc=shawnguo@kernel.org \
--cc=shawnx.tu@intel.com \
--cc=skhan@linuxfoundation.org \
--cc=skomatineni@nvidia.com \
--cc=slongerbeam@gmail.com \
--cc=sylvester.nawrocki@gmail.com \
--cc=tharvey@gateworks.com \
--cc=thierry.reding@gmail.com \
--cc=tian.shu.qiu@intel.com \
--cc=todor.too@gmail.com \
--cc=wens@csie.org \
--cc=wenyou.yang@microchip.com \
--cc=yong.zhi@intel.com \
--cc=zhengsq@rock-chips.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).