Re: [PATCH] tools/power/acpi/os_specific/service_layers/osunixxf.c:remove unnecessary (void*) conversions

From: Rafael J. Wysocki
Date: Tue Aug 23 2022 - 14:50:05 EST


On Mon, Aug 22, 2022 at 8:24 AM Dong Chuanjian <chuanjian@xxxxxxxxxxxx> wrote:
>
> remove unnecessary void* type casting
>
> Signed-off-by: Dong Chuanjian <chuanjian@xxxxxxxxxxxx>

This is ACPICA code, so please submit changes against it to the
upstream ACPICA project on GitHub. Thanks!

> diff --git a/tools/power/acpi/os_specific/service_layers/osunixxf.c b/tools/power/acpi/os_specific/service_layers/osunixxf.c
> index b3651a04d68c..a7e65fb95caf 100644
> --- a/tools/power/acpi/os_specific/service_layers/osunixxf.c
> +++ b/tools/power/acpi/os_specific/service_layers/osunixxf.c
> @@ -679,7 +679,7 @@ acpi_os_create_semaphore(u32 max_units,
>
> acpi_status acpi_os_delete_semaphore(acpi_handle handle)
> {
> - sem_t *sem = (sem_t *) handle;
> + sem_t *sem = handle;
>
> if (!sem) {
> return (AE_BAD_PARAMETER);
> @@ -715,7 +715,7 @@ acpi_status
> acpi_os_wait_semaphore(acpi_handle handle, u32 units, u16 msec_timeout)
> {
> acpi_status status = AE_OK;
> - sem_t *sem = (sem_t *) handle;
> + sem_t *sem = handle;
> int ret_val;
> #ifndef ACPI_USE_ALTERNATE_TIMEOUT
> struct timespec time;
> @@ -832,7 +832,7 @@ acpi_os_wait_semaphore(acpi_handle handle, u32 units, u16 msec_timeout)
>
> acpi_status acpi_os_signal_semaphore(acpi_handle handle, u32 units)
> {
> - sem_t *sem = (sem_t *) handle;
> + sem_t *sem = handle;
>
> if (!sem) {
> return (AE_BAD_PARAMETER);
> --
> 2.18.2
>