[PATCH 3/3] powerpc: setup_64 comment cleanup.

Michael Ellerman michael at ellerman.id.au
Fri Sep 7 09:54:18 EST 2007


On Thu, 2007-09-06 at 12:47 -0500, Linas Vepstas wrote:
> Gramatical corrections to comments.
> 
> Signed-off-by: Linas Vepstas <linas at austin.ibm.com>
> 
> ----
> 
>  arch/powerpc/kernel/prom.c     |    8 +++++---
>  arch/powerpc/kernel/setup_64.c |    6 +++---
>  2 files changed, 8 insertions(+), 6 deletions(-)
> 
> Index: linux-2.6.22-git2/arch/powerpc/kernel/setup_64.c
> ===================================================================
> --- linux-2.6.22-git2.orig/arch/powerpc/kernel/setup_64.c	2007-09-04 17:29:36.000000000 -0500
> +++ linux-2.6.22-git2/arch/powerpc/kernel/setup_64.c	2007-09-05 14:12:23.000000000 -0500
> @@ -181,9 +181,9 @@ void __init early_setup(unsigned long dt
>   	DBG(" -> early_setup(), dt_ptr: 0x%lx\n", dt_ptr);
>  
>  	/*
> -	 * Do early initializations using the flattened device
> -	 * tree, like retreiving the physical memory map or
> -	 * calculating/retreiving the hash table size
> +	 * Do early initialization using the flattened device
> +	 * tree, such as retrieving the physical memory map or
> +	 * calculating/retrieving the hash table size.
>  	 */
>  	early_init_devtree(__va(dt_ptr));

That's a little pedantic ..

> Index: linux-2.6.22-git2/arch/powerpc/kernel/prom.c
> ===================================================================
> --- linux-2.6.22-git2.orig/arch/powerpc/kernel/prom.c	2007-09-05 14:23:06.000000000 -0500
> +++ linux-2.6.22-git2/arch/powerpc/kernel/prom.c	2007-09-05 14:24:49.000000000 -0500
> @@ -433,9 +433,11 @@ static int __init early_parse_mem(char *
>  }
>  early_param("mem", early_parse_mem);
>  
> -/*
> - * The device tree may be allocated below our memory limit, or inside the
> - * crash kernel region for kdump. If so, move it out now.
> +/**
> + * move_device_tree - move tree to an unused area, if needed.
> + *
> + * The device tree may be allocated beyond our memory limit, or inside the
> + * crash kernel region for kdump. If so, move it out of the way.
>   */
>  static void move_device_tree(void)

.. but that looks good. Although it's a semantic change, not just
grammatical as your changelog says.

cheers

-- 
Michael Ellerman
OzLabs, IBM Australia Development Lab

wwweb: http://michael.ellerman.id.au
phone: +61 2 6212 1183 (tie line 70 21183)

We do not inherit the earth from our ancestors,
we borrow it from our children. - S.M.A.R.T Person
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.ozlabs.org/pipermail/linuxppc-dev/attachments/20070907/e22d7d0d/attachment.pgp>


More information about the Linuxppc-dev mailing list