On Wed, Jun 09, 2021 at 07:51:30PM +0200, Hans de Goede wrote: > context->ret.pointer already gets set to NULL at the beginning of > acpi_run_osc() and it only gets assigned a new value in the success > path near the end of acpi_run_osc(), so the clearing of > context->ret.pointer (when status != AE_OK) at the end of > acpi_run_osc() is redundant since it will always already be NULL when > status != AE_OK. > > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> Reviewed-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>