lkml.org 
[lkml]   [2000]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: PATCH 2.3.41.2: Replace Via support for ACPI
    Martin Mares wrote:
    >
    > Hi Jeff,
    >
    > > Attached is a patch against 2.3.41-pre2 which adds back Via support. It
    > > does so using the new PCI table-driven device matching code, which
    > > should make it easy to add other chipsets.
    >
    > I don't understand why did you introduce the acpi_chip_info[] table
    > and especially why does it contain CH_xxx constants which are equivalent
    > to entry indices. To me, it looks best to use the driver data field
    > in acpi_pci_tbl[] to point directly to the init function.

    My approach is due to future expandability -- if you ever need ANY
    per-chip information besides the init function, then you have to go my
    way of setting up the PCI table. Therefore, why not use my method in
    the first place, and make the maintainer's life easier in the long run?
    The extra table is __initdata so the extra couple bytes doesn't even
    matter...

    Jeff




    --
    Jeff Garzik | Andre the Giant has a posse.
    Building 1024 |
    MandrakeSoft, Inc. |

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

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