ACPICA: Rename some function and variable names
Subsystems (2)
PATCH v2 2/8 ACPICA: Rename some function and variable names
Some function and variable names are renamed to be consistent with ACPICA code base.
acpi_raw_enable_gpe -> acpi_ev_add_gpe_reference acpi_raw_disable_gpe -> acpi_ev_remove_gpe_reference acpi_gpe_can_wake -> acpi_setup_gpe_for_wake acpi_gpe_wakeup -> acpi_set_gpe_wake_mask acpi_update_gpes -> acpi_update_all_gpes acpi_all_gpes_initialized -> acpi_gbl_all_gpes_initialized acpi_handler_info -> acpi_gpe_handler_info
...
Signed-off-by Lin Ming <[email protected]>
--- drivers/acpi/acpica/acevents.h | 6 ++-- drivers/acpi/acpica/acglobal.h | 2 +- drivers/acpi/acpica/aclocal.h | 8 +++--- drivers/acpi/acpica/evgpe.c | 14 +++++++---- drivers/acpi/acpica/evgpeblk.c | 4 +- drivers/acpi/acpica/evxface.c | 16 ++++++------ drivers/acpi/acpica/evxfgpe.c | 53 +++++++++++++++++++++++++--------------- drivers/acpi/acpica/hwgpe.c | 2 +- drivers/acpi/acpica/utglobal.c | 2 +- drivers/acpi/dock.c | 2 +- drivers/acpi/scan.c | 6 ++-- drivers/acpi/wakeup.c | 4 +- include/acpi/acpixf.h | 6 ++-- include/acpi/actypes.h | 4 +- 14 files changed, 73 insertions(+), 56 deletions(-)
drivers/acpi/acpica/acevents.h +3 -3
@@ -82,9 +82,9 @@ acpi_ev_update_gpe_enable_mask(struct acpi_gpe_event_info *gpe_event_info);acpi_status acpi_ev_enable_gpe (struct acpi_gpe_event_info *gpe_event_info); -acpi_status acpi_raw_enable_gpe (struct acpi_gpe_event_info *gpe_event_info); +acpi_status acpi_ev_add_gpe_reference (struct acpi_gpe_event_info *gpe_event_info); -acpi_status acpi_raw_disable_gpe (struct acpi_gpe_event_info *gpe_event_info); +acpi_status acpi_ev_remove_gpe_reference (struct acpi_gpe_event_info *gpe_event_info);struct acpi_gpe_event_info *acpi_ev_get_gpe_event_info (acpi_handle gpe_device,u32 gpe_number);
@@ -107,7 +107,7 @@ acpi_ev_create_gpe_block(struct acpi_namespace_node *gpe_device,acpi_status acpi_ev_initialize_gpe_block (struct acpi_gpe_xrupt_info *gpe_xrupt_info,struct acpi_gpe_block_info *gpe_block, -void *ignored); +void *context);acpi_status acpi_ev_delete_gpe_block (struct acpi_gpe_block_info *gpe_block);
drivers/acpi/acpica/acglobal.h +1 -1
@@ -370,7 +370,7 @@ ACPI_EXTERN struct acpi_fixed_event_handler ACPI_EXTERN struct acpi_gpe_xrupt_info *acpi_gbl_gpe_xrupt_list_head; ACPI_EXTERN struct acpi_gpe_block_info *acpi_gbl_gpe_fadt_blocks[ACPI_MAX_GPE_BLOCKS ]; -ACPI_EXTERN u8 acpi_all_gpes_initialized; +ACPI_EXTERN u8 acpi_gbl_all_gpes_initialized;/***************************************************************************** *
drivers/acpi/acpica/aclocal.h +4 -4
@@ -412,13 +412,13 @@ struct acpi_handler_info {acpi_event_handler address;/* Address of handler, if any */ void *context;/* Context to be passed to handler */ struct acpi_namespace_node *method_node;/* Method node for this GPE level (saved) */ -u8 orig_flags;/* Original misc info about this GPE */ -u8 orig_enabled;/* Set if the GPE was originally enabled */ +u8 original_flags;/* Original (pre-handler) GPE info */ +u8 originally_enabled;/* True if GPE was originally enabled */ };union acpi_gpe_dispatch_info {struct acpi_namespace_node *method_node;/* Method node for this GPE level */ -struct acpi_handler_info *handler; +struct acpi_handler_info *handler;/* Installed GPE handler */ };/*
@@ -458,7 +458,7 @@ struct acpi_gpe_block_info {u32 register_count;/* Number of register pairs in block */ u16 gpe_count;/* Number of individual GPEs in block */ u8 block_base_number;/* Base GPE number for this block */ -u8 initialized;/* If set, the GPE block has been initialized */ +u8 initialized;/* TRUE if this block is initialized */ };/* Information about GPE interrupt handlers, one per each interrupt level used for GPEs */
drivers/acpi/acpica/evgpe.c +9 -5
@@ -137,7 +137,7 @@ acpi_ev_enable_gpe(struct acpi_gpe_event_info *gpe_event_info)/******************************************************************************* * -* FUNCTION: acpi_raw_enable_gpe +* FUNCTION: acpi_ev_add_gpe_reference * * PARAMETERS: gpe_event_info - GPE to enable *
@@ -148,10 +148,12 @@ acpi_ev_enable_gpe(struct acpi_gpe_event_info *gpe_event_info)* ******************************************************************************/ -acpi_status acpi_raw_enable_gpe (struct acpi_gpe_event_info *gpe_event_info) +acpi_status acpi_ev_add_gpe_reference (struct acpi_gpe_event_info *gpe_event_info) {acpi_status status =AE_OK ; +ACPI_FUNCTION_TRACE (ev_add_gpe_reference); +if (gpe_event_info->runtime_count ==ACPI_UINT8_MAX ) {return_ACPI_STATUS (AE_LIMIT ); }
@@ -173,7 +175,7 @@ acpi_status acpi_raw_enable_gpe(struct acpi_gpe_event_info *gpe_event_info)/******************************************************************************* * -* FUNCTION: acpi_raw_disable_gpe +* FUNCTION: acpi_ev_remove_gpe_reference * * PARAMETERS: gpe_event_info - GPE to disable *
@@ -184,10 +186,12 @@ acpi_status acpi_raw_enable_gpe(struct acpi_gpe_event_info *gpe_event_info)* ******************************************************************************/ -acpi_status acpi_raw_disable_gpe (struct acpi_gpe_event_info *gpe_event_info) +acpi_status acpi_ev_remove_gpe_reference (struct acpi_gpe_event_info *gpe_event_info) {acpi_status status =AE_OK ; +ACPI_FUNCTION_TRACE (ev_remove_gpe_reference); +if (!gpe_event_info->runtime_count) {return_ACPI_STATUS (AE_LIMIT ); }
@@ -529,7 +533,7 @@ static void acpi_ev_asynch_enable_gpe(void *context)* Enable this GPE, conditionally. This means that the GPE will only be * physically enabled if the enable_for_run bit is set in the event_info */ - (void )acpi_hw_low_set_gpe (gpe_event_info,ACPI_GPE_COND_ENABLE ); + (void )acpi_hw_low_set_gpe (gpe_event_info,ACPI_GPE_CONDITIONAL_ENABLE ); return_VOID; }
drivers/acpi/acpica/evgpeblk.c +2 -2
@@ -386,7 +386,7 @@ acpi_ev_create_gpe_block(struct acpi_namespace_node *gpe_device,return_ACPI_STATUS (status); } - acpi_all_gpes_initialized = FALSE; + acpi_gbl_all_gpes_initialized = FALSE;/* Find all GPE methods (_Lxx or_Exx) for this block */
@@ -479,7 +479,7 @@ acpi_ev_initialize_gpe_block(struct acpi_gpe_xrupt_info *gpe_xrupt_info,continue ; } - status =acpi_raw_enable_gpe (gpe_event_info); + status =acpi_ev_add_gpe_reference (gpe_event_info);if (ACPI_FAILURE (status)) { ACPI_EXCEPTION((AE_INFO, status, "Could not enable GPE 0x%02X",
drivers/acpi/acpica/evxface.c +8 -8
@@ -722,7 +722,7 @@ acpi_install_gpe_handler(acpi_handle gpe_device, handler->address = address; handler->context = context; handler->method_node = gpe_event_info->dispatch.method_node; - handler->orig_flags = gpe_event_info->flags & + handler->original_flags = gpe_event_info->flags & (ACPI_GPE_XRUPT_TYPE_MASK |ACPI_GPE_DISPATCH_MASK );/*
@@ -731,10 +731,10 @@ acpi_install_gpe_handler(acpi_handle gpe_device,* disabled now to avoid spurious execution of the handler. */ -if ((handler->orig_flags &ACPI_GPE_DISPATCH_METHOD ) +if ((handler->original_flags &ACPI_GPE_DISPATCH_METHOD ) && gpe_event_info->runtime_count) { - handler->orig_enabled =1 ; - (void )acpi_raw_disable_gpe (gpe_event_info); + handler->originally_enabled =1 ; + (void )acpi_ev_remove_gpe_reference (gpe_event_info); }/* Install the handler */
@@ -835,7 +835,7 @@ acpi_remove_gpe_handler(acpi_handle gpe_device, gpe_event_info->dispatch.method_node = handler->method_node; gpe_event_info->flags &= ~(ACPI_GPE_XRUPT_TYPE_MASK |ACPI_GPE_DISPATCH_MASK ); - gpe_event_info->flags |= handler->orig_flags; + gpe_event_info->flags |= handler->original_flags;/* * If the GPE was previously associated with a method and it was
@@ -843,9 +843,9 @@ acpi_remove_gpe_handler(acpi_handle gpe_device,* post-initialization configuration. */ -if ((handler->orig_flags &ACPI_GPE_DISPATCH_METHOD ) - && handler->orig_enabled) - (void )acpi_raw_enable_gpe (gpe_event_info); +if ((handler->original_flags &ACPI_GPE_DISPATCH_METHOD ) + && handler->originally_enabled) + (void )acpi_ev_add_gpe_reference (gpe_event_info);/* Now we can free the handler object */
drivers/acpi/acpica/evxfgpe.c +33 -20
173 lines not on this page · raw message
drivers/acpi/acpica/hwgpe.c +1 -1
@@ -118,7 +118,7 @@ acpi_hw_low_set_gpe(struct acpi_gpe_event_info *gpe_event_info, u8 action) register_bit =acpi_hw_get_gpe_register_bit (gpe_event_info, gpe_register_info);switch (action) { -case ACPI_GPE_COND_ENABLE : +case ACPI_GPE_CONDITIONAL_ENABLE :if (!(register_bit & gpe_register_info->enable_for_run))return (AE_BAD_PARAMETER );
drivers/acpi/acpica/utglobal.c +1 -1
@@ -768,7 +768,7 @@ acpi_status acpi_ut_init_globals(void) acpi_gbl_gpe_fadt_blocks[0 ] =NULL ; acpi_gbl_gpe_fadt_blocks[1 ] =NULL ; acpi_current_gpe_count =0 ; - acpi_all_gpes_initialized = FALSE; + acpi_gbl_all_gpes_initialized = FALSE;/* Global handlers */
drivers/acpi/dock.c +1 -1
@@ -725,7 +725,7 @@ static void dock_notify(acpi_handle handle, u32 event, void *data)complete_dock (ds);dock_event (ds, event,DOCK_EVENT );dock_lock (ds,1 ); -acpi_update_gpes (); +acpi_update_all_gpes ();break ; }if (dock_present (ds) ||dock_in_progress (ds))
drivers/acpi/scan.c +3 -3
@@ -750,7 +750,7 @@ acpi_bus_extract_wakeup_device_power_package(struct acpi_device *device, device->wakeup.resources.handles[i] = element->reference.handle; } -acpi_gpe_can_wake (device->wakeup.gpe_device, device->wakeup.gpe_number); +acpi_setup_gpe_for_wake (device->wakeup.gpe_device, device->wakeup.gpe_number);return AE_OK ; }
@@ -1444,7 +1444,7 @@ int acpi_bus_start(struct acpi_device *device) result =acpi_bus_scan (device->handle, &ops,NULL ); -acpi_update_gpes (); +acpi_update_all_gpes ();return result; }
@@ -1561,7 +1561,7 @@ int __init acpi_scan_init(void)if (result)acpi_device_unregister (acpi_root,ACPI_BUS_REMOVAL_NORMAL );else -acpi_update_gpes (); +acpi_update_all_gpes ();return result; }
drivers/acpi/wakeup.c +2 -2
@@ -45,7 +45,7 @@ void acpi_enable_wakeup_devices(u8 sleep_state)acpi_enable_wakeup_device_power (dev, sleep_state);/* The wake-up power should have been enabled already. */ -acpi_gpe_wakeup (dev->wakeup.gpe_device, dev->wakeup.gpe_number, +acpi_set_gpe_wake_mask (dev->wakeup.gpe_device, dev->wakeup.gpe_number,ACPI_GPE_ENABLE ); } }
@@ -67,7 +67,7 @@ void acpi_disable_wakeup_devices(u8 sleep_state) || (sleep_state > (u32) dev->wakeup.sleep_state))continue ; -acpi_gpe_wakeup (dev->wakeup.gpe_device, dev->wakeup.gpe_number, +acpi_set_gpe_wake_mask (dev->wakeup.gpe_device, dev->wakeup.gpe_number,ACPI_GPE_DISABLE );if (dev->wakeup.state.enabled)
include/acpi/acpixf.h +3 -3
@@ -292,11 +292,11 @@ acpi_status acpi_enable_gpe(acpi_handle gpe_device, u32 gpe_number);acpi_status acpi_disable_gpe (acpi_handle gpe_device,u32 gpe_number); -acpi_status acpi_gpe_can_wake (acpi_handle gpe_device,u32 gpe_number); +acpi_status acpi_setup_gpe_for_wake (acpi_handle gpe_device,u32 gpe_number);acpi_status acpi_clear_gpe (acpi_handle gpe_device,u32 gpe_number); -acpi_status acpi_gpe_wakeup (acpi_handle gpe_device,u32 gpe_number,u8 action); +acpi_status acpi_set_gpe_wake_mask (acpi_handle gpe_device,u32 gpe_number,u8 action); acpi_status acpi_get_gpe_status(acpi_handle gpe_device,
@@ -315,7 +315,7 @@ acpi_install_gpe_block(acpi_handle gpe_device,acpi_status acpi_remove_gpe_block (acpi_handle gpe_device); -acpi_status acpi_update_gpes (void ); +acpi_status acpi_update_all_gpes (void );/* * Resource interfaces
include/acpi/actypes.h +2 -2
@@ -656,11 +656,11 @@ typedef u32 acpi_event_status;#define ACPI_GPE_MAX 0xFF#define ACPI_NUM_GPE 256 -/* Actions for acpi_gpe_wakeup, acpi_hw_low_set_gpe */ +/* Actions for acpi_set_gpe_wake_mask, acpi_hw_low_set_gpe */ #define ACPI_GPE_ENABLE 0#define ACPI_GPE_DISABLE 1 -#define ACPI_GPE_COND_ENABLE 2 +#define ACPI_GPE_CONDITIONAL_ENABLE 2/* * GPE info flags - Per GPE
Signature · 1 line
1.5.3