On Fri, Jun 5, 2020 at 9:40 PM Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > > On Fri, Jun 5, 2020 at 5:11 PM Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote: > > ... > > > + if (!refcount) { > > + write_lock_irq(&acpi_ioremaps_list_lock); > > + > > + list_del(&map->list); > > + > > + write_unlock_irq(&acpi_ioremaps_list_lock); > > + } > > return refcount; > > It seems we can decrease indentation level at the same time: > > if (refcount) > return refcount; > > ... > return 0; Right, but the patch will need to be dropped anyway I think.