[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)
- 2012-11-22 · Re: [PATCH 1/4] OF: Add helper for matching against linux,stdout-path · Grant Likely <hidden>
- 2012-11-22 · Re: [PATCH 1/4] OF: Add helper for matching against linux,stdout-path · Jean-Christophe PLAGNIOL-VILLARD <hidden>
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.