2014-07-23 06:42:42

by Nicholas Krause

[permalink] [raw]
Subject: [PATCH] avr32: Remove unnessary variables from page.h

This removes the unneeded variables memory_start and memory_end in
order to fix this fix me message.

Signed-off-by: Nicholas Krause <[email protected]>
---
arch/avr32/include/asm/page.h | 2 --
1 file changed, 2 deletions(-)

diff --git a/arch/avr32/include/asm/page.h b/arch/avr32/include/asm/page.h
index f805d1c..701006f 100644
--- a/arch/avr32/include/asm/page.h
+++ b/arch/avr32/include/asm/page.h
@@ -42,8 +42,6 @@ typedef struct page *pgtable_t;
#define __pgd(x) ((pgd_t) { (x) })
#define __pgprot(x) ((pgprot_t) { (x) })

-/* FIXME: These should be removed soon */
-extern unsigned long memory_start, memory_end;

/* Pure 2^n version of get_order */
static inline int get_order(unsigned long size)
--
1.9.1


Subject: Re: [PATCH] avr32: Remove unnessary variables from page.h

Around Wed 23 Jul 2014 02:42:36 -0400 or thereabout, Nicholas Krause wrote:
> This removes the unneeded variables memory_start and memory_end in
> order to fix this fix me message.
>
> Signed-off-by: Nicholas Krause <[email protected]>

This is already applied to my local for-linus branch, it will be pushed once
Linus opens up for merges after 3.16 is released.

> ---
> arch/avr32/include/asm/page.h | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/arch/avr32/include/asm/page.h b/arch/avr32/include/asm/page.h
> index f805d1c..701006f 100644
> --- a/arch/avr32/include/asm/page.h
> +++ b/arch/avr32/include/asm/page.h
> @@ -42,8 +42,6 @@ typedef struct page *pgtable_t;
> #define __pgd(x) ((pgd_t) { (x) })
> #define __pgprot(x) ((pgprot_t) { (x) })
>
> -/* FIXME: These should be removed soon */
> -extern unsigned long memory_start, memory_end;
>
> /* Pure 2^n version of get_order */
> static inline int get_order(unsigned long size)
--
mvh
Hans-Christian Egtvedt

2014-07-23 15:06:07

by Nicholas Krause

[permalink] [raw]
Subject: Re: [PATCH] avr32: Remove unnessary variables from page.h

On Wed, Jul 23, 2014 at 2:50 AM, Hans-Christian Egtvedt
<[email protected]> wrote:
> Around Wed 23 Jul 2014 02:42:36 -0400 or thereabout, Nicholas Krause wrote:
>> This removes the unneeded variables memory_start and memory_end in
>> order to fix this fix me message.
>>
>> Signed-off-by: Nicholas Krause <[email protected]>
>
> This is already applied to my local for-linus branch, it will be pushed once
> Linus opens up for merges after 3.16 is released.
>
>> ---
>> arch/avr32/include/asm/page.h | 2 --
>> 1 file changed, 2 deletions(-)
>>
>> diff --git a/arch/avr32/include/asm/page.h b/arch/avr32/include/asm/page.h
>> index f805d1c..701006f 100644
>> --- a/arch/avr32/include/asm/page.h
>> +++ b/arch/avr32/include/asm/page.h
>> @@ -42,8 +42,6 @@ typedef struct page *pgtable_t;
>> #define __pgd(x) ((pgd_t) { (x) })
>> #define __pgprot(x) ((pgprot_t) { (x) })
>>
>> -/* FIXME: These should be removed soon */
>> -extern unsigned long memory_start, memory_end;
>>
>> /* Pure 2^n version of get_order */
>> static inline int get_order(unsigned long size)
> --
> mvh
> Hans-Christian Egtvedt


I wasn't sure if I had sent this out. Seems I have :).
Cheers Nick