On Thu, Jun 10, 2021 at 9:01 AM Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> wrote: > > 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> Applied as 5.14 material, thanks!
