lkml.org 
[lkml]   [2006]   [Mar]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] DMI: move dmi_scan.c from arch/i386 to drivers/firmware/
Bjorn Helgaas <bjorn.helgaas@hp.com> wrote:
>
> On Thursday 23 March 2006 13:41, Andi Kleen wrote:
> > On Thursday 23 March 2006 21:37, Bjorn Helgaas wrote:
> > > dmi_scan.c is arch-independent and is used by i386, x86_64, and ia64.
> > > Currently all three arches compile it from arch/i386, which means that
> > > ia64 and x86_64 depend on things in arch/i386 that they wouldn't
> > > otherwise care about.
> > >
> > > This is simply "mv arch/i386/kernel/dmi_scan.c drivers/firmware/"
> > > (removing trailing whitespace) and the associated Makefile changes.
> > > All three architectures already set CONFIG_DMI in their top-level
> > > Kconfig files.
> > >
> > > Built and booted on ia64 and i386.
> >
> > This conflicts with at least three patches in my upcomming x86-64 patchkit.

akpm:/usr/src/25> grep -l arch/i386/kernel/dmi_scan.c patches/*.patch
patches/dmi-only-ioremap-stuff-we-actually-need.patch
patches/efi-fixes.patch
patches/efi-keep-physical-table-addresses-in-efi-structure.patch
patches/ia64-use-i386-dmi_scanc.patch
patches/x86_64-mm-dmi-early.patch
patches/x86_64-mm-dmi-year.patch


> > Since it doesn't fix anything I would request that such cleanup patches be
> > delayed.
>
> Sure, that's fine with me. Do you want to let me know when your
> patches are in? Or maybe it would be easier for you to incorporate
> my patch into your patchkit. I just don't know how to tell when
> the cleanup window is open.
>

I'm reluctant to attempt to merge the efi patches until ia64, acpi and
x86_64 have merged. Only ia64 has merged at this time. acpi might need to
be bypassed, dunno.

I'd expect things to be settled down in a couple of weeks. Patches which
move files around are hard.
-
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-03-24 00:51    [W:0.048 / U:2.612 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site