Hi Geert,

On 01/06/12 06:23, Geert Uytterhoeven wrote:
   - Remove casts and unneeded address-of ('&') operators,
   - Use %p to format pointers, %lx to format unsigned longs.

Signed-off-by: Geert Uytterhoeven<ge...@linux-m68k.org>

Looks good. Compile and run tested, all ok.

Acked-by: Greg Ungerer <g...@uclinux.org>

Regards
Greg


---
Not even compile tested

  arch/m68k/kernel/setup_no.c |   11 ++++-------
  1 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/arch/m68k/kernel/setup_no.c b/arch/m68k/kernel/setup_no.c
index 7dc186b..8a624ec 100644
--- a/arch/m68k/kernel/setup_no.c
+++ b/arch/m68k/kernel/setup_no.c
@@ -218,13 +218,10 @@ void __init setup_arch(char **cmdline_p)
        printk(KERN_INFO "Motorola M5235EVB support (C)2005 Syn-tech Systems, Inc. 
(Jate Sujjavanich)\n");
  #endif

-       pr_debug("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);
-       pr_debug("MEMORY ->  ROMFS=0x%06x-0x%06x MEM=0x%06x-0x%06x\n ",
-                (int)&_ebss, (int) memory_start,
-                (int) memory_start, (int) memory_end);
+       pr_debug("KERNEL ->  TEXT=0x%p-0x%p DATA=0x%p-0x%p BSS=0x%p-0x%p\n",
+                _stext, _etext, _sdata, _edata, _sbss, _ebss);
+       pr_debug("MEMORY ->  ROMFS=0x%p-0x%06lx MEM=0x%06lx-0x%06lx\n ",
+                _ebss, memory_start, memory_start, memory_end);

        /* Keep a copy of command line */
        *cmdline_p =&command_line[0];


--
------------------------------------------------------------------------
Greg Ungerer  --  Principal Engineer        EMAIL:     g...@snapgear.com
SnapGear Group, McAfee                      PHONE:       +61 7 3435 2888
8 Gardner Close                             FAX:         +61 7 3217 5323
Milton, QLD, 4064, Australia                WEB: http://www.SnapGear.com
_______________________________________________
uClinux-dev mailing list
uClinux-dev@uclinux.org
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by uclinux-dev@uclinux.org
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev

Reply via email to