* [PATCH] remoteproc: Introduce rproc handle accessor for children
@ 2017-08-28 5:34 Bjorn Andersson
2017-09-01 21:00 ` Suman Anna
0 siblings, 1 reply; 3+ messages in thread
From: Bjorn Andersson @ 2017-08-28 5:34 UTC (permalink / raw)
To: Ohad Ben-Cohen, Bjorn Andersson; +Cc: linux-remoteproc, linux-kernel
In certain circumstances rpmsg devices needs to acquire a handle to the
ancestor remoteproc instance, e.g. to invoke rproc_report_crash() when a
fatal error is detected. Introduce an interface that walks the device
tree in search for a remoteproc instance and return this.
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
---
drivers/remoteproc/remoteproc_core.c | 18 ++++++++++++++++++
include/linux/remoteproc.h | 2 ++
2 files changed, 20 insertions(+)
diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c
index 564061dcc019..5b1b19519275 100644
--- a/drivers/remoteproc/remoteproc_core.c
+++ b/drivers/remoteproc/remoteproc_core.c
@@ -1296,6 +1296,23 @@ struct rproc *rproc_get_by_phandle(phandle phandle)
EXPORT_SYMBOL(rproc_get_by_phandle);
/**
+ * rproc_get_by_child() - acquire rproc handle of @dev's ancestor
+ * @dev: child device to find ancestor of
+ *
+ * Returns the ancestor rproc instance, or NULL if not found.
+ */
+struct rproc *rproc_get_by_child(struct device *dev)
+{
+ for (dev = dev->parent; dev; dev = dev->parent) {
+ if (dev->type && !strcmp(dev->type->name, "remoteproc"))
+ return dev->driver_data;
+ }
+
+ return NULL;
+}
+EXPORT_SYMBOL(rproc_get_by_child);
+
+/**
* rproc_add() - register a remote processor
* @rproc: the remote processor handle to register
*
@@ -1440,6 +1457,7 @@ struct rproc *rproc_alloc(struct device *dev, const char *name,
rproc->dev.parent = dev;
rproc->dev.type = &rproc_type;
rproc->dev.class = &rproc_class;
+ rproc->dev.driver_data = rproc;
/* Assign a unique device index and name */
rproc->index = ida_simple_get(&rproc_dev_index, 0, 0, GFP_KERNEL);
diff --git a/include/linux/remoteproc.h b/include/linux/remoteproc.h
index 81da49564ff4..44e630eb3d94 100644
--- a/include/linux/remoteproc.h
+++ b/include/linux/remoteproc.h
@@ -510,6 +510,8 @@ struct rproc_vdev {
};
struct rproc *rproc_get_by_phandle(phandle phandle);
+struct rproc *rproc_get_by_child(struct device *dev);
+
struct rproc *rproc_alloc(struct device *dev, const char *name,
const struct rproc_ops *ops,
const char *firmware, int len);
--
2.12.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] remoteproc: Introduce rproc handle accessor for children
2017-08-28 5:34 [PATCH] remoteproc: Introduce rproc handle accessor for children Bjorn Andersson
@ 2017-09-01 21:00 ` Suman Anna
0 siblings, 0 replies; 3+ messages in thread
From: Suman Anna @ 2017-09-01 21:00 UTC (permalink / raw)
To: Bjorn Andersson, Ohad Ben-Cohen; +Cc: linux-remoteproc, linux-kernel
Hi Bjorn,
On 08/28/2017 12:34 AM, Bjorn Andersson wrote:
> In certain circumstances rpmsg devices needs to acquire a handle to the
> ancestor remoteproc instance, e.g. to invoke rproc_report_crash() when a
> fatal error is detected. Introduce an interface that walks the device
> tree in search for a remoteproc instance and return this.
Thanks for the patch. I also have a need to be able to get a rproc
handle from rpmsg device for some buffer translations. I have been using
a somewhat similar solution downstream and was able to use this patch
just fine.
>
> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> ---
> drivers/remoteproc/remoteproc_core.c | 18 ++++++++++++++++++
> include/linux/remoteproc.h | 2 ++
> 2 files changed, 20 insertions(+)
>
> diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c
> index 564061dcc019..5b1b19519275 100644
> --- a/drivers/remoteproc/remoteproc_core.c
> +++ b/drivers/remoteproc/remoteproc_core.c
> @@ -1296,6 +1296,23 @@ struct rproc *rproc_get_by_phandle(phandle phandle)
> EXPORT_SYMBOL(rproc_get_by_phandle);
>
> /**
> + * rproc_get_by_child() - acquire rproc handle of @dev's ancestor
> + * @dev: child device to find ancestor of
> + *
> + * Returns the ancestor rproc instance, or NULL if not found.
> + */
> +struct rproc *rproc_get_by_child(struct device *dev)
> +{
> + for (dev = dev->parent; dev; dev = dev->parent) {
> + if (dev->type && !strcmp(dev->type->name, "remoteproc"))
It's much simpler to just check the type pointer,
if (dev->type && dev->type == &rproc_type)
Tested both versions of the patch.
regards
Suman
> + return dev->driver_data;
> + }
> +
> + return NULL;
> +}
> +EXPORT_SYMBOL(rproc_get_by_child);
> +
> +/**
> * rproc_add() - register a remote processor
> * @rproc: the remote processor handle to register
> *
> @@ -1440,6 +1457,7 @@ struct rproc *rproc_alloc(struct device *dev, const char *name,
> rproc->dev.parent = dev;
> rproc->dev.type = &rproc_type;
> rproc->dev.class = &rproc_class;
> + rproc->dev.driver_data = rproc;
>
> /* Assign a unique device index and name */
> rproc->index = ida_simple_get(&rproc_dev_index, 0, 0, GFP_KERNEL);
> diff --git a/include/linux/remoteproc.h b/include/linux/remoteproc.h
> index 81da49564ff4..44e630eb3d94 100644
> --- a/include/linux/remoteproc.h
> +++ b/include/linux/remoteproc.h
> @@ -510,6 +510,8 @@ struct rproc_vdev {
> };
>
> struct rproc *rproc_get_by_phandle(phandle phandle);
> +struct rproc *rproc_get_by_child(struct device *dev);
> +
> struct rproc *rproc_alloc(struct device *dev, const char *name,
> const struct rproc_ops *ops,
> const char *firmware, int len);
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] remoteproc: Introduce rproc handle accessor for children
@ 2017-09-01 21:00 ` Suman Anna
0 siblings, 0 replies; 3+ messages in thread
From: Suman Anna @ 2017-09-01 21:00 UTC (permalink / raw)
To: Bjorn Andersson, Ohad Ben-Cohen; +Cc: linux-remoteproc, linux-kernel
Hi Bjorn,
On 08/28/2017 12:34 AM, Bjorn Andersson wrote:
> In certain circumstances rpmsg devices needs to acquire a handle to the
> ancestor remoteproc instance, e.g. to invoke rproc_report_crash() when a
> fatal error is detected. Introduce an interface that walks the device
> tree in search for a remoteproc instance and return this.
Thanks for the patch. I also have a need to be able to get a rproc
handle from rpmsg device for some buffer translations. I have been using
a somewhat similar solution downstream and was able to use this patch
just fine.
>
> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> ---
> drivers/remoteproc/remoteproc_core.c | 18 ++++++++++++++++++
> include/linux/remoteproc.h | 2 ++
> 2 files changed, 20 insertions(+)
>
> diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c
> index 564061dcc019..5b1b19519275 100644
> --- a/drivers/remoteproc/remoteproc_core.c
> +++ b/drivers/remoteproc/remoteproc_core.c
> @@ -1296,6 +1296,23 @@ struct rproc *rproc_get_by_phandle(phandle phandle)
> EXPORT_SYMBOL(rproc_get_by_phandle);
>
> /**
> + * rproc_get_by_child() - acquire rproc handle of @dev's ancestor
> + * @dev: child device to find ancestor of
> + *
> + * Returns the ancestor rproc instance, or NULL if not found.
> + */
> +struct rproc *rproc_get_by_child(struct device *dev)
> +{
> + for (dev = dev->parent; dev; dev = dev->parent) {
> + if (dev->type && !strcmp(dev->type->name, "remoteproc"))
It's much simpler to just check the type pointer,
if (dev->type && dev->type == &rproc_type)
Tested both versions of the patch.
regards
Suman
> + return dev->driver_data;
> + }
> +
> + return NULL;
> +}
> +EXPORT_SYMBOL(rproc_get_by_child);
> +
> +/**
> * rproc_add() - register a remote processor
> * @rproc: the remote processor handle to register
> *
> @@ -1440,6 +1457,7 @@ struct rproc *rproc_alloc(struct device *dev, const char *name,
> rproc->dev.parent = dev;
> rproc->dev.type = &rproc_type;
> rproc->dev.class = &rproc_class;
> + rproc->dev.driver_data = rproc;
>
> /* Assign a unique device index and name */
> rproc->index = ida_simple_get(&rproc_dev_index, 0, 0, GFP_KERNEL);
> diff --git a/include/linux/remoteproc.h b/include/linux/remoteproc.h
> index 81da49564ff4..44e630eb3d94 100644
> --- a/include/linux/remoteproc.h
> +++ b/include/linux/remoteproc.h
> @@ -510,6 +510,8 @@ struct rproc_vdev {
> };
>
> struct rproc *rproc_get_by_phandle(phandle phandle);
> +struct rproc *rproc_get_by_child(struct device *dev);
> +
> struct rproc *rproc_alloc(struct device *dev, const char *name,
> const struct rproc_ops *ops,
> const char *firmware, int len);
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-09-01 21:00 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-28 5:34 [PATCH] remoteproc: Introduce rproc handle accessor for children Bjorn Andersson
2017-09-01 21:00 ` Suman Anna
2017-09-01 21:00 ` Suman Anna
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.