platform/x86/intel/vsec: Remove nuisance message
authorDavid E. Box <david.e.box@linux.intel.com>
Tue, 27 Feb 2024 19:01:32 +0000 (11:01 -0800)
committerIlpo Järvinen <ilpo.jarvinen@linux.intel.com>
Tue, 12 Mar 2024 10:47:49 +0000 (12:47 +0200)
intel_vsec_walk_header() is used to configure features from devices that
don't provide a PCI VSEC or DVSEC structure. Some of these features may
be unsupported and fail to load. Ignore them silently as we do for
unsupported features described by VSEC/DVSEC.

Signed-off-by: "David E. Box" <david.e.box@linux.intel.com>
Reviewed-by: Kuppuswamy Sathyanarayanan <sathyanarayanan.kuppuswamy@linux.intel.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://lore.kernel.org/r/20240227190134.1592072-1-david.e.box@linux.intel.com
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
drivers/platform/x86/intel/vsec.c

index 778eb0aa3479a5bcdd31ceef5dec4446db55069e..0fdfaf3a4f5cde4a2945a3a8860b1eb220c4f265 100644 (file)
@@ -236,10 +236,7 @@ static bool intel_vsec_walk_header(struct pci_dev *pdev,
 
        for ( ; *header; header++) {
                ret = intel_vsec_add_dev(pdev, *header, info);
-               if (ret)
-                       dev_info(&pdev->dev, "Could not add device for VSEC id %d\n",
-                                (*header)->id);
-               else
+               if (!ret)
                        have_devices = true;
        }