lkml.org 
[lkml]   [2006]   [Aug]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH for review] [65/145] i386: Clean up code style in mpparse.c ACPI code
On Wed, 16 Aug 2006 12:08:43 -0400
Len Brown <len.brown@intel.com> wrote:

> On Thursday 10 August 2006 15:36, Andi Kleen wrote:
> >
> > Remove some unlinuxy ways to write function parameter definitions.
> > Remove some stray "return;"s
> >
> > No functional change.
> >
> > Cc: len.brown@intel.com
> > Signed-off-by: Andi Kleen <ak@suse.de>
> >
> > ---
> > arch/i386/kernel/mpparse.c | 52 ++++++++++++++-------------------------------
> > 1 files changed, 17 insertions(+), 35 deletions(-)
>
> Maybe it is time to just Lindent the file?
> When I Lindented the ACPI sub-system, I stopped short of mpparse.c.

I think except for the ACPI bits which I already fixed in this patch
everything else was linuxy already.

>
> As you know, I'd like to see the ACPI part of mpparse.c split out into a different file
> that can be shared by i386 and x86_64.

I'm for splitting out in ACPI/non ACPI (with CONFIG for mpparse), but not for
sharing. I want the freedom to change mpparse's internal data structures without
caring for i386.

The first step would be to get rid some of the hacks that convert ACPI into
mpparse.

-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-08-16 18:35    [W:0.329 / U:0.008 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site