[PATCH 1/4] dt-bindings: misc: pci1de4,1: add required reg property for endpoint

Andrea della Porta posted 4 patches 1 month, 3 weeks ago
[PATCH 1/4] dt-bindings: misc: pci1de4,1: add required reg property for endpoint
Posted by Andrea della Porta 1 month, 3 weeks ago
The PCI subsystem links an endpoint Device Tree node to its corresponding
pci_dev structure only if the Bus/Device/Function (BDF) encoded in the
'reg' property matches the actual hardware topology.

Add the 'reg' property and mark it as required to ensure proper binding
between the device_node and the pci_dev.

Update the example to reflect this requirement.

Signed-off-by: Andrea della Porta <andrea.porta@suse.com>
---
 Documentation/devicetree/bindings/misc/pci1de4,1.yaml | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/misc/pci1de4,1.yaml b/Documentation/devicetree/bindings/misc/pci1de4,1.yaml
index 2f9a7a554ed8a..17a8c19af8cc2 100644
--- a/Documentation/devicetree/bindings/misc/pci1de4,1.yaml
+++ b/Documentation/devicetree/bindings/misc/pci1de4,1.yaml
@@ -25,6 +25,10 @@ properties:
     items:
       - const: pci1de4,1
 
+  reg:
+    maxItems: 1
+    description: The PCI Bus-Device-Function address.
+
   '#interrupt-cells':
     const: 2
     description: |
@@ -101,6 +105,7 @@ unevaluatedProperties: false
 
 required:
   - compatible
+  - reg
   - '#interrupt-cells'
   - interrupt-controller
   - pci-ep-bus@1
@@ -111,8 +116,9 @@ examples:
         #address-cells = <3>;
         #size-cells = <2>;
 
-        rp1@0,0 {
+        dev@0,0 {
             compatible = "pci1de4,1";
+            reg = <0x10000 0x0 0x0 0x0 0x0>;
             ranges = <0x01 0x00 0x00000000  0x82010000 0x00 0x00  0x00 0x400000>;
             #address-cells = <3>;
             #size-cells = <2>;
-- 
2.35.3
Re: [PATCH 1/4] dt-bindings: misc: pci1de4,1: add required reg property for endpoint
Posted by Florian Fainelli 1 month, 2 weeks ago
From: Florian Fainelli <f.fainelli@gmail.com>

On Thu, 18 Dec 2025 20:09:06 +0100, Andrea della Porta <andrea.porta@suse.com> wrote:
> The PCI subsystem links an endpoint Device Tree node to its corresponding
> pci_dev structure only if the Bus/Device/Function (BDF) encoded in the
> 'reg' property matches the actual hardware topology.
> 
> Add the 'reg' property and mark it as required to ensure proper binding
> between the device_node and the pci_dev.
> 
> Update the example to reflect this requirement.
> 
> Signed-off-by: Andrea della Porta <andrea.porta@suse.com>
> ---

Applied to https://github.com/Broadcom/stblinux/commits/devicetree-arm64/fixes, thanks!
--
Florian