Thread (7 messages) 7 messages, 5 authors, 2012-11-22

[PATCH 1/4] OF: Add helper for matching against linux, stdout-path

From: Grant Likely <hidden>
Date: 2012-11-21 18:03:29
Also in: linux-devicetree, linux-serial

Possibly related (same subject, not in this thread)

On Wed, Nov 21, 2012 at 3:57 PM, Jean-Christophe PLAGNIOL-VILLARD
[off-list ref] wrote:
From: Sascha Hauer <s.hauer@pengutronix.de>

devicetrees may have a linux,stdout-path or stdout-path property
in the chosen node describing the console device. This adds a helper
function to match a device against this property and retrieve the options
so a driver can call add_preferred_console for a matching device.
NIce. Looks like the right behaviour, but is a little inefficient.
Since the stdout property doesn't change this code could be called
once right after the tree is unflattened and then cache the pointer to
the node and arguments. Can you add the code to of_alias_scan()?

g.
quoted hunk ↗ jump to hunk
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Signed-off-by: Jean-Christophe PLAGNIOL-VILLARD <redacted>
Cc: linux-serial at vger.kernel.org
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: kernel at pengutronix.de
Cc: Alan Cox <redacted>
Cc: Grant Likely <redacted>
Cc: Nicolas Ferre <redacted>
---
 drivers/of/base.c  |   46 ++++++++++++++++++++++++++++++++++++++++++++++
 include/linux/of.h |    7 +++++++
 2 files changed, 53 insertions(+)
diff --git a/drivers/of/base.c b/drivers/of/base.c
index f2f63c8..72c49ce 100644
--- a/drivers/of/base.c
+++ b/drivers/of/base.c
@@ -1470,3 +1470,49 @@ const char *of_prop_next_string(struct property *prop, const char *cur)
        return curv;
 }
 EXPORT_SYMBOL_GPL(of_prop_next_string);
+
+/**
+ * of_device_is_stdout_path - check if a device node matches the
+ *                           linux,stdout-path property
+ * @np:                Pointer to the given device_node
+ * @option:    parsed option
+ *
+ * Check if this device node matches the linux,stdout-path property
+ * in the chosen node. return true if yes, false otherwise.
+ */
+int of_device_is_stdout_path(struct device_node *dn, char **option)
+{
+       const char *name;
+       struct device_node *dn_stdout;
+       bool is_stdout = 0;
+       const char *tmp;
+       const char *tmp_option;
+
+       name = of_get_property(of_chosen, "linux,stdout-path", NULL);
+       if (name == NULL)
+               name = of_get_property(of_chosen, "stdout-path", NULL);
+
+       if (name == NULL)
+               return 0;
+
+       tmp_option = strchr(name, ':');
+
+       tmp = kstrndup(name, strlen(name) - strlen(tmp_option), GFP_KERNEL);
+       if (!tmp)
+               return 0;
+
+       dn_stdout = of_find_node_by_path(tmp);
+
+       if (dn_stdout && dn_stdout == dn) {
+               is_stdout = 1;
+               tmp_option++;
+               *option = kstrdup(tmp_option, GFP_KERNEL);
+       }
+
+       of_node_put(dn_stdout);
+
+       kfree(tmp);
+
+       return is_stdout;
+}
+EXPORT_SYMBOL_GPL(of_device_is_stdout_path);
diff --git a/include/linux/of.h b/include/linux/of.h
index 681a6c8..6a82e3f 100644
--- a/include/linux/of.h
+++ b/include/linux/of.h
@@ -323,6 +323,8 @@ const char *of_prop_next_string(struct property *prop, const char *cur);
                s;                                              \
                s = of_prop_next_string(prop, s))

+int of_device_is_stdout_path(struct device_node *dn, char **option);
+
 #else /* CONFIG_OF */

 static inline const char* of_node_full_name(struct device_node *np)
@@ -450,6 +452,11 @@ static inline int of_machine_is_compatible(const char *compat)
        return 0;
 }

+static inline int of_device_is_stdout_path(struct device_node *dn, char **option)
+{
+       return 0;
+}
+
 #define of_match_ptr(_ptr)     NULL
 #define of_match_node(_matches, _node) NULL
 #define of_property_for_each_u32(np, propname, prop, p, u) \
--
1.7.10.4


--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help