Re: [PATCH v2 0/4] PCI: of: warn on bogus device_type property

Hi Alex,

Can you add me in Cc in all patches of the series for next iterations?

Best regards,
Hervé

On Wed, 12 Aug 2026 12:22:43 -0500
Alex Elder <[email protected]> wrote:

> The purpose of this series lies in its final patch, where a new check
> is added whenever a PCI devicetree node is found to already exist when
> one might otherwise be dynamically created.
> 
> 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.
> 
> The last patch in this series implements that check and warning.
> The first three patches are fairly trivial changes to clean up
> some related code.  This is the only changed patch since v1.
> 
> 					-Alex
> 
> 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/[email protected]/
> 
> 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: make a flags argument optional
>   PCI: of: introduce of_pci_verify_node()
> 
>  drivers/pci/bus.c         |  1 +
>  drivers/pci/of.c          | 26 ++++++++++++++++++++++++++
>  drivers/pci/of_property.c | 20 ++++++++++----------
>  drivers/pci/pci.h         |  3 +++
>  4 files changed, 40 insertions(+), 10 deletions(-)
> 
> 
> base-commit: db2ddb87143519e20a95aa36c60b36107b736a58