* [PATCH 1/2] frv: Fix use of extinct _sbss and _ebss in debug code
@ 2012-06-10 14:37 Geert Uytterhoeven
2012-06-10 14:37 ` [PATCH 2/2] frv: Properly use the declarations provided by <asm/sections.h> Geert Uytterhoeven
0 siblings, 1 reply; 2+ messages in thread
From: Geert Uytterhoeven @ 2012-06-10 14:37 UTC (permalink / raw)
To: David Howells; +Cc: linux-kernel, Geert Uytterhoeven
Nowadays it should probably use __bss_start and __bss_stop
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
arch/frv/kernel/setup.c | 2 +-
arch/frv/mm/init.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/frv/kernel/setup.c b/arch/frv/kernel/setup.c
index 75cf7f4..1294fdf 100644
--- a/arch/frv/kernel/setup.c
+++ b/arch/frv/kernel/setup.c
@@ -815,7 +815,7 @@ void __init setup_arch(char **cmdline_p)
printk("KERNEL -> TEXT=0x%06x-0x%06x DATA=0x%06x-0x%06x BSS=0x%06x-0x%06x\n",
(int) &_stext, (int) &_etext,
(int) &_sdata, (int) &_edata,
- (int) &_sbss, (int) &_ebss);
+ (int) &__bss_start, (int) &__bss_stop);
#endif
#ifdef CONFIG_VT
diff --git a/arch/frv/mm/init.c b/arch/frv/mm/init.c
index a19effc..92e97b0 100644
--- a/arch/frv/mm/init.c
+++ b/arch/frv/mm/init.c
@@ -146,7 +146,7 @@ void __init mem_init(void)
#else
codek = (_etext - _stext) >> 10;
- datak = 0; //(_ebss - _sdata) >> 10;
+ datak = 0; //(__bss_stop - _sdata) >> 10;
#endif
tmp = nr_free_pages() << PAGE_SHIFT;
--
1.7.0.4
^ permalink raw reply [flat|nested] 2+ messages in thread
* [PATCH 2/2] frv: Properly use the declarations provided by <asm/sections.h>
2012-06-10 14:37 [PATCH 1/2] frv: Fix use of extinct _sbss and _ebss in debug code Geert Uytterhoeven
@ 2012-06-10 14:37 ` Geert Uytterhoeven
0 siblings, 0 replies; 2+ messages in thread
From: Geert Uytterhoeven @ 2012-06-10 14:37 UTC (permalink / raw)
To: David Howells; +Cc: linux-kernel, Geert Uytterhoeven
- Remove the superfluous address-of ('&') operators,
- Remove the unneeded casts, use %p to format pointers instead.
Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
---
arch/frv/kernel/setup.c | 12 +++++-------
1 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/arch/frv/kernel/setup.c b/arch/frv/kernel/setup.c
index 1294fdf..16a6ac9 100644
--- a/arch/frv/kernel/setup.c
+++ b/arch/frv/kernel/setup.c
@@ -802,9 +802,9 @@ void __init setup_arch(char **cmdline_p)
BUG_ON(memory_start == memory_end);
- init_mm.start_code = (unsigned long) &_stext;
- init_mm.end_code = (unsigned long) &_etext;
- init_mm.end_data = (unsigned long) &_edata;
+ init_mm.start_code = (unsigned long) _stext;
+ init_mm.end_code = (unsigned long) _etext;
+ init_mm.end_data = (unsigned long) _edata;
#if 0 /* DAVIDM - don't set brk just incase someone decides to use it */
init_mm.brk = (unsigned long) &_end;
#else
@@ -812,10 +812,8 @@ void __init setup_arch(char **cmdline_p)
#endif
#ifdef DEBUG
- printk("KERNEL -> TEXT=0x%06x-0x%06x DATA=0x%06x-0x%06x BSS=0x%06x-0x%06x\n",
- (int) &_stext, (int) &_etext,
- (int) &_sdata, (int) &_edata,
- (int) &__bss_start, (int) &__bss_stop);
+ printk("KERNEL -> TEXT=0x%p-0x%p DATA=0x%p-0x%p BSS=0x%p-0x%p\n",
+ _stext, _etext, _sdata, _edata, __bss_start, __bss_stop);
#endif
#ifdef CONFIG_VT
--
1.7.0.4
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-06-10 14:38 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-06-10 14:37 [PATCH 1/2] frv: Fix use of extinct _sbss and _ebss in debug code Geert Uytterhoeven
2012-06-10 14:37 ` [PATCH 2/2] frv: Properly use the declarations provided by <asm/sections.h> Geert Uytterhoeven
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®