summaryrefslogtreecommitdiffstats
path: root/drivers/acpi
diff options
context:
space:
mode:
authorBob Moore <robert.moore@intel.com>2007-02-02 19:48:20 +0300
committerLen Brown <len.brown@intel.com>2007-02-02 21:14:24 -0500
commitb89b71a0019660d73e3c9671205c49e443d7085c (patch)
tree562463b563af49731b894c76acde6173015dbd86 /drivers/acpi
parent0eaa14c02809cc93386b907846da5c024fd73012 (diff)
downloadop-kernel-dev-b89b71a0019660d73e3c9671205c49e443d7085c.zip
op-kernel-dev-b89b71a0019660d73e3c9671205c49e443d7085c.tar.gz
ACPICA: Cleanup of FADT verification function.
Removed offset display, not needed. Signed-off-by: Alexey Starikovskiy <alexey.y.starikovskiy@intel.com> Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi')
-rw-r--r--drivers/acpi/utilities/utinit.c65
1 files changed, 23 insertions, 42 deletions
diff --git a/drivers/acpi/utilities/utinit.c b/drivers/acpi/utilities/utinit.c
index 2d2c4a3..5079f19 100644
--- a/drivers/acpi/utilities/utinit.c
+++ b/drivers/acpi/utilities/utinit.c
@@ -50,8 +50,7 @@
ACPI_MODULE_NAME("utinit")
/* Local prototypes */
-static void
-acpi_ut_fadt_register_error(char *register_name, u32 value, u8 offset);
+static void acpi_ut_fadt_register_error(char *register_name, u32 value);
static void acpi_ut_terminate(void);
@@ -61,21 +60,18 @@ static void acpi_ut_terminate(void);
*
* PARAMETERS: register_name - Pointer to string identifying register
* Value - Actual register contents value
- * Offset - Byte offset in the FADT
*
- * RETURN: AE_BAD_VALUE
+ * RETURN: None
*
* DESCRIPTION: Display failure message
*
******************************************************************************/
-static void
-acpi_ut_fadt_register_error(char *register_name, u32 value, u8 offset)
+static void acpi_ut_fadt_register_error(char *register_name, u32 value)
{
- ACPI_WARNING((AE_INFO,
- "Invalid FADT value %s=%X at offset %X in FADT=%p",
- register_name, value, offset, &acpi_gbl_FADT));
+ ACPI_WARNING((AE_INFO, "Invalid FADT value %s = %X",
+ register_name, value));
}
/******************************************************************************
@@ -98,69 +94,54 @@ acpi_status acpi_ut_validate_fadt(void)
* but don't abort on any problems, just display error
*/
if (acpi_gbl_FADT.pm1_event_length < 4) {
- acpi_ut_fadt_register_error("PM1_EVT_LEN",
+ acpi_ut_fadt_register_error("Pm1EventLength",
(u32) acpi_gbl_FADT.
- pm1_event_length,
- ACPI_FADT_OFFSET(pm1_event_length));
+ pm1_event_length);
+ }
+
+ if (acpi_gbl_FADT.pm_timer_length < 4) {
+ acpi_ut_fadt_register_error("PmTimerLength",
+ (u32) acpi_gbl_FADT.
+ pm_timer_length);
}
if (!acpi_gbl_FADT.pm1_control_length) {
- acpi_ut_fadt_register_error("PM1_CNT_LEN", 0,
- ACPI_FADT_OFFSET
- (pm1_control_length));
+ acpi_ut_fadt_register_error("Pm1ControlLength", 0);
}
if (!acpi_gbl_FADT.xpm1a_event_block.address) {
- acpi_ut_fadt_register_error("X_PM1a_EVT_BLK", 0,
- ACPI_FADT_OFFSET(xpm1a_event_block.
- address));
+ acpi_ut_fadt_register_error("XPm1aEventBlock.Address", 0);
}
if (!acpi_gbl_FADT.xpm1a_control_block.address) {
- acpi_ut_fadt_register_error("X_PM1a_CNT_BLK", 0,
- ACPI_FADT_OFFSET
- (xpm1a_control_block.address));
+ acpi_ut_fadt_register_error("XPm1aControlBlock.Address", 0);
}
if (!acpi_gbl_FADT.xpm_timer_block.address) {
- acpi_ut_fadt_register_error("X_PM_TMR_BLK", 0,
- ACPI_FADT_OFFSET(xpm_timer_block.
- address));
+ acpi_ut_fadt_register_error("XPmTimerBlock.Address", 0);
}
if ((acpi_gbl_FADT.xpm2_control_block.address &&
!acpi_gbl_FADT.pm2_control_length)) {
- acpi_ut_fadt_register_error("PM2_CNT_LEN",
+ acpi_ut_fadt_register_error("Pm2ControlLength",
(u32) acpi_gbl_FADT.
- pm2_control_length,
- ACPI_FADT_OFFSET
- (pm2_control_length));
- }
-
- if (acpi_gbl_FADT.pm_timer_length < 4) {
- acpi_ut_fadt_register_error("PM_TM_LEN",
- (u32) acpi_gbl_FADT.pm_timer_length,
- ACPI_FADT_OFFSET(pm_timer_length));
+ pm2_control_length);
}
/* Length of GPE blocks must be a multiple of 2 */
if (acpi_gbl_FADT.xgpe0_block.address &&
(acpi_gbl_FADT.gpe0_block_length & 1)) {
- acpi_ut_fadt_register_error("(x)GPE0_BLK_LEN",
+ acpi_ut_fadt_register_error("Gpe0BlockLength",
(u32) acpi_gbl_FADT.
- gpe0_block_length,
- ACPI_FADT_OFFSET
- (gpe0_block_length));
+ gpe0_block_length);
}
if (acpi_gbl_FADT.xgpe1_block.address &&
(acpi_gbl_FADT.gpe1_block_length & 1)) {
- acpi_ut_fadt_register_error("(x)GPE1_BLK_LEN",
+ acpi_ut_fadt_register_error("Gpe1BlockLength",
(u32) acpi_gbl_FADT.
- gpe1_block_length,
- ACPI_FADT_OFFSET
- (gpe1_block_length));
+ gpe1_block_length);
}
return (AE_OK);
OpenPOWER on IntegriCloud