All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2 1/2] ACPICA: Update table load object initialization
@ 2019-07-11 16:58 Erik Schmauss
  2019-07-18  8:43 ` Rafael J. Wysocki
  0 siblings, 1 reply; 2+ messages in thread
From: Erik Schmauss @ 2019-07-11 16:58 UTC (permalink / raw)
  To: rjw, linux-acpi; +Cc: Erik Schmauss, Bob Moore

ACPICA commit c7ef9f3526765bed8930825dda1eed1a274b9668

Use the common internal "initialize objects" interface
Affects:
Load()
load_table()
acpi_load_table

Link: https://github.com/acpica/acpica/commit/c7ef9f35

Tested-by: Rong Chen <rong.a.chen@intel.com>
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Erik Schmauss <erik.schmauss@intel.com>
---
 drivers/acpi/acpica/exconfig.c | 18 ++++++++----------
 drivers/acpi/acpica/tbxfload.c | 10 +++-------
 2 files changed, 11 insertions(+), 17 deletions(-)

diff --git a/drivers/acpi/acpica/exconfig.c b/drivers/acpi/acpica/exconfig.c
index 587aeeeb5070..46a8baf28bd0 100644
--- a/drivers/acpi/acpica/exconfig.c
+++ b/drivers/acpi/acpica/exconfig.c
@@ -174,12 +174,11 @@ acpi_ex_load_table_op(struct acpi_walk_state *walk_state,
 		return_ACPI_STATUS(status);
 	}
 
-	/* Complete the initialization/resolution of package objects */
+	/* Complete the initialization/resolution of new objects */
 
-	status = acpi_ns_walk_namespace(ACPI_TYPE_PACKAGE, ACPI_ROOT_OBJECT,
-					ACPI_UINT32_MAX, 0,
-					acpi_ns_init_one_package, NULL, NULL,
-					NULL);
+	acpi_ex_exit_interpreter();
+	acpi_ns_initialize_objects();
+	acpi_ex_enter_interpreter();
 
 	/* Parameter Data (optional) */
 
@@ -437,12 +436,11 @@ acpi_ex_load_op(union acpi_operand_object *obj_desc,
 		return_ACPI_STATUS(status);
 	}
 
-	/* Complete the initialization/resolution of package objects */
+	/* Complete the initialization/resolution of new objects */
 
-	status = acpi_ns_walk_namespace(ACPI_TYPE_PACKAGE, ACPI_ROOT_OBJECT,
-					ACPI_UINT32_MAX, 0,
-					acpi_ns_init_one_package, NULL, NULL,
-					NULL);
+	acpi_ex_exit_interpreter();
+	acpi_ns_initialize_objects();
+	acpi_ex_enter_interpreter();
 
 	/* Store the ddb_handle into the Target operand */
 
diff --git a/drivers/acpi/acpica/tbxfload.c b/drivers/acpi/acpica/tbxfload.c
index ef8f8a9f3c9c..86f1693f6d29 100644
--- a/drivers/acpi/acpica/tbxfload.c
+++ b/drivers/acpi/acpica/tbxfload.c
@@ -297,15 +297,11 @@ acpi_status acpi_load_table(struct acpi_table_header *table)
 	status = acpi_tb_install_and_load_table(ACPI_PTR_TO_PHYSADDR(table),
 						ACPI_TABLE_ORIGIN_EXTERNAL_VIRTUAL,
 						FALSE, &table_index);
-
 	if (ACPI_SUCCESS(status)) {
-		/* Complete the initialization/resolution of package objects */
 
-		status = acpi_ns_walk_namespace(ACPI_TYPE_PACKAGE,
-						ACPI_ROOT_OBJECT,
-						ACPI_UINT32_MAX, 0,
-						acpi_ns_init_one_package,
-						NULL, NULL, NULL);
+		/* Complete the initialization/resolution of new objects */
+
+		acpi_ns_initialize_objects();
 	}
 
 	return_ACPI_STATUS(status);
-- 
2.17.2


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH v2 1/2] ACPICA: Update table load object initialization
  2019-07-11 16:58 [PATCH v2 1/2] ACPICA: Update table load object initialization Erik Schmauss
@ 2019-07-18  8:43 ` Rafael J. Wysocki
  0 siblings, 0 replies; 2+ messages in thread
