linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: ppc-dev <linuxppc-dev@ozlabs.org>,
	paulus@samba.org, "David S. Miller" <davem@davemloft.net>
Subject: Re: [PATCH 6/6] Consolidate of_find_node_by routines
Date: Wed, 25 Apr 2007 10:01:52 +1000	[thread overview]
Message-ID: <1177459313.14873.157.camel@localhost.localdomain> (raw)
In-Reply-To: <20070424224349.e760efe2.sfr@canb.auug.org.au>

On Tue, 2007-04-24 at 22:43 +1000, Stephen Rothwell wrote:
> This consolidates the routines of_find_node_by_path, of_find_node_by_name,
> of_find_node_by_type and of_find_compatible_device.  Again, the comparison
> of strings are done differently by Sparc and PowerPC and also these add
> readlocks around the iterations.

Same question about how should we consolidate the string comparisons ?

In addition, we should really have a more intelligent path walker than
supports going down one level by name, unitaddress or both.

Ben.

> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
>  arch/powerpc/kernel/prom.c |  113 +-------------------------------------------
>  arch/sparc/kernel/prom.c   |   61 +-----------------------
>  arch/sparc64/kernel/prom.c |   61 +-----------------------
>  drivers/openfw/base.c      |  114 ++++++++++++++++++++++++++++++++++++++++++++
>  include/asm-powerpc/prom.h |    1 +
>  include/asm-sparc/prom.h   |    1 +
>  include/asm-sparc64/prom.h |    1 +
>  7 files changed, 120 insertions(+), 232 deletions(-)
> 
> -- 
> Cheers,
> Stephen Rothwell                    sfr@canb.auug.org.au
> 
> diff --git a/arch/powerpc/kernel/prom.c b/arch/powerpc/kernel/prom.c
> index 90901e4..fc64bcb 100644
> --- a/arch/powerpc/kernel/prom.c
> +++ b/arch/powerpc/kernel/prom.c
> @@ -77,7 +77,7 @@ static struct boot_param_header *initial_boot_params __initdata;
>  struct boot_param_header *initial_boot_params;
>  #endif
>  
> -static struct device_node *allnodes = NULL;
> +extern struct device_node *allnodes;	/* temporary while merging */
>  
>  extern rwlock_t devtree_lock;	/* temporary while merging */
>  
> @@ -1068,117 +1068,6 @@ EXPORT_SYMBOL(machine_is_compatible);
>   *
>   *******/
>  
> -/**
> - *	of_find_node_by_name - Find a node by its "name" property
> - *	@from:	The node to start searching from or NULL, the node
> - *		you pass will not be searched, only the next one
> - *		will; typically, you pass what the previous call
> - *		returned. of_node_put() will be called on it
> - *	@name:	The name string to match against
> - *
> - *	Returns a node pointer with refcount incremented, use
> - *	of_node_put() on it when done.
> - */
> -struct device_node *of_find_node_by_name(struct device_node *from,
> -	const char *name)
> -{
> -	struct device_node *np;
> -
> -	read_lock(&devtree_lock);
> -	np = from ? from->allnext : allnodes;
> -	for (; np != NULL; np = np->allnext)
> -		if (np->name != NULL && strcasecmp(np->name, name) == 0
> -		    && of_node_get(np))
> -			break;
> -	of_node_put(from);
> -	read_unlock(&devtree_lock);
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_name);
> -
> -/**
> - *	of_find_node_by_type - Find a node by its "device_type" property
> - *	@from:	The node to start searching from or NULL, the node
> - *		you pass will not be searched, only the next one
> - *		will; typically, you pass what the previous call
> - *		returned. of_node_put() will be called on it
> - *	@name:	The type string to match against
> - *
> - *	Returns a node pointer with refcount incremented, use
> - *	of_node_put() on it when done.
> - */
> -struct device_node *of_find_node_by_type(struct device_node *from,
> -	const char *type)
> -{
> -	struct device_node *np;
> -
> -	read_lock(&devtree_lock);
> -	np = from ? from->allnext : allnodes;
> -	for (; np != 0; np = np->allnext)
> -		if (np->type != 0 && strcasecmp(np->type, type) == 0
> -		    && of_node_get(np))
> -			break;
> -	of_node_put(from);
> -	read_unlock(&devtree_lock);
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_type);
> -
> -/**
> - *	of_find_compatible_node - Find a node based on type and one of the
> - *                                tokens in its "compatible" property
> - *	@from:		The node to start searching from or NULL, the node
> - *			you pass will not be searched, only the next one
> - *			will; typically, you pass what the previous call
> - *			returned. of_node_put() will be called on it
> - *	@type:		The type string to match "device_type" or NULL to ignore
> - *	@compatible:	The string to match to one of the tokens in the device
> - *			"compatible" list.
> - *
> - *	Returns a node pointer with refcount incremented, use
> - *	of_node_put() on it when done.
> - */
> -struct device_node *of_find_compatible_node(struct device_node *from,
> -	const char *type, const char *compatible)
> -{
> -	struct device_node *np;
> -
> -	read_lock(&devtree_lock);
> -	np = from ? from->allnext : allnodes;
> -	for (; np != 0; np = np->allnext) {
> -		if (type != NULL
> -		    && !(np->type != 0 && strcasecmp(np->type, type) == 0))
> -			continue;
> -		if (of_device_is_compatible(np, compatible) && of_node_get(np))
> -			break;
> -	}
> -	of_node_put(from);
> -	read_unlock(&devtree_lock);
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_compatible_node);
> -
> -/**
> - *	of_find_node_by_path - Find a node matching a full OF path
> - *	@path:	The full path to match
> - *
> - *	Returns a node pointer with refcount incremented, use
> - *	of_node_put() on it when done.
> - */
> -struct device_node *of_find_node_by_path(const char *path)
> -{
> -	struct device_node *np = allnodes;
> -
> -	read_lock(&devtree_lock);
> -	for (; np != 0; np = np->allnext) {
> -		if (np->full_name != 0 && strcasecmp(np->full_name, path) == 0
> -		    && of_node_get(np))
> -			break;
> -	}
> -	read_unlock(&devtree_lock);
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_path);
>  
>  /**
>   *	of_find_node_by_phandle - Find a node given a phandle
> diff --git a/arch/sparc/kernel/prom.c b/arch/sparc/kernel/prom.c
> index 3f8ccfa..012f983 100644
> --- a/arch/sparc/kernel/prom.c
> +++ b/arch/sparc/kernel/prom.c
> @@ -25,23 +25,10 @@
>  #include <asm/prom.h>
>  #include <asm/oplib.h>
>  
> -static struct device_node *allnodes;
> +extern struct device_node *allnodes;	/* temporary while merging */
>  
>  extern rwlock_t devtree_lock;	/* temporary while merging */
>  
> -struct device_node *of_find_node_by_path(const char *path)
> -{
> -	struct device_node *np = allnodes;
> -
> -	for (; np != 0; np = np->allnext) {
> -		if (np->full_name != 0 && strcmp(np->full_name, path) == 0)
> -			break;
> -	}
> -
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_path);
> -
>  struct device_node *of_find_node_by_phandle(phandle handle)
>  {
>  	struct device_node *np;
> @@ -54,52 +41,6 @@ struct device_node *of_find_node_by_phandle(phandle handle)
>  }
>  EXPORT_SYMBOL(of_find_node_by_phandle);
>  
> -struct device_node *of_find_node_by_name(struct device_node *from,
> -	const char *name)
> -{
> -	struct device_node *np;
> -
> -	np = from ? from->allnext : allnodes;
> -	for (; np != NULL; np = np->allnext)
> -		if (np->name != NULL && strcmp(np->name, name) == 0)
> -			break;
> -
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_name);
> -
> -struct device_node *of_find_node_by_type(struct device_node *from,
> -	const char *type)
> -{
> -	struct device_node *np;
> -
> -	np = from ? from->allnext : allnodes;
> -	for (; np != 0; np = np->allnext)
> -		if (np->type != 0 && strcmp(np->type, type) == 0)
> -			break;
> -
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_type);
> -
> -struct device_node *of_find_compatible_node(struct device_node *from,
> -	const char *type, const char *compatible)
> -{
> -	struct device_node *np;
> -
> -	np = from ? from->allnext : allnodes;
> -	for (; np != 0; np = np->allnext) {
> -		if (type != NULL
> -		    && !(np->type != 0 && strcmp(np->type, type) == 0))
> -			continue;
> -		if (of_device_is_compatible(np, compatible))
> -			break;
> -	}
> -
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_compatible_node);
> -
>  int of_getintprop_default(struct device_node *np, const char *name, int def)
>  {
>  	struct property *prop;
> diff --git a/arch/sparc64/kernel/prom.c b/arch/sparc64/kernel/prom.c
> index be3b958..ae2065c 100644
> --- a/arch/sparc64/kernel/prom.c
> +++ b/arch/sparc64/kernel/prom.c
> @@ -29,23 +29,10 @@
>  #include <asm/asi.h>
>  #include <asm/upa.h>
>  
> -static struct device_node *allnodes;
> +extern struct device_node *allnodes;	/* temporary while merging */
>  
>  extern rwlock_t devtree_lock;	/* temporary while merging */
>  
> -struct device_node *of_find_node_by_path(const char *path)
> -{
> -	struct device_node *np = allnodes;
> -
> -	for (; np != 0; np = np->allnext) {
> -		if (np->full_name != 0 && strcmp(np->full_name, path) == 0)
> -			break;
> -	}
> -
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_path);
> -
>  struct device_node *of_find_node_by_phandle(phandle handle)
>  {
>  	struct device_node *np;
> @@ -58,52 +45,6 @@ struct device_node *of_find_node_by_phandle(phandle handle)
>  }
>  EXPORT_SYMBOL(of_find_node_by_phandle);
>  
> -struct device_node *of_find_node_by_name(struct device_node *from,
> -	const char *name)
> -{
> -	struct device_node *np;
> -
> -	np = from ? from->allnext : allnodes;
> -	for (; np != NULL; np = np->allnext)
> -		if (np->name != NULL && strcmp(np->name, name) == 0)
> -			break;
> -
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_name);
> -
> -struct device_node *of_find_node_by_type(struct device_node *from,
> -	const char *type)
> -{
> -	struct device_node *np;
> -
> -	np = from ? from->allnext : allnodes;
> -	for (; np != 0; np = np->allnext)
> -		if (np->type != 0 && strcmp(np->type, type) == 0)
> -			break;
> -
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_node_by_type);
> -
> -struct device_node *of_find_compatible_node(struct device_node *from,
> -	const char *type, const char *compatible)
> -{
> -	struct device_node *np;
> -
> -	np = from ? from->allnext : allnodes;
> -	for (; np != 0; np = np->allnext) {
> -		if (type != NULL
> -		    && !(np->type != 0 && strcmp(np->type, type) == 0))
> -			continue;
> -		if (of_device_is_compatible(np, compatible))
> -			break;
> -	}
> -
> -	return np;
> -}
> -EXPORT_SYMBOL(of_find_compatible_node);
> -
>  int of_getintprop_default(struct device_node *np, const char *name, int def)
>  {
>  	struct property *prop;
> diff --git a/drivers/openfw/base.c b/drivers/openfw/base.c
> index 60f7bd4..bb66e48 100644
> --- a/drivers/openfw/base.c
> +++ b/drivers/openfw/base.c
> @@ -20,6 +20,8 @@
>  #include <linux/openfw.h>
>  #include <linux/spinlock.h>
>  
> +struct device_node *allnodes;
> +
>  /* use when traversing tree through the allnext, child, sibling,
>   * or parent members of struct device_node.
>   */
> @@ -158,3 +160,115 @@ struct device_node *of_get_next_child(const struct device_node *node,
>  	return next;
>  }
>  EXPORT_SYMBOL(of_get_next_child);
> +
> +/**
> + *	of_find_node_by_path - Find a node matching a full OF path
> + *	@path:	The full path to match
> + *
> + *	Returns a node pointer with refcount incremented, use
> + *	of_node_put() on it when done.
> + */
> +struct device_node *of_find_node_by_path(const char *path)
> +{
> +	struct device_node *np = allnodes;
> +
> +	read_lock(&devtree_lock);
> +	for (; np; np = np->allnext) {
> +		if (np->full_name && (of_node_cmp(np->full_name, path) == 0)
> +		    && of_node_get(np))
> +			break;
> +	}
> +	read_unlock(&devtree_lock);
> +	return np;
> +}
> +EXPORT_SYMBOL(of_find_node_by_path);
> +
> +/**
> + *	of_find_node_by_name - Find a node by its "name" property
> + *	@from:	The node to start searching from or NULL, the node
> + *		you pass will not be searched, only the next one
> + *		will; typically, you pass what the previous call
> + *		returned. of_node_put() will be called on it
> + *	@name:	The name string to match against
> + *
> + *	Returns a node pointer with refcount incremented, use
> + *	of_node_put() on it when done.
> + */
> +struct device_node *of_find_node_by_name(struct device_node *from,
> +	const char *name)
> +{
> +	struct device_node *np;
> +
> +	read_lock(&devtree_lock);
> +	np = from ? from->allnext : allnodes;
> +	for (; np; np = np->allnext)
> +		if (np->name && (of_node_cmp(np->name, name) == 0)
> +		    && of_node_get(np))
> +			break;
> +	of_node_put(from);
> +	read_unlock(&devtree_lock);
> +	return np;
> +}
> +EXPORT_SYMBOL(of_find_node_by_name);
> +
> +/**
> + *	of_find_node_by_type - Find a node by its "device_type" property
> + *	@from:	The node to start searching from or NULL, the node
> + *		you pass will not be searched, only the next one
> + *		will; typically, you pass what the previous call
> + *		returned. of_node_put() will be called on it
> + *	@name:	The type string to match against
> + *
> + *	Returns a node pointer with refcount incremented, use
> + *	of_node_put() on it when done.
> + */
> +struct device_node *of_find_node_by_type(struct device_node *from,
> +	const char *type)
> +{
> +	struct device_node *np;
> +
> +	read_lock(&devtree_lock);
> +	np = from ? from->allnext : allnodes;
> +	for (; np; np = np->allnext)
> +		if (np->type && (of_node_cmp(np->type, type) == 0)
> +		    && of_node_get(np))
> +			break;
> +	of_node_put(from);
> +	read_unlock(&devtree_lock);
> +	return np;
> +}
> +EXPORT_SYMBOL(of_find_node_by_type);
> +
> +/**
> + *	of_find_compatible_node - Find a node based on type and one of the
> + *                                tokens in its "compatible" property
> + *	@from:		The node to start searching from or NULL, the node
> + *			you pass will not be searched, only the next one
> + *			will; typically, you pass what the previous call
> + *			returned. of_node_put() will be called on it
> + *	@type:		The type string to match "device_type" or NULL to ignore
> + *	@compatible:	The string to match to one of the tokens in the device
> + *			"compatible" list.
> + *
> + *	Returns a node pointer with refcount incremented, use
> + *	of_node_put() on it when done.
> + */
> +struct device_node *of_find_compatible_node(struct device_node *from,
> +	const char *type, const char *compatible)
> +{
> +	struct device_node *np;
> +
> +	read_lock(&devtree_lock);
> +	np = from ? from->allnext : allnodes;
> +	for (; np; np = np->allnext) {
> +		if (type
> +		    && !(np->type && (of_node_cmp(np->type, type) == 0)))
> +			continue;
> +		if (of_device_is_compatible(np, compatible) && of_node_get(np))
> +			break;
> +	}
> +	of_node_put(from);
> +	read_unlock(&devtree_lock);
> +	return np;
> +}
> +EXPORT_SYMBOL(of_find_compatible_node);
> diff --git a/include/asm-powerpc/prom.h b/include/asm-powerpc/prom.h
> index a0679be..d0ff917 100644
> --- a/include/asm-powerpc/prom.h
> +++ b/include/asm-powerpc/prom.h
> @@ -26,6 +26,7 @@
>  
>  #define of_compat_cmp(s1, s2, l)	strncasecmp((s1), (s2), (l))
>  #define of_prop_cmp(s1, s2)		strcmp((s1), (s2))
> +#define of_node_cmp(s1, s2)		strcasecmp((s1), (s2))
>  
>  /* Definitions used by the flattened device tree */
>  #define OF_DT_HEADER		0xd00dfeed	/* marker */
> diff --git a/include/asm-sparc/prom.h b/include/asm-sparc/prom.h
> index 4621af6..92919ac 100644
> --- a/include/asm-sparc/prom.h
> +++ b/include/asm-sparc/prom.h
> @@ -25,6 +25,7 @@
>  
>  #define of_compat_cmp(s1, s2, l)	strncmp((s1), (s2), (l))
>  #define of_prop_cmp(s1, s2)		strcasecmp((s1), (s2))
> +#define of_node_cmp(s1, s2)		strcmp((s1), (s2))
>  
>  typedef u32 phandle;
>  typedef u32 ihandle;
> diff --git a/include/asm-sparc64/prom.h b/include/asm-sparc64/prom.h
> index c36b884..d2123c2 100644
> --- a/include/asm-sparc64/prom.h
> +++ b/include/asm-sparc64/prom.h
> @@ -25,6 +25,7 @@
>  
>  #define of_compat_cmp(s1, s2, l)	strncmp((s1), (s2), (l))
>  #define of_prop_cmp(s1, s2)		strcasecmp((s1), (s2))
> +#define of_node_cmp(s1, s2)		strcmp((s1), (s2))
>  
>  typedef u32 phandle;
>  typedef u32 ihandle;

  reply	other threads:[~2007-04-25  0:02 UTC|newest]

Thread overview: 109+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-03  0:52 [PATCH 1/6] [POWERPC] get_property returns const Stephen Rothwell
2007-04-03  0:54 ` [PATCH 2/6] [POWERPC] Rename get_property to of_get_property Stephen Rothwell
2007-04-03  0:55   ` [PATCH 3/6] [POWERPC] Rename device_is_compatible to of_device_is_compatible Stephen Rothwell
2007-04-03  0:56     ` [PATCH 4/6] [POWERPC] Rename prom_n_addr_cells to of_n_addr_cells Stephen Rothwell
2007-04-03  0:57       ` [PATCH 5/6] [POWERPC] Rename prom_n_size_cells to of_n_size_cells Stephen Rothwell
2007-04-03  0:58         ` [PATCH 6/6] [POWERPC] make struct property's value a void * Stephen Rothwell
2007-04-03  3:37           ` David Miller
2007-04-03 15:40             ` Segher Boessenkool
2007-04-03 12:24           ` [PATCH 01/17] [POWERPC] Rename get_property to of_get_property: include Stephen Rothwell
2007-04-03 12:26             ` [PATCH 02/17] [POWERPC] Rename get_property to of_get_property: arch/powerpc/kernel Stephen Rothwell
2007-04-03 12:28               ` [PATCH 03/17] [POWERPC] Rename get_property to of_get_property: arch/powerpc/mm Stephen Rothwell
2007-04-03 12:30                 ` [PATCH 04/17] [POWERPC] Rename get_property to of_get_property: arch/powerpc/sysdev Stephen Rothwell
2007-04-03 12:31                   ` [PATCH 05/17] [POWERPC] Rename get_property to of_get_property: arch/powerpc/platforms/pseries Stephen Rothwell
2007-04-03 12:32                     ` [PATCH 06/17] [POWERPC] Rename get_property to of_get_property: arch/powerpc/platforms/powermac Stephen Rothwell
2007-04-03 12:35                       ` [PATCH 07/17] [POWERPC] Rename get_property to of_get_property: arch/powerpc/platforms/cell Stephen Rothwell
2007-04-03 12:37                         ` [PATCH 08/17] [POWERPC] Rename get_property to of_get_property: arch/powerpc/platforms Stephen Rothwell
2007-04-03 12:39                           ` [PATCH 09/17] [POWERPC] Rename get_property to of_get_property: sound Stephen Rothwell
2007-04-03 12:40                             ` [PATCH 10/17] [POWERPC] Rename get_property to of_get_property: drivers/macintosh Stephen Rothwell
2007-04-03 12:42                               ` [PATCH 11/17] [POWERPC] Rename get_property to of_get_property: drivers/char Stephen Rothwell
2007-04-03 12:43                                 ` [PATCH 12/17] [POWERPC] Rename get_property to of_get_property: drivers/net Stephen Rothwell
2007-04-03 12:45                                   ` [PATCH 13/17] [POWERPC] Rename get_property to of_get_property: drivers/video Stephen Rothwell
2007-04-03 12:46                                     ` [PATCH 14/17] [POWERPC] Rename get_property to of_get_property: drivers/scsi Stephen Rothwell
2007-04-03 12:49                                       ` [PATCH 15/17] [POWERPC] Rename get_property to of_get_property: drivers Stephen Rothwell
2007-04-03 12:50                                         ` [PATCH 16/17] [POWERPC] Rename get_property to of_get_property: the last one Stephen Rothwell
2007-04-03 12:52                                           ` [PATCH 17/17] [POWERPC] Remove get_property Stephen Rothwell
2007-04-03 13:05                                             ` [PATCH] [SPARC/64] " Stephen Rothwell
2007-04-03 13:24                                               ` [PATCH] [SPARC/64] Make device_node name and type const Stephen Rothwell
2007-04-12  4:19                                                 ` [PATCH,RFC] Split out common parts of prom.h Stephen Rothwell
2007-04-12  4:25                                                   ` David Miller
2007-04-12 16:13                                                     ` Kumar Gala
2007-04-12 19:19                                                       ` Segher Boessenkool
2007-04-12  5:34                                                   ` [PATCH] Start split out of common open firmware code Stephen Rothwell
     [not found]                                                     ` <20070424223245.78f4fdfb.sfr@canb.auug .org.au>
2007-04-12  5:41                                                     ` David Miller
2007-04-12  5:50                                                     ` Benjamin Herrenschmidt
2007-04-12  6:25                                                       ` David Miller
2007-04-12  6:36                                                         ` Benjamin Herrenschmidt
2007-04-12  6:40                                                           ` David Miller
2007-04-12  7:00                                                         ` Segher Boessenkool
2007-04-23  7:43                                                           ` Stephen Rothwell
2007-04-23 18:30                                                             ` Segher Boessenkool
2007-04-24 12:32                                                     ` [PATCH 1/6] " Stephen Rothwell
2007-04-24 12:38                                                       ` [PATCH 2/6] Consolidate of_device_is_compatible Stephen Rothwell
2007-04-24 13:36                                                         ` Segher Boessenkool
2007-04-24 23:59                                                         ` Benjamin Herrenschmidt
2007-04-25  4:23                                                           ` David Miller
2007-04-24 12:39                                                       ` [PATCH 3/6] Consolidate of_find_property Stephen Rothwell
2007-04-24 13:41                                                         ` Segher Boessenkool
2007-04-25  1:43                                                           ` Paul Mackerras
2007-04-25  2:10                                                             ` Segher Boessenkool
2007-04-25  4:31                                                             ` David Miller
2007-04-25  5:07                                                             ` Benjamin Herrenschmidt
2007-04-25 13:13                                                               ` Segher Boessenkool
2007-04-25 17:46                                                                 ` Matt Sealey
2007-04-25 19:02                                                                   ` Segher Boessenkool
2007-04-25 22:16                                                                   ` Benjamin Herrenschmidt
2007-04-25 22:24                                                                     ` David Miller
2007-04-25 22:47                                                                       ` Benjamin Herrenschmidt
2007-04-25 23:13                                                                         ` David Miller
2007-04-25 23:18                                                                           ` Segher Boessenkool
2007-04-25 23:28                                                                             ` David Miller
2007-04-25 23:21                                                                           ` Benjamin Herrenschmidt
2007-04-26  5:54                                                                           ` Stephen Rothwell
2007-04-26  5:58                                                                             ` David Miller
2007-04-25  0:00                                                         ` Benjamin Herrenschmidt
2007-04-25  0:43                                                           ` Segher Boessenkool
2007-04-25  1:12                                                             ` Benjamin Herrenschmidt
2007-04-25  2:03                                                               ` Segher Boessenkool
2007-04-25  4:24                                                           ` David Miller
2007-04-24 12:40                                                       ` [PATCH 4/6] Consolidate of_get_parent Stephen Rothwell
2007-04-24 14:42                                                         ` Loeliger Jon-LOELIGER
2007-04-26  1:34                                                           ` Stephen Rothwell
2007-04-24 12:42                                                       ` [PATCH 5/6] Consolidate of_get_next_child Stephen Rothwell
2007-04-24 12:43                                                       ` [PATCH 6/6] Consolidate of_find_node_by routines Stephen Rothwell
2007-04-25  0:01                                                         ` Benjamin Herrenschmidt [this message]
2007-04-24 13:27                                                       ` [PATCH 1/6] Start split out of common open firmware code Segher Boessenkool
2007-04-24 14:47                                                         ` Loeliger Jon-LOELIGER
2007-04-24 18:04                                                           ` David Miller
2007-04-24 18:18                                                             ` Segher Boessenkool
2007-04-24 19:14                                                               ` Jon Loeliger
2007-04-25  0:07                                                             ` Benjamin Herrenschmidt
2007-04-25  1:03                                                               ` Segher Boessenkool
2007-04-25  4:29                                                                 ` David Miller
2007-04-25 13:11                                                                   ` Segher Boessenkool
2007-04-25  0:03                                                         ` Benjamin Herrenschmidt
2007-04-25  0:50                                                           ` Segher Boessenkool
2007-04-25  1:51                                                             ` Josh Boyer
2007-04-25  2:07                                                               ` Segher Boessenkool
2007-04-25  4:27                                                             ` David Miller
2007-04-25 13:07                                                               ` Segher Boessenkool
2007-04-24 23:35                                                       ` David Miller
2007-04-12  6:58                                                   ` [PATCH,RFC] Split out common parts of prom.h Segher Boessenkool
2007-04-12 21:41                                                 ` [PATCH] [SPARC/64] Make device_node name and type const David Miller
2007-04-12 21:42                                               ` [PATCH] [SPARC/64] Remove get_property David Miller
2007-04-27  3:41                                 ` [PATCH] [POWERPC] Rename get_property to of_get_property: partial drivers Stephen Rothwell
2007-04-03 17:36             ` [PATCH 01/17] [POWERPC] Rename get_property to of_get_property: include Olof Johansson
2007-04-05  1:49             ` Benjamin Herrenschmidt
2007-04-03 15:39           ` [PATCH 6/6] [POWERPC] make struct property's value a void * Segher Boessenkool
2007-04-03 18:28             ` David Miller
2007-04-04 11:11               ` Segher Boessenkool
2007-04-04 19:21                 ` David Miller
2007-04-05  1:48                   ` Benjamin Herrenschmidt
2007-04-05 18:12                   ` Segher Boessenkool
2007-04-05 23:01                     ` Benjamin Herrenschmidt
2007-04-05 23:12                       ` David Miller
2007-04-05 23:26                       ` Segher Boessenkool
2007-04-09 18:20                         ` Paul Mackerras
2007-04-09 22:11                           ` Segher Boessenkool
2007-04-09 23:17                           ` David Miller
2007-04-27  4:32 ` [PATCH] Partialially revert a7edd0e676d51145ae634a2acf7a447e319200fa Stephen Rothwell

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=1177459313.14873.157.camel@localhost.localdomain \
    --to=benh@kernel.crashing.org \
    --cc=davem@davemloft.net \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.org \
    --cc=sfr@canb.auug.org.au \
    /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).