From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
To: linux-renesas-soc@vger.kernel.org,
"Jacopo Mondi" <jacopo@jmondi.org>,
"Niklas Söderlund" <niklas.soderlund@ragnatech.se>,
"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>
Cc: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
Subject: [PATCH 1/2] max9286: Split out async registration
Date: Wed, 12 Feb 2020 17:37:26 +0000 [thread overview]
Message-ID: <20200212173727.19476-2-kieran.bingham+renesas@ideasonboard.com> (raw)
In-Reply-To: <20200212173727.19476-1-kieran.bingham+renesas@ideasonboard.com>
Move all the V4L2 Subdev Async registration so that it can only happen once
we know we will not need to -EPROBE_DEFER...
Signed-off-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>
---
drivers/media/i2c/max9286.c | 88 +++++++++++++++++++++++--------------
1 file changed, 55 insertions(+), 33 deletions(-)
diff --git a/drivers/media/i2c/max9286.c b/drivers/media/i2c/max9286.c
index 1b4ff3533795..03c5fa232b6d 100644
--- a/drivers/media/i2c/max9286.c
+++ b/drivers/media/i2c/max9286.c
@@ -503,6 +503,49 @@ static const struct v4l2_async_notifier_operations max9286_notify_ops = {
.unbind = max9286_notify_unbind,
};
+static int max9286_v4l2_async_register(struct max9286_priv *priv)
+{
+ struct device *dev = &priv->client->dev;
+ struct max9286_source *source = NULL;
+ int ret;
+
+ v4l2_async_notifier_init(&priv->notifier);
+
+ for_each_source(priv, source) {
+ unsigned int i = to_index(priv, source);
+
+ dev_err(dev, "Registering v4l2-async for source %d\n", i);
+
+ source->asd.match_type = V4L2_ASYNC_MATCH_FWNODE;
+ source->asd.match.fwnode = source->fwnode;
+
+ ret = v4l2_async_notifier_add_subdev(&priv->notifier,
+ &source->asd);
+ if (ret) {
+ dev_err(dev, "Failed to add subdev for source %d", i);
+ v4l2_async_notifier_cleanup(&priv->notifier);
+ return ret;
+ }
+ }
+
+ priv->notifier.ops = &max9286_notify_ops;
+
+ ret = v4l2_async_subdev_notifier_register(&priv->sd, &priv->notifier);
+ if (ret) {
+ dev_err(dev, "Failed to register subdev_notifier");
+ v4l2_async_notifier_cleanup(&priv->notifier);
+ return ret;
+ }
+
+ return 0;
+}
+
+static void max9286_v4l2_async_unregister(struct max9286_priv *priv)
+{
+ v4l2_async_notifier_unregister(&priv->notifier);
+ v4l2_async_notifier_cleanup(&priv->notifier);
+}
+
static int max9286_s_stream(struct v4l2_subdev *sd, int enable)
{
struct max9286_priv *priv = sd_to_max9286(sd);
@@ -870,6 +913,13 @@ static int max9286_init(struct device *dev)
goto err_regulator;
}
+ /* Register v4l2 async notifiers */
+ ret = max9286_v4l2_async_register(priv);
+ if (ret) {
+ dev_err(dev, "Unable to register V4L2 async notifiers\n");
+ goto err_regulator;
+ }
+
v4l2_i2c_subdev_init(&priv->sd, client, &max9286_subdev_ops);
priv->sd.internal_ops = &max9286_subdev_internal_ops;
priv->sd.flags |= V4L2_SUBDEV_FL_HAS_DEVNODE;
@@ -884,7 +934,7 @@ static int max9286_init(struct device *dev)
priv->sd.ctrl_handler = &priv->ctrls;
ret = priv->ctrls.error;
if (ret)
- goto err_regulator;
+ goto err_async;
priv->sd.entity.function = MEDIA_ENT_F_VID_IF_BRIDGE;
@@ -927,6 +977,8 @@ static int max9286_init(struct device *dev)
max9286_i2c_mux_close(priv);
err_put_node:
fwnode_handle_put(ep);
+err_async:
+ max9286_v4l2_async_unregister(priv);
err_regulator:
regulator_disable(priv->regulator);
priv->poc_enabled = false;
@@ -938,14 +990,6 @@ static void max9286_cleanup_dt(struct max9286_priv *priv)
{
struct max9286_source *source;
- /*
- * Not strictly part of the DT, but the notifier is registered during
- * max9286_parse_dt(), and the notifier holds references to the fwnodes
- * thus the cleanup is here to mirror the registration.
- */
- v4l2_async_notifier_unregister(&priv->notifier);
- v4l2_async_notifier_cleanup(&priv->notifier);
-
for_each_source(priv, source) {
fwnode_handle_put(source->fwnode);
source->fwnode = NULL;
@@ -958,7 +1002,6 @@ static int max9286_parse_dt(struct max9286_priv *priv)
struct device_node *i2c_mux;
struct device_node *node = NULL;
unsigned int i2c_mux_mask = 0;
- int ret;
of_node_get(dev->of_node);
i2c_mux = of_find_node_by_name(dev->of_node, "i2c-mux");
@@ -986,8 +1029,6 @@ static int max9286_parse_dt(struct max9286_priv *priv)
of_node_put(node);
of_node_put(i2c_mux);
- v4l2_async_notifier_init(&priv->notifier);
-
/* Parse the endpoints */
for_each_endpoint_of_node(dev->of_node, node) {
struct max9286_source *source;
@@ -1056,34 +1097,14 @@ static int max9286_parse_dt(struct max9286_priv *priv)
continue;
}
- source->asd.match_type = V4L2_ASYNC_MATCH_FWNODE;
- source->asd.match.fwnode = source->fwnode;
-
- ret = v4l2_async_notifier_add_subdev(&priv->notifier,
- &source->asd);
- if (ret) {
- v4l2_async_notifier_cleanup(&priv->notifier);
- of_node_put(node);
- return ret;
- }
-
priv->source_mask |= BIT(ep.port);
priv->nsources++;
}
of_node_put(node);
- /* Do not register the subdev notifier if there are no devices. */
- if (!priv->nsources)
- return 0;
-
priv->route_mask = priv->source_mask;
- priv->notifier.ops = &max9286_notify_ops;
-
- ret = v4l2_async_subdev_notifier_register(&priv->sd, &priv->notifier);
- if (ret)
- v4l2_async_notifier_cleanup(&priv->notifier);
- return ret;
+ return 0;
}
static int max9286_probe(struct i2c_client *client)
@@ -1182,6 +1203,7 @@ static int max9286_remove(struct i2c_client *client)
fwnode_handle_put(priv->sd.fwnode);
v4l2_async_unregister_subdev(&priv->sd);
+ max9286_v4l2_async_unregister(priv);
if (priv->poc_enabled)
regulator_disable(priv->regulator);
--
2.20.1
next prev parent reply other threads:[~2020-02-12 17:37 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-12 17:37 [PATCH 0/2] max9286: Refactor V4L2 support to prevent EPROBE_DEFER failures Kieran Bingham
2020-02-12 17:37 ` Kieran Bingham [this message]
2020-02-12 17:39 ` [PATCH 1/2] max9286: Split out async registration Kieran Bingham
2020-02-13 9:46 ` Jacopo Mondi
2020-02-13 10:07 ` Kieran Bingham
2020-02-13 10:15 ` Kieran Bingham
2020-02-13 10:20 ` Jacopo Mondi
2020-02-13 10:27 ` Kieran Bingham
2020-02-13 11:41 ` Jacopo Mondi
2020-02-12 17:37 ` [PATCH 2/2] max9286: Collect all V4L2 registrations Kieran Bingham
2020-02-13 10:21 ` [PATCH] max9286: balance v4l2_async refcnting Kieran Bingham
2020-02-13 13:06 ` Kieran Bingham
2020-02-13 13:11 ` Jacopo Mondi
2020-02-13 13:28 ` Kieran Bingham
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=20200212173727.19476-2-kieran.bingham+renesas@ideasonboard.com \
--to=kieran.bingham+renesas@ideasonboard.com \
--cc=jacopo@jmondi.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=niklas.soderlund@ragnatech.se \
/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).