From: Dong Aisheng <b29396@freescale.com>
To: devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org
Cc: grant.likely@linaro.org, rob.herring@calxeda.com,
rob@landley.net, linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] of: add device node status update APIs
Date: Thu, 15 Aug 2013 18:55:31 +0800 [thread overview]
Message-ID: <1376564133-11286-2-git-send-email-b29396@freescale.com> (raw)
In-Reply-To: <1376564133-11286-1-git-send-email-b29396@freescale.com>
Used for convineniently update the device node status.
Signed-off-by: Dong Aisheng <b29396@freescale.com>
---
drivers/of/base.c | 74 ++++++++++++++++++++++++++++++++++++++++++++++++++++
include/linux/of.h | 5 +++
2 files changed, 79 insertions(+), 0 deletions(-)
diff --git a/drivers/of/base.c b/drivers/of/base.c
index 5c54279..f944a54 100644
--- a/drivers/of/base.c
+++ b/drivers/of/base.c
@@ -1424,6 +1424,80 @@ int of_update_property(struct device_node *np, struct property *newprop)
return 0;
}
+/* update the node status property to be "enabled" or "disabled" */
+static int of_node_status_update(struct device_node *np, bool enable)
+{
+ struct property *oldprop;
+ struct property *newprop;
+ int ret;
+
+ if (of_device_is_available(np) == enable)
+ return 0;
+
+ oldprop = of_find_property(np, "status", NULL);
+
+ if (enable) {
+ /* for disabled node, there must be a status property with "disabled" state */
+ ret = of_remove_property(np, oldprop);
+ } else {
+ newprop = kzalloc(sizeof(*newprop), GFP_KERNEL);
+ if (!newprop)
+ return -ENOMEM;
+
+ newprop->name = kstrdup("status", GFP_KERNEL);
+ newprop->value = kstrdup("disabled", GFP_KERNEL);
+ newprop->length = 9;
+ if (!newprop->name || !newprop->value) {
+ kfree(newprop->name);
+ kfree(newprop->value);
+ kfree(newprop);
+ return -ENOMEM;
+ }
+
+ ret = of_update_property(np, newprop);
+ }
+
+ pr_debug("%s: %s --> %s %s\n", np->full_name, enable ?
+ "disabled" : "enabled", enable ? "enabled" : "disabled",
+ ret ? "failed" : "okay");
+
+ return ret;
+}
+
+int of_node_status_disable(struct device_node *np)
+{
+ return of_node_status_update(np, 0);
+}
+
+int of_node_status_enable(struct device_node *np)
+{
+ return of_node_status_update(np, 1);
+}
+
+int of_node_status_disable_by_path(const char *path)
+{
+ struct device_node *np;
+
+ pr_debug("disable node: %s\n", path);
+ np = of_find_node_by_path(path);
+ if (!np)
+ return -ENODEV;
+
+ return of_node_status_update(np, 0);
+}
+
+int of_node_status_enable_by_path(const char *path)
+{
+ struct device_node *np;
+
+ pr_debug("enable node: %s\n", path);
+ np = of_find_node_by_path(path);
+ if (!np)
+ return -ENODEV;
+
+ return of_node_status_update(np, 1);
+}
+
#if defined(CONFIG_OF_DYNAMIC)
/*
* Support for dynamic device trees.
diff --git a/include/linux/of.h b/include/linux/of.h
index 1fd08ca..61b35fe 100644
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@ -292,6 +292,11 @@ extern int of_add_property(struct device_node *np, struct property *prop);
extern int of_remove_property(struct device_node *np, struct property *prop);
extern int of_update_property(struct device_node *np, struct property *newprop);
+extern int of_node_status_disable(struct device_node *np);
+extern int of_node_status_enable(struct device_node *np);
+extern int of_node_status_disable_by_path(const char *path);
+extern int of_node_status_enable_by_path(const char *path);
+
/* For updating the device tree at runtime */
#define OF_RECONFIG_ATTACH_NODE 0x0001
#define OF_RECONFIG_DETACH_NODE 0x0002
--
1.7.1
next prev parent reply other threads:[~2013-08-15 10:55 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-15 10:55 [PATCH 0/3] of: add update device node status via cmdline feature Dong Aisheng
2013-08-15 10:55 ` Dong Aisheng [this message]
2013-08-15 10:55 ` [PATCH 2/3] " Dong Aisheng
2013-08-15 10:55 ` [PATCH 3/3] of: add node status update via name format with cmdline Dong Aisheng
2013-08-15 12:37 ` [PATCH 0/3] of: add update device node status via cmdline feature Rob Herring
2013-08-15 12:46 ` Grant Likely
2013-08-21 12:47 ` Dong Aisheng
2013-08-21 13:23 ` Rob Herring
2013-08-23 7:13 ` Dong Aisheng
2013-08-15 12:45 ` Grant Likely
2013-08-21 12:37 ` Dong Aisheng
2013-08-23 7:09 ` Dong Aisheng
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=1376564133-11286-2-git-send-email-b29396@freescale.com \
--to=b29396@freescale.com \
--cc=devicetree-discuss@lists.ozlabs.org \
--cc=grant.likely@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rob.herring@calxeda.com \
--cc=rob@landley.net \
/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).