| Message ID | 20221107204934.32655-10-Sergey.Semin@baikalelectronics.ru |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | PCI: dwc: Add generic resources and Baikal-T1 support | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | warning | total: 0 errors, 1 warnings, 211 lines checked |
| robh/patch-applied | success | |
| robh/dtbs-check | warning | build log |
| robh/dt-meta-schema | success |
Hi Serge, > From: Serge Semin, Sent: Tuesday, November 8, 2022 5:49 AM > > Currently the 'interrupts' and 'interrupt-names' properties are defined > being too generic to really describe any actual IRQ interface. Moreover > the DW PCIe End-point devices are left with no IRQ signals. All of that > can be fixed by adding the IRQ-related properties to the common DW PCIe > DT-schemas in accordance with the hardware reference manual. The DW PCIe > common DT-schema will contain the generic properties definitions with just > a number of entries per property, while the DW PCIe RP/EP-specific schemas > will have the particular number of items and the generic resource names > listed. > > Note since there are DW PCI-based vendor-specific DT-bindings with the > custom names assigned to the same IRQ resources we have no much choice but > to add them to the generic DT-schemas in order to have the schemas being > applicable for such devices. These names are marked as vendor-specific and > should be avoided being used in new bindings in favor of the generic > names. > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > --- > > Note without the next dtschema tool fix > > --- a/lib.py 2022-09-29 15:17:13.100033810 +0300 > +++ b/lib.py 2022-09-29 15:19:54.886172794 +0300 JFYI. git am command could not work correctly by this lib.py file: --- Applying: dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties error: lib.py: does not exist in index Patch failed at 0001 dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties --- If I used patch command and skipped the lib.py, it could apply this patch correctly. Best regards, Yoshihiro Shimoda
Hi Yoshihiro On Tue, Nov 08, 2022 at 12:40:54PM +0000, Yoshihiro Shimoda wrote: > Hi Serge, > > > From: Serge Semin, Sent: Tuesday, November 8, 2022 5:49 AM > > > > Currently the 'interrupts' and 'interrupt-names' properties are defined > > being too generic to really describe any actual IRQ interface. Moreover > > the DW PCIe End-point devices are left with no IRQ signals. All of that > > can be fixed by adding the IRQ-related properties to the common DW PCIe > > DT-schemas in accordance with the hardware reference manual. The DW PCIe > > common DT-schema will contain the generic properties definitions with just > > a number of entries per property, while the DW PCIe RP/EP-specific schemas > > will have the particular number of items and the generic resource names > > listed. > > > > Note since there are DW PCI-based vendor-specific DT-bindings with the > > custom names assigned to the same IRQ resources we have no much choice but > > to add them to the generic DT-schemas in order to have the schemas being > > applicable for such devices. These names are marked as vendor-specific and > > should be avoided being used in new bindings in favor of the generic > > names. > > > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > > > --- > > > > Note without the next dtschema tool fix > > > > --- a/lib.py 2022-09-29 15:17:13.100033810 +0300 > > +++ b/lib.py 2022-09-29 15:19:54.886172794 +0300 > > JFYI. > > git am command could not work correctly by this lib.py file: > --- > Applying: dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties > error: lib.py: does not exist in index > Patch failed at 0001 dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties > --- > > If I used patch command and skipped the lib.py, it could apply this patch correctly. Got it. Thanks for the note. I'll either drop this part on the next patchset revision (hopefully Rob will do something about that by then) or make it less looking like a patch so git am wouldn't be confused. -Sergey > > Best regards, > Yoshihiro Shimoda >
On Tue, Nov 8, 2022 at 7:52 AM Serge Semin <fancer.lancer@gmail.com> wrote: > > Hi Yoshihiro > > On Tue, Nov 08, 2022 at 12:40:54PM +0000, Yoshihiro Shimoda wrote: > > Hi Serge, > > > > > From: Serge Semin, Sent: Tuesday, November 8, 2022 5:49 AM > > > > > > Currently the 'interrupts' and 'interrupt-names' properties are defined > > > being too generic to really describe any actual IRQ interface. Moreover > > > the DW PCIe End-point devices are left with no IRQ signals. All of that > > > can be fixed by adding the IRQ-related properties to the common DW PCIe > > > DT-schemas in accordance with the hardware reference manual. The DW PCIe > > > common DT-schema will contain the generic properties definitions with just > > > a number of entries per property, while the DW PCIe RP/EP-specific schemas > > > will have the particular number of items and the generic resource names > > > listed. > > > > > > Note since there are DW PCI-based vendor-specific DT-bindings with the > > > custom names assigned to the same IRQ resources we have no much choice but > > > to add them to the generic DT-schemas in order to have the schemas being > > > applicable for such devices. These names are marked as vendor-specific and > > > should be avoided being used in new bindings in favor of the generic > > > names. > > > > > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > > > > > --- > > > > > > Note without the next dtschema tool fix > > > > > > --- a/lib.py 2022-09-29 15:17:13.100033810 +0300 > > > +++ b/lib.py 2022-09-29 15:19:54.886172794 +0300 > > > > > JFYI. > > > > git am command could not work correctly by this lib.py file: > > --- > > Applying: dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties > > error: lib.py: does not exist in index > > Patch failed at 0001 dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties > > --- > > > > If I used patch command and skipped the lib.py, it could apply this patch correctly. > > Got it. Thanks for the note. I'll either drop this part on the next > patchset revision (hopefully Rob will do something about that by then) > or make it less looking like a patch so git am wouldn't be confused. Now fixed in main branch. Thanks for the report. Rob
On Tue, Nov 08, 2022 at 04:32:22PM -0600, Rob Herring wrote: > On Tue, Nov 8, 2022 at 7:52 AM Serge Semin <fancer.lancer@gmail.com> wrote: > > > > Hi Yoshihiro > > > > On Tue, Nov 08, 2022 at 12:40:54PM +0000, Yoshihiro Shimoda wrote: > > > Hi Serge, > > > > > > > From: Serge Semin, Sent: Tuesday, November 8, 2022 5:49 AM > > > > > > > > Currently the 'interrupts' and 'interrupt-names' properties are defined > > > > being too generic to really describe any actual IRQ interface. Moreover > > > > the DW PCIe End-point devices are left with no IRQ signals. All of that > > > > can be fixed by adding the IRQ-related properties to the common DW PCIe > > > > DT-schemas in accordance with the hardware reference manual. The DW PCIe > > > > common DT-schema will contain the generic properties definitions with just > > > > a number of entries per property, while the DW PCIe RP/EP-specific schemas > > > > will have the particular number of items and the generic resource names > > > > listed. > > > > > > > > Note since there are DW PCI-based vendor-specific DT-bindings with the > > > > custom names assigned to the same IRQ resources we have no much choice but > > > > to add them to the generic DT-schemas in order to have the schemas being > > > > applicable for such devices. These names are marked as vendor-specific and > > > > should be avoided being used in new bindings in favor of the generic > > > > names. > > > > > > > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > > > > > > > --- > > > > > > > > Note without the next dtschema tool fix > > > > > > > > --- a/lib.py 2022-09-29 15:17:13.100033810 +0300 > > > > +++ b/lib.py 2022-09-29 15:19:54.886172794 +0300 > > > > > > > > JFYI. > > > > > > git am command could not work correctly by this lib.py file: > > > --- > > > Applying: dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties > > > error: lib.py: does not exist in index > > > Patch failed at 0001 dt-bindings: PCI: dwc: Add interrupts/interrupt-names common properties > > > --- > > > > > > If I used patch command and skipped the lib.py, it could apply this patch correctly. > > > > Got it. Thanks for the note. I'll either drop this part on the next > > patchset revision (hopefully Rob will do something about that by then) > > or make it less looking like a patch so git am wouldn't be confused. > > Now fixed in main branch. Thanks for the report. Ok. I'll drop that chunk from v7 then. -Sergey > > Rob
On Mon, 07 Nov 2022 23:49:23 +0300, Serge Semin wrote: > Currently the 'interrupts' and 'interrupt-names' properties are defined > being too generic to really describe any actual IRQ interface. Moreover > the DW PCIe End-point devices are left with no IRQ signals. All of that > can be fixed by adding the IRQ-related properties to the common DW PCIe > DT-schemas in accordance with the hardware reference manual. The DW PCIe > common DT-schema will contain the generic properties definitions with just > a number of entries per property, while the DW PCIe RP/EP-specific schemas > will have the particular number of items and the generic resource names > listed. > > Note since there are DW PCI-based vendor-specific DT-bindings with the > custom names assigned to the same IRQ resources we have no much choice but > to add them to the generic DT-schemas in order to have the schemas being > applicable for such devices. These names are marked as vendor-specific and > should be avoided being used in new bindings in favor of the generic > names. > > Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> > > --- > > Note without the next dtschema tool fix > > --- a/lib.py 2022-09-29 15:17:13.100033810 +0300 > +++ b/lib.py 2022-09-29 15:19:54.886172794 +0300 > @@ -1307,7 +1307,7 @@ > def format_error(filename, error, prefix="", nodename=None, verbose=False): > src = prefix + os.path.abspath(filename) + ':' > > - if error.linecol[0] >= 0: > + if hasattr(error, 'linecol') and error.linecol[0] >= 0: > src = src + '%i:%i: ' % (error.linecol[0]+1, error.linecol[1]+1) > else: > src += ' ' > @@ -1342,10 +1342,10 @@ > else: > msg = error.message > > - if error.note: > + if hasattr(error, 'note') and error.note: > msg += '\n\t' + prefix + 'hint: ' + error.note > > - if error.schema_file: > + if hasattr(error, 'schema_file') and error.schema_file: > msg += '\n\t' + prefix + 'from schema $id: ' + error.schema_file > > return src + msg > > any DT-bindings error will cause the dt-schema script crash: > > Traceback (most recent call last): > File "/home/fancer/.local/bin/dt-validate", line 175, in <module> > sg.check_trees(filename, testtree) > File "/home/fancer/.local/bin/dt-validate", line 122, in check_trees > self.check_subtree(dt, subtree, False, "/", "/", filename) > File "/home/fancer/.local/bin/dt-validate", line 111, in check_subtree > self.check_subtree(tree, value, disabled, name, fullname + name, filename) > File "/home/fancer/.local/bin/dt-validate", line 111, in check_subtree > self.check_subtree(tree, value, disabled, name, fullname + name, filename) > File "/home/fancer/.local/bin/dt-validate", line 106, in check_subtree > self.check_node(tree, subtree, disabled, nodename, fullname, filename) > File "/home/fancer/.local/bin/dt-validate", line 84, in check_node > print(dtschema.format_error(filename, error, nodename=nodename, verbose=verbose) + > File "/home/fancer/.local/lib/python3.8/site-packages/dtschema/lib.py", line 1332, in format_error > msg += '\n' + format_error(filename, suberror, prefix=prefix+"\t", nodename=nodename, verbose=verbose) > File "/home/fancer/.local/lib/python3.8/site-packages/dtschema/lib.py", line 1310, in format_error > if error.linecol[0] >= 0: > AttributeError: 'ValidationError' object has no attribute 'linecol' > > Changelog v3: > - This is a new patch unpinned from the next one: > https://lore.kernel.org/linux-pci/20220503214638.1895-2-Sergey.Semin@baikalelectronics.ru/ > by the Rob' request. (@Rob) > > Changelog v5: > - Add platform-specific interrupt names, but mark them as deprecated. > > Changelog v6: > - Move the common interrupt-names definitions to the RP/EP schemas. > Thus drop the 'definitions' property. (@Rob) > - Drop the 'deprecated' keywords from the vendor-specific names. (@Rob) > --- > .../bindings/pci/snps,dw-pcie-common.yaml | 19 ++++ > .../bindings/pci/snps,dw-pcie-ep.yaml | 52 +++++++++++ > .../devicetree/bindings/pci/snps,dw-pcie.yaml | 90 ++++++++++++++++++- > 3 files changed, 158 insertions(+), 3 deletions(-) > Reviewed-by: Rob Herring <robh@kernel.org>
--- a/lib.py 2022-09-29 15:17:13.100033810 +0300 +++ b/lib.py 2022-09-29 15:19:54.886172794 +0300 @@ -1307,7 +1307,7 @@ def format_error(filename, error, prefix="", nodename=None, verbose=False): src = prefix + os.path.abspath(filename) + ':' - if error.linecol[0] >= 0: + if hasattr(error, 'linecol') and error.linecol[0] >= 0: src = src + '%i:%i: ' % (error.linecol[0]+1, error.linecol[1]+1) else: src += ' ' @@ -1342,10 +1342,10 @@ else: msg = error.message - if error.note: + if hasattr(error, 'note') and error.note: msg += '\n\t' + prefix + 'hint: ' + error.note - if error.schema_file: + if hasattr(error, 'schema_file') and error.schema_file: msg += '\n\t' + prefix + 'from schema $id: ' + error.schema_file return src + msg any DT-bindings error will cause the dt-schema script crash: Traceback (most recent call last): File "/home/fancer/.local/bin/dt-validate", line 175, in <module> sg.check_trees(filename, testtree) File "/home/fancer/.local/bin/dt-validate", line 122, in check_trees self.check_subtree(dt, subtree, False, "/", "/", filename) File "/home/fancer/.local/bin/dt-validate", line 111, in check_subtree self.check_subtree(tree, value, disabled, name, fullname + name, filename) File "/home/fancer/.local/bin/dt-validate", line 111, in check_subtree self.check_subtree(tree, value, disabled, name, fullname + name, filename) File "/home/fancer/.local/bin/dt-validate", line 106, in check_subtree self.check_node(tree, subtree, disabled, nodename, fullname, filename) File "/home/fancer/.local/bin/dt-validate", line 84, in check_node print(dtschema.format_error(filename, error, nodename=nodename, verbose=verbose) + File "/home/fancer/.local/lib/python3.8/site-packages/dtschema/lib.py", line 1332, in format_error msg += '\n' + format_error(filename, suberror, prefix=prefix+"\t", nodename=nodename, verbose=verbose) File "/home/fancer/.local/lib/python3.8/site-packages/dtschema/lib.py", line 1310, in format_error if error.linecol[0] >= 0: AttributeError: 'ValidationError' object has no attribute 'linecol' Changelog v3: - This is a new patch unpinned from the next one: https://lore.kernel.org/linux-pci/20220503214638.1895-2-Sergey.Semin@baikalelectronics.ru/ by the Rob' request. (@Rob) Changelog v5: - Add platform-specific interrupt names, but mark them as deprecated. Changelog v6: - Move the common interrupt-names definitions to the RP/EP schemas. Thus drop the 'definitions' property. (@Rob) - Drop the 'deprecated' keywords from the vendor-specific names. (@Rob) --- .../bindings/pci/snps,dw-pcie-common.yaml | 19 ++++ .../bindings/pci/snps,dw-pcie-ep.yaml | 52 +++++++++++ .../devicetree/bindings/pci/snps,dw-pcie.yaml | 90 ++++++++++++++++++- 3 files changed, 158 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/pci/snps,dw-pcie-common.yaml b/Documentation/devicetree/bindings/pci/snps,dw-pcie-common.yaml index e63c21783fc1..4646fb14e817 100644 --- a/Documentation/devicetree/bindings/pci/snps,dw-pcie-common.yaml +++ b/Documentation/devicetree/bindings/pci/snps,dw-pcie-common.yaml @@ -17,6 +17,25 @@ description: select: false properties: + interrupts: + description: + There are two main sub-blocks which are normally capable of + generating interrupts. It's System Information Interface and MSI + interface. While the former one has some common for the Host and + Endpoint controllers IRQ-signals, the later interface is obviously + Root Complex specific since it's responsible for the incoming MSI + messages signalling. The System Information IRQ signals are mainly + responsible for reporting the generic PCIe hierarchy and Root + Complex events like VPD IO request, general AER, PME, Hot-plug, link + bandwidth change, link equalization request, INTx asserted/deasserted + Message detection, embedded DMA Tx/Rx/Error. + minItems: 1 + maxItems: 26 + + interrupt-names: + minItems: 1 + maxItems: 26 + phys: description: There can be up to the number of possible lanes PHYs specified placed in diff --git a/Documentation/devicetree/bindings/pci/snps,dw-pcie-ep.yaml b/Documentation/devicetree/bindings/pci/snps,dw-pcie-ep.yaml index 71dd19ae1060..7d3f8fc8b7b4 100644 --- a/Documentation/devicetree/bindings/pci/snps,dw-pcie-ep.yaml +++ b/Documentation/devicetree/bindings/pci/snps,dw-pcie-ep.yaml @@ -41,6 +41,55 @@ properties: items: enum: [dbi, dbi2, config, atu, addr_space, link, atu_dma, appl] + interrupts: + description: + There is no mandatory IRQ signals for the normal controller functioning, + but in addition to the native set the platforms may have a link- or + PM-related IRQs specified. + minItems: 1 + maxItems: 20 + + interrupt-names: + minItems: 1 + maxItems: 20 + items: + oneOf: + - description: + Controller request to read or write virtual product data + from/to the VPD capability registers. + const: vpd + - description: + Link Equalization Request flag is set in the Link Status 2 + register (applicable if the corresponding IRQ is enabled in + the Link Control 3 register). + const: l_eq + - description: + Indicates that the eDMA Tx/Rx transfer is complete or that an + error has occurred on the corresponding channel. eDMA can have + eight Tx (Write) and Rx (Read) eDMA channels thus supporting up + to 16 IRQ signals all together. Write eDMA channels shall go + first in the ordered row as per default edma_int[*] bus setup. + pattern: '^dma([0-9]|1[0-5])?$' + - description: + PCIe protocol correctable error or a Data Path protection + correctable error is detected by the automotive/safety + feature. + const: sft_ce + - description: + Indicates that the internal safety mechanism has detected an + uncorrectable error. + const: sft_ue + - description: + Application-specific IRQ raised depending on the vendor-specific + events basis. + const: app + - description: + Vendor-specific IRQ names. Consider using the generic names above + for new bindings. + oneOf: + - description: See native "app" IRQ for details + enum: [ intr ] + max-functions: maximum: 32 @@ -60,6 +109,9 @@ examples: <0xd0000000 0x2000000>; /* Configuration space */ reg-names = "dbi", "dbi2", "addr_space"; + interrupts = <23>, <24>; + interrupt-names = "dma0", "dma1"; + phys = <&pcie_phy0>, <&pcie_phy1>, <&pcie_phy2>, <&pcie_phy3>; phy-names = "pcie0", "pcie1", "pcie2", "pcie3"; diff --git a/Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml b/Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml index 85861b71d9ff..fa1db57b2b97 100644 --- a/Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml +++ b/Documentation/devicetree/bindings/pci/snps,dw-pcie.yaml @@ -42,9 +42,92 @@ properties: enum: [ dbi, dbi2, config, atu, atu_dma, app, appl, elbi, mgmt, ctrl, parf, cfg, link, ulreg, smu, mpu, apb, phy ] - interrupts: true - - interrupt-names: true + interrupts: + description: + DWC PCIe Root Port/Complex specific IRQ signals. At least MSI interrupt + signal is supposed to be specified for the host controller. + minItems: 1 + maxItems: 26 + + interrupt-names: + minItems: 1 + maxItems: 26 + items: + oneOf: + - description: + Controller request to read or write virtual product data + from/to the VPD capability registers. + const: vpd + - description: + Link Equalization Request flag is set in the Link Status 2 + register (applicable if the corresponding IRQ is enabled in + the Link Control 3 register). + const: l_eq + - description: + Indicates that the eDMA Tx/Rx transfer is complete or that an + error has occurred on the corresponding channel. eDMA can have + eight Tx (Write) and Rx (Read) eDMA channels thus supporting up + to 16 IRQ signals all together. Write eDMA channels shall go + first in the ordered row as per default edma_int[*] bus setup. + pattern: '^dma([0-9]|1[0-5])?$' + - description: + PCIe protocol correctable error or a Data Path protection + correctable error is detected by the automotive/safety + feature. + const: sft_ce + - description: + Indicates that the internal safety mechanism has detected an + uncorrectable error. + const: sft_ue + - description: + Application-specific IRQ raised depending on the vendor-specific + events basis. + const: app + - description: + DSP AXI MSI Interrupt detected. It gets de-asserted when there is + no more MSI interrupt pending. The interrupt is relevant to the + iMSI-RX - Integrated MSI Receiver (AXI bridge). + const: msi + - description: + Legacy A/B/C/D interrupt signal. Basically it's triggered by + receiving a Assert_INT{A,B,C,D}/Desassert_INT{A,B,C,D} message + from the downstream device. + pattern: "^int(a|b|c|d)$" + - description: + Error condition detected and a flag is set in the Root Error Status + register of the AER capability. It's asserted when the RC + internally generated an error or an error message is received by + the RC. + const: aer + - description: + PME message is received by the port. That means having the PME + status bit set in the Root Status register (the event is + supposed to be unmasked in the Root Control register). + const: pme + - description: + Hot-plug event is detected. That is a bit has been set in the + Slot Status register and the corresponding event is enabled in + the Slot Control register. + const: hp + - description: + Link Autonomous Bandwidth Status flag has been set in the Link + Status register (the event is supposed to be unmasked in the + Link Control register). + const: bw_au + - description: + Bandwidth Management Status flag has been set in the Link + Status register (the event is supposed to be unmasked in the + Link Control register). + const: bw_mg + - description: + Vendor-specific IRQ names. Consider using the generic names above + for new bindings. + oneOf: + - description: See native "app" IRQ for details + enum: [ intr ] + allOf: + - contains: + const: msi clocks: true @@ -70,6 +153,7 @@ examples: bus-range = <0x0 0xff>; interrupts = <25>, <24>; + interrupt-names = "msi", "hp"; #interrupt-cells = <1>; reset-gpios = <&port0 0 1>;
Currently the 'interrupts' and 'interrupt-names' properties are defined being too generic to really describe any actual IRQ interface. Moreover the DW PCIe End-point devices are left with no IRQ signals. All of that can be fixed by adding the IRQ-related properties to the common DW PCIe DT-schemas in accordance with the hardware reference manual. The DW PCIe common DT-schema will contain the generic properties definitions with just a number of entries per property, while the DW PCIe RP/EP-specific schemas will have the particular number of items and the generic resource names listed. Note since there are DW PCI-based vendor-specific DT-bindings with the custom names assigned to the same IRQ resources we have no much choice but to add them to the generic DT-schemas in order to have the schemas being applicable for such devices. These names are marked as vendor-specific and should be avoided being used in new bindings in favor of the generic names. Signed-off-by: Serge Semin <Sergey.Semin@baikalelectronics.ru> --- Note without the next dtschema tool fix