From: Rob Clark <robclark@freedesktop.org>
Date: Sun, 15 Feb 2015 06:59:17 +0000 (-0500)
Subject: loader: not having a pci-id should not be a warn
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=9153dd4b7eb95728aa9746a45a9dd136458d0213;p=mesa.git

loader: not having a pci-id should not be a warn

If there is no pci-id, which is valid for vc4 and freedreno, just emit
an info msg.  Keep malformed but existing pci-id's as a warning.

Mostly just to clean up a warning that confuses users for the non-pci
devices.

Signed-off-by: Rob Clark <robclark@freedesktop.org>
---

diff --git a/src/loader/loader.c b/src/loader/loader.c
index 94c993ac841..9ff5115221f 100644
--- a/src/loader/loader.c
+++ b/src/loader/loader.c
@@ -207,9 +207,12 @@ libudev_get_pci_id_for_fd(int fd, int *vendor_id, int *chip_id)
    }
 
    pci_id = udev_device_get_property_value(parent, "PCI_ID");
-   if (pci_id == NULL ||
-       sscanf(pci_id, "%x:%x", vendor_id, chip_id) != 2) {
-      log_(_LOADER_WARNING, "MESA-LOADER: malformed or no PCI ID\n");
+   if (pci_id == NULL) {
+      log_(_LOADER_INFO, "MESA-LOADER: no PCI ID\n");
+      *chip_id = -1;
+      goto out;
+   } else if (sscanf(pci_id, "%x:%x", vendor_id, chip_id) != 2) {
+      log_(_LOADER_WARNING, "MESA-LOADER: malformed PCI ID\n");
       *chip_id = -1;
       goto out;
    }