ecore_drm2: simplify device selection code
authorDerek Foreman <derekf@osg.samsung.com>
Wed, 10 Jan 2018 18:54:42 +0000 (12:54 -0600)
committerWonki Kim <wonki_.kim@samsung.com>
Wed, 17 Jan 2018 09:19:26 +0000 (18:19 +0900)
The platform check was added for systems (like ARM) that don't generally
have PCI graphics devices.  However, now we pick a fallback device that
doesn't have a PCI constraint, so the platform check should no longer be
necessary.

src/lib/ecore_drm2/ecore_drm2_device.c

index dea67b1..e04f0d3 100644 (file)
@@ -97,7 +97,6 @@ _drm2_device_find(Elput_Manager *em, const char *seat)
    Eina_List *devs, *l;
    const char *dev, *ret = NULL, *chosen_dev = NULL;
    Eina_Bool found = EINA_FALSE;
-   Eina_Bool platform = EINA_FALSE;
    Eina_Bool modeset;
    int fd;
 
@@ -130,28 +129,16 @@ _drm2_device_find(Elput_Manager *em, const char *seat)
         chosen_dev = dev;
 
         dparent = eeze_udev_syspath_get_parent_filtered(dev, "pci", NULL);
-        if (!dparent)
-          {
-             dparent =
-               eeze_udev_syspath_get_parent_filtered(dev, "platform", NULL);
-             platform = EINA_TRUE;
-          }
-
         if (dparent)
           {
-             if (!platform)
-               {
-                  const char *id;
+             const char *id;
 
-                  id = eeze_udev_syspath_get_sysattr(dparent, "boot_vga");
-                  if (id)
-                    {
-                       if (!strcmp(id, "1")) found = EINA_TRUE;
-                       eina_stringshare_del(id);
-                    }
+             id = eeze_udev_syspath_get_sysattr(dparent, "boot_vga");
+             if (id)
+               {
+                  if (!strcmp(id, "1")) found = EINA_TRUE;
+                  eina_stringshare_del(id);
                }
-             else
-               found = EINA_TRUE;
 
              eina_stringshare_del(dparent);
           }