[lkml]   [2002]   [Jun]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH: NEW SUBARCHITECTURE FOR 2.5.21] support for NCR voyager (3/4/5xxx series)
    On Thu, Jun 13, 2002 at 08:13:26PM -0400, James Bottomley wrote:
    > > Would it make sense for the subarchs to use the generic code where
    > > possible, and only reimplement it's own (for eg) apic.c as and when it
    > > actually *needs* to be different ?
    > That is really the way I've implemented it.

    Ah, good.

    > The only PC specific file in the
    > generic directory is mpparse.c (since neither visws nor voyager has an MP
    > compliant bios). All the shareable files are kept in `kernel' and activated
    > by config options.

    Another piece of low hanging fruit is probably dmi_scan.c
    There are no workarounds there for either (are they even DMI compliant?)
    so compiling it in doesn't make much sense.

    > I can certainly move mpparse.c back to kernel and add an extra (non user
    > visible) config option.

    if neither visws or voyager need it, I'd say it doesn't belong in the
    respective subarch directories period.

    > > Sounds quite logical. What does the current patches you have do ? I've
    > > not had chance to look at them yet.
    > It creates directories `generic' for the standard pc and `visws'. The voyager
    > patch creates a `voyager' directory. Alternatively, these could be `mach-pc',
    > `mach-visws' and `mach-voyager'.

    Yeah, I think mach-foo would be more aesthetically pleasing, so I'll
    cast my vote for that one. If nothing else, it makes it obvious that
    the subdir isn't important if you don't care about $subarch


    | Dave Jones.
    | SuSE Labs
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:26    [W:0.049 / U:2.504 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site