lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240703091535.3531-1-spasswolf@web.de>
Date: Wed,  3 Jul 2024 11:15:30 +0200
From: Bert Karwatzki <spasswolf@....de>
To: Bartosz Golaszewski <bartosz.golaszewski@...aro.org>
Cc: Bert Karwatzki <spasswolf@....de>,
	caleb.connolly@...aro.org,
	bhelgaas@...gle.com,
	brgl@...ev.pl,
	amit.pundir@...aro.org,
	neil.armstrong@...aro.org,
	linux-kernel@...r.kernel.org,
	linux-pci@...r.kernel.org
Subject: [PATCH] pci: bus: only call of_platform_populate() if CONFIG_OF is defined

If of_platform_populate() is called when CONFIG_OF is not defined this
leads to spurious error messages of the following type:
 pci 0000:00:01.1: failed to populate child OF nodes (-19)
 pci 0000:00:02.1: failed to populate child OF nodes (-19)

Fixes: 8fb18619d910 ("PCI/pwrctl: Create platform devices for child OF nodes of the port node")
Signed-off-by: Bert Karwatzki <spasswolf@....de>
---
 drivers/pci/bus.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/pci/bus.c b/drivers/pci/bus.c
index e4735428814d..b363010664cd 100644
--- a/drivers/pci/bus.c
+++ b/drivers/pci/bus.c
@@ -350,6 +350,7 @@ void pci_bus_add_device(struct pci_dev *dev)

 	pci_dev_assign_added(dev, true);

+#ifdef CONFIG_OF
 	if (pci_is_bridge(dev)) {
 		retval = of_platform_populate(dev->dev.of_node, NULL, NULL,
 					      &dev->dev);
@@ -357,6 +358,7 @@ void pci_bus_add_device(struct pci_dev *dev)
 			pci_err(dev, "failed to populate child OF nodes (%d)\n",
 				retval);
 	}
+#endif
 }
 EXPORT_SYMBOL_GPL(pci_bus_add_device);

--
2.45.2

The mentioned error message occur on systems without CONFIG_OF, i.e.
x86_64. The call to of_platform_depopulate() in drivers/pci/remove.c
does not need #ifdef CONFIG_OF as the return value is not checked (it
will most likely be optimized away on platforms witout OF where the
of_platform_{de,}populate() functions just return -ENODEV)

Please CC me when replying, I'm not subscribed to the lists.

Bert Karwatzki

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