Re: [PATCH 02/11] ACPI: acpica: Constify pathname argument for acpi_get_handle()

Rafael J. Wysocki
2022-05-17T16:21:59Z

On Fri, May 6, 2022 at 2:58 PM Sakari Ailus
<sakari.ailus@linux.intel.com> wrote:
>
> acpi_get_handle() uses the pathname argument to find a handle related to
> that pathname but it does not need to modify it. Make it const, in order
> to be able to pass const pathname to it.
>
> Cc: "Moore, Robert" <robert.moore@intel.com>
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>

Which patches in the rest of the series depend on this one?

> ---
> drivers/acpi/acpica/nsxfname.c | 2 +-
> include/acpi/acpixf.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/acpi/acpica/nsxfname.c b/drivers/acpi/acpica/nsxfname.c
> index b2cfdfef31947..a0592d15dd37c 100644
> --- a/drivers/acpi/acpica/nsxfname.c
> +++ b/drivers/acpi/acpica/nsxfname.c
> @@ -44,7 +44,7 @@ static char *acpi_ns_copy_device_id(struct acpi_pnp_device_id *dest,
>
> acpi_status
> acpi_get_handle(acpi_handle parent,
> - acpi_string pathname, acpi_handle *ret_handle)
> + const char *pathname, acpi_handle *ret_handle)
> {
> acpi_status status;
> struct acpi_namespace_node *node = NULL;
> diff --git a/include/acpi/acpixf.h b/include/acpi/acpixf.h
> index 67c0b9e734b64..085f23d833349 100644
> --- a/include/acpi/acpixf.h
> +++ b/include/acpi/acpixf.h
> @@ -526,7 +526,7 @@ ACPI_EXTERNAL_RETURN_STATUS(acpi_status
> struct acpi_buffer *ret_path_ptr))
> ACPI_EXTERNAL_RETURN_STATUS(acpi_status
> acpi_get_handle(acpi_handle parent,
> - acpi_string pathname,
> + const char *pathname,
> acpi_handle *ret_handle))
> ACPI_EXTERNAL_RETURN_STATUS(acpi_status
> acpi_attach_data(acpi_handle object,
> --
> 2.30.2
>