patch-2.4.0-test3 linux/drivers/acpi/interpreter/amcreate.c

Next file: linux/drivers/acpi/interpreter/amdyadic.c
Previous file: linux/drivers/acpi/interpreter/amconfig.c
Back to the patch index
Back to the overall index

diff -u --recursive --new-file v2.4.0-test2/linux/drivers/acpi/interpreter/amcreate.c linux/drivers/acpi/interpreter/amcreate.c
@@ -0,0 +1,871 @@
+
+/******************************************************************************
+ *
+ * Module Name: amcreate - Named object creation
+ *
+ *****************************************************************************/
+
+/*
+ *  Copyright (C) 2000 R. Byron Moore
+ *
+ *  This program is free software; you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation; either version 2 of the License, or
+ *  (at your option) any later version.
+ *
+ *  This program is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with this program; if not, write to the Free Software
+ *  Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ */
+
+
+#include "acpi.h"
+#include "parser.h"
+#include "interp.h"
+#include "amlcode.h"
+#include "namesp.h"
+#include "events.h"
+#include "dispatch.h"
+
+
+#define _COMPONENT          INTERPRETER
+	 MODULE_NAME         ("amcreate");
+
+
+/*****************************************************************************
+ *
+ * FUNCTION:    Acpi_aml_exec_create_field
+ *
+ * PARAMETERS:  Opcode              - The opcode to be executed
+ *              Operands            - List of operands for the opcode
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Execute Create_field operators: Create_bit_field_op,
+ *              Create_byte_field_op, Create_word_field_op, Create_dWord_field_op,
+ *              Create_field_op (which define fields in buffers)
+ *
+ * ALLOCATION:  Deletes Create_field_op's count operand descriptor
+ *
+ *
+ *  ACPI SPECIFICATION REFERENCES:
+ *  Def_create_bit_field := Create_bit_field_op Src_buf Bit_idx   Name_string
+ *  Def_create_byte_field := Create_byte_field_op Src_buf Byte_idx Name_string
+ *  Def_create_dWord_field := Create_dWord_field_op Src_buf Byte_idx Name_string
+ *  Def_create_field    :=  Create_field_op     Src_buf Bit_idx   Num_bits    Name_string
+ *  Def_create_word_field := Create_word_field_op Src_buf Byte_idx Name_string
+ *  Bit_index           :=  Term_arg=>Integer
+ *  Byte_index          :=  Term_arg=>Integer
+ *  Num_bits            :=  Term_arg=>Integer
+ *  Source_buff         :=  Term_arg=>Buffer
+ *
+ ****************************************************************************/
+
+ACPI_STATUS
+acpi_aml_exec_create_field (
+	u16                     opcode,
+	ACPI_WALK_STATE         *walk_state)
+{
+	ACPI_OBJECT_INTERNAL    *res_desc = NULL;
+	ACPI_OBJECT_INTERNAL    *cnt_desc = NULL;
+	ACPI_OBJECT_INTERNAL    *off_desc = NULL;
+	ACPI_OBJECT_INTERNAL    *src_desc = NULL;
+	ACPI_OBJECT_INTERNAL    *field_desc;
+	ACPI_OBJECT_INTERNAL    *obj_desc;
+	OBJECT_TYPE_INTERNAL    res_type;
+	ACPI_STATUS             status;
+	u32                     num_operands = 3;
+	u32                     offset;
+	u32                     bit_offset;
+	u16                     bit_count;
+	u8                      type_found;
+
+
+	/* Resolve the operands */
+
+	status = acpi_aml_resolve_operands (opcode, WALK_OPERANDS);
+
+	/* Get the operands */
+
+	status |= acpi_ds_obj_stack_pop_object (&res_desc, walk_state);
+	if (AML_CREATE_FIELD_OP == opcode) {
+		num_operands = 4;
+		status |= acpi_ds_obj_stack_pop_object (&cnt_desc, walk_state);
+	}
+
+	status |= acpi_ds_obj_stack_pop_object (&off_desc, walk_state);
+	status |= acpi_ds_obj_stack_pop_object (&src_desc, walk_state);
+
+	if (status != AE_OK) {
+		/* Invalid parameters on object stack  */
+
+		acpi_aml_append_operand_diag (_THIS_MODULE, __LINE__, opcode,
+				  WALK_OPERANDS, 3);
+		goto cleanup;
+	}
+
+
+	offset = off_desc->number.value;
+
+
+	/*
+	 * If Res_desc is a Name, it will be a direct name pointer after
+	 * Acpi_aml_resolve_operands()
+	 */
+
+	if (!VALID_DESCRIPTOR_TYPE (res_desc, ACPI_DESC_TYPE_NAMED)) {
+		status = AE_AML_OPERAND_TYPE;
+		goto cleanup;
+	}
+
+
+	/*
+	 * Setup the Bit offsets and counts, according to the opcode
+	 */
+
+	switch (opcode)
+	{
+
+	/* Def_create_bit_field */
+
+	case AML_BIT_FIELD_OP:
+
+		/* Offset is in bits, Field is a bit */
+
+		bit_offset = offset;
+		bit_count = 1;
+		break;
+
+
+	/* Def_create_byte_field */
+
+	case AML_BYTE_FIELD_OP:
+
+		/* Offset is in bytes, field is a byte */
+
+		bit_offset = 8 * offset;
+		bit_count = 8;
+		break;
+
+
+	/* Def_create_word_field */
+
+	case AML_WORD_FIELD_OP:
+
+		/* Offset is in bytes, field is a word */
+
+		bit_offset = 8 * offset;
+		bit_count = 16;
+		break;
+
+
+	/* Def_create_dWord_field */
+
+	case AML_DWORD_FIELD_OP:
+
+		/* Offset is in bytes, field is a dword */
+
+		bit_offset = 8 * offset;
+		bit_count = 32;
+		break;
+
+
+	/* Def_create_field */
+
+	case AML_CREATE_FIELD_OP:
+
+		/* Offset is in bits, count is in bits */
+
+		bit_offset = offset;
+		bit_count = (u16) cnt_desc->number.value;
+		break;
+
+
+	default:
+
+		status = AE_AML_BAD_OPCODE;
+		goto cleanup;
+	}
+
+
+	/*
+	 * Setup field according to the object type
+	 */
+
+	switch (src_desc->common.type)
+	{
+
+	/* Source_buff :=  Term_arg=>Buffer */
+
+	case ACPI_TYPE_BUFFER:
+
+		if (bit_offset + (u32) bit_count >
+			(8 * (u32) src_desc->buffer.length))
+		{
+			status = AE_AML_BUFFER_LIMIT;
+			goto cleanup;
+		}
+
+
+		/* Allocate an object for the field */
+
+		field_desc = acpi_cm_create_internal_object (ACPI_TYPE_FIELD_UNIT);
+		if (!field_desc) {
+			status = AE_NO_MEMORY;
+			goto cleanup;
+		}
+
+		/* Construct the field object */
+
+		field_desc->field_unit.access     = (u8) ACCESS_ANY_ACC;
+		field_desc->field_unit.lock_rule  = (u8) GLOCK_NEVER_LOCK;
+		field_desc->field_unit.update_rule = (u8) UPDATE_PRESERVE;
+		field_desc->field_unit.length     = bit_count;
+		field_desc->field_unit.bit_offset = (u8) (bit_offset % 8);
+		field_desc->field_unit.offset     = DIV_8 (bit_offset);
+		field_desc->field_unit.container  = src_desc;
+		field_desc->field_unit.sequence   = src_desc->buffer.sequence;
+
+		/* An additional reference for Src_desc */
+
+		acpi_cm_add_reference (src_desc);
+
+		break;
+
+
+	/* Improper object type */
+
+	default:
+
+		type_found = src_desc->common.type;
+
+		if ((type_found > (u8) INTERNAL_TYPE_REFERENCE) ||
+			!acpi_cm_valid_object_type (type_found))
+
+
+		status = AE_AML_OPERAND_TYPE;
+		goto cleanup;
+	}
+
+
+	if (AML_CREATE_FIELD_OP == opcode) {
+		/* Delete object descriptor unique to Create_field */
+
+		acpi_cm_remove_reference (cnt_desc);
+		cnt_desc = NULL;
+	}
+
+	/*
+	 * This operation is supposed to cause the destination Name to refer
+	 * to the defined Field_unit -- it must not store the constructed
+	 * Field_unit object (or its current value) in some location that the
+	 * Name may already be pointing to.  So, if the Name currently contains
+	 * a reference which would cause Acpi_aml_exec_store() to perform an indirect
+	 * store rather than setting the value of the Name itself, clobber that
+	 * reference before calling Acpi_aml_exec_store().
+	 */
+
+	res_type = acpi_ns_get_type (res_desc);
+
+	/* Type of Name's existing value */
+
+	switch (res_type)
+	{
+
+	case ACPI_TYPE_FIELD_UNIT:
+
+	case INTERNAL_TYPE_ALIAS:
+	case INTERNAL_TYPE_BANK_FIELD:
+	case INTERNAL_TYPE_DEF_FIELD:
+	case INTERNAL_TYPE_INDEX_FIELD:
+
+		obj_desc = acpi_ns_get_attached_object (res_desc);
+		if (obj_desc) {
+			/*
+			 * There is an existing object here;  delete it and zero out the
+			 * NTE
+			 */
+
+			acpi_cm_remove_reference (obj_desc);
+			acpi_ns_attach_object (res_desc, NULL, ACPI_TYPE_ANY);
+		}
+
+		/* Set the type to ANY (or the store below will fail) */
+
+		((ACPI_NAMED_OBJECT*) res_desc)->type = ACPI_TYPE_ANY;
+
+		break;
+
+
+	default:
+
+		break;
+	}
+
+
+	/* Store constructed field descriptor in result location */
+
+	status = acpi_aml_exec_store (field_desc, res_desc);
+
+	/*
+	 * If the field descriptor was not physically stored (or if a failure
+	 * above), we must delete it
+	 */
+	if (field_desc->common.reference_count <= 1) {
+		acpi_cm_remove_reference (field_desc);
+	}
+
+
+cleanup:
+
+	/* Always delete the operands */
+
+	acpi_cm_remove_reference (off_desc);
+	acpi_cm_remove_reference (src_desc);
+
+	if (AML_CREATE_FIELD_OP == opcode) {
+		acpi_cm_remove_reference (cnt_desc);
+	}
+
+	/* On failure, delete the result descriptor */
+
+	if (ACPI_FAILURE (status)) {
+		acpi_cm_remove_reference (res_desc); /* Result descriptor */
+	}
+
+	return (status);
+}
+
+
+/*****************************************************************************
+ *
+ * FUNCTION:    Acpi_aml_exec_create_alias
+ *
+ * PARAMETERS:  Operands            - List of operands for the opcode
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Create a new named alias
+ *
+ ****************************************************************************/
+
+ACPI_STATUS
+acpi_aml_exec_create_alias (
+	ACPI_WALK_STATE         *walk_state)
+{
+	ACPI_NAMED_OBJECT       *src_entry;
+	ACPI_NAMED_OBJECT       *alias_entry;
+	ACPI_STATUS             status;
+
+
+	/* Get the source/alias operands (both NTEs) */
+
+	status = acpi_ds_obj_stack_pop_object ((ACPI_OBJECT_INTERNAL **) &src_entry,
+			 walk_state);
+	if (ACPI_FAILURE (status)) {
+		return (status);
+	}
+
+	/* Don't pop it, it gets popped later */
+
+	alias_entry = acpi_ds_obj_stack_get_value (0, walk_state);
+
+	/* Add an additional reference to the object */
+
+	acpi_cm_add_reference (src_entry->object);
+
+	/*
+	 * Attach the original source NTE to the new Alias NTE.
+	 */
+	status = acpi_ns_attach_object (alias_entry, src_entry->object,
+			   src_entry->type);
+
+
+	/*
+	 * The new alias assumes the type of the source, but it points
+	 * to the same object.  The reference count of the object has two
+	 * additional references to prevent deletion out from under either the
+	 * source or the alias NTE
+	 */
+
+	/* Since both operands are NTEs, we don't need to delete them */
+
+	return (status);
+}
+
+
+/*****************************************************************************
+ *
+ * FUNCTION:    Acpi_aml_exec_create_event
+ *
+ * PARAMETERS:  None
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Create a new event object
+ *
+ ****************************************************************************/
+
+ACPI_STATUS
+acpi_aml_exec_create_event (
+	ACPI_WALK_STATE         *walk_state)
+{
+	ACPI_STATUS             status;
+	ACPI_OBJECT_INTERNAL    *obj_desc;
+
+
+ BREAKPOINT3;
+
+	obj_desc = acpi_cm_create_internal_object (ACPI_TYPE_EVENT);
+	if (!obj_desc) {
+		status = AE_NO_MEMORY;
+		goto cleanup;
+	}
+
+	/* Create the actual OS semaphore */
+
+	/* TBD: [Investigate] should be created with 0 or 1 units? */
+
+	status = acpi_os_create_semaphore (ACPI_NO_UNIT_LIMIT, 1,
+			   &obj_desc->event.semaphore);
+	if (ACPI_FAILURE (status)) {
+		acpi_cm_remove_reference (obj_desc);
+		goto cleanup;
+	}
+
+	/* Attach object to the NTE */
+
+	status = acpi_ns_attach_object (acpi_ds_obj_stack_get_value (0, walk_state),
+			   obj_desc, (u8) ACPI_TYPE_EVENT);
+	if (ACPI_FAILURE (status)) {
+		acpi_os_delete_semaphore (obj_desc->event.semaphore);
+		acpi_cm_remove_reference (obj_desc);
+		goto cleanup;
+	}
+
+
+cleanup:
+
+	return (status);
+}
+
+
+/*****************************************************************************
+ *
+ * FUNCTION:    Acpi_aml_exec_create_mutex
+ *
+ * PARAMETERS:  Interpreter_mode    - Current running mode (load1/Load2/Exec)
+ *              Operands            - List of operands for the opcode
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Create a new mutex object
+ *
+ ****************************************************************************/
+
+ACPI_STATUS
+acpi_aml_exec_create_mutex (
+	ACPI_WALK_STATE         *walk_state)
+{
+	ACPI_STATUS             status = AE_OK;
+	ACPI_OBJECT_INTERNAL    *sync_desc;
+	ACPI_OBJECT_INTERNAL    *obj_desc;
+
+
+	/* Get the operand */
+
+	status = acpi_ds_obj_stack_pop_object (&sync_desc, walk_state);
+	if (ACPI_FAILURE (status)) {
+		return (status);
+	}
+
+	/* Attempt to allocate a new object */
+
+	obj_desc = acpi_cm_create_internal_object (ACPI_TYPE_MUTEX);
+	if (!obj_desc) {
+		status = AE_NO_MEMORY;
+		goto cleanup;
+	}
+
+	/* Create the actual OS semaphore */
+
+	status = acpi_os_create_semaphore (1, 1, &obj_desc->mutex.semaphore);
+	if (ACPI_FAILURE (status)) {
+		acpi_cm_remove_reference (obj_desc);
+		goto cleanup;
+	}
+
+	obj_desc->mutex.sync_level = (u8) sync_desc->number.value;
+
+	/* Obj_desc was on the stack top, and the name is below it */
+
+	status = acpi_ns_attach_object (acpi_ds_obj_stack_get_value (0, walk_state),
+			  obj_desc, (u8) ACPI_TYPE_MUTEX);
+	if (ACPI_FAILURE (status)) {
+		acpi_os_delete_semaphore (obj_desc->mutex.semaphore);
+		acpi_cm_remove_reference (obj_desc);
+		goto cleanup;
+	}
+
+
+cleanup:
+
+	/* Always delete the operand */
+
+	acpi_cm_remove_reference (sync_desc);
+
+	return (status);
+}
+
+
+/*****************************************************************************
+ *
+ * FUNCTION:    Acpi_aml_exec_create_region
+ *
+ * PARAMETERS:  Aml_ptr             - Pointer to the region declaration AML
+ *              Aml_length          - Max length of the declaration AML
+ *              Operands            - List of operands for the opcode
+ *              Interpreter_mode    - Load1/Load2/Execute
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Create a new operation region object
+ *
+ ****************************************************************************/
+
+ACPI_STATUS
+acpi_aml_exec_create_region (
+	u8                      *aml_ptr,
+	u32                     aml_length,
+	u32                     region_space,
+	ACPI_WALK_STATE         *walk_state)
+{
+	ACPI_STATUS             status;
+	ACPI_OBJECT_INTERNAL    *obj_desc_region;
+	ACPI_HANDLE             *entry;
+
+
+	if (region_space >= NUM_REGION_TYPES) {
+		/* TBD: [Errors] should this return an error, or should we just keep
+		 * going? */
+
+		REPORT_WARNING ("Unable to decode the Region_space");
+	}
+
+
+	/* Get the NTE from the object stack  */
+
+	entry = acpi_ds_obj_stack_get_value (0, walk_state);
+
+
+	/* Create the region descriptor */
+
+	obj_desc_region = acpi_cm_create_internal_object (ACPI_TYPE_REGION);
+	if (!obj_desc_region) {
+		status = AE_NO_MEMORY;
+		goto cleanup;
+	}
+
+	/*
+	 * Allocate a method object for this region.
+	 */
+	obj_desc_region->region.method = acpi_cm_create_internal_object (
+			 ACPI_TYPE_METHOD);
+	if (!obj_desc_region->region.method) {
+		status = AE_NO_MEMORY;
+		goto cleanup;
+	}
+
+	/* Init the region from the operands */
+
+	obj_desc_region->region.space_id    = (u16) region_space;
+	obj_desc_region->region.address     = 0;
+	obj_desc_region->region.length      = 0;
+	obj_desc_region->region.region_flags = 0;
+
+	/*
+	 * Remember location in AML stream of address & length
+	 * operands since they need to be evaluated at run time.
+	 */
+	obj_desc_region->region.method->method.pcode     = aml_ptr;
+	obj_desc_region->region.method->method.pcode_length = aml_length;
+
+
+	/* Install the new region object in the parent NTE */
+
+	obj_desc_region->region.nte = (ACPI_NAMED_OBJECT*) entry;
+
+	status = acpi_ns_attach_object (entry, obj_desc_region,
+			  (u8) ACPI_TYPE_REGION);
+	if (ACPI_FAILURE (status)) {
+		goto cleanup;
+	}
+
+cleanup:
+
+	if (status != AE_OK) {
+		/* Delete region object and method subobject */
+
+		if (obj_desc_region) {
+			/* Remove deletes both objects! */
+
+			acpi_cm_remove_reference (obj_desc_region);
+			obj_desc_region = NULL;
+		}
+	}
+
+
+	/*
+	 * If we have a valid region, initialize it
+	 */
+	if (obj_desc_region) {
+		/*
+		 *  TBD: [Errors] Is there anything we can or could do when this
+		 *          fails?
+		 *  We need to do something useful with a failure.
+		 */
+		/* Namespace IS locked */
+
+		(void *) acpi_ev_initialize_region (obj_desc_region, TRUE);
+
+	}
+
+	return (status);
+}
+
+
+/*****************************************************************************
+ *
+ * FUNCTION:    Acpi_aml_exec_create_processor
+ *
+ * PARAMETERS:  Op              - Op containing the Processor definition and
+ *                                args
+ *              Processor_nTE   - NTE for the containing NTE
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Create a new processor object and populate the fields
+ *
+ ****************************************************************************/
+
+ACPI_STATUS
+acpi_aml_exec_create_processor (
+	ACPI_GENERIC_OP         *op,
+	ACPI_HANDLE             processor_nTE)
+{
+	ACPI_STATUS             status;
+	ACPI_GENERIC_OP         *arg;
+	ACPI_OBJECT_INTERNAL    *obj_desc;
+
+
+	obj_desc = acpi_cm_create_internal_object (ACPI_TYPE_PROCESSOR);
+	if (!obj_desc) {
+		status = AE_NO_MEMORY;
+		return (status);
+	}
+
+	/* Install the new processor object in the parent NTE */
+
+	status = acpi_ns_attach_object (processor_nTE, obj_desc,
+			   (u8) ACPI_TYPE_PROCESSOR);
+	if (ACPI_FAILURE (status)) {
+		return(status);
+	}
+
+	arg = op->value.arg;
+
+	/* check existence */
+
+	if (!arg) {
+		status = AE_AML_NO_OPERAND;
+		return (status);
+	}
+
+	/* First arg is the Processor ID */
+
+	obj_desc->processor.proc_id = (u8) arg->value.integer;
+
+	/* Move to next arg and check existence */
+
+	arg = arg->next;
+	if (!arg) {
+		status = AE_AML_NO_OPERAND;
+		return (status);
+	}
+
+	/* Second arg is the PBlock Address */
+
+	obj_desc->processor.pblk_address = (ACPI_IO_ADDRESS) arg->value.integer;
+
+	/* Move to next arg and check existence */
+
+	arg = arg->next;
+	if (!arg) {
+		status = AE_AML_NO_OPERAND;
+		return (status);
+	}
+
+	/* Third arg is the PBlock Length */
+
+	obj_desc->processor.pblk_length = (u8) arg->value.integer;
+
+	return (AE_OK);
+}
+
+
+/*****************************************************************************
+ *
+ * FUNCTION:    Acpi_aml_exec_create_power_resource
+ *
+ * PARAMETERS:  Op              - Op containing the Power_resource definition
+ *                                and args
+ *              Power_res_nTE   - NTE for the containing NTE
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Create a new Power_resource object and populate the fields
+ *
+ ****************************************************************************/
+
+ACPI_STATUS
+acpi_aml_exec_create_power_resource (
+	ACPI_GENERIC_OP         *op,
+	ACPI_HANDLE             power_res_nTE)
+{
+	ACPI_STATUS             status;
+	ACPI_GENERIC_OP         *arg;
+	ACPI_OBJECT_INTERNAL    *obj_desc;
+
+
+	obj_desc = acpi_cm_create_internal_object (ACPI_TYPE_POWER);
+	if (!obj_desc) {
+		status = AE_NO_MEMORY;
+		return (status);
+	}
+
+	/* Install the new power resource object in the parent NTE */
+
+	status = acpi_ns_attach_object (power_res_nTE, obj_desc,
+			  (u8) ACPI_TYPE_POWER);
+	if (ACPI_FAILURE (status)) {
+		return(status);
+	}
+
+	arg = op->value.arg;
+
+	/* check existence */
+
+	if (!arg) {
+		status = AE_AML_NO_OPERAND;
+		return (status);
+	}
+
+	/* First arg is the System_level */
+
+	obj_desc->power_resource.system_level = (u8) arg->value.integer;
+
+	/* Move to next arg and check existence */
+
+	arg = arg->next;
+	if (!arg) {
+		status = AE_AML_NO_OPERAND;
+		return (status);
+	}
+
+	/* Second arg is the PBlock Address */
+
+	obj_desc->power_resource.resource_order = (u16) arg->value.integer;
+
+	return (AE_OK);
+}
+
+
+/*****************************************************************************
+ *
+ * FUNCTION:    Acpi_aml_exec_create_method
+ *
+ * PARAMETERS:  Interpreter_mode    - Current running mode (load1/Load2/Exec)
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Create a new mutex object
+ *
+ ****************************************************************************/
+
+ACPI_STATUS
+acpi_aml_exec_create_method (
+	u8                      *aml_ptr,
+	u32                     aml_length,
+	u32                     method_flags,
+	ACPI_HANDLE             method)
+{
+	ACPI_OBJECT_INTERNAL    *obj_desc;
+	ACPI_STATUS             status;
+
+
+	/* Create a new method object */
+
+	obj_desc = acpi_cm_create_internal_object (ACPI_TYPE_METHOD);
+	if (!obj_desc) {
+	   return (AE_NO_MEMORY);
+	}
+
+	/* Get the method's AML pointer/length from the Op */
+
+	obj_desc->method.pcode      = aml_ptr;
+	obj_desc->method.pcode_length = aml_length;
+
+	/*
+	 * First argument is the Method Flags (contains parameter count for the
+	 * method)
+	 */
+
+	obj_desc->method.method_flags = (u8) method_flags;
+	obj_desc->method.param_count = (u8) (method_flags &
+			  METHOD_FLAGS_ARG_COUNT);
+
+	/*
+	 * Get the concurrency count
+	 * If required, a semaphore will be created for this method when it is
+	 * parsed.
+	 *
+	 * TBD: [Future]  for APCI 2.0, there will be a Sync_level value, not
+	 * just a flag
+	 * Concurrency = Sync_level + 1;.
+	 */
+
+	if (method_flags & METHOD_FLAGS_SERIALIZED) {
+		obj_desc->method.concurrency = 1;
+	}
+	else {
+		obj_desc->method.concurrency = INFINITE_CONCURRENCY;
+	}
+
+	/* Mark the Method as not parsed yet */
+
+	obj_desc->method.parser_op  = NULL;
+
+	/*
+	 * Another  +1 gets added when Acpi_psx_execute is called,
+	 * no need for: Obj_desc->Method.Pcode++;
+	 */
+
+	obj_desc->method.acpi_table = NULL; /* TBD: [Restructure] was (u8 *) Pcode_addr; */
+	obj_desc->method.table_length = 0;  /* TBD: [Restructure] needed? (u32) (Walk_state->aml_end - Pcode_addr); */
+
+	/* Attach the new object to the method NTE */
+
+	status = acpi_ns_attach_object (method, obj_desc, (u8) ACPI_TYPE_METHOD);
+	if (ACPI_FAILURE (status)) {
+		acpi_cm_free (obj_desc);
+	}
+
+	return (status);
+}
+
+
FUNET's LINUX-ADM group, linux-adm@nic.funet.fi
TCL-scripts by Sam Shen (who was at: slshen@lbl.gov)