On Sun, Mar 6, 2022 at 9:37 AM Tom Rix <trix@xxxxxxxxxx> wrote: > > Lizhi, > > Sorry for the delay, I am fighting with checking this with 'make > dt_binding_check' > > There is a recent failure in linux-next around display/mediatek,* > between next-20220301 and next-20220302 that I am bisecting. There's already patches for that posted. Just use 'make -k'. > > There are a couple of checkpatch --strict warnings for this set, the > obvious one is adding to the MAINTAINERS for new files. > > Tom > > On 3/4/22 9:23 PM, Lizhi Hou wrote: > > Create device tree binding document for PCIe endpoint bus. > > > > Signed-off-by: Sonal Santan <sonal.santan@xxxxxxxxxx> > > Signed-off-by: Max Zhen <max.zhen@xxxxxxxxxx> > > Signed-off-by: Lizhi Hou <lizhi.hou@xxxxxxxxxx> > > --- > > .../devicetree/bindings/bus/pci-ep-bus.yaml | 72 +++++++++++++++++++ > > 1 file changed, 72 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/bus/pci-ep-bus.yaml > > > > diff --git a/Documentation/devicetree/bindings/bus/pci-ep-bus.yaml b/Documentation/devicetree/bindings/bus/pci-ep-bus.yaml > > new file mode 100644 > > index 000000000000..0ca96298db6f > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/bus/pci-ep-bus.yaml > > @@ -0,0 +1,72 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/bus/pci-ep-bus.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: PCIe Endpoint Bus binding > > + > > +description: | > > + PCIe device may use flattened device tree to describe apertures in its > > + PCIe BARs. The Bus PCIe endpoint node is created and attached under the > > + device tree root node for this kind of device. Then the flatten device > > + tree overlay for this device is attached under the endpoint node. > > + > > + The aperture address which is under the endpoint node consists of BAR > > + index and offset. It uses the following encoding: > > + > > + 0xIooooooo 0xoooooooo > > + > > + Where: > > + > > + I = BAR index > > + oooooo oooooooo = BAR offset > > + > > + The endpoint is compatible with 'simple-bus' and contains 'ranges' > > + property for translating aperture address to CPU address. This binding is completely confusing because 'PCIe endpoint' is generally used (in context of bindings and Linux) for describing the endpoint's system (i.e. the internal structure of a PCIe device (e.g. add-in card) from the view of its own processor (not the host system)). This binding seems to be describing the host system's view of a PCIe device. We already have that! That's just the PCI bus binding[1] which has existed for ~25 years. For a non-DT system, what you are going to need here is some way to create DT nodes of the PCI bus hierarchy or at least from your device back up to the host bridge. I would suggest you solve that problem separately from implementing the FPGA driver by making it work first on a DT based system. Rob [1] https://www.devicetree.org/open-firmware/bindings/pci/pci2_1.pdf
