[lkml]   [2000]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: PATCH 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


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
Please read the FAQ at

 \ /
  Last update: 2005-03-22 13:56    [W:0.095 / U:0.200 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site