[PATCH v3 0/4] PCI: of: warn on bogus device_type property
From: Alex Elder
Date: Mon Aug 31 2026 - 21:14:02 EST
Add a check when adding a PCI device to ensure the device_type
property is (or is not) correctly defined when the device has
a non-null devicetree node pointer.
PCI has a well-defined bus and device discovery process. The
PCI_DYNAMIC_OF_NODES Kconfig option allows PCI devices to *also*
have a devicetree node. This enables certain things that are
not possible with PCI enumeration alone.
While working on a Qualcomm platform, I learned that some PCI
endpoint nodes were defined with device_type = "pci" properties.
Herve Codina pointed out that this was not correct. Rob Herring
indicated that people seem to have trouble getting the PCI
devicetree nodes right, and asked whether we could warn if this
particular problem occurred. During review, Herve Codina also
suggested that bridge nodes be checked to ensure they *do*
have the proper device_type property.
This series implements these checks. A new patch in this version
removes a (now duplicate) warning issued by the devicetree code.
The first two patches are simple cleanups. The third adds the new
PCI devicetree node checks and warnings, and the last removes the
duplicate warning message.
-Alex
Between version 2 and version 3:
- Drop a patch that made a change only needed by a different series
- Switch a function header to use kernel-doc format
- Add a warning if a PCI bridge node has no device_type property
- Added a patch to remove a duplicate warning in the devicetree code
Version 2 is available here:
https://lore.kernel.org/lkml/20260812172247.276554-1-elder@xxxxxxxxxxxx/
Between version 1 and version 2:
- Check the PCI devicetree node even when PCI_DYNAMIC_OF_NODES is
not enabled
Version 1 is available here:
https://lore.kernel.org/lkml/20260807194100.455599-1-elder@xxxxxxxxxxxx/
Alex Elder (4):
PCI: of: drop the reg_num argument to of_pci_set_address()
PCI: of: don't zero flags in of_pci_get_addr_flags()
PCI: of: introduce of_pci_verify_node()
of: address: kill of_node_is_pcie()
drivers/of/address.c | 12 +-----------
drivers/pci/bus.c | 1 +
drivers/pci/of.c | 31 +++++++++++++++++++++++++++++++
drivers/pci/of_property.c | 14 ++++++--------
drivers/pci/pci.h | 3 +++
5 files changed, 42 insertions(+), 19 deletions(-)
base-commit: cee9395acd8043be0644b25c34bfa86623f2b935
--
2.53.0