Re: [PATCH resend] nosave: Consolidate __nosave_{begin,end} in <asm/sections.h>

[Geert Uytterhoeven]

Hi Stephen,

On Fri, Aug 29, 2014 at 10:51 PM, Stephen Boyd <sboyd@xxxxxxxxxxxxxx> wrote:
> On 08/29/14 13:22, Geert Uytterhoeven wrote:
>> The different architectures used their own (and different) declarations:
>>
>>     extern __visible const void __nosave_begin, __nosave_end;
>>     extern const void __nosave_begin, __nosave_end;
>>     extern long __nosave_begin, __nosave_end;
>>
>> Consolidate them using the first variant in <asm/sections.h>.
>>
>> Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
>> ---
>> This has been sent before, as part of the series "[PATCH 00/17] <asm/sections.h>
>> related cleanups", almost one year ago (https://lkml.org/lkml/2013/11/12/510).
>>
>>  arch/mips/include/asm/suspend.h   | 7 -------
>>  arch/mips/power/cpu.c             | 2 +-
>>  arch/powerpc/kernel/suspend.c     | 4 +---
>>  arch/s390/kernel/suspend.c        | 6 +-----
>>  arch/sh/include/asm/sections.h    | 1 -
>>  arch/sparc/power/hibernate.c      | 4 +---
>>  arch/unicore32/include/mach/pm.h  | 3 ---
>>  arch/unicore32/kernel/hibernate.c | 1 +
>>  arch/x86/power/hibernate_32.c     | 4 +---
>>  arch/x86/power/hibernate_64.c     | 4 +---
>>  include/asm-generic/sections.h    | 4 ++++
>>  11 files changed, 11 insertions(+), 29 deletions(-)
>>  delete mode 100644 arch/mips/include/asm/suspend.h
>
> There's one in arch/arm/kernel/hibernate.c now. Can we update ARM too?

Thanks, I'll fix that.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/