From: Saravana Kannan <saravanak@google.com>
To: Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Frank Rowand <frowand.list@gmail.com>
Cc: Saravana Kannan <saravanak@google.com>,
David Collins <collinsd@codeaurora.org>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
kernel-team@android.com
Subject: [RESEND PATCH v1 1/5] of/platform: Speed up of_find_device_by_node()
Date: Mon, 3 Jun 2019 17:32:14 -0700 [thread overview]
Message-ID: <20190604003218.241354-2-saravanak@google.com> (raw)
In-Reply-To: <20190604003218.241354-1-saravanak@google.com>
Add a pointer from device tree node to the device created from it.
This allows us to find the device corresponding to a device tree node
without having to loop through all the platform devices.
However, fallback to looping through the platform devices to handle
any devices that might set their own of_node.
Signed-off-by: Saravana Kannan <saravanak@google.com>
---
drivers/of/platform.c | 20 +++++++++++++++++++-
include/linux/of.h | 3 +++
2 files changed, 22 insertions(+), 1 deletion(-)
diff --git a/drivers/of/platform.c b/drivers/of/platform.c
index 04ad312fd85b..1115a8d80a33 100644
--- a/drivers/of/platform.c
+++ b/drivers/of/platform.c
@@ -42,6 +42,8 @@ static int of_dev_node_match(struct device *dev, void *data)
return dev->of_node == data;
}
+static DEFINE_SPINLOCK(of_dev_lock);
+
/**
* of_find_device_by_node - Find the platform_device associated with a node
* @np: Pointer to device tree node
@@ -55,7 +57,18 @@ struct platform_device *of_find_device_by_node(struct device_node *np)
{
struct device *dev;
- dev = bus_find_device(&platform_bus_type, NULL, np, of_dev_node_match);
+ /*
+ * Spinlock needed to make sure np->dev doesn't get freed between NULL
+ * check inside and kref count increment inside get_device(). This is
+ * achieved by grabbing the spinlock before setting np->dev = NULL in
+ * of_platform_device_destroy().
+ */
+ spin_lock(&of_dev_lock);
+ dev = get_device(np->dev);
+ spin_unlock(&of_dev_lock);
+ if (!dev)
+ dev = bus_find_device(&platform_bus_type, NULL, np,
+ of_dev_node_match);
return dev ? to_platform_device(dev) : NULL;
}
EXPORT_SYMBOL(of_find_device_by_node);
@@ -196,6 +209,7 @@ static struct platform_device *of_platform_device_create_pdata(
platform_device_put(dev);
goto err_clear_flag;
}
+ np->dev = &dev->dev;
return dev;
@@ -556,6 +570,10 @@ int of_platform_device_destroy(struct device *dev, void *data)
if (of_node_check_flag(dev->of_node, OF_POPULATED_BUS))
device_for_each_child(dev, NULL, of_platform_device_destroy);
+ /* Spinlock is needed for of_find_device_by_node() to work */
+ spin_lock(&of_dev_lock);
+ dev->of_node->dev = NULL;
+ spin_unlock(&of_dev_lock);
of_node_clear_flag(dev->of_node, OF_POPULATED);
of_node_clear_flag(dev->of_node, OF_POPULATED_BUS);
diff --git a/include/linux/of.h b/include/linux/of.h
index 0cf857012f11..f2b4912cbca1 100644
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@ -48,6 +48,8 @@ struct property {
struct of_irq_controller;
#endif
+struct device;
+
struct device_node {
const char *name;
phandle phandle;
@@ -68,6 +70,7 @@ struct device_node {
unsigned int unique_id;
struct of_irq_controller *irq_trans;
#endif
+ struct device *dev; /* Device created from this node */
};
#define MAX_PHANDLE_ARGS 16
--
2.22.0.rc1.257.g3120a18244-goog
next prev parent reply other threads:[~2019-06-04 0:32 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-04 0:32 [RESEND PATCH v1 0/5] Solve postboot supplier cleanup and optimize probe ordering Saravana Kannan
2019-06-04 0:32 ` Saravana Kannan [this message]
2019-06-10 17:36 ` [RESEND PATCH v1 1/5] of/platform: Speed up of_find_device_by_node() Rob Herring
[not found] ` <CAGETcx_Kb3+fFYHhBUdVbCSNTk4v5j1Ket=Et2ZQY0SHUgLLMw@mail.gmail.com>
2019-06-10 21:07 ` Rob Herring
2019-06-11 15:18 ` Frank Rowand
2019-06-11 20:56 ` Saravana Kannan
2019-06-11 21:52 ` Sandeep Patil
2019-06-12 13:53 ` Rob Herring
2019-06-12 14:21 ` Greg Kroah-Hartman
2019-06-12 16:53 ` Rob Herring
2019-06-12 17:08 ` Greg Kroah-Hartman
2019-06-12 18:19 ` Rob Herring
2019-06-12 19:29 ` Saravana Kannan
2019-06-12 20:23 ` Frank Rowand
2019-06-12 21:12 ` Rob Herring
2019-06-12 22:10 ` Saravana Kannan
2019-06-18 20:47 ` Sandeep Patil
2019-06-18 21:22 ` Saravana Kannan
2019-06-12 17:03 ` Frank Rowand
2019-06-12 16:07 ` Frank Rowand
2019-06-12 16:47 ` Frank Rowand
2019-06-12 19:03 ` Frank Rowand
2019-06-04 0:32 ` [RESEND PATCH v1 2/5] driver core: Add device links support for pending links to suppliers Saravana Kannan
2019-06-04 0:32 ` [RESEND PATCH v1 3/5] dt-bindings: Add depends-on property Saravana Kannan
2019-06-12 14:45 ` Sudeep Holla
2019-06-04 0:32 ` [RESEND PATCH v1 4/5] of/platform: Add functional dependency link from "depends-on" property Saravana Kannan
2019-06-04 0:32 ` [RESEND PATCH v1 5/5] driver core: Add sync_state driver/bus callback Saravana Kannan
2019-06-12 21:21 ` [RESEND PATCH v1 0/5] Solve postboot supplier cleanup and optimize probe ordering Frank Rowand
2019-06-13 13:19 ` Rob Herring
2019-06-24 22:37 ` Sandeep Patil
2019-06-25 3:53 ` Greg Kroah-Hartman
2019-06-26 4:30 ` Sandeep Patil
2019-06-26 5:49 ` Frank Rowand
2019-06-26 21:30 ` Rob Herring
2019-06-28 2:36 ` Saravana Kannan
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=20190604003218.241354-2-saravanak@google.com \
--to=saravanak@google.com \
--cc=collinsd@codeaurora.org \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=kernel-team@android.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=rafael@kernel.org \
--cc=robh+dt@kernel.org \
/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).