Re: [PATCH] m68k: merge arch free memory functions into a single function

Finn Thain
2012-12-01T03:49:56Z

On Fri, 30 Nov 2012, gerg@snapgear.com wrote:

> - pr_notice("Freeing unused kernel memory: %luk freed (0x%x - 0x%x)\n",
> - (addr - (unsigned long) __init_begin) >> 10,
> - (unsigned int) __init_begin, (unsigned int) __init_end);
> + pr_notice("Freeing %s memory: %dk freed (0x%lx-0x%lx)\n", name,
> + pages << (PAGE_SHIFT - 10), start, end);

Perhaps "Freed %d KiB %s memory: 0x%lx - 0x%lx\n" is better (a tad shorter
and has the correct unit).

Finn
_______________________________________________
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