On Thu, Jun 25, 2020 at 2:06 PM kernel test robot <lkp@intel.com> wrote:

>     50          } else if (is_acpi_node(dev->parent->fwnode)) {

Hmm... Is it a device or data node?

>     51                  device_for_each_child_node(dev->parent, dpmac) {
>   > 52                          status = acpi_evaluate_integer(ACPI_HANDLE_FWNODE(dpmac),
>     53                                                         "_ADR", NULL, &adr);
>     54                          if (ACPI_FAILURE(status)) {
>     55                                  dev_info(dev, "_ADR returned status 0x%x\n", status);
>     56                                  continue;
>     57                          } else {
>     58                                  id = (u32)adr;
>     59                                  if (id == dpmac_id)
>     60                                          return dpmac;
>     61                          }
>     62                  }

Can't you use

 adev = acpi_find_child_device(ACPI_COMPANION(dev->parent), dpmac_id, false);
 if (adev)
  return ...

?

>     63          }


-- 
With Best Regards,
Andy Shevchenko