From: Rafael J. Wysocki @ 2019-07-18  8:43 UTC (permalink / raw)
  To: Erik Schmauss; +Cc: linux-acpi, Bob Moore

On Thursday, July 11, 2019 6:58:21 PM CEST Erik Schmauss wrote:
> ACPICA commit c7ef9f3526765bed8930825dda1eed1a274b9668
> 
> Use the common internal "initialize objects" interface
> Affects:
> Load()
> load_table()
> acpi_load_table
> 
> Link: https://github.com/acpica/acpica/commit/c7ef9f35
> 
> Tested-by: Rong Chen <rong.a.chen@intel.com>
> Signed-off-by: Bob Moore <robert.moore@intel.com>
> Signed-off-by: Erik Schmauss <erik.schmauss@intel.com>
> ---
>  drivers/acpi/acpica/exconfig.c | 18 ++++++++----------
>  drivers/acpi/acpica/tbxfload.c | 10 +++-------
>  2 files changed, 11 insertions(+), 17 deletions(-)
> 
> diff --git a/drivers/acpi/acpica/exconfig.c b/drivers/acpi/acpica/exconfig.c
> index 587aeeeb5070..46a8baf28bd0 100644
> --- a/drivers/acpi/acpica/exconfig.c
> +++ b/drivers/acpi/acpica/exconfig.c
> @@ -174,12 +174,11 @@ acpi_ex_load_table_op(struct acpi_walk_state *walk_state,
>  		return_ACPI_STATUS(status);
>  	}
>  
> -	/* Complete the initialization/resolution of package objects */
> +	/* Complete the initialization/resolution of new objects */
>  
> -	status = acpi_ns_walk_namespace(ACPI_TYPE_PACKAGE, ACPI_ROOT_OBJECT,
> -					ACPI_UINT32_MAX, 0,
> -					acpi_ns_init_one_package, NULL, NULL,
> -					NULL);
> +	acpi_ex_exit_interpreter();
> +	acpi_ns_initialize_objects();
> +	acpi_ex_enter_interpreter();
>  
>  	/* Parameter Data (optional) */
>  
> @@ -437,12 +436,11 @@ acpi_ex_load_op(union acpi_operand_object *obj_desc,
>  		return_ACPI_STATUS(status);
>  	}
>  
> -	/* Complete the initialization/resolution of package objects */
> +	/* Complete the initialization/resolution of new objects */
>  
> -	status = acpi_ns_walk_namespace(ACPI_TYPE_PACKAGE, ACPI_ROOT_OBJECT,
> -					ACPI_UINT32_MAX, 0,
> -					acpi_ns_init_one_package, NULL, NULL,
> -					NULL);
> +	acpi_ex_exit_interpreter();
> +	acpi_ns_initialize_objects();
> +	acpi_ex_enter_interpreter();
>  
>  	/* Store the ddb_handle into the Target operand */
>  
> diff --git a/drivers/acpi/acpica/tbxfload.c b/drivers/acpi/acpica/tbxfload.c
> index ef8f8a9f3c9c..86f1693f6d29 100644
> --- a/drivers/acpi/acpica/tbxfload.c
> +++ b/drivers/acpi/acpica/tbxfload.c
> @@ -297,15 +297,11 @@ acpi_status acpi_load_table(struct acpi_table_header *table)
>  	status = acpi_tb_install_and_load_table(ACPI_PTR_TO_PHYSADDR(table),
>  						ACPI_TABLE_ORIGIN_EXTERNAL_VIRTUAL,
>  						FALSE, &table_index);
> -
>  	if (ACPI_SUCCESS(status)) {
> -		/* Complete the initialization/resolution of package objects */
>  
> -		status = acpi_ns_walk_namespace(ACPI_TYPE_PACKAGE,
> -						ACPI_ROOT_OBJECT,
> -						ACPI_UINT32_MAX, 0,
> -						acpi_ns_init_one_package,
> -						NULL, NULL, NULL);
> +		/* Complete the initialization/resolution of new objects */
> +
> +		acpi_ns_initialize_objects();
>  	}
>  
>  	return_ACPI_STATUS(status);
> 

Applied, thanks!





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-07-18  8:43 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-11 16:58 [PATCH v2 1/2] ACPICA: Update table load object initialization Erik Schmauss
2019-07-18  8:43 ` Rafael J. Wysocki

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.