New ACPI blacklist format

M. Warner Losh imp at bsdimp.com
Sun May 9 08:35:34 PDT 2004


In message: <20040507231846.F52653 at root.org>
            Nate Lawson <nate at root.org> writes:
: I have extracted a set of known-broken tables/versions from various
: sources.  Since. as far as I know, C does not allow variable length
: initializers, I've settled on the following format:

It does.

: struct acpi_table_desc {
:     char        *signature;
:     char        *oem_id;
:     char        *oem_table_id;
:     char        *oem_rev_op;
:     char        *oem_revision;
:     char        *creator_id;
:     char        *creator_rev_op;
:     char        *creator_revision;
: };
: 
: struct acpi_blacklist {
:     int         quirk;
:     struct      acpi_table_desc *match;
: };
: 
: #define ACPI_BROKEN     0x1
: 
: static struct acpi_table_desc Abit_BP6[] = {
:     { "FACP", "AWARD", "AWRDACPI", "<=", "30302e31", "", "", "" },
: };

{ .signature = "FACP", .oem_id="AWARD", .oem_table_id="AWARDACPI",
  .oem_rev_op = "<=", }

:The op values will be "<=", "=", and ">=".

These are likely better as a enum.

: Is there any better way to compact this?

Using shorter structure names would get it all onthe same line.

Warner


More information about the freebsd-acpi mailing list