[microblaze-uclinux] Re: [PATCH] Actually show KiB rather than pages in "Freeing initrd memory:" message on microblaze.

[Michal Simek <monstr@xxxxxxxxx>]

Lennart Sorensen wrote:
> On Mon, Sep 21, 2009 at 02:39:18PM +0200, Michal Simek wrote:
> > Added to next branch.
> >
> > There were some problems which I fixed.
> > 1. long first description
> > 2. missing microblaze: at the beginning

I want to have "microblaze:" at the beginning of email/commit message.
It is much clear for mailing list, Linus and me.

Thanks,
Michal

> Missing what?
>
> > 3. line longer than 80chars
>
> Hmm, I thought for sure I had checkpatch.pl claiming all my lines
> were OK.  Odd.  Clearly the one below is longer though.  How strange.
>
> > Lennart Sorensen wrote:
> > > Fix "Freeing initrd memory:" message on microblaze to show kilobytes as
> > > claimed rather than number of pages.
> > >
> > > Signed-off-by: Lennart Sorensen <lsorense@xxxxxxxxxxxxxxxxxxx>
> > >
> > > diff --git a/arch/microblaze/mm/init.c b/arch/microblaze/mm/init.c
> > > index f207f1a..ed5f96b 100644
> > > --- a/arch/microblaze/mm/init.c
> > > +++ b/arch/microblaze/mm/init.c
> > > @@ -180,7 +180,7 @@ void free_initrd_mem(unsigned long start, unsigned long end)
> > >  		totalram_pages++;
> > >  		pages++;
> > >  	}
> > > -	printk(KERN_NOTICE "Freeing initrd memory: %dk freed\n", pages);
> > > +	printk(KERN_NOTICE "Freeing initrd memory: %dk freed\n", pages * (PAGE_SIZE / 1024));
> > >  }
> > >  #endif
> > >  

___________________________
microblaze-uclinux mailing list
microblaze-uclinux@xxxxxxxxxxxxxx
Project Home Page : http://www.itee.uq.edu.au/~jwilliams/mblaze-uclinux
Mailing List Archive : http://www.itee.uq.edu.au/~listarch/microblaze-uclinux/