Re: [PATCH v2 1/2] ACPICA: Update table load object initialization

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx>
> Signed-off-by: Bob Moore <robert.moore@xxxxxxxxx>
> Signed-off-by: Erik Schmauss <erik.schmauss@xxxxxxxxx>
> ---
>  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!







[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux