On Sat, Nov 21, 2020 at 9:31 PM Hans de Goede <hdegoede@xxxxxxxxxx> wrote: > > When battery devices get added during the second step of the now 2 step > scan-process, then acpi_walk_dep_device_list() may have already been > called for some deps during the first step. > > In this case acpi_device_dep_initialize() should not add these deps to > the acpi_dep_list; and it should not increase adev->dep_unmet. > > Add a check for already registered (and bound to a driver) devices to > acpi_device_dep_initialize(). This fixes battery devices (which honor the > dep_unmet value) not working with acpi.defer_scan_based_on_dep=1. I'd rather avoid having to fix this issue at all ...