devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rui Miguel Silva <rui.silva@linaro.org>
To: mchehab@kernel.org, sakari.ailus@linux.intel.com,
	Steve Longerbeam <slongerbeam@gmail.com>,
	Philipp Zabel <p.zabel@pengutronix.de>,
	Rob Herring <robh+dt@kernel.org>
Cc: devel@driverdev.osuosl.org, devicetree@vger.kernel.org,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Ryan Harkin <ryan.harkin@linaro.org>,
	Rui Miguel Silva <rui.silva@linaro.org>,
	Fabio Estevam <fabio.estevam@nxp.com>,
	Shawn Guo <shawnguo@kernel.org>,
	linux-media@vger.kernel.org
Subject: [PATCH 01/15] media: staging/imx: add support to media dev for no IPU systems
Date: Thu, 19 Apr 2018 11:17:58 +0100	[thread overview]
Message-ID: <20180419101812.30688-2-rui.silva@linaro.org> (raw)
In-Reply-To: <20180419101812.30688-1-rui.silva@linaro.org>

Some i.MX SoC do not have IPU, like the i.MX7, add to the the media device
infrastructure support to be used in this type of systems that do not have
internal subdevices besides the CSI.

Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org>
---
 drivers/staging/media/imx/imx-media-dev.c        | 16 +++++++++++-----
 .../staging/media/imx/imx-media-internal-sd.c    |  3 +++
 drivers/staging/media/imx/imx-media.h            |  3 +++
 3 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/media/imx/imx-media-dev.c b/drivers/staging/media/imx/imx-media-dev.c
index f67ec8e27093..a8afe0ec4134 100644
--- a/drivers/staging/media/imx/imx-media-dev.c
+++ b/drivers/staging/media/imx/imx-media-dev.c
@@ -92,6 +92,9 @@ static int imx_media_get_ipu(struct imx_media_dev *imxmd,
 	struct ipu_soc *ipu;
 	int ipu_id;
 
+	if (imxmd->no_ipu_present)
+		return 0;
+
 	ipu = dev_get_drvdata(csi_sd->dev->parent);
 	if (!ipu) {
 		v4l2_err(&imxmd->v4l2_dev,
@@ -481,16 +484,19 @@ static int imx_media_probe(struct platform_device *pdev)
 		goto notifier_cleanup;
 	}
 
-	ret = imx_media_add_internal_subdevs(imxmd);
-	if (ret) {
-		v4l2_err(&imxmd->v4l2_dev,
-			 "add_internal_subdevs failed with %d\n", ret);
-		goto notifier_cleanup;
+	if (!imxmd->no_ipu_present) {
+		ret = imx_media_add_internal_subdevs(imxmd);
+		if (ret) {
+			v4l2_err(&imxmd->v4l2_dev,
+				 "add_internal_subdevs failed with %d\n", ret);
+			goto notifier_cleanup;
+		}
 	}
 
 	/* no subdevs? just bail */
 	if (imxmd->notifier.num_subdevs == 0) {
 		ret = -ENODEV;
+		v4l2_err(&imxmd->v4l2_dev, "no subdevs\n");
 		goto notifier_cleanup;
 	}
 
diff --git a/drivers/staging/media/imx/imx-media-internal-sd.c b/drivers/staging/media/imx/imx-media-internal-sd.c
index 0fdc45dbfb76..4a246813b4e1 100644
--- a/drivers/staging/media/imx/imx-media-internal-sd.c
+++ b/drivers/staging/media/imx/imx-media-internal-sd.c
@@ -238,6 +238,9 @@ int imx_media_create_internal_links(struct imx_media_dev *imxmd,
 	struct media_pad *pad;
 	int i, j, ret;
 
+	if (imxmd->no_ipu_present)
+		return 0;
+
 	intsd = find_intsd_by_grp_id(sd->grp_id);
 	if (!intsd)
 		return -ENODEV;
diff --git a/drivers/staging/media/imx/imx-media.h b/drivers/staging/media/imx/imx-media.h
index 44532cd5b812..0c63132861a0 100644
--- a/drivers/staging/media/imx/imx-media.h
+++ b/drivers/staging/media/imx/imx-media.h
@@ -147,6 +147,9 @@ struct imx_media_dev {
 
 	/* for async subdev registration */
 	struct v4l2_async_notifier notifier;
+
+	/* indicator to if the system lack IPU */
+	bool no_ipu_present;
 };
 
 enum codespace_sel {
-- 
2.17.0

  reply	other threads:[~2018-04-19 10:17 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-19 10:17 [PATCH 00/15] media: staging/imx7: add i.MX7 media driver Rui Miguel Silva
2018-04-19 10:17 ` Rui Miguel Silva [this message]
2018-04-19 12:06   ` [PATCH 01/15] media: staging/imx: add support to media dev for no IPU systems Dan Carpenter
2018-04-19 14:02     ` Rui Miguel Silva
2018-04-19 10:17 ` [PATCH 02/15] media: staging/imx7: add imx7 CSI subdev driver Rui Miguel Silva
2018-04-19 12:22   ` Dan Carpenter
2018-04-19 14:03     ` Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 03/15] clk: imx7d: fix mipi dphy div parent Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 04/15] clk: imx7d: reset parent for mipi csi root Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 05/15] media: staging/imx7: add MIPI CSI-2 receiver subdev for i.MX7 Rui Miguel Silva
2018-04-19 12:45   ` Dan Carpenter
2018-04-19 14:09     ` Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 06/15] media: staging/imx: add imx7 capture subsystem Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 07/15] media: staging/imx: add 10 bit bayer support Rui Miguel Silva
2018-04-19 13:38   ` Philipp Zabel
2018-04-19 14:20     ` Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 08/15] ARM: dts: increase default cma size to 40MB Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 09/15] media: dt-bindings: add bindings for i.MX7 media driver Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 10/15] ARM: dts: imx7s: add mipi phy power domain Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 11/15] ARM: dts: imx7s: add multiplexer controls Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 12/15] ARM: dts: imx7: Add video mux, csi and mipi_csi and connections Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 13/15] ARM: dts: imx7s: add capture subsystem Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 14/15] media: imx7.rst: add documentation for i.MX7 media driver Rui Miguel Silva
2018-04-19 10:18 ` [PATCH 15/15] media: staging/imx: add i.MX7 entries to TODO file Rui Miguel Silva

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=20180419101812.30688-2-rui.silva@linaro.org \
    --to=rui.silva@linaro.org \
    --cc=devel@driverdev.osuosl.org \
    --cc=devicetree@vger.kernel.org \
    --cc=fabio.estevam@nxp.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@kernel.org \
    --cc=p.zabel@pengutronix.de \
    --cc=robh+dt@kernel.org \
    --cc=ryan.harkin@linaro.org \
    --cc=sakari.ailus@linux.intel.com \
    --cc=shawnguo@kernel.org \
    --cc=slongerbeam@gmail.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).