Patchwork [1/2] powerpc/mm: remove obsolete comment about page size name array

login
register
mail settings
Submitter Scott Wood
Date June 14, 2012, 11:40 p.m.
Message ID <20120614234055.GA17147@tyr.buserror.net>
Download mbox | patch
Permalink /patch/165027/
State Accepted, archived
Delegated to: Michael Ellerman
Headers show

Comments

Scott Wood - June 14, 2012, 11:40 p.m.
The array of names in hugetlbpage.c no longer exists.

Signed-off-by: Scott Wood <scottwood@freescale.com>
---
 arch/powerpc/include/asm/mmu.h |    7 +------
 1 files changed, 1 insertions(+), 6 deletions(-)
Michael Ellerman - June 15, 2012, 2:50 a.m.
On Thu, 2012-06-14 at 18:40 -0500, Scott Wood wrote:
> The array of names in hugetlbpage.c no longer exists.
> 
> Signed-off-by: Scott Wood <scottwood@freescale.com>
> ---
>  arch/powerpc/include/asm/mmu.h |    7 +------
>  1 files changed, 1 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/powerpc/include/asm/mmu.h b/arch/powerpc/include/asm/mmu.h
> index f014552..a9e9ec6 100644
> --- a/arch/powerpc/include/asm/mmu.h
> +++ b/arch/powerpc/include/asm/mmu.h
> @@ -163,12 +163,7 @@ extern u64 ppc64_rma_size;
>   * to think about, feedback welcome. --BenH.
>   */
>  
> -/* There are #define as they have to be used in assembly
> - *
> - * WARNING: If you change this list, make sure to update the array of
> - * names currently in arch/powerpc/mm/hugetlbpage.c or bad things will
> - * happen
> - */
> +/* There are #define as they have to be used in assembly */
     ^^^^^^

Should be "These", I'll fix it up.

cheers

Patch

diff --git a/arch/powerpc/include/asm/mmu.h b/arch/powerpc/include/asm/mmu.h
index f014552..a9e9ec6 100644
--- a/arch/powerpc/include/asm/mmu.h
+++ b/arch/powerpc/include/asm/mmu.h
@@ -163,12 +163,7 @@  extern u64 ppc64_rma_size;
  * to think about, feedback welcome. --BenH.
  */
 
-/* There are #define as they have to be used in assembly
- *
- * WARNING: If you change this list, make sure to update the array of
- * names currently in arch/powerpc/mm/hugetlbpage.c or bad things will
- * happen
- */
+/* There are #define as they have to be used in assembly */
 #define MMU_PAGE_4K	0
 #define MMU_PAGE_16K	1
 #define MMU_PAGE_64K	2