* ACPICA Release 20090123 linuxized patches
@ 2009-02-04 1:42 Lin Ming
2009-02-04 5:05 ` Len Brown
0 siblings, 1 reply; 2+ messages in thread
From: Lin Ming @ 2009-02-04 1:42 UTC (permalink / raw)
To: Len Brown, Moore, Robert; +Cc: linux-acpi
[-- Attachment #1: Type: text/plain, Size: 376 bytes --]
mbox file attached which can be applied to Linus tree cleanly.
Build & test on a x86_64 machine.
Lin Ming
---
ACPICA: Add error check to debug object dump routine
ACPICA: Allow OS override of all ACPI tables
ACPICA: Remove extraneous parameter in table manager
ACPICA: Add override for dynamic tables
ACPICA: Update FADT flag definitions
ACPICA: Update version to 20090123
[-- Attachment #2: R01_23_09.mbox --]
[-- Type: application/mbox, Size: 25505 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: ACPICA Release 20090123 linuxized patches
2009-02-04 1:42 ACPICA Release 20090123 linuxized patches Lin Ming
@ 2009-02-04 5:05 ` Len Brown
0 siblings, 0 replies; 2+ messages in thread
From: Len Brown @ 2009-02-04 5:05 UTC (permalink / raw)
To: Lin Ming; +Cc: Moore, Robert, linux-acpi
> ACPICA: Add error check to debug object dump routine
> ACPICA: Allow OS override of all ACPI tables
We'll need to expand acpi_os_table_override()
if we want to take advantage of this new ACPICA feature.
I suppose we'll do it the first time we need it:-)
> ACPICA: Remove extraneous parameter in table manager
> ACPICA: Add override for dynamic tables
ditto above.
Note that the reason that I didn't accept Thomas' proposal
to add SSDT overrides to Linux was because if you're doing
an override, you can always just combine the SSDT into the
DSDT override and disable loading SSDTs.
Also, the mechanism to identify the table
was the signature, and it wasn't immediately obvious how to identify
one of several SSDTs.
> ACPICA: Update FADT flag definitions
Thanks for digging through this. It seems that sins
like the undocumented FADT v2 simply can't be swept under the rug:-)
> ACPICA: Update version to 20090123
I'm going to ignore these warnings from checkpatch.pl,
but I'd rather not have to...
applied to acpica branch.
thanks,
Len Brown, Intel Open Source Technology Center
---
scripts/checkpatch.pl
ERROR: return is not a function, parentheses are not required
#25: FILE: drivers/acpi/acpica/nsdump.c:187:
+ return (AE_OK);
WARNING: braces {} are not necessary for single statement blocks
#98: FILE: drivers/acpi/acpica/tbutils.c:317:
+ if (!mapped_table) {
return;
}
WARNING: line over 80 characters
#106: FILE: drivers/acpi/acpica/tbutils.c:323:
+ if (signature && !ACPI_COMPARE_NAME(mapped_table->signature,
signature)) {
WARNING: line over 80 characters
#110: FILE: drivers/acpi/acpica/tbutils.c:325:
+ "Invalid signature 0x%X for ACPI table,
expected [%s]",
WARNING: line over 80 characters
#516: FILE: include/acpi/actbl.h:217:
+ u8 duty_width; /* Processor duty cycle value bit width in
P_CNT register */
WARNING: line over 80 characters
#521: FILE: include/acpi/actbl.h:221:
+ u16 boot_flags; /* IA-PC Boot Architecture Flags (see
below for individual flags) */
WARNING: line over 80 characters
#531: FILE: include/acpi/actbl.h:241:
+#define ACPI_FADT_LEGACY_DEVICES (1) /* 00: [V2] System has LPC
or ISA bus devices */
WARNING: line over 80 characters
#532: FILE: include/acpi/actbl.h:242:
+#define ACPI_FADT_8042 (1<<1) /* 01: [V3] System has an
8042 controller on port 60/64 */
WARNING: line over 80 characters
#533: FILE: include/acpi/actbl.h:243:
+#define ACPI_FADT_NO_VGA (1<<2) /* 02: [V4] It is not safe
to probe for VGA hardware */
WARNING: line over 80 characters
#534: FILE: include/acpi/actbl.h:244:
+#define ACPI_FADT_NO_MSI (1<<3) /* 03: [V4] Message
Signaled Interrupts (MSI) must not be enabled */
WARNING: line over 80 characters
#535: FILE: include/acpi/actbl.h:245:
+#define ACPI_FADT_NO_ASPM (1<<4) /* 04: [V4] PCIe ASPM
control must not be enabled */
WARNING: line over 80 characters
#561: FILE: include/acpi/actbl.h:251:
+#define ACPI_FADT_WBINVD (1) /* 00: [V1] The wbinvd
instruction works properly */
WARNING: line over 80 characters
#562: FILE: include/acpi/actbl.h:252:
+#define ACPI_FADT_WBINVD_FLUSH (1<<1) /* 01: [V1] wbinvd flushes
but does not invalidate caches */
WARNING: line over 80 characters
#563: FILE: include/acpi/actbl.h:253:
+#define ACPI_FADT_C1_SUPPORTED (1<<2) /* 02: [V1] All processors
support C1 state */
WARNING: line over 80 characters
#564: FILE: include/acpi/actbl.h:254:
+#define ACPI_FADT_C2_MP_SUPPORTED (1<<3) /* 03: [V1] C2 state works
on MP system */
WARNING: line over 80 characters
#565: FILE: include/acpi/actbl.h:255:
+#define ACPI_FADT_POWER_BUTTON (1<<4) /* 04: [V1] Power button
is handled as a control method device */
WARNING: line over 80 characters
#566: FILE: include/acpi/actbl.h:256:
+#define ACPI_FADT_SLEEP_BUTTON (1<<5) /* 05: [V1] Sleep button
is handled as a control method device */
WARNING: line over 80 characters
#567: FILE: include/acpi/actbl.h:257:
+#define ACPI_FADT_FIXED_RTC (1<<6) /* 06: [V1] RTC wakeup
status not in fixed register space */
WARNING: line over 80 characters
#568: FILE: include/acpi/actbl.h:258:
+#define ACPI_FADT_S4_RTC_WAKE (1<<7) /* 07: [V1] RTC alarm can
wake system from S4 */
WARNING: line over 80 characters
#569: FILE: include/acpi/actbl.h:259:
+#define ACPI_FADT_32BIT_TIMER (1<<8) /* 08: [V1] ACPI timer
width is 32-bit (0=24-bit) */
WARNING: line over 80 characters
#571: FILE: include/acpi/actbl.h:261:
+#define ACPI_FADT_RESET_REGISTER (1<<10) /* 10: [V2] System reset
via the FADT RESET_REG supported */
WARNING: line over 80 characters
#572: FILE: include/acpi/actbl.h:262:
+#define ACPI_FADT_SEALED_CASE (1<<11) /* 11: [V3] No internal
expansion capabilities and case is sealed */
WARNING: line over 80 characters
#573: FILE: include/acpi/actbl.h:263:
+#define ACPI_FADT_HEADLESS (1<<12) /* 12: [V3] No local video
capabilities or local input devices */
WARNING: line over 80 characters
#574: FILE: include/acpi/actbl.h:264:
+#define ACPI_FADT_SLEEP_TYPE (1<<13) /* 13: [V3] Must execute
native instruction after writing SLP_TYPx register */
WARNING: line over 80 characters
#575: FILE: include/acpi/actbl.h:265:
+#define ACPI_FADT_PCI_EXPRESS_WAKE (1<<14) /* 14: [V4] System
supports PCIEXP_WAKE (STS/EN) bits (ACPI 3.0) */
WARNING: line over 80 characters
#576: FILE: include/acpi/actbl.h:266:
+#define ACPI_FADT_PLATFORM_CLOCK (1<<15) /* 15: [V4] OSPM should
use platform-provided timer (ACPI 3.0) */
WARNING: line over 80 characters
#577: FILE: include/acpi/actbl.h:267:
+#define ACPI_FADT_S4_RTC_VALID (1<<16) /* 16: [V4] Contents of
RTC_STS valid after S4 wake (ACPI 3.0) */
WARNING: line over 80 characters
#578: FILE: include/acpi/actbl.h:268:
+#define ACPI_FADT_REMOTE_POWER_ON (1<<17) /* 17: [V4] System is
compatible with remote power on (ACPI 3.0) */
WARNING: line over 80 characters
#579: FILE: include/acpi/actbl.h:269:
+#define ACPI_FADT_APIC_CLUSTER (1<<18) /* 18: [V4] All local
APICs must use cluster model (ACPI 3.0) */
WARNING: line over 80 characters
#580: FILE: include/acpi/actbl.h:270:
+#define ACPI_FADT_APIC_PHYSICAL (1<<19) /* 19: [V4] All local
x_aPICs must use physical dest mode (ACPI 3.0) */
total: 1 errors, 29 warnings, 458 lines checked
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-02-04 5:05 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-02-04 1:42 ACPICA Release 20090123 linuxized patches Lin Ming
2009-02-04 5:05 ` Len Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox