On Thu, Jun 9, 2022 at 5:26 PM Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > > On Thu, Jun 09, 2022 at 03:54:40PM +0200, Rafael J. Wysocki wrote: > > From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx> > > > > Instead of walking the list of children of an ACPI device directly > > in order to find the child matching a given bus address, use > > acpi_find_child_by_adr() for this purpose. > > ... > > > if (!adev) > > return NULL; > > Already checked in the below call, IIUC. Hence can be removed. Yes, it can, will update. > > > + return acpi_find_child_by_adr(adev, port->port); > > --