On 2015/7/22 7:02, Bjorn Helgaas wrote: > [+cc Rob] > > On Tue, Jul 21, 2015 at 02:48:42PM +0800, Zhou Wang wrote: >> This patch adds related DTS binding document for Hisilicon PCIe host driver. >> >> Signed-off-by: Zhou Wang <wangzhou1@xxxxxxxxxxxxx> > > If I merge this via my tree, I'm looking for an ack from Arnd and/or Rob > here. > >> --- >> .../devicetree/bindings/pci/hisilicon-pcie.txt | 46 ++++++++++++++++++++++ >> 1 file changed, 46 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/pci/hisilicon-pcie.txt >> >> diff --git a/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt b/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt >> new file mode 100644 >> index 0000000..6c9b827 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/pci/hisilicon-pcie.txt >> @@ -0,0 +1,46 @@ >> +Hisilicon PCIe host bridge DT description > > Even the website at http://hisilicon.com isn't consistent, but there is > some indication that the correct capitalization would be "HiSilicon". > Since this is English text, feel free to capitalize it correctly here :) > Hi Bjorn, I checked with related colleagues about this. It should be "HiSilicon" as formal name. Thanks for pointing this :) > Similarly, Synopsys seems to use "DesignWare," so I try to use that when > it makes sense. > >> +Hisilicon PCIe host controller is based on Designware PCI core. >> +It shares common functions with PCIe Designware core driver and inherits >> +common properties defined in >> +Documentation/devicetree/bindings/pci/designware-pci.txt. >> + >> +Additional properties are described here: >> + >> +Required properties: >> +- compatible: Should contain "hisilicon,hip05-pcie". >> +- reg: Should contain rc_dbi, subctrl, config registers location and length. >> +- reg-names: Must include the following entries: >> + "rc_dbi": controller configuration registers; >> + "subctrl": whole PCIe hosts configuration registers; >> + "config": PCIe configuration space registers. >> +- msi-parent: Should be its_pcie which is an its receiving MSI interrupts. > > I guess "its" here is an abbreviation for something; if so, this would read > better as "... which is an ITS receiving ..." > It is ITS(Interrupt Translation Service) in GIC v3. Will modify it. >> +- port-id: Should be 0, 1, 2 or 3. >> + >> +Optional properties: >> +- status: Either "ok" or "disabled". >> +- dma-coherent: Present if dma operations are coherent. > > "if DMA operations" > Thanks, will modify this. Best regards, Zhou >> + >> +Example: >> + pcie@0xb0080000 { >> + compatible = "hisilicon,hip05-pcie", "snps,dw-pcie"; >> + reg = <0 0xb0080000 0 0x10000>, <0 0xb0000000 0 0x10000>, >> + <0x220 0x00000000 0 0x2000>; >> + reg-names = "rc_dbi", "subctrl", "config"; >> + bus-range = <0 15>; >> + msi-parent = <&its_pcie>; >> + #address-cells = <3>; >> + #size-cells = <2>; >> + device_type = "pci"; >> + dma-coherent; >> + ranges = <0x82000000 0 0x00000000 0x220 0x00000000 0 0x10000000>; >> + num-lanes = <8>; >> + port-id = <1>; >> + #interrupts-cells = <1>; >> + interrupts-map-mask = <0xf800 0 0 7>; >> + interrupts-map = <0x0 0 0 1 &mbigen_pcie 1 10 >> + 0x0 0 0 2 &mbigen_pcie 2 11 >> + 0x0 0 0 3 &mbigen_pcie 3 12 >> + 0x0 0 0 4 &mbigen_pcie 4 13>; >> + status = "ok"; >> + }; >> -- >> 1.9.1 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-pci" in >> the body of a message to majordomo@xxxxxxxxxxxxxxx >> More majordomo info at http://vger.kernel.org/majordomo-info.html > > . > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html