nouveau: ACPI: Use the ACPI_COMPANION() macro directly
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 13 Oct 2021 16:08:06 +0000 (18:08 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 26 Oct 2021 13:48:49 +0000 (15:48 +0200)
The ACPI_HANDLE() macro is a wrapper arond the ACPI_COMPANION()
macro and the ACPI handle produced by the former comes from the
ACPI device object produced by the latter, so it is way more
straightforward to evaluate the latter directly instead of passing
the handle produced by the former to acpi_bus_get_device().

Modify nouveau_acpi_edid() accordingly (no intentional functional
impact).

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nouveau_acpi.c

index 7c15f64484281ede2a8ca79ebcc5267a4271b9fb..6140db756d063babccda081bbc77deaedf61b890 100644 (file)
@@ -364,7 +364,6 @@ void *
 nouveau_acpi_edid(struct drm_device *dev, struct drm_connector *connector)
 {
        struct acpi_device *acpidev;
-       acpi_handle handle;
        int type, ret;
        void *edid;
 
@@ -377,12 +376,8 @@ nouveau_acpi_edid(struct drm_device *dev, struct drm_connector *connector)
                return NULL;
        }
 
-       handle = ACPI_HANDLE(dev->dev);
-       if (!handle)
-               return NULL;
-
-       ret = acpi_bus_get_device(handle, &acpidev);
-       if (ret)
+       acpidev = ACPI_COMPANION(dev->dev);
+       if (!acpidev)
                return NULL;
 
        ret = acpi_video_get_edid(acpidev, type, -1, &edid);