* [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy [not found] <CGME20220328084319epcms2p3670d7836bb111e0d9c648aeebfde8603@epcms2p3> @ 2022-03-28 8:43 ` 이왕석 2022-03-28 12:51 ` Rob Herring ` (2 more replies) 0 siblings, 3 replies; 6+ messages in thread From: 이왕석 @ 2022-03-28 8:43 UTC (permalink / raw) To: robh+dt, krzk+dt, kishon, vkoul, linux-kernel, jesper.nilsson, lars.persson Cc: bhelgaas, linux-phy, linux-pci, devicetree, lorenzo.pieralisi, kw, linux-arm-kernel, kernel, 전문기 Add description to support Axis, ARTPEC-8 SoC. ARTPEC-8 is the SoC platform of Axis Communications and PCIe phy is designed based on SAMSUNG PHY. Signed-off-by: Wangseok Lee <wangseok.lee@samsung.com> --- .../bindings/phy/axis,artpec8-pcie-phy.yaml | 67 ++++++++++++++++++++++ 1 file changed, 67 insertions(+) create mode 100644 Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml diff --git a/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml b/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml new file mode 100644 index 0000000..f5f4166 --- /dev/null +++ b/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml @@ -0,0 +1,67 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/phy/axis,artpec8-pcie-phy.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: ARTPEC-8 SoC PCIe PHY Device Tree Bindings + +maintainers: + - Jesper Nilsson <jesper.nilsson@axis.com> + +properties: + compatible: + const: axis,artpec8-pcie-phy + + reg: + items: + - description: PHY registers. + - description: PHY coding sublayer registers. + + reg-names: + items: + - const: phy + - const: pcs + + clocks: + items: + - description: PCIe PHY reference clock + + clock-names: + items: + - const: ref_clk + +required: + - compatible + - "#phy-cells" + - reg + - reg-names + - clocks + - clock-names + - samsung,fsys-sysreg + - num-lanes + + +additionalProperties: false + +examples: + - | + #include <dt-bindings/interrupt-controller/irq.h> + #include <dt-bindings/interrupt-controller/arm-gic.h> + + artec8 { + #address-cells = <2>; + #size-cells = <2>; + pcie_phy: pcie-phy@16c80000 { + compatible = "samsung,artpec8-pcie-phy"; + #phy-cells = <0>; + reg = <0x0 0x16c80000 0x0 0x2000>, + <0x0 0x16c90000 0x0 0x1000>; + reg-names = "phy", "pcs"; + clocks = <&clock_cmu_fsys 53>; + clock-names = "ref_clk"; + samsung,fsys-sysreg = <&syscon_fsys>; + num-lanes = <2>; + }; + }; +... -- 2.9.5 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy 2022-03-28 8:43 ` [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy 이왕석 @ 2022-03-28 12:51 ` Rob Herring 2022-03-28 18:32 ` Krzysztof Kozlowski [not found] ` <CGME20220328084319epcms2p3670d7836bb111e0d9c648aeebfde8603@epcms2p6> 2 siblings, 0 replies; 6+ messages in thread From: Rob Herring @ 2022-03-28 12:51 UTC (permalink / raw) To: 이왕석 Cc: kernel, vkoul, bhelgaas, devicetree, lorenzo.pieralisi, krzk+dt, kw, linux-pci, 전문기, linux-phy, linux-arm-kernel, robh+dt, jesper.nilsson, linux-kernel, kishon, lars.persson [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #1: Type: text/plain, Size: 1266 bytes --] On Mon, 28 Mar 2022 17:43:19 +0900, 이왕석 wrote: > Add description to support Axis, ARTPEC-8 SoC. > ARTPEC-8 is the SoC platform of Axis Communications > and PCIe phy is designed based on SAMSUNG PHY. > > Signed-off-by: Wangseok Lee <wangseok.lee@samsung.com> > --- > .../bindings/phy/axis,artpec8-pcie-phy.yaml | 67 ++++++++++++++++++++++ > 1 file changed, 67 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' on your patch (DT_CHECKER_FLAGS is new in v5.13): yamllint warnings/errors: dtschema/dtc warnings/errors: Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.example.dt.yaml:0:0: /example-0/artec8/pcie-phy@16c80000: failed to match any schema with compatible: ['samsung,artpec8-pcie-phy'] doc reference errors (make refcheckdocs): See https://patchwork.ozlabs.org/patch/1610068 This check can fail if there are any dependencies. The base for a patch series is generally the most recent rc1. If you already ran 'make dt_binding_check' and didn't see the above error(s), then make sure 'yamllint' is installed and dt-schema is up to date: pip3 install dtschema --upgrade Please check and re-submit. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy 2022-03-28 8:43 ` [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy 이왕석 2022-03-28 12:51 ` Rob Herring @ 2022-03-28 18:32 ` Krzysztof Kozlowski [not found] ` <CGME20220328084319epcms2p3670d7836bb111e0d9c648aeebfde8603@epcms2p6> 2 siblings, 0 replies; 6+ messages in thread From: Krzysztof Kozlowski @ 2022-03-28 18:32 UTC (permalink / raw) To: wangseok.lee, robh+dt, krzk+dt, kishon, vkoul, linux-kernel, jesper.nilsson, lars.persson Cc: bhelgaas, linux-phy, linux-pci, devicetree, lorenzo.pieralisi, kw, linux-arm-kernel, kernel, 전문기 On 28/03/2022 10:43, 이왕석 wrote: > Add description to support Axis, ARTPEC-8 SoC. > ARTPEC-8 is the SoC platform of Axis Communications > and PCIe phy is designed based on SAMSUNG PHY. > > Signed-off-by: Wangseok Lee <wangseok.lee@samsung.com> > --- > .../bindings/phy/axis,artpec8-pcie-phy.yaml | 67 ++++++++++++++++++++++ > 1 file changed, 67 insertions(+) > create mode 100644 Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml > > diff --git a/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml b/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml > new file mode 100644 > index 0000000..f5f4166 > --- /dev/null > +++ b/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml > @@ -0,0 +1,67 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/phy/axis,artpec8-pcie-phy.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: ARTPEC-8 SoC PCIe PHY Device Tree Bindings s/Device Tree Bindings// > + > +maintainers: > + - Jesper Nilsson <jesper.nilsson@axis.com> > + > +properties: > + compatible: > + const: axis,artpec8-pcie-phy > + > + reg: > + items: > + - description: PHY registers. > + - description: PHY coding sublayer registers. > + > + reg-names: > + items: > + - const: phy > + - const: pcs > + > + clocks: > + items: > + - description: PCIe PHY reference clock > + > + clock-names: > + items: > + - const: ref_clk > + > +required: > + - compatible > + - "#phy-cells" > + - reg > + - reg-names > + - clocks > + - clock-names > + - samsung,fsys-sysreg It seems you copied some things from some other bindings, so please clean it up to really fit your device. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CGME20220328084319epcms2p3670d7836bb111e0d9c648aeebfde8603@epcms2p6>]
* Re: [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy [not found] ` <CGME20220328084319epcms2p3670d7836bb111e0d9c648aeebfde8603@epcms2p6> @ 2022-03-29 5:20 ` 이왕석 0 siblings, 0 replies; 6+ messages in thread From: 이왕석 @ 2022-03-29 5:20 UTC (permalink / raw) To: Rob Herring Cc: kernel, vkoul, bhelgaas, devicetree, lorenzo.pieralisi, krzk+dt, kw, linux-pci, 전문기, linux-phy, linux-arm-kernel, robh+dt, jesper.nilsson, linux-kernel, kishon, lars.persson > --------- Original Message --------- > Sender : Rob Herring <robh@kernel.org> > Date : 2022-03-28 21:51 (GMT+9) > Title : Re: [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy > > On Mon, 28 Mar 2022 17:43:19 +0900, �댁���� wrote: >> Add description to support Axis, ARTPEC-8 SoC. >> ARTPEC-8 is the SoC platform of Axis Communications >> and PCIe phy is designed based on SAMSUNG PHY. >> >> Signed-off-by: Wangseok Lee <wangseok.lee@samsung.com> >> --- >> .../bindings/phy/axis,artpec8-pcie-phy.yaml | 67 ++++++++++++++++++++++ >> 1 file changed, 67 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml >> > > My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' > on your patch (DT_CHECKER_FLAGS is new in v5.13): > > yamllint warnings/errors: > > dtschema/dtc warnings/errors: > Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.example.dt.yaml:0:0: /example-0/artec8/pcie-phy@16c80000: failed to match any schema with compatible: ['samsung,artpec8-pcie-phy'] > > doc reference errors (make refcheckdocs): > > See https://protect2.fireeye.com/v1/url?k=0fdd5edc-6f3fc381-0fdcd593-000babd9f1ba-cd731013fdd72389&q=1&e=df20f229-606a-4e6a-bcb2-2cbdc399248e&u=https%3A%2F%2Fpatchwork.ozlabs.org%2Fpatch%2F1610068 > > This check can fail if there are any dependencies. The base for a patch > series is generally the most recent rc1. > > If you already ran 'make dt_binding_check' and didn't see the above > error(s), then make sure 'yamllint' is installed and dt-schema is up to > date: > > pip3 install dtschema --upgrade > > Please check and re-submit. Hello Rob Herring Thanks for your review. I wrote the wrong "compatible" value. so i will modify this part and summit it again. also i checked the result by performing "make dt_binding_check DT_SCHEMA_FILES= Documentation/devicetree/bindings/pci/axis,artpec8-pcie.yaml". Thanks. ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CGME20220328020947epcms2p2525f40ac5157735063927aedfbf6fc4f@epcms2p2>]
* [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy [not found] <CGME20220328020947epcms2p2525f40ac5157735063927aedfbf6fc4f@epcms2p2> @ 2022-03-28 2:09 ` 이왕석 2022-03-28 7:08 ` Krzysztof Kozlowski 0 siblings, 1 reply; 6+ messages in thread From: 이왕석 @ 2022-03-28 2:09 UTC (permalink / raw) To: robh+dt, krzk+dt, kishon, vkoul, linux-kernel, jesper.nilsson, lars.persson Cc: bhelgaas, linux-phy, linux-pci, devicetree, lorenzo.pieralisi, kw, linux-arm-kernel, kernel, 전문기 Add description to support Axis, ARTPEC-8 SoC. ARTPEC-8 is the SoC platform of Axis Communications and PCIe phy is designed based on SAMSUNG PHY. Signed-off-by: Wangseok Lee <wangseok.lee@samsung.com> --- .../bindings/phy/axis,artpec8-pcie-phy.yaml | 67 ++++++++++++++++++++++ 1 file changed, 67 insertions(+) create mode 100644 Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml diff --git a/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml b/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml new file mode 100644 index 0000000..f5f4166 --- /dev/null +++ b/Documentation/devicetree/bindings/phy/axis,artpec8-pcie-phy.yaml @@ -0,0 +1,67 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/phy/axis,artpec8-pcie-phy.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: ARTPEC-8 SoC PCIe PHY Device Tree Bindings + +maintainers: + - Jesper Nilsson <jesper.nilsson@axis.com> + +properties: + compatible: + const: axis,artpec8-pcie-phy + + reg: + items: + - description: PHY registers. + - description: PHY coding sublayer registers. + + reg-names: + items: + - const: phy + - const: pcs + + clocks: + items: + - description: PCIe PHY reference clock + + clock-names: + items: + - const: ref_clk + +required: + - compatible + - "#phy-cells" + - reg + - reg-names + - clocks + - clock-names + - samsung,fsys-sysreg + - num-lanes + + +additionalProperties: false + +examples: + - | + #include <dt-bindings/interrupt-controller/irq.h> + #include <dt-bindings/interrupt-controller/arm-gic.h> + + artec8 { + #address-cells = <2>; + #size-cells = <2>; + pcie_phy: pcie-phy@16c80000 { + compatible = "samsung,artpec8-pcie-phy"; + #phy-cells = <0>; + reg = <0x0 0x16c80000 0x0 0x2000>, + <0x0 0x16c90000 0x0 0x1000>; + reg-names = "phy", "pcs"; + clocks = <&clock_cmu_fsys 53>; + clock-names = "ref_clk"; + samsung,fsys-sysreg = <&syscon_fsys>; + num-lanes = <2>; + }; + }; +... -- 2.9.5 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy 2022-03-28 2:09 ` 이왕석 @ 2022-03-28 7:08 ` Krzysztof Kozlowski 0 siblings, 0 replies; 6+ messages in thread From: Krzysztof Kozlowski @ 2022-03-28 7:08 UTC (permalink / raw) To: wangseok.lee, robh+dt, krzk+dt, kishon, vkoul, linux-kernel, jesper.nilsson, lars.persson Cc: bhelgaas, linux-phy, linux-pci, devicetree, lorenzo.pieralisi, kw, linux-arm-kernel, kernel, 전문기 On 28/03/2022 04:09, 이왕석 wrote: > Add description to support Axis, ARTPEC-8 SoC. > > ARTPEC-8 is the SoC platform of Axis Communications > > and PCIe phy is designed based on SAMSUNG PHY. This email is corrupted. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-03-29 5:21 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <CGME20220328084319epcms2p3670d7836bb111e0d9c648aeebfde8603@epcms2p3> 2022-03-28 8:43 ` [PATCH 2/5] dt-bindings: phy: Add ARTPEC-8 PCIe phy 이왕석 2022-03-28 12:51 ` Rob Herring 2022-03-28 18:32 ` Krzysztof Kozlowski [not found] ` <CGME20220328084319epcms2p3670d7836bb111e0d9c648aeebfde8603@epcms2p6> 2022-03-29 5:20 ` 이왕석 [not found] <CGME20220328020947epcms2p2525f40ac5157735063927aedfbf6fc4f@epcms2p2> 2022-03-28 2:09 ` 이왕석 2022-03-28 7:08 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).