Messages in this thread | | | Date | Thu, 17 Sep 2009 11:50:15 -0400 | Subject | Re: [microblaze-uclinux] Re: =?us-ascii?Q?=5Bmicroblaze=2Duclinux=5D=20Re =3A=20=5BPATCH=5D=20Actually=20show=20KiB=20r =?us-ascii?Q?ather=3D20than=3D20pages=3D20in=3D20=3D22Fre | From | (Lennart Sorensen) |
| |
On Thu, Sep 17, 2009 at 05:39:10PM +0200, Michal Simek wrote: > > > lsorense@csclub.uwaterloo.ca wrote: > > On Thu, Sep 17, 2009 at 10:56:16AM +0200, Michal Simek wrote: > >> Greg Ungerer wrote: > >>> Hi Lennart, > >>> > >>> Lennart Sorensen wrote: > >>>> Fix "Freeing initrd memory:" message on h3800, m68knommu and microblaze > >>>> to show kilobytes as claimed rather than number of pages. > >>>> > >>>> Signed-off-by: Lennart Sorensen <lsorense@csclub.uwaterloo.ca> > >>> Looks ok to me. I can take and push the m68knommu part separately. > >>> Or if Andrew wants to take it all as one, here is an: > >>> > >>> Acked-by: Greg Ungerer <gerg@uclinux.org> > >> As Greg wrote. It will be the best to push it separately. IMHO it > >> will be faster. It is up to you. If you want to add it to microblaze > >> branch please send only patch for microblaze. > >> > >> If not. > >> > >> Acked-by: Michal Simek <monstr@monstr.eu> > > > > Oh right how silly of me. 3 seperate patches would have been nicer I > > guess. The original code is certainly copied from one place to the others > > although I have no idea which architecture was the first to get it wrong. > > > > Would everyone prefer I made 3 patches instead and resent them? > > Just do it for microblaze
Well I just sent a seperate one for each. I misspelled h8300 (as h3800) in the comment before.
-- Len Sorensen
| |