* [PATCH 1/6] dt-bindings: usb: Convert USB HCD generic binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel The USB HCD generic binding is used by many USB host bindings. In order to allow the DT validation to happen on those, let's create a YAML description for that generic binding that can be referenced later on. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/usb-hcd.txt | 9 +----- Documentation/devicetree/bindings/usb/usb-hcd.yaml | 25 +++++++++++++++- 2 files changed, 25 insertions(+), 9 deletions(-) delete mode 100644 Documentation/devicetree/bindings/usb/usb-hcd.txt create mode 100644 Documentation/devicetree/bindings/usb/usb-hcd.yaml diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.txt b/Documentation/devicetree/bindings/usb/usb-hcd.txt deleted file mode 100644 index 50529b838c9c..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-hcd.txt +++ /dev/null @@ -1,9 +0,0 @@ -Generic USB HCD (Host Controller Device) Properties - -Optional properties: -- phys: a list of all USB PHYs on this HCD - -Example: - &usb1 { - phys = <&usb2_phy1>, <&usb3_phy1>; - }; diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.yaml b/Documentation/devicetree/bindings/usb/usb-hcd.yaml new file mode 100644 index 000000000000..9c8c56d3a792 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/usb-hcd.yaml @@ -0,0 +1,25 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/usb-hcd.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Generic USB Host Controller Device Tree Bindings + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + $nodename: + pattern: "^usb(@.*)?" + + phys: + $ref: /schemas/types.yaml#/definitions/phandle-array + description: + List of all the USB PHYs on this HCD + +examples: + - | + usb { + phys = <&usb2_phy1>, <&usb3_phy1>; + }; base-commit: f9221a7a1014d8a047b277a73289678646ddc110 -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 1/6] dt-bindings: usb: Convert USB HCD generic binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel The USB HCD generic binding is used by many USB host bindings. In order to allow the DT validation to happen on those, let's create a YAML description for that generic binding that can be referenced later on. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/usb-hcd.txt | 9 +----- Documentation/devicetree/bindings/usb/usb-hcd.yaml | 25 +++++++++++++++- 2 files changed, 25 insertions(+), 9 deletions(-) delete mode 100644 Documentation/devicetree/bindings/usb/usb-hcd.txt create mode 100644 Documentation/devicetree/bindings/usb/usb-hcd.yaml diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.txt b/Documentation/devicetree/bindings/usb/usb-hcd.txt deleted file mode 100644 index 50529b838c9c..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-hcd.txt +++ /dev/null @@ -1,9 +0,0 @@ -Generic USB HCD (Host Controller Device) Properties - -Optional properties: -- phys: a list of all USB PHYs on this HCD - -Example: - &usb1 { - phys = <&usb2_phy1>, <&usb3_phy1>; - }; diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.yaml b/Documentation/devicetree/bindings/usb/usb-hcd.yaml new file mode 100644 index 000000000000..9c8c56d3a792 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/usb-hcd.yaml @@ -0,0 +1,25 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/usb-hcd.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Generic USB Host Controller Device Tree Bindings + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + $nodename: + pattern: "^usb(@.*)?" + + phys: + $ref: /schemas/types.yaml#/definitions/phandle-array + description: + List of all the USB PHYs on this HCD + +examples: + - | + usb { + phys = <&usb2_phy1>, <&usb3_phy1>; + }; base-commit: f9221a7a1014d8a047b277a73289678646ddc110 -- git-series 0.9.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 1/6] dt-bindings: usb: Convert USB HCD generic binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel The USB HCD generic binding is used by many USB host bindings. In order to allow the DT validation to happen on those, let's create a YAML description for that generic binding that can be referenced later on. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/usb-hcd.txt | 9 +----- Documentation/devicetree/bindings/usb/usb-hcd.yaml | 25 +++++++++++++++- 2 files changed, 25 insertions(+), 9 deletions(-) delete mode 100644 Documentation/devicetree/bindings/usb/usb-hcd.txt create mode 100644 Documentation/devicetree/bindings/usb/usb-hcd.yaml diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.txt b/Documentation/devicetree/bindings/usb/usb-hcd.txt deleted file mode 100644 index 50529b838c9c..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-hcd.txt +++ /dev/null @@ -1,9 +0,0 @@ -Generic USB HCD (Host Controller Device) Properties - -Optional properties: -- phys: a list of all USB PHYs on this HCD - -Example: - &usb1 { - phys = <&usb2_phy1>, <&usb3_phy1>; - }; diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.yaml b/Documentation/devicetree/bindings/usb/usb-hcd.yaml new file mode 100644 index 000000000000..9c8c56d3a792 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/usb-hcd.yaml @@ -0,0 +1,25 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/usb-hcd.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Generic USB Host Controller Device Tree Bindings + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + $nodename: + pattern: "^usb(@.*)?" + + phys: + $ref: /schemas/types.yaml#/definitions/phandle-array + description: + List of all the USB PHYs on this HCD + +examples: + - | + usb { + phys = <&usb2_phy1>, <&usb3_phy1>; + }; base-commit: f9221a7a1014d8a047b277a73289678646ddc110 -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [1/6] dt-bindings: usb: Convert USB HCD generic binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel The USB HCD generic binding is used by many USB host bindings. In order to allow the DT validation to happen on those, let's create a YAML description for that generic binding that can be referenced later on. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/usb-hcd.txt | 9 +----- Documentation/devicetree/bindings/usb/usb-hcd.yaml | 25 +++++++++++++++- 2 files changed, 25 insertions(+), 9 deletions(-) delete mode 100644 Documentation/devicetree/bindings/usb/usb-hcd.txt create mode 100644 Documentation/devicetree/bindings/usb/usb-hcd.yaml base-commit: f9221a7a1014d8a047b277a73289678646ddc110 diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.txt b/Documentation/devicetree/bindings/usb/usb-hcd.txt deleted file mode 100644 index 50529b838c9c..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-hcd.txt +++ /dev/null @@ -1,9 +0,0 @@ -Generic USB HCD (Host Controller Device) Properties - -Optional properties: -- phys: a list of all USB PHYs on this HCD - -Example: - &usb1 { - phys = <&usb2_phy1>, <&usb3_phy1>; - }; diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.yaml b/Documentation/devicetree/bindings/usb/usb-hcd.yaml new file mode 100644 index 000000000000..9c8c56d3a792 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/usb-hcd.yaml @@ -0,0 +1,25 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/usb-hcd.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Generic USB Host Controller Device Tree Bindings + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + $nodename: + pattern: "^usb(@.*)?" + + phys: + $ref: /schemas/types.yaml#/definitions/phandle-array + description: + List of all the USB PHYs on this HCD + +examples: + - | + usb { + phys = <&usb2_phy1>, <&usb3_phy1>; + }; ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 2/6] dt-bindings: usb: Convert the generic OHCI binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel The generic OHCI binding is used by many controllers that are using the OHCI spec. Convert that binding to a YAML description to enable the validation on all the nodes using that binding. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/generic-ohci.yaml | 89 ++++++++++- Documentation/devicetree/bindings/usb/usb-ohci.txt | 35 +---- 2 files changed, 89 insertions(+), 35 deletions(-) create mode 100644 Documentation/devicetree/bindings/usb/generic-ohci.yaml delete mode 100644 Documentation/devicetree/bindings/usb/usb-ohci.txt diff --git a/Documentation/devicetree/bindings/usb/generic-ohci.yaml b/Documentation/devicetree/bindings/usb/generic-ohci.yaml new file mode 100644 index 000000000000..da5a14becbe5 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/generic-ohci.yaml @@ -0,0 +1,89 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/generic-ohci.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: USB OHCI Controller Device Tree Bindings + +allOf: + - $ref: "usb-hcd.yaml" + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + compatible: + contains: + const: generic-ohci + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + resets: + minItems: 1 + maxItems: 2 + + clocks: + minItems: 1 + maxItems: 3 + description: | + In case the Renesas R-Car Gen3 SoCs: + - if a host only channel: first clock should be host. + - if a USB DRD channel: first clock should be host and second + one should be peripheral + + big-endian: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors and big + endian registers. + + big-endian-desc: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors. + + big-endian-regs: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian registers. + + remote-wakeup-connected: + $ref: /schemas/types.yaml#/definitions/flag + description: + Remote wakeup is wired on the platform. + + no-big-frame-no: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set if frame_no lives in bits [15:0] of HCCA + + num-ports: + $ref: /schemas/types.yaml#/definitions/uint32 + description: + Overrides the detected port count + + phys: true + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + ohci0: usb@1c14400 { + compatible = "allwinner,sun4i-a10-ohci", "generic-ohci"; + reg = <0x01c14400 0x100>; + interrupts = <64>; + clocks = <&usb_clk 6>, <&ahb_gates 2>; + phys = <&usbphy 1>; + }; + +... diff --git a/Documentation/devicetree/bindings/usb/usb-ohci.txt b/Documentation/devicetree/bindings/usb/usb-ohci.txt deleted file mode 100644 index aaaa5255c972..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-ohci.txt +++ /dev/null @@ -1,35 +0,0 @@ -USB OHCI controllers - -Required properties: -- compatible : "generic-ohci" -- reg : ohci controller register range (address and length) -- interrupts : ohci controller interrupt - -Optional properties: -- big-endian-regs : boolean, set this for hcds with big-endian registers -- big-endian-desc : boolean, set this for hcds with big-endian descriptors -- big-endian : boolean, for hcds with big-endian-regs + big-endian-desc -- no-big-frame-no : boolean, set if frame_no lives in bits [15:0] of HCCA -- remote-wakeup-connected: remote wakeup is wired on the platform -- num-ports : u32, to override the detected port count -- clocks : a list of phandle + clock specifier pairs. In case of Renesas - R-Car Gen3 SoCs: - - if a host only channel: first clock should be host. - - if a USB DRD channel: first clock should be host and second one - should be peripheral. -- phys : see usb-hcd.txt in the current directory -- resets : a list of phandle + reset specifier pairs - -additionally the properties from usb-hcd.txt (in the current directory) are -supported. - -Example: - - ohci0: usb@1c14400 { - compatible = "allwinner,sun4i-a10-ohci", "generic-ohci"; - reg = <0x01c14400 0x100>; - interrupts = <64>; - clocks = <&usb_clk 6>, <&ahb_gates 2>; - phys = <&usbphy 1>; - phy-names = "usb"; - }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 2/6] dt-bindings: usb: Convert the generic OHCI binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel The generic OHCI binding is used by many controllers that are using the OHCI spec. Convert that binding to a YAML description to enable the validation on all the nodes using that binding. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/generic-ohci.yaml | 89 ++++++++++- Documentation/devicetree/bindings/usb/usb-ohci.txt | 35 +---- 2 files changed, 89 insertions(+), 35 deletions(-) create mode 100644 Documentation/devicetree/bindings/usb/generic-ohci.yaml delete mode 100644 Documentation/devicetree/bindings/usb/usb-ohci.txt diff --git a/Documentation/devicetree/bindings/usb/generic-ohci.yaml b/Documentation/devicetree/bindings/usb/generic-ohci.yaml new file mode 100644 index 000000000000..da5a14becbe5 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/generic-ohci.yaml @@ -0,0 +1,89 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/generic-ohci.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: USB OHCI Controller Device Tree Bindings + +allOf: + - $ref: "usb-hcd.yaml" + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + compatible: + contains: + const: generic-ohci + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + resets: + minItems: 1 + maxItems: 2 + + clocks: + minItems: 1 + maxItems: 3 + description: | + In case the Renesas R-Car Gen3 SoCs: + - if a host only channel: first clock should be host. + - if a USB DRD channel: first clock should be host and second + one should be peripheral + + big-endian: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors and big + endian registers. + + big-endian-desc: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors. + + big-endian-regs: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian registers. + + remote-wakeup-connected: + $ref: /schemas/types.yaml#/definitions/flag + description: + Remote wakeup is wired on the platform. + + no-big-frame-no: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set if frame_no lives in bits [15:0] of HCCA + + num-ports: + $ref: /schemas/types.yaml#/definitions/uint32 + description: + Overrides the detected port count + + phys: true + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + ohci0: usb@1c14400 { + compatible = "allwinner,sun4i-a10-ohci", "generic-ohci"; + reg = <0x01c14400 0x100>; + interrupts = <64>; + clocks = <&usb_clk 6>, <&ahb_gates 2>; + phys = <&usbphy 1>; + }; + +... diff --git a/Documentation/devicetree/bindings/usb/usb-ohci.txt b/Documentation/devicetree/bindings/usb/usb-ohci.txt deleted file mode 100644 index aaaa5255c972..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-ohci.txt +++ /dev/null @@ -1,35 +0,0 @@ -USB OHCI controllers - -Required properties: -- compatible : "generic-ohci" -- reg : ohci controller register range (address and length) -- interrupts : ohci controller interrupt - -Optional properties: -- big-endian-regs : boolean, set this for hcds with big-endian registers -- big-endian-desc : boolean, set this for hcds with big-endian descriptors -- big-endian : boolean, for hcds with big-endian-regs + big-endian-desc -- no-big-frame-no : boolean, set if frame_no lives in bits [15:0] of HCCA -- remote-wakeup-connected: remote wakeup is wired on the platform -- num-ports : u32, to override the detected port count -- clocks : a list of phandle + clock specifier pairs. In case of Renesas - R-Car Gen3 SoCs: - - if a host only channel: first clock should be host. - - if a USB DRD channel: first clock should be host and second one - should be peripheral. -- phys : see usb-hcd.txt in the current directory -- resets : a list of phandle + reset specifier pairs - -additionally the properties from usb-hcd.txt (in the current directory) are -supported. - -Example: - - ohci0: usb@1c14400 { - compatible = "allwinner,sun4i-a10-ohci", "generic-ohci"; - reg = <0x01c14400 0x100>; - interrupts = <64>; - clocks = <&usb_clk 6>, <&ahb_gates 2>; - phys = <&usbphy 1>; - phy-names = "usb"; - }; -- git-series 0.9.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 2/6] dt-bindings: usb: Convert the generic OHCI binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel The generic OHCI binding is used by many controllers that are using the OHCI spec. Convert that binding to a YAML description to enable the validation on all the nodes using that binding. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/generic-ohci.yaml | 89 ++++++++++- Documentation/devicetree/bindings/usb/usb-ohci.txt | 35 +---- 2 files changed, 89 insertions(+), 35 deletions(-) create mode 100644 Documentation/devicetree/bindings/usb/generic-ohci.yaml delete mode 100644 Documentation/devicetree/bindings/usb/usb-ohci.txt diff --git a/Documentation/devicetree/bindings/usb/generic-ohci.yaml b/Documentation/devicetree/bindings/usb/generic-ohci.yaml new file mode 100644 index 000000000000..da5a14becbe5 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/generic-ohci.yaml @@ -0,0 +1,89 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/generic-ohci.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: USB OHCI Controller Device Tree Bindings + +allOf: + - $ref: "usb-hcd.yaml" + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + compatible: + contains: + const: generic-ohci + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + resets: + minItems: 1 + maxItems: 2 + + clocks: + minItems: 1 + maxItems: 3 + description: | + In case the Renesas R-Car Gen3 SoCs: + - if a host only channel: first clock should be host. + - if a USB DRD channel: first clock should be host and second + one should be peripheral + + big-endian: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors and big + endian registers. + + big-endian-desc: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors. + + big-endian-regs: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian registers. + + remote-wakeup-connected: + $ref: /schemas/types.yaml#/definitions/flag + description: + Remote wakeup is wired on the platform. + + no-big-frame-no: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set if frame_no lives in bits [15:0] of HCCA + + num-ports: + $ref: /schemas/types.yaml#/definitions/uint32 + description: + Overrides the detected port count + + phys: true + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + ohci0: usb@1c14400 { + compatible = "allwinner,sun4i-a10-ohci", "generic-ohci"; + reg = <0x01c14400 0x100>; + interrupts = <64>; + clocks = <&usb_clk 6>, <&ahb_gates 2>; + phys = <&usbphy 1>; + }; + +... diff --git a/Documentation/devicetree/bindings/usb/usb-ohci.txt b/Documentation/devicetree/bindings/usb/usb-ohci.txt deleted file mode 100644 index aaaa5255c972..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-ohci.txt +++ /dev/null @@ -1,35 +0,0 @@ -USB OHCI controllers - -Required properties: -- compatible : "generic-ohci" -- reg : ohci controller register range (address and length) -- interrupts : ohci controller interrupt - -Optional properties: -- big-endian-regs : boolean, set this for hcds with big-endian registers -- big-endian-desc : boolean, set this for hcds with big-endian descriptors -- big-endian : boolean, for hcds with big-endian-regs + big-endian-desc -- no-big-frame-no : boolean, set if frame_no lives in bits [15:0] of HCCA -- remote-wakeup-connected: remote wakeup is wired on the platform -- num-ports : u32, to override the detected port count -- clocks : a list of phandle + clock specifier pairs. In case of Renesas - R-Car Gen3 SoCs: - - if a host only channel: first clock should be host. - - if a USB DRD channel: first clock should be host and second one - should be peripheral. -- phys : see usb-hcd.txt in the current directory -- resets : a list of phandle + reset specifier pairs - -additionally the properties from usb-hcd.txt (in the current directory) are -supported. - -Example: - - ohci0: usb@1c14400 { - compatible = "allwinner,sun4i-a10-ohci", "generic-ohci"; - reg = <0x01c14400 0x100>; - interrupts = <64>; - clocks = <&usb_clk 6>, <&ahb_gates 2>; - phys = <&usbphy 1>; - phy-names = "usb"; - }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [2/6] dt-bindings: usb: Convert the generic OHCI binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel The generic OHCI binding is used by many controllers that are using the OHCI spec. Convert that binding to a YAML description to enable the validation on all the nodes using that binding. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/generic-ohci.yaml | 89 ++++++++++- Documentation/devicetree/bindings/usb/usb-ohci.txt | 35 +---- 2 files changed, 89 insertions(+), 35 deletions(-) create mode 100644 Documentation/devicetree/bindings/usb/generic-ohci.yaml delete mode 100644 Documentation/devicetree/bindings/usb/usb-ohci.txt diff --git a/Documentation/devicetree/bindings/usb/generic-ohci.yaml b/Documentation/devicetree/bindings/usb/generic-ohci.yaml new file mode 100644 index 000000000000..da5a14becbe5 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/generic-ohci.yaml @@ -0,0 +1,89 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/generic-ohci.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: USB OHCI Controller Device Tree Bindings + +allOf: + - $ref: "usb-hcd.yaml" + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + compatible: + contains: + const: generic-ohci + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + resets: + minItems: 1 + maxItems: 2 + + clocks: + minItems: 1 + maxItems: 3 + description: | + In case the Renesas R-Car Gen3 SoCs: + - if a host only channel: first clock should be host. + - if a USB DRD channel: first clock should be host and second + one should be peripheral + + big-endian: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors and big + endian registers. + + big-endian-desc: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors. + + big-endian-regs: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian registers. + + remote-wakeup-connected: + $ref: /schemas/types.yaml#/definitions/flag + description: + Remote wakeup is wired on the platform. + + no-big-frame-no: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set if frame_no lives in bits [15:0] of HCCA + + num-ports: + $ref: /schemas/types.yaml#/definitions/uint32 + description: + Overrides the detected port count + + phys: true + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + ohci0: usb@1c14400 { + compatible = "allwinner,sun4i-a10-ohci", "generic-ohci"; + reg = <0x01c14400 0x100>; + interrupts = <64>; + clocks = <&usb_clk 6>, <&ahb_gates 2>; + phys = <&usbphy 1>; + }; + +... diff --git a/Documentation/devicetree/bindings/usb/usb-ohci.txt b/Documentation/devicetree/bindings/usb/usb-ohci.txt deleted file mode 100644 index aaaa5255c972..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-ohci.txt +++ /dev/null @@ -1,35 +0,0 @@ -USB OHCI controllers - -Required properties: -- compatible : "generic-ohci" -- reg : ohci controller register range (address and length) -- interrupts : ohci controller interrupt - -Optional properties: -- big-endian-regs : boolean, set this for hcds with big-endian registers -- big-endian-desc : boolean, set this for hcds with big-endian descriptors -- big-endian : boolean, for hcds with big-endian-regs + big-endian-desc -- no-big-frame-no : boolean, set if frame_no lives in bits [15:0] of HCCA -- remote-wakeup-connected: remote wakeup is wired on the platform -- num-ports : u32, to override the detected port count -- clocks : a list of phandle + clock specifier pairs. In case of Renesas - R-Car Gen3 SoCs: - - if a host only channel: first clock should be host. - - if a USB DRD channel: first clock should be host and second one - should be peripheral. -- phys : see usb-hcd.txt in the current directory -- resets : a list of phandle + reset specifier pairs - -additionally the properties from usb-hcd.txt (in the current directory) are -supported. - -Example: - - ohci0: usb@1c14400 { - compatible = "allwinner,sun4i-a10-ohci", "generic-ohci"; - reg = <0x01c14400 0x100>; - interrupts = <64>; - clocks = <&usb_clk 6>, <&ahb_gates 2>; - phys = <&usbphy 1>; - phy-names = "usb"; - }; ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 3/6] dt-bindings: usb: Convert the generic EHCI binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel The generic EHCI binding is used by many controllers that are using the EHCI spec. Convert that binding to a YAML description to enable the validation on all the nodes using that binding. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/generic-ehci.yaml | 95 ++++++++++- Documentation/devicetree/bindings/usb/usb-ehci.txt | 46 +----- 2 files changed, 95 insertions(+), 46 deletions(-) create mode 100644 Documentation/devicetree/bindings/usb/generic-ehci.yaml delete mode 100644 Documentation/devicetree/bindings/usb/usb-ehci.txt diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml new file mode 100644 index 000000000000..d3b4f6415920 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml @@ -0,0 +1,95 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/generic-ehci.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: USB EHCI Controller Device Tree Bindings + +allOf: + - $ref: "usb-hcd.yaml" + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + compatible: + contains: + const: generic-ehci + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + resets: + minItems: 1 + maxItems: 4 + + clocks: + minItems: 1 + maxItems: 4 + description: | + In case the Renesas R-Car Gen3 SoCs: + - if a host only channel: first clock should be host. + - if a USB DRD channel: first clock should be host and second + one should be peripheral + + big-endian: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors and big + endian registers. + + big-endian-desc: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors. + + big-endian-regs: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian registers. + + has-transaction-translator: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag if EHCI has a Transaction Translator built into + the root hub. + + needs-reset-on-resume: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag to force EHCI reset after resume. + + phys: true + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + ehci@e0000300 { + compatible = "ibm,usb-ehci-440epx", "generic-ehci"; + interrupt-parent = <&UIC0>; + interrupts = <0x1a 4>; + reg = <0 0xe0000300 90 0 0xe0000390 70>; + big-endian; + }; + + - | + ehci0: usb@1c14000 { + compatible = "allwinner,sun4i-a10-ehci", "generic-ehci"; + reg = <0x01c14000 0x100>; + interrupts = <39>; + clocks = <&ahb_gates 1>; + phys = <&usbphy 1>; + phy-names = "usb"; + }; + +... diff --git a/Documentation/devicetree/bindings/usb/usb-ehci.txt b/Documentation/devicetree/bindings/usb/usb-ehci.txt deleted file mode 100644 index 406252d14c6b..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-ehci.txt +++ /dev/null @@ -1,46 +0,0 @@ -USB EHCI controllers - -Required properties: - - compatible : should be "generic-ehci". - - reg : should contain at least address and length of the standard EHCI - register set for the device. Optional platform-dependent registers - (debug-port or other) can be also specified here, but only after - definition of standard EHCI registers. - - interrupts : one EHCI interrupt should be described here. - -Optional properties: - - big-endian-regs : boolean, set this for hcds with big-endian registers - - big-endian-desc : boolean, set this for hcds with big-endian descriptors - - big-endian : boolean, for hcds with big-endian-regs + big-endian-desc - - needs-reset-on-resume : boolean, set this to force EHCI reset after resume - - has-transaction-translator : boolean, set this if EHCI have a Transaction - Translator built into the root hub. - - clocks : a list of phandle + clock specifier pairs. In case of Renesas - R-Car Gen3 SoCs: - - if a host only channel: first clock should be host. - - if a USB DRD channel: first clock should be host and second one - should be peripheral. - - phys : see usb-hcd.txt in the current directory - - resets : phandle + reset specifier pair - -additionally the properties from usb-hcd.txt (in the current directory) are -supported. - -Example (Sequoia 440EPx): - ehci@e0000300 { - compatible = "ibm,usb-ehci-440epx", "usb-ehci"; - interrupt-parent = <&UIC0>; - interrupts = <1a 4>; - reg = <0 e0000300 90 0 e0000390 70>; - big-endian; - }; - -Example (Allwinner sun4i A10 SoC): - ehci0: usb@1c14000 { - compatible = "allwinner,sun4i-a10-ehci", "generic-ehci"; - reg = <0x01c14000 0x100>; - interrupts = <39>; - clocks = <&ahb_gates 1>; - phys = <&usbphy 1>; - phy-names = "usb"; - }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 3/6] dt-bindings: usb: Convert the generic EHCI binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel The generic EHCI binding is used by many controllers that are using the EHCI spec. Convert that binding to a YAML description to enable the validation on all the nodes using that binding. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/generic-ehci.yaml | 95 ++++++++++- Documentation/devicetree/bindings/usb/usb-ehci.txt | 46 +----- 2 files changed, 95 insertions(+), 46 deletions(-) create mode 100644 Documentation/devicetree/bindings/usb/generic-ehci.yaml delete mode 100644 Documentation/devicetree/bindings/usb/usb-ehci.txt diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml new file mode 100644 index 000000000000..d3b4f6415920 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml @@ -0,0 +1,95 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/generic-ehci.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: USB EHCI Controller Device Tree Bindings + +allOf: + - $ref: "usb-hcd.yaml" + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + compatible: + contains: + const: generic-ehci + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + resets: + minItems: 1 + maxItems: 4 + + clocks: + minItems: 1 + maxItems: 4 + description: | + In case the Renesas R-Car Gen3 SoCs: + - if a host only channel: first clock should be host. + - if a USB DRD channel: first clock should be host and second + one should be peripheral + + big-endian: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors and big + endian registers. + + big-endian-desc: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors. + + big-endian-regs: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian registers. + + has-transaction-translator: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag if EHCI has a Transaction Translator built into + the root hub. + + needs-reset-on-resume: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag to force EHCI reset after resume. + + phys: true + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + ehci@e0000300 { + compatible = "ibm,usb-ehci-440epx", "generic-ehci"; + interrupt-parent = <&UIC0>; + interrupts = <0x1a 4>; + reg = <0 0xe0000300 90 0 0xe0000390 70>; + big-endian; + }; + + - | + ehci0: usb@1c14000 { + compatible = "allwinner,sun4i-a10-ehci", "generic-ehci"; + reg = <0x01c14000 0x100>; + interrupts = <39>; + clocks = <&ahb_gates 1>; + phys = <&usbphy 1>; + phy-names = "usb"; + }; + +... diff --git a/Documentation/devicetree/bindings/usb/usb-ehci.txt b/Documentation/devicetree/bindings/usb/usb-ehci.txt deleted file mode 100644 index 406252d14c6b..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-ehci.txt +++ /dev/null @@ -1,46 +0,0 @@ -USB EHCI controllers - -Required properties: - - compatible : should be "generic-ehci". - - reg : should contain at least address and length of the standard EHCI - register set for the device. Optional platform-dependent registers - (debug-port or other) can be also specified here, but only after - definition of standard EHCI registers. - - interrupts : one EHCI interrupt should be described here. - -Optional properties: - - big-endian-regs : boolean, set this for hcds with big-endian registers - - big-endian-desc : boolean, set this for hcds with big-endian descriptors - - big-endian : boolean, for hcds with big-endian-regs + big-endian-desc - - needs-reset-on-resume : boolean, set this to force EHCI reset after resume - - has-transaction-translator : boolean, set this if EHCI have a Transaction - Translator built into the root hub. - - clocks : a list of phandle + clock specifier pairs. In case of Renesas - R-Car Gen3 SoCs: - - if a host only channel: first clock should be host. - - if a USB DRD channel: first clock should be host and second one - should be peripheral. - - phys : see usb-hcd.txt in the current directory - - resets : phandle + reset specifier pair - -additionally the properties from usb-hcd.txt (in the current directory) are -supported. - -Example (Sequoia 440EPx): - ehci@e0000300 { - compatible = "ibm,usb-ehci-440epx", "usb-ehci"; - interrupt-parent = <&UIC0>; - interrupts = <1a 4>; - reg = <0 e0000300 90 0 e0000390 70>; - big-endian; - }; - -Example (Allwinner sun4i A10 SoC): - ehci0: usb@1c14000 { - compatible = "allwinner,sun4i-a10-ehci", "generic-ehci"; - reg = <0x01c14000 0x100>; - interrupts = <39>; - clocks = <&ahb_gates 1>; - phys = <&usbphy 1>; - phy-names = "usb"; - }; -- git-series 0.9.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 3/6] dt-bindings: usb: Convert the generic EHCI binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel The generic EHCI binding is used by many controllers that are using the EHCI spec. Convert that binding to a YAML description to enable the validation on all the nodes using that binding. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/generic-ehci.yaml | 95 ++++++++++- Documentation/devicetree/bindings/usb/usb-ehci.txt | 46 +----- 2 files changed, 95 insertions(+), 46 deletions(-) create mode 100644 Documentation/devicetree/bindings/usb/generic-ehci.yaml delete mode 100644 Documentation/devicetree/bindings/usb/usb-ehci.txt diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml new file mode 100644 index 000000000000..d3b4f6415920 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml @@ -0,0 +1,95 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/generic-ehci.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: USB EHCI Controller Device Tree Bindings + +allOf: + - $ref: "usb-hcd.yaml" + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + compatible: + contains: + const: generic-ehci + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + resets: + minItems: 1 + maxItems: 4 + + clocks: + minItems: 1 + maxItems: 4 + description: | + In case the Renesas R-Car Gen3 SoCs: + - if a host only channel: first clock should be host. + - if a USB DRD channel: first clock should be host and second + one should be peripheral + + big-endian: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors and big + endian registers. + + big-endian-desc: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors. + + big-endian-regs: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian registers. + + has-transaction-translator: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag if EHCI has a Transaction Translator built into + the root hub. + + needs-reset-on-resume: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag to force EHCI reset after resume. + + phys: true + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + ehci@e0000300 { + compatible = "ibm,usb-ehci-440epx", "generic-ehci"; + interrupt-parent = <&UIC0>; + interrupts = <0x1a 4>; + reg = <0 0xe0000300 90 0 0xe0000390 70>; + big-endian; + }; + + - | + ehci0: usb@1c14000 { + compatible = "allwinner,sun4i-a10-ehci", "generic-ehci"; + reg = <0x01c14000 0x100>; + interrupts = <39>; + clocks = <&ahb_gates 1>; + phys = <&usbphy 1>; + phy-names = "usb"; + }; + +... diff --git a/Documentation/devicetree/bindings/usb/usb-ehci.txt b/Documentation/devicetree/bindings/usb/usb-ehci.txt deleted file mode 100644 index 406252d14c6b..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-ehci.txt +++ /dev/null @@ -1,46 +0,0 @@ -USB EHCI controllers - -Required properties: - - compatible : should be "generic-ehci". - - reg : should contain at least address and length of the standard EHCI - register set for the device. Optional platform-dependent registers - (debug-port or other) can be also specified here, but only after - definition of standard EHCI registers. - - interrupts : one EHCI interrupt should be described here. - -Optional properties: - - big-endian-regs : boolean, set this for hcds with big-endian registers - - big-endian-desc : boolean, set this for hcds with big-endian descriptors - - big-endian : boolean, for hcds with big-endian-regs + big-endian-desc - - needs-reset-on-resume : boolean, set this to force EHCI reset after resume - - has-transaction-translator : boolean, set this if EHCI have a Transaction - Translator built into the root hub. - - clocks : a list of phandle + clock specifier pairs. In case of Renesas - R-Car Gen3 SoCs: - - if a host only channel: first clock should be host. - - if a USB DRD channel: first clock should be host and second one - should be peripheral. - - phys : see usb-hcd.txt in the current directory - - resets : phandle + reset specifier pair - -additionally the properties from usb-hcd.txt (in the current directory) are -supported. - -Example (Sequoia 440EPx): - ehci@e0000300 { - compatible = "ibm,usb-ehci-440epx", "usb-ehci"; - interrupt-parent = <&UIC0>; - interrupts = <1a 4>; - reg = <0 e0000300 90 0 e0000390 70>; - big-endian; - }; - -Example (Allwinner sun4i A10 SoC): - ehci0: usb@1c14000 { - compatible = "allwinner,sun4i-a10-ehci", "generic-ehci"; - reg = <0x01c14000 0x100>; - interrupts = <39>; - clocks = <&ahb_gates 1>; - phys = <&usbphy 1>; - phy-names = "usb"; - }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [3/6] dt-bindings: usb: Convert the generic EHCI binding to YAML @ 2019-04-16 8:27 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:27 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel The generic EHCI binding is used by many controllers that are using the EHCI spec. Convert that binding to a YAML description to enable the validation on all the nodes using that binding. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- Documentation/devicetree/bindings/usb/generic-ehci.yaml | 95 ++++++++++- Documentation/devicetree/bindings/usb/usb-ehci.txt | 46 +----- 2 files changed, 95 insertions(+), 46 deletions(-) create mode 100644 Documentation/devicetree/bindings/usb/generic-ehci.yaml delete mode 100644 Documentation/devicetree/bindings/usb/usb-ehci.txt diff --git a/Documentation/devicetree/bindings/usb/generic-ehci.yaml b/Documentation/devicetree/bindings/usb/generic-ehci.yaml new file mode 100644 index 000000000000..d3b4f6415920 --- /dev/null +++ b/Documentation/devicetree/bindings/usb/generic-ehci.yaml @@ -0,0 +1,95 @@ +# SPDX-License-Identifier: GPL-2.0 +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/usb/generic-ehci.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: USB EHCI Controller Device Tree Bindings + +allOf: + - $ref: "usb-hcd.yaml" + +maintainers: + - Greg Kroah-Hartman <gregkh@linuxfoundation.org> + +properties: + compatible: + contains: + const: generic-ehci + + reg: + maxItems: 1 + + interrupts: + maxItems: 1 + + resets: + minItems: 1 + maxItems: 4 + + clocks: + minItems: 1 + maxItems: 4 + description: | + In case the Renesas R-Car Gen3 SoCs: + - if a host only channel: first clock should be host. + - if a USB DRD channel: first clock should be host and second + one should be peripheral + + big-endian: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors and big + endian registers. + + big-endian-desc: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian descriptors. + + big-endian-regs: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag for HCDs with big endian registers. + + has-transaction-translator: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag if EHCI has a Transaction Translator built into + the root hub. + + needs-reset-on-resume: + $ref: /schemas/types.yaml#/definitions/flag + description: + Set this flag to force EHCI reset after resume. + + phys: true + +required: + - compatible + - reg + - interrupts + +additionalProperties: false + +examples: + - | + ehci@e0000300 { + compatible = "ibm,usb-ehci-440epx", "generic-ehci"; + interrupt-parent = <&UIC0>; + interrupts = <0x1a 4>; + reg = <0 0xe0000300 90 0 0xe0000390 70>; + big-endian; + }; + + - | + ehci0: usb@1c14000 { + compatible = "allwinner,sun4i-a10-ehci", "generic-ehci"; + reg = <0x01c14000 0x100>; + interrupts = <39>; + clocks = <&ahb_gates 1>; + phys = <&usbphy 1>; + phy-names = "usb"; + }; + +... diff --git a/Documentation/devicetree/bindings/usb/usb-ehci.txt b/Documentation/devicetree/bindings/usb/usb-ehci.txt deleted file mode 100644 index 406252d14c6b..000000000000 --- a/Documentation/devicetree/bindings/usb/usb-ehci.txt +++ /dev/null @@ -1,46 +0,0 @@ -USB EHCI controllers - -Required properties: - - compatible : should be "generic-ehci". - - reg : should contain at least address and length of the standard EHCI - register set for the device. Optional platform-dependent registers - (debug-port or other) can be also specified here, but only after - definition of standard EHCI registers. - - interrupts : one EHCI interrupt should be described here. - -Optional properties: - - big-endian-regs : boolean, set this for hcds with big-endian registers - - big-endian-desc : boolean, set this for hcds with big-endian descriptors - - big-endian : boolean, for hcds with big-endian-regs + big-endian-desc - - needs-reset-on-resume : boolean, set this to force EHCI reset after resume - - has-transaction-translator : boolean, set this if EHCI have a Transaction - Translator built into the root hub. - - clocks : a list of phandle + clock specifier pairs. In case of Renesas - R-Car Gen3 SoCs: - - if a host only channel: first clock should be host. - - if a USB DRD channel: first clock should be host and second one - should be peripheral. - - phys : see usb-hcd.txt in the current directory - - resets : phandle + reset specifier pair - -additionally the properties from usb-hcd.txt (in the current directory) are -supported. - -Example (Sequoia 440EPx): - ehci@e0000300 { - compatible = "ibm,usb-ehci-440epx", "usb-ehci"; - interrupt-parent = <&UIC0>; - interrupts = <1a 4>; - reg = <0 e0000300 90 0 e0000390 70>; - big-endian; - }; - -Example (Allwinner sun4i A10 SoC): - ehci0: usb@1c14000 { - compatible = "allwinner,sun4i-a10-ehci", "generic-ehci"; - reg = <0x01c14000 0x100>; - interrupts = <39>; - clocks = <&ahb_gates 1>; - phys = <&usbphy 1>; - phy-names = "usb"; - }; ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 4/6] ARM: dts: sunxi: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm/boot/dts/sun4i-a10.dtsi | 4 ---- arch/arm/boot/dts/sun5i.dtsi | 2 -- arch/arm/boot/dts/sun6i-a31.dtsi | 4 ---- arch/arm/boot/dts/sun7i-a20.dtsi | 4 ---- arch/arm/boot/dts/sun8i-a23-a33.dtsi | 2 -- arch/arm/boot/dts/sun8i-a83t.dtsi | 3 --- arch/arm/boot/dts/sun8i-r40.dtsi | 4 ---- arch/arm/boot/dts/sun9i-a80.dtsi | 5 ----- 8 files changed, 28 deletions(-) diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi index ef6ec526f394..e88daa4ef1af 100644 --- a/arch/arm/boot/dts/sun4i-a10.dtsi +++ b/arch/arm/boot/dts/sun4i-a10.dtsi @@ -520,7 +520,6 @@ interrupts = <39>; clocks = <&ccu CLK_AHB_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -530,7 +529,6 @@ interrupts = <64>; clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -610,7 +608,6 @@ interrupts = <40>; clocks = <&ccu CLK_AHB_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -620,7 +617,6 @@ interrupts = <65>; clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi index cb820bd7974c..0d71b01967a3 100644 --- a/arch/arm/boot/dts/sun5i.dtsi +++ b/arch/arm/boot/dts/sun5i.dtsi @@ -391,7 +391,6 @@ interrupts = <39>; clocks = <&ccu CLK_AHB_EHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -401,7 +400,6 @@ interrupts = <40>; clocks = <&ccu CLK_USB_OHCI>, <&ccu CLK_AHB_OHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun6i-a31.dtsi b/arch/arm/boot/dts/sun6i-a31.dtsi index fa983f9ff5f5..c04efad81bbc 100644 --- a/arch/arm/boot/dts/sun6i-a31.dtsi +++ b/arch/arm/boot/dts/sun6i-a31.dtsi @@ -543,7 +543,6 @@ clocks = <&ccu CLK_AHB1_EHCI0>; resets = <&ccu RST_AHB1_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -554,7 +553,6 @@ clocks = <&ccu CLK_AHB1_OHCI0>, <&ccu CLK_USB_OHCI0>; resets = <&ccu RST_AHB1_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -565,7 +563,6 @@ clocks = <&ccu CLK_AHB1_EHCI1>; resets = <&ccu RST_AHB1_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -576,7 +573,6 @@ clocks = <&ccu CLK_AHB1_OHCI1>, <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_AHB1_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi index 794c915f504b..9ad8e445b240 100644 --- a/arch/arm/boot/dts/sun7i-a20.dtsi +++ b/arch/arm/boot/dts/sun7i-a20.dtsi @@ -612,7 +612,6 @@ interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_AHB_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -622,7 +621,6 @@ interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -705,7 +703,6 @@ interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_AHB_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -715,7 +712,6 @@ interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-a23-a33.dtsi b/arch/arm/boot/dts/sun8i-a23-a33.dtsi index f76aad0c5d4d..af2fa694a467 100644 --- a/arch/arm/boot/dts/sun8i-a23-a33.dtsi +++ b/arch/arm/boot/dts/sun8i-a23-a33.dtsi @@ -307,7 +307,6 @@ clocks = <&ccu CLK_BUS_EHCI>; resets = <&ccu RST_BUS_EHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -318,7 +317,6 @@ clocks = <&ccu CLK_BUS_OHCI>, <&ccu CLK_USB_OHCI>; resets = <&ccu RST_BUS_OHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi index b105a85467b3..392b0cabbf0d 100644 --- a/arch/arm/boot/dts/sun8i-a83t.dtsi +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi @@ -632,7 +632,6 @@ clocks = <&ccu CLK_BUS_EHCI0>; resets = <&ccu RST_BUS_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -644,7 +643,6 @@ clocks = <&ccu CLK_BUS_OHCI0>, <&ccu CLK_USB_OHCI0>; resets = <&ccu RST_BUS_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -656,7 +654,6 @@ clocks = <&ccu CLK_BUS_EHCI1>; resets = <&ccu RST_BUS_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-r40.dtsi b/arch/arm/boot/dts/sun8i-r40.dtsi index 56c6885b02d1..bb856e53b806 100644 --- a/arch/arm/boot/dts/sun8i-r40.dtsi +++ b/arch/arm/boot/dts/sun8i-r40.dtsi @@ -273,7 +273,6 @@ clocks = <&ccu CLK_BUS_EHCI1>; resets = <&ccu RST_BUS_EHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -285,7 +284,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -296,7 +294,6 @@ clocks = <&ccu CLK_BUS_EHCI2>; resets = <&ccu RST_BUS_EHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -308,7 +305,6 @@ <&ccu CLK_USB_OHCI2>; resets = <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun9i-a80.dtsi b/arch/arm/boot/dts/sun9i-a80.dtsi index c1aa26db44ae..0c1eec9000e3 100644 --- a/arch/arm/boot/dts/sun9i-a80.dtsi +++ b/arch/arm/boot/dts/sun9i-a80.dtsi @@ -342,7 +342,6 @@ clocks = <&usb_clocks CLK_BUS_HCI0>; resets = <&usb_clocks RST_USB0_HCI>; phys = <&usbphy1>; - phy-names = "usb"; status = "disabled"; }; @@ -354,7 +353,6 @@ <&usb_clocks CLK_USB_OHCI0>; resets = <&usb_clocks RST_USB0_HCI>; phys = <&usbphy1>; - phy-names = "usb"; status = "disabled"; }; @@ -376,7 +374,6 @@ clocks = <&usb_clocks CLK_BUS_HCI1>; resets = <&usb_clocks RST_USB1_HCI>; phys = <&usbphy2>; - phy-names = "usb"; status = "disabled"; }; @@ -406,7 +403,6 @@ clocks = <&usb_clocks CLK_BUS_HCI2>; resets = <&usb_clocks RST_USB2_HCI>; phys = <&usbphy3>; - phy-names = "usb"; status = "disabled"; }; @@ -418,7 +414,6 @@ <&usb_clocks CLK_USB_OHCI2>; resets = <&usb_clocks RST_USB2_HCI>; phys = <&usbphy3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 4/6] ARM: dts: sunxi: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm/boot/dts/sun4i-a10.dtsi | 4 ---- arch/arm/boot/dts/sun5i.dtsi | 2 -- arch/arm/boot/dts/sun6i-a31.dtsi | 4 ---- arch/arm/boot/dts/sun7i-a20.dtsi | 4 ---- arch/arm/boot/dts/sun8i-a23-a33.dtsi | 2 -- arch/arm/boot/dts/sun8i-a83t.dtsi | 3 --- arch/arm/boot/dts/sun8i-r40.dtsi | 4 ---- arch/arm/boot/dts/sun9i-a80.dtsi | 5 ----- 8 files changed, 28 deletions(-) diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi index ef6ec526f394..e88daa4ef1af 100644 --- a/arch/arm/boot/dts/sun4i-a10.dtsi +++ b/arch/arm/boot/dts/sun4i-a10.dtsi @@ -520,7 +520,6 @@ interrupts = <39>; clocks = <&ccu CLK_AHB_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -530,7 +529,6 @@ interrupts = <64>; clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -610,7 +608,6 @@ interrupts = <40>; clocks = <&ccu CLK_AHB_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -620,7 +617,6 @@ interrupts = <65>; clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi index cb820bd7974c..0d71b01967a3 100644 --- a/arch/arm/boot/dts/sun5i.dtsi +++ b/arch/arm/boot/dts/sun5i.dtsi @@ -391,7 +391,6 @@ interrupts = <39>; clocks = <&ccu CLK_AHB_EHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -401,7 +400,6 @@ interrupts = <40>; clocks = <&ccu CLK_USB_OHCI>, <&ccu CLK_AHB_OHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun6i-a31.dtsi b/arch/arm/boot/dts/sun6i-a31.dtsi index fa983f9ff5f5..c04efad81bbc 100644 --- a/arch/arm/boot/dts/sun6i-a31.dtsi +++ b/arch/arm/boot/dts/sun6i-a31.dtsi @@ -543,7 +543,6 @@ clocks = <&ccu CLK_AHB1_EHCI0>; resets = <&ccu RST_AHB1_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -554,7 +553,6 @@ clocks = <&ccu CLK_AHB1_OHCI0>, <&ccu CLK_USB_OHCI0>; resets = <&ccu RST_AHB1_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -565,7 +563,6 @@ clocks = <&ccu CLK_AHB1_EHCI1>; resets = <&ccu RST_AHB1_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -576,7 +573,6 @@ clocks = <&ccu CLK_AHB1_OHCI1>, <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_AHB1_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi index 794c915f504b..9ad8e445b240 100644 --- a/arch/arm/boot/dts/sun7i-a20.dtsi +++ b/arch/arm/boot/dts/sun7i-a20.dtsi @@ -612,7 +612,6 @@ interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_AHB_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -622,7 +621,6 @@ interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -705,7 +703,6 @@ interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_AHB_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -715,7 +712,6 @@ interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-a23-a33.dtsi b/arch/arm/boot/dts/sun8i-a23-a33.dtsi index f76aad0c5d4d..af2fa694a467 100644 --- a/arch/arm/boot/dts/sun8i-a23-a33.dtsi +++ b/arch/arm/boot/dts/sun8i-a23-a33.dtsi @@ -307,7 +307,6 @@ clocks = <&ccu CLK_BUS_EHCI>; resets = <&ccu RST_BUS_EHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -318,7 +317,6 @@ clocks = <&ccu CLK_BUS_OHCI>, <&ccu CLK_USB_OHCI>; resets = <&ccu RST_BUS_OHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi index b105a85467b3..392b0cabbf0d 100644 --- a/arch/arm/boot/dts/sun8i-a83t.dtsi +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi @@ -632,7 +632,6 @@ clocks = <&ccu CLK_BUS_EHCI0>; resets = <&ccu RST_BUS_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -644,7 +643,6 @@ clocks = <&ccu CLK_BUS_OHCI0>, <&ccu CLK_USB_OHCI0>; resets = <&ccu RST_BUS_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -656,7 +654,6 @@ clocks = <&ccu CLK_BUS_EHCI1>; resets = <&ccu RST_BUS_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-r40.dtsi b/arch/arm/boot/dts/sun8i-r40.dtsi index 56c6885b02d1..bb856e53b806 100644 --- a/arch/arm/boot/dts/sun8i-r40.dtsi +++ b/arch/arm/boot/dts/sun8i-r40.dtsi @@ -273,7 +273,6 @@ clocks = <&ccu CLK_BUS_EHCI1>; resets = <&ccu RST_BUS_EHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -285,7 +284,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -296,7 +294,6 @@ clocks = <&ccu CLK_BUS_EHCI2>; resets = <&ccu RST_BUS_EHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -308,7 +305,6 @@ <&ccu CLK_USB_OHCI2>; resets = <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun9i-a80.dtsi b/arch/arm/boot/dts/sun9i-a80.dtsi index c1aa26db44ae..0c1eec9000e3 100644 --- a/arch/arm/boot/dts/sun9i-a80.dtsi +++ b/arch/arm/boot/dts/sun9i-a80.dtsi @@ -342,7 +342,6 @@ clocks = <&usb_clocks CLK_BUS_HCI0>; resets = <&usb_clocks RST_USB0_HCI>; phys = <&usbphy1>; - phy-names = "usb"; status = "disabled"; }; @@ -354,7 +353,6 @@ <&usb_clocks CLK_USB_OHCI0>; resets = <&usb_clocks RST_USB0_HCI>; phys = <&usbphy1>; - phy-names = "usb"; status = "disabled"; }; @@ -376,7 +374,6 @@ clocks = <&usb_clocks CLK_BUS_HCI1>; resets = <&usb_clocks RST_USB1_HCI>; phys = <&usbphy2>; - phy-names = "usb"; status = "disabled"; }; @@ -406,7 +403,6 @@ clocks = <&usb_clocks CLK_BUS_HCI2>; resets = <&usb_clocks RST_USB2_HCI>; phys = <&usbphy3>; - phy-names = "usb"; status = "disabled"; }; @@ -418,7 +414,6 @@ <&usb_clocks CLK_USB_OHCI2>; resets = <&usb_clocks RST_USB2_HCI>; phys = <&usbphy3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 4/6] ARM: dts: sunxi: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm/boot/dts/sun4i-a10.dtsi | 4 ---- arch/arm/boot/dts/sun5i.dtsi | 2 -- arch/arm/boot/dts/sun6i-a31.dtsi | 4 ---- arch/arm/boot/dts/sun7i-a20.dtsi | 4 ---- arch/arm/boot/dts/sun8i-a23-a33.dtsi | 2 -- arch/arm/boot/dts/sun8i-a83t.dtsi | 3 --- arch/arm/boot/dts/sun8i-r40.dtsi | 4 ---- arch/arm/boot/dts/sun9i-a80.dtsi | 5 ----- 8 files changed, 28 deletions(-) diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi index ef6ec526f394..e88daa4ef1af 100644 --- a/arch/arm/boot/dts/sun4i-a10.dtsi +++ b/arch/arm/boot/dts/sun4i-a10.dtsi @@ -520,7 +520,6 @@ interrupts = <39>; clocks = <&ccu CLK_AHB_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -530,7 +529,6 @@ interrupts = <64>; clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -610,7 +608,6 @@ interrupts = <40>; clocks = <&ccu CLK_AHB_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -620,7 +617,6 @@ interrupts = <65>; clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi index cb820bd7974c..0d71b01967a3 100644 --- a/arch/arm/boot/dts/sun5i.dtsi +++ b/arch/arm/boot/dts/sun5i.dtsi @@ -391,7 +391,6 @@ interrupts = <39>; clocks = <&ccu CLK_AHB_EHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -401,7 +400,6 @@ interrupts = <40>; clocks = <&ccu CLK_USB_OHCI>, <&ccu CLK_AHB_OHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun6i-a31.dtsi b/arch/arm/boot/dts/sun6i-a31.dtsi index fa983f9ff5f5..c04efad81bbc 100644 --- a/arch/arm/boot/dts/sun6i-a31.dtsi +++ b/arch/arm/boot/dts/sun6i-a31.dtsi @@ -543,7 +543,6 @@ clocks = <&ccu CLK_AHB1_EHCI0>; resets = <&ccu RST_AHB1_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -554,7 +553,6 @@ clocks = <&ccu CLK_AHB1_OHCI0>, <&ccu CLK_USB_OHCI0>; resets = <&ccu RST_AHB1_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -565,7 +563,6 @@ clocks = <&ccu CLK_AHB1_EHCI1>; resets = <&ccu RST_AHB1_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -576,7 +573,6 @@ clocks = <&ccu CLK_AHB1_OHCI1>, <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_AHB1_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi index 794c915f504b..9ad8e445b240 100644 --- a/arch/arm/boot/dts/sun7i-a20.dtsi +++ b/arch/arm/boot/dts/sun7i-a20.dtsi @@ -612,7 +612,6 @@ interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_AHB_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -622,7 +621,6 @@ interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -705,7 +703,6 @@ interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_AHB_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -715,7 +712,6 @@ interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-a23-a33.dtsi b/arch/arm/boot/dts/sun8i-a23-a33.dtsi index f76aad0c5d4d..af2fa694a467 100644 --- a/arch/arm/boot/dts/sun8i-a23-a33.dtsi +++ b/arch/arm/boot/dts/sun8i-a23-a33.dtsi @@ -307,7 +307,6 @@ clocks = <&ccu CLK_BUS_EHCI>; resets = <&ccu RST_BUS_EHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -318,7 +317,6 @@ clocks = <&ccu CLK_BUS_OHCI>, <&ccu CLK_USB_OHCI>; resets = <&ccu RST_BUS_OHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi index b105a85467b3..392b0cabbf0d 100644 --- a/arch/arm/boot/dts/sun8i-a83t.dtsi +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi @@ -632,7 +632,6 @@ clocks = <&ccu CLK_BUS_EHCI0>; resets = <&ccu RST_BUS_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -644,7 +643,6 @@ clocks = <&ccu CLK_BUS_OHCI0>, <&ccu CLK_USB_OHCI0>; resets = <&ccu RST_BUS_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -656,7 +654,6 @@ clocks = <&ccu CLK_BUS_EHCI1>; resets = <&ccu RST_BUS_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-r40.dtsi b/arch/arm/boot/dts/sun8i-r40.dtsi index 56c6885b02d1..bb856e53b806 100644 --- a/arch/arm/boot/dts/sun8i-r40.dtsi +++ b/arch/arm/boot/dts/sun8i-r40.dtsi @@ -273,7 +273,6 @@ clocks = <&ccu CLK_BUS_EHCI1>; resets = <&ccu RST_BUS_EHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -285,7 +284,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -296,7 +294,6 @@ clocks = <&ccu CLK_BUS_EHCI2>; resets = <&ccu RST_BUS_EHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -308,7 +305,6 @@ <&ccu CLK_USB_OHCI2>; resets = <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun9i-a80.dtsi b/arch/arm/boot/dts/sun9i-a80.dtsi index c1aa26db44ae..0c1eec9000e3 100644 --- a/arch/arm/boot/dts/sun9i-a80.dtsi +++ b/arch/arm/boot/dts/sun9i-a80.dtsi @@ -342,7 +342,6 @@ clocks = <&usb_clocks CLK_BUS_HCI0>; resets = <&usb_clocks RST_USB0_HCI>; phys = <&usbphy1>; - phy-names = "usb"; status = "disabled"; }; @@ -354,7 +353,6 @@ <&usb_clocks CLK_USB_OHCI0>; resets = <&usb_clocks RST_USB0_HCI>; phys = <&usbphy1>; - phy-names = "usb"; status = "disabled"; }; @@ -376,7 +374,6 @@ clocks = <&usb_clocks CLK_BUS_HCI1>; resets = <&usb_clocks RST_USB1_HCI>; phys = <&usbphy2>; - phy-names = "usb"; status = "disabled"; }; @@ -406,7 +403,6 @@ clocks = <&usb_clocks CLK_BUS_HCI2>; resets = <&usb_clocks RST_USB2_HCI>; phys = <&usbphy3>; - phy-names = "usb"; status = "disabled"; }; @@ -418,7 +414,6 @@ <&usb_clocks CLK_USB_OHCI2>; resets = <&usb_clocks RST_USB2_HCI>; phys = <&usbphy3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [4/6] ARM: dts: sunxi: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm/boot/dts/sun4i-a10.dtsi | 4 ---- arch/arm/boot/dts/sun5i.dtsi | 2 -- arch/arm/boot/dts/sun6i-a31.dtsi | 4 ---- arch/arm/boot/dts/sun7i-a20.dtsi | 4 ---- arch/arm/boot/dts/sun8i-a23-a33.dtsi | 2 -- arch/arm/boot/dts/sun8i-a83t.dtsi | 3 --- arch/arm/boot/dts/sun8i-r40.dtsi | 4 ---- arch/arm/boot/dts/sun9i-a80.dtsi | 5 ----- 8 files changed, 28 deletions(-) diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi index ef6ec526f394..e88daa4ef1af 100644 --- a/arch/arm/boot/dts/sun4i-a10.dtsi +++ b/arch/arm/boot/dts/sun4i-a10.dtsi @@ -520,7 +520,6 @@ interrupts = <39>; clocks = <&ccu CLK_AHB_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -530,7 +529,6 @@ interrupts = <64>; clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -610,7 +608,6 @@ interrupts = <40>; clocks = <&ccu CLK_AHB_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -620,7 +617,6 @@ interrupts = <65>; clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi index cb820bd7974c..0d71b01967a3 100644 --- a/arch/arm/boot/dts/sun5i.dtsi +++ b/arch/arm/boot/dts/sun5i.dtsi @@ -391,7 +391,6 @@ interrupts = <39>; clocks = <&ccu CLK_AHB_EHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -401,7 +400,6 @@ interrupts = <40>; clocks = <&ccu CLK_USB_OHCI>, <&ccu CLK_AHB_OHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun6i-a31.dtsi b/arch/arm/boot/dts/sun6i-a31.dtsi index fa983f9ff5f5..c04efad81bbc 100644 --- a/arch/arm/boot/dts/sun6i-a31.dtsi +++ b/arch/arm/boot/dts/sun6i-a31.dtsi @@ -543,7 +543,6 @@ clocks = <&ccu CLK_AHB1_EHCI0>; resets = <&ccu RST_AHB1_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -554,7 +553,6 @@ clocks = <&ccu CLK_AHB1_OHCI0>, <&ccu CLK_USB_OHCI0>; resets = <&ccu RST_AHB1_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -565,7 +563,6 @@ clocks = <&ccu CLK_AHB1_EHCI1>; resets = <&ccu RST_AHB1_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -576,7 +573,6 @@ clocks = <&ccu CLK_AHB1_OHCI1>, <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_AHB1_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi index 794c915f504b..9ad8e445b240 100644 --- a/arch/arm/boot/dts/sun7i-a20.dtsi +++ b/arch/arm/boot/dts/sun7i-a20.dtsi @@ -612,7 +612,6 @@ interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_AHB_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -622,7 +621,6 @@ interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -705,7 +703,6 @@ interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_AHB_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -715,7 +712,6 @@ interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>; clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-a23-a33.dtsi b/arch/arm/boot/dts/sun8i-a23-a33.dtsi index f76aad0c5d4d..af2fa694a467 100644 --- a/arch/arm/boot/dts/sun8i-a23-a33.dtsi +++ b/arch/arm/boot/dts/sun8i-a23-a33.dtsi @@ -307,7 +307,6 @@ clocks = <&ccu CLK_BUS_EHCI>; resets = <&ccu RST_BUS_EHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -318,7 +317,6 @@ clocks = <&ccu CLK_BUS_OHCI>, <&ccu CLK_USB_OHCI>; resets = <&ccu RST_BUS_OHCI>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi index b105a85467b3..392b0cabbf0d 100644 --- a/arch/arm/boot/dts/sun8i-a83t.dtsi +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi @@ -632,7 +632,6 @@ clocks = <&ccu CLK_BUS_EHCI0>; resets = <&ccu RST_BUS_EHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -644,7 +643,6 @@ clocks = <&ccu CLK_BUS_OHCI0>, <&ccu CLK_USB_OHCI0>; resets = <&ccu RST_BUS_OHCI0>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -656,7 +654,6 @@ clocks = <&ccu CLK_BUS_EHCI1>; resets = <&ccu RST_BUS_EHCI1>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun8i-r40.dtsi b/arch/arm/boot/dts/sun8i-r40.dtsi index 56c6885b02d1..bb856e53b806 100644 --- a/arch/arm/boot/dts/sun8i-r40.dtsi +++ b/arch/arm/boot/dts/sun8i-r40.dtsi @@ -273,7 +273,6 @@ clocks = <&ccu CLK_BUS_EHCI1>; resets = <&ccu RST_BUS_EHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -285,7 +284,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -296,7 +294,6 @@ clocks = <&ccu CLK_BUS_EHCI2>; resets = <&ccu RST_BUS_EHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -308,7 +305,6 @@ <&ccu CLK_USB_OHCI2>; resets = <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm/boot/dts/sun9i-a80.dtsi b/arch/arm/boot/dts/sun9i-a80.dtsi index c1aa26db44ae..0c1eec9000e3 100644 --- a/arch/arm/boot/dts/sun9i-a80.dtsi +++ b/arch/arm/boot/dts/sun9i-a80.dtsi @@ -342,7 +342,6 @@ clocks = <&usb_clocks CLK_BUS_HCI0>; resets = <&usb_clocks RST_USB0_HCI>; phys = <&usbphy1>; - phy-names = "usb"; status = "disabled"; }; @@ -354,7 +353,6 @@ <&usb_clocks CLK_USB_OHCI0>; resets = <&usb_clocks RST_USB0_HCI>; phys = <&usbphy1>; - phy-names = "usb"; status = "disabled"; }; @@ -376,7 +374,6 @@ clocks = <&usb_clocks CLK_BUS_HCI1>; resets = <&usb_clocks RST_USB1_HCI>; phys = <&usbphy2>; - phy-names = "usb"; status = "disabled"; }; @@ -406,7 +403,6 @@ clocks = <&usb_clocks CLK_BUS_HCI2>; resets = <&usb_clocks RST_USB2_HCI>; phys = <&usbphy3>; - phy-names = "usb"; status = "disabled"; }; @@ -418,7 +414,6 @@ <&usb_clocks CLK_USB_OHCI2>; resets = <&usb_clocks RST_USB2_HCI>; phys = <&usbphy3>; - phy-names = "usb"; status = "disabled"; }; ^ permalink raw reply related [flat|nested] 34+ messages in thread
* Re: [PATCH 4/6] ARM: dts: sunxi: Remove useless phy-names from EHCI and OHCI 2019-04-16 8:28 ` [4/6] " Maxime Ripard (?) @ 2019-10-01 20:42 ` Emmanuel Vadot -1 siblings, 0 replies; 34+ messages in thread From: Emmanuel Vadot @ 2019-10-01 20:42 UTC (permalink / raw) To: Maxime Ripard Cc: Mark Rutland, devicetree, Greg Kroah-Hartman, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Hi Maxime, On Tue, 16 Apr 2019 10:28:00 +0200 Maxime Ripard <maxime.ripard@bootlin.com> wrote: > Neither the OHCI or EHCI bindings are using the phy-names property, so we > can just drop it. > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > --- > arch/arm/boot/dts/sun4i-a10.dtsi | 4 ---- > arch/arm/boot/dts/sun5i.dtsi | 2 -- > arch/arm/boot/dts/sun6i-a31.dtsi | 4 ---- > arch/arm/boot/dts/sun7i-a20.dtsi | 4 ---- > arch/arm/boot/dts/sun8i-a23-a33.dtsi | 2 -- > arch/arm/boot/dts/sun8i-a83t.dtsi | 3 --- > arch/arm/boot/dts/sun8i-r40.dtsi | 4 ---- > arch/arm/boot/dts/sun9i-a80.dtsi | 5 ----- > 8 files changed, 28 deletions(-) > > diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi > index ef6ec526f394..e88daa4ef1af 100644 > --- a/arch/arm/boot/dts/sun4i-a10.dtsi > +++ b/arch/arm/boot/dts/sun4i-a10.dtsi > @@ -520,7 +520,6 @@ > interrupts = <39>; > clocks = <&ccu CLK_AHB_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -530,7 +529,6 @@ > interrupts = <64>; > clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -610,7 +608,6 @@ > interrupts = <40>; > clocks = <&ccu CLK_AHB_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -620,7 +617,6 @@ > interrupts = <65>; > clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi > index cb820bd7974c..0d71b01967a3 100644 > --- a/arch/arm/boot/dts/sun5i.dtsi > +++ b/arch/arm/boot/dts/sun5i.dtsi > @@ -391,7 +391,6 @@ > interrupts = <39>; > clocks = <&ccu CLK_AHB_EHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -401,7 +400,6 @@ > interrupts = <40>; > clocks = <&ccu CLK_USB_OHCI>, <&ccu CLK_AHB_OHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun6i-a31.dtsi b/arch/arm/boot/dts/sun6i-a31.dtsi > index fa983f9ff5f5..c04efad81bbc 100644 > --- a/arch/arm/boot/dts/sun6i-a31.dtsi > +++ b/arch/arm/boot/dts/sun6i-a31.dtsi > @@ -543,7 +543,6 @@ > clocks = <&ccu CLK_AHB1_EHCI0>; > resets = <&ccu RST_AHB1_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -554,7 +553,6 @@ > clocks = <&ccu CLK_AHB1_OHCI0>, <&ccu CLK_USB_OHCI0>; > resets = <&ccu RST_AHB1_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -565,7 +563,6 @@ > clocks = <&ccu CLK_AHB1_EHCI1>; > resets = <&ccu RST_AHB1_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -576,7 +573,6 @@ > clocks = <&ccu CLK_AHB1_OHCI1>, <&ccu CLK_USB_OHCI1>; > resets = <&ccu RST_AHB1_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi > index 794c915f504b..9ad8e445b240 100644 > --- a/arch/arm/boot/dts/sun7i-a20.dtsi > +++ b/arch/arm/boot/dts/sun7i-a20.dtsi > @@ -612,7 +612,6 @@ > interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_AHB_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -622,7 +621,6 @@ > interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -705,7 +703,6 @@ > interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_AHB_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -715,7 +712,6 @@ > interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-a23-a33.dtsi b/arch/arm/boot/dts/sun8i-a23-a33.dtsi > index f76aad0c5d4d..af2fa694a467 100644 > --- a/arch/arm/boot/dts/sun8i-a23-a33.dtsi > +++ b/arch/arm/boot/dts/sun8i-a23-a33.dtsi > @@ -307,7 +307,6 @@ > clocks = <&ccu CLK_BUS_EHCI>; > resets = <&ccu RST_BUS_EHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -318,7 +317,6 @@ > clocks = <&ccu CLK_BUS_OHCI>, <&ccu CLK_USB_OHCI>; > resets = <&ccu RST_BUS_OHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi > index b105a85467b3..392b0cabbf0d 100644 > --- a/arch/arm/boot/dts/sun8i-a83t.dtsi > +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi > @@ -632,7 +632,6 @@ > clocks = <&ccu CLK_BUS_EHCI0>; > resets = <&ccu RST_BUS_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -644,7 +643,6 @@ > clocks = <&ccu CLK_BUS_OHCI0>, <&ccu CLK_USB_OHCI0>; > resets = <&ccu RST_BUS_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -656,7 +654,6 @@ > clocks = <&ccu CLK_BUS_EHCI1>; > resets = <&ccu RST_BUS_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-r40.dtsi b/arch/arm/boot/dts/sun8i-r40.dtsi > index 56c6885b02d1..bb856e53b806 100644 > --- a/arch/arm/boot/dts/sun8i-r40.dtsi > +++ b/arch/arm/boot/dts/sun8i-r40.dtsi > @@ -273,7 +273,6 @@ > clocks = <&ccu CLK_BUS_EHCI1>; > resets = <&ccu RST_BUS_EHCI1>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -285,7 +284,6 @@ > <&ccu CLK_USB_OHCI1>; > resets = <&ccu RST_BUS_OHCI1>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -296,7 +294,6 @@ > clocks = <&ccu CLK_BUS_EHCI2>; > resets = <&ccu RST_BUS_EHCI2>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -308,7 +305,6 @@ > <&ccu CLK_USB_OHCI2>; > resets = <&ccu RST_BUS_OHCI2>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun9i-a80.dtsi b/arch/arm/boot/dts/sun9i-a80.dtsi > index c1aa26db44ae..0c1eec9000e3 100644 > --- a/arch/arm/boot/dts/sun9i-a80.dtsi > +++ b/arch/arm/boot/dts/sun9i-a80.dtsi > @@ -342,7 +342,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI0>; > resets = <&usb_clocks RST_USB0_HCI>; > phys = <&usbphy1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -354,7 +353,6 @@ > <&usb_clocks CLK_USB_OHCI0>; > resets = <&usb_clocks RST_USB0_HCI>; > phys = <&usbphy1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -376,7 +374,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI1>; > resets = <&usb_clocks RST_USB1_HCI>; > phys = <&usbphy2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -406,7 +403,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI2>; > resets = <&usb_clocks RST_USB2_HCI>; > phys = <&usbphy3>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -418,7 +414,6 @@ > <&usb_clocks CLK_USB_OHCI2>; > resets = <&usb_clocks RST_USB2_HCI>; > phys = <&usbphy3>; > - phy-names = "usb"; > status = "disabled"; > }; > > -- > git-series 0.9.1 Sorry for the late reply on this mail but it seems that it was only applied for 5.2 and I've just imported those DTS for FreeBSD recently. So, this is not correct, the phy-bindings.txt documents says that phy-names is a required property for any phy user node. I agree that the generic-{o,e}hci doesn't say what name the phy should be represented but almost all DTS seems to agree on the "usb" name (at least when they have only one phy). All the BSDs requires (or used to since this change) the phy-names to be present since it's a required property. I've badly patched FreeBSD since I though this was an Allwinner thing before someone directed me to the phy-bindings.txt document. Please revert this patch or send a patch to update the documentation. Thank you. -- Emmanuel Vadot <manu@bidouilliste.com> <manu@freebsd.org> ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH 4/6] ARM: dts: sunxi: Remove useless phy-names from EHCI and OHCI @ 2019-10-01 20:42 ` Emmanuel Vadot 0 siblings, 0 replies; 34+ messages in thread From: Emmanuel Vadot @ 2019-10-01 20:42 UTC (permalink / raw) To: Maxime Ripard Cc: Mark Rutland, devicetree, Greg Kroah-Hartman, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Hi Maxime, On Tue, 16 Apr 2019 10:28:00 +0200 Maxime Ripard <maxime.ripard@bootlin.com> wrote: > Neither the OHCI or EHCI bindings are using the phy-names property, so we > can just drop it. > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > --- > arch/arm/boot/dts/sun4i-a10.dtsi | 4 ---- > arch/arm/boot/dts/sun5i.dtsi | 2 -- > arch/arm/boot/dts/sun6i-a31.dtsi | 4 ---- > arch/arm/boot/dts/sun7i-a20.dtsi | 4 ---- > arch/arm/boot/dts/sun8i-a23-a33.dtsi | 2 -- > arch/arm/boot/dts/sun8i-a83t.dtsi | 3 --- > arch/arm/boot/dts/sun8i-r40.dtsi | 4 ---- > arch/arm/boot/dts/sun9i-a80.dtsi | 5 ----- > 8 files changed, 28 deletions(-) > > diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi > index ef6ec526f394..e88daa4ef1af 100644 > --- a/arch/arm/boot/dts/sun4i-a10.dtsi > +++ b/arch/arm/boot/dts/sun4i-a10.dtsi > @@ -520,7 +520,6 @@ > interrupts = <39>; > clocks = <&ccu CLK_AHB_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -530,7 +529,6 @@ > interrupts = <64>; > clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -610,7 +608,6 @@ > interrupts = <40>; > clocks = <&ccu CLK_AHB_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -620,7 +617,6 @@ > interrupts = <65>; > clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi > index cb820bd7974c..0d71b01967a3 100644 > --- a/arch/arm/boot/dts/sun5i.dtsi > +++ b/arch/arm/boot/dts/sun5i.dtsi > @@ -391,7 +391,6 @@ > interrupts = <39>; > clocks = <&ccu CLK_AHB_EHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -401,7 +400,6 @@ > interrupts = <40>; > clocks = <&ccu CLK_USB_OHCI>, <&ccu CLK_AHB_OHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun6i-a31.dtsi b/arch/arm/boot/dts/sun6i-a31.dtsi > index fa983f9ff5f5..c04efad81bbc 100644 > --- a/arch/arm/boot/dts/sun6i-a31.dtsi > +++ b/arch/arm/boot/dts/sun6i-a31.dtsi > @@ -543,7 +543,6 @@ > clocks = <&ccu CLK_AHB1_EHCI0>; > resets = <&ccu RST_AHB1_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -554,7 +553,6 @@ > clocks = <&ccu CLK_AHB1_OHCI0>, <&ccu CLK_USB_OHCI0>; > resets = <&ccu RST_AHB1_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -565,7 +563,6 @@ > clocks = <&ccu CLK_AHB1_EHCI1>; > resets = <&ccu RST_AHB1_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -576,7 +573,6 @@ > clocks = <&ccu CLK_AHB1_OHCI1>, <&ccu CLK_USB_OHCI1>; > resets = <&ccu RST_AHB1_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi > index 794c915f504b..9ad8e445b240 100644 > --- a/arch/arm/boot/dts/sun7i-a20.dtsi > +++ b/arch/arm/boot/dts/sun7i-a20.dtsi > @@ -612,7 +612,6 @@ > interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_AHB_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -622,7 +621,6 @@ > interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -705,7 +703,6 @@ > interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_AHB_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -715,7 +712,6 @@ > interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-a23-a33.dtsi b/arch/arm/boot/dts/sun8i-a23-a33.dtsi > index f76aad0c5d4d..af2fa694a467 100644 > --- a/arch/arm/boot/dts/sun8i-a23-a33.dtsi > +++ b/arch/arm/boot/dts/sun8i-a23-a33.dtsi > @@ -307,7 +307,6 @@ > clocks = <&ccu CLK_BUS_EHCI>; > resets = <&ccu RST_BUS_EHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -318,7 +317,6 @@ > clocks = <&ccu CLK_BUS_OHCI>, <&ccu CLK_USB_OHCI>; > resets = <&ccu RST_BUS_OHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi > index b105a85467b3..392b0cabbf0d 100644 > --- a/arch/arm/boot/dts/sun8i-a83t.dtsi > +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi > @@ -632,7 +632,6 @@ > clocks = <&ccu CLK_BUS_EHCI0>; > resets = <&ccu RST_BUS_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -644,7 +643,6 @@ > clocks = <&ccu CLK_BUS_OHCI0>, <&ccu CLK_USB_OHCI0>; > resets = <&ccu RST_BUS_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -656,7 +654,6 @@ > clocks = <&ccu CLK_BUS_EHCI1>; > resets = <&ccu RST_BUS_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-r40.dtsi b/arch/arm/boot/dts/sun8i-r40.dtsi > index 56c6885b02d1..bb856e53b806 100644 > --- a/arch/arm/boot/dts/sun8i-r40.dtsi > +++ b/arch/arm/boot/dts/sun8i-r40.dtsi > @@ -273,7 +273,6 @@ > clocks = <&ccu CLK_BUS_EHCI1>; > resets = <&ccu RST_BUS_EHCI1>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -285,7 +284,6 @@ > <&ccu CLK_USB_OHCI1>; > resets = <&ccu RST_BUS_OHCI1>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -296,7 +294,6 @@ > clocks = <&ccu CLK_BUS_EHCI2>; > resets = <&ccu RST_BUS_EHCI2>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -308,7 +305,6 @@ > <&ccu CLK_USB_OHCI2>; > resets = <&ccu RST_BUS_OHCI2>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun9i-a80.dtsi b/arch/arm/boot/dts/sun9i-a80.dtsi > index c1aa26db44ae..0c1eec9000e3 100644 > --- a/arch/arm/boot/dts/sun9i-a80.dtsi > +++ b/arch/arm/boot/dts/sun9i-a80.dtsi > @@ -342,7 +342,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI0>; > resets = <&usb_clocks RST_USB0_HCI>; > phys = <&usbphy1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -354,7 +353,6 @@ > <&usb_clocks CLK_USB_OHCI0>; > resets = <&usb_clocks RST_USB0_HCI>; > phys = <&usbphy1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -376,7 +374,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI1>; > resets = <&usb_clocks RST_USB1_HCI>; > phys = <&usbphy2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -406,7 +403,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI2>; > resets = <&usb_clocks RST_USB2_HCI>; > phys = <&usbphy3>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -418,7 +414,6 @@ > <&usb_clocks CLK_USB_OHCI2>; > resets = <&usb_clocks RST_USB2_HCI>; > phys = <&usbphy3>; > - phy-names = "usb"; > status = "disabled"; > }; > > -- > git-series 0.9.1 Sorry for the late reply on this mail but it seems that it was only applied for 5.2 and I've just imported those DTS for FreeBSD recently. So, this is not correct, the phy-bindings.txt documents says that phy-names is a required property for any phy user node. I agree that the generic-{o,e}hci doesn't say what name the phy should be represented but almost all DTS seems to agree on the "usb" name (at least when they have only one phy). All the BSDs requires (or used to since this change) the phy-names to be present since it's a required property. I've badly patched FreeBSD since I though this was an Allwinner thing before someone directed me to the phy-bindings.txt document. Please revert this patch or send a patch to update the documentation. Thank you. -- Emmanuel Vadot <manu@bidouilliste.com> <manu@freebsd.org> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH 4/6] ARM: dts: sunxi: Remove useless phy-names from EHCI and OHCI @ 2019-10-01 20:42 ` Emmanuel Vadot 0 siblings, 0 replies; 34+ messages in thread From: Emmanuel Vadot @ 2019-10-01 20:42 UTC (permalink / raw) To: Maxime Ripard Cc: Greg Kroah-Hartman, Mark Rutland, devicetree, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Hi Maxime, On Tue, 16 Apr 2019 10:28:00 +0200 Maxime Ripard <maxime.ripard@bootlin.com> wrote: > Neither the OHCI or EHCI bindings are using the phy-names property, so we > can just drop it. > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > --- > arch/arm/boot/dts/sun4i-a10.dtsi | 4 ---- > arch/arm/boot/dts/sun5i.dtsi | 2 -- > arch/arm/boot/dts/sun6i-a31.dtsi | 4 ---- > arch/arm/boot/dts/sun7i-a20.dtsi | 4 ---- > arch/arm/boot/dts/sun8i-a23-a33.dtsi | 2 -- > arch/arm/boot/dts/sun8i-a83t.dtsi | 3 --- > arch/arm/boot/dts/sun8i-r40.dtsi | 4 ---- > arch/arm/boot/dts/sun9i-a80.dtsi | 5 ----- > 8 files changed, 28 deletions(-) > > diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi > index ef6ec526f394..e88daa4ef1af 100644 > --- a/arch/arm/boot/dts/sun4i-a10.dtsi > +++ b/arch/arm/boot/dts/sun4i-a10.dtsi > @@ -520,7 +520,6 @@ > interrupts = <39>; > clocks = <&ccu CLK_AHB_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -530,7 +529,6 @@ > interrupts = <64>; > clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -610,7 +608,6 @@ > interrupts = <40>; > clocks = <&ccu CLK_AHB_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -620,7 +617,6 @@ > interrupts = <65>; > clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun5i.dtsi b/arch/arm/boot/dts/sun5i.dtsi > index cb820bd7974c..0d71b01967a3 100644 > --- a/arch/arm/boot/dts/sun5i.dtsi > +++ b/arch/arm/boot/dts/sun5i.dtsi > @@ -391,7 +391,6 @@ > interrupts = <39>; > clocks = <&ccu CLK_AHB_EHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -401,7 +400,6 @@ > interrupts = <40>; > clocks = <&ccu CLK_USB_OHCI>, <&ccu CLK_AHB_OHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun6i-a31.dtsi b/arch/arm/boot/dts/sun6i-a31.dtsi > index fa983f9ff5f5..c04efad81bbc 100644 > --- a/arch/arm/boot/dts/sun6i-a31.dtsi > +++ b/arch/arm/boot/dts/sun6i-a31.dtsi > @@ -543,7 +543,6 @@ > clocks = <&ccu CLK_AHB1_EHCI0>; > resets = <&ccu RST_AHB1_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -554,7 +553,6 @@ > clocks = <&ccu CLK_AHB1_OHCI0>, <&ccu CLK_USB_OHCI0>; > resets = <&ccu RST_AHB1_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -565,7 +563,6 @@ > clocks = <&ccu CLK_AHB1_EHCI1>; > resets = <&ccu RST_AHB1_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -576,7 +573,6 @@ > clocks = <&ccu CLK_AHB1_OHCI1>, <&ccu CLK_USB_OHCI1>; > resets = <&ccu RST_AHB1_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun7i-a20.dtsi b/arch/arm/boot/dts/sun7i-a20.dtsi > index 794c915f504b..9ad8e445b240 100644 > --- a/arch/arm/boot/dts/sun7i-a20.dtsi > +++ b/arch/arm/boot/dts/sun7i-a20.dtsi > @@ -612,7 +612,6 @@ > interrupts = <GIC_SPI 39 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_AHB_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -622,7 +621,6 @@ > interrupts = <GIC_SPI 64 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_USB_OHCI0>, <&ccu CLK_AHB_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -705,7 +703,6 @@ > interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_AHB_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -715,7 +712,6 @@ > interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>; > clocks = <&ccu CLK_USB_OHCI1>, <&ccu CLK_AHB_OHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-a23-a33.dtsi b/arch/arm/boot/dts/sun8i-a23-a33.dtsi > index f76aad0c5d4d..af2fa694a467 100644 > --- a/arch/arm/boot/dts/sun8i-a23-a33.dtsi > +++ b/arch/arm/boot/dts/sun8i-a23-a33.dtsi > @@ -307,7 +307,6 @@ > clocks = <&ccu CLK_BUS_EHCI>; > resets = <&ccu RST_BUS_EHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -318,7 +317,6 @@ > clocks = <&ccu CLK_BUS_OHCI>, <&ccu CLK_USB_OHCI>; > resets = <&ccu RST_BUS_OHCI>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi > index b105a85467b3..392b0cabbf0d 100644 > --- a/arch/arm/boot/dts/sun8i-a83t.dtsi > +++ b/arch/arm/boot/dts/sun8i-a83t.dtsi > @@ -632,7 +632,6 @@ > clocks = <&ccu CLK_BUS_EHCI0>; > resets = <&ccu RST_BUS_EHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -644,7 +643,6 @@ > clocks = <&ccu CLK_BUS_OHCI0>, <&ccu CLK_USB_OHCI0>; > resets = <&ccu RST_BUS_OHCI0>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -656,7 +654,6 @@ > clocks = <&ccu CLK_BUS_EHCI1>; > resets = <&ccu RST_BUS_EHCI1>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun8i-r40.dtsi b/arch/arm/boot/dts/sun8i-r40.dtsi > index 56c6885b02d1..bb856e53b806 100644 > --- a/arch/arm/boot/dts/sun8i-r40.dtsi > +++ b/arch/arm/boot/dts/sun8i-r40.dtsi > @@ -273,7 +273,6 @@ > clocks = <&ccu CLK_BUS_EHCI1>; > resets = <&ccu RST_BUS_EHCI1>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -285,7 +284,6 @@ > <&ccu CLK_USB_OHCI1>; > resets = <&ccu RST_BUS_OHCI1>; > phys = <&usbphy 1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -296,7 +294,6 @@ > clocks = <&ccu CLK_BUS_EHCI2>; > resets = <&ccu RST_BUS_EHCI2>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -308,7 +305,6 @@ > <&ccu CLK_USB_OHCI2>; > resets = <&ccu RST_BUS_OHCI2>; > phys = <&usbphy 2>; > - phy-names = "usb"; > status = "disabled"; > }; > > diff --git a/arch/arm/boot/dts/sun9i-a80.dtsi b/arch/arm/boot/dts/sun9i-a80.dtsi > index c1aa26db44ae..0c1eec9000e3 100644 > --- a/arch/arm/boot/dts/sun9i-a80.dtsi > +++ b/arch/arm/boot/dts/sun9i-a80.dtsi > @@ -342,7 +342,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI0>; > resets = <&usb_clocks RST_USB0_HCI>; > phys = <&usbphy1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -354,7 +353,6 @@ > <&usb_clocks CLK_USB_OHCI0>; > resets = <&usb_clocks RST_USB0_HCI>; > phys = <&usbphy1>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -376,7 +374,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI1>; > resets = <&usb_clocks RST_USB1_HCI>; > phys = <&usbphy2>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -406,7 +403,6 @@ > clocks = <&usb_clocks CLK_BUS_HCI2>; > resets = <&usb_clocks RST_USB2_HCI>; > phys = <&usbphy3>; > - phy-names = "usb"; > status = "disabled"; > }; > > @@ -418,7 +414,6 @@ > <&usb_clocks CLK_USB_OHCI2>; > resets = <&usb_clocks RST_USB2_HCI>; > phys = <&usbphy3>; > - phy-names = "usb"; > status = "disabled"; > }; > > -- > git-series 0.9.1 Sorry for the late reply on this mail but it seems that it was only applied for 5.2 and I've just imported those DTS for FreeBSD recently. So, this is not correct, the phy-bindings.txt documents says that phy-names is a required property for any phy user node. I agree that the generic-{o,e}hci doesn't say what name the phy should be represented but almost all DTS seems to agree on the "usb" name (at least when they have only one phy). All the BSDs requires (or used to since this change) the phy-names to be present since it's a required property. I've badly patched FreeBSD since I though this was an Allwinner thing before someone directed me to the phy-bindings.txt document. Please revert this patch or send a patch to update the documentation. Thank you. -- Emmanuel Vadot <manu@bidouilliste.com> <manu@freebsd.org> ^ permalink raw reply [flat|nested] 34+ messages in thread
* [PATCH 5/6] ARM: dts: sunxi: h3/h5: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm/boot/dts/sunxi-h3-h5.dtsi | 6 ------ 1 file changed, 6 deletions(-) diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 3b18fd71efc1..84977d4eb97a 100644 --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi @@ -304,7 +304,6 @@ clocks = <&ccu CLK_BUS_EHCI1>, <&ccu CLK_BUS_OHCI1>; resets = <&ccu RST_BUS_EHCI1>, <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -316,7 +315,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_EHCI1>, <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -327,7 +325,6 @@ clocks = <&ccu CLK_BUS_EHCI2>, <&ccu CLK_BUS_OHCI2>; resets = <&ccu RST_BUS_EHCI2>, <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -339,7 +336,6 @@ <&ccu CLK_USB_OHCI2>; resets = <&ccu RST_BUS_EHCI2>, <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -350,7 +346,6 @@ clocks = <&ccu CLK_BUS_EHCI3>, <&ccu CLK_BUS_OHCI3>; resets = <&ccu RST_BUS_EHCI3>, <&ccu RST_BUS_OHCI3>; phys = <&usbphy 3>; - phy-names = "usb"; status = "disabled"; }; @@ -362,7 +357,6 @@ <&ccu CLK_USB_OHCI3>; resets = <&ccu RST_BUS_EHCI3>, <&ccu RST_BUS_OHCI3>; phys = <&usbphy 3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 5/6] ARM: dts: sunxi: h3/h5: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm/boot/dts/sunxi-h3-h5.dtsi | 6 ------ 1 file changed, 6 deletions(-) diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 3b18fd71efc1..84977d4eb97a 100644 --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi @@ -304,7 +304,6 @@ clocks = <&ccu CLK_BUS_EHCI1>, <&ccu CLK_BUS_OHCI1>; resets = <&ccu RST_BUS_EHCI1>, <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -316,7 +315,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_EHCI1>, <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -327,7 +325,6 @@ clocks = <&ccu CLK_BUS_EHCI2>, <&ccu CLK_BUS_OHCI2>; resets = <&ccu RST_BUS_EHCI2>, <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -339,7 +336,6 @@ <&ccu CLK_USB_OHCI2>; resets = <&ccu RST_BUS_EHCI2>, <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -350,7 +346,6 @@ clocks = <&ccu CLK_BUS_EHCI3>, <&ccu CLK_BUS_OHCI3>; resets = <&ccu RST_BUS_EHCI3>, <&ccu RST_BUS_OHCI3>; phys = <&usbphy 3>; - phy-names = "usb"; status = "disabled"; }; @@ -362,7 +357,6 @@ <&ccu CLK_USB_OHCI3>; resets = <&ccu RST_BUS_EHCI3>, <&ccu RST_BUS_OHCI3>; phys = <&usbphy 3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 5/6] ARM: dts: sunxi: h3/h5: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm/boot/dts/sunxi-h3-h5.dtsi | 6 ------ 1 file changed, 6 deletions(-) diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 3b18fd71efc1..84977d4eb97a 100644 --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi @@ -304,7 +304,6 @@ clocks = <&ccu CLK_BUS_EHCI1>, <&ccu CLK_BUS_OHCI1>; resets = <&ccu RST_BUS_EHCI1>, <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -316,7 +315,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_EHCI1>, <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -327,7 +325,6 @@ clocks = <&ccu CLK_BUS_EHCI2>, <&ccu CLK_BUS_OHCI2>; resets = <&ccu RST_BUS_EHCI2>, <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -339,7 +336,6 @@ <&ccu CLK_USB_OHCI2>; resets = <&ccu RST_BUS_EHCI2>, <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -350,7 +346,6 @@ clocks = <&ccu CLK_BUS_EHCI3>, <&ccu CLK_BUS_OHCI3>; resets = <&ccu RST_BUS_EHCI3>, <&ccu RST_BUS_OHCI3>; phys = <&usbphy 3>; - phy-names = "usb"; status = "disabled"; }; @@ -362,7 +357,6 @@ <&ccu CLK_USB_OHCI3>; resets = <&ccu RST_BUS_EHCI3>, <&ccu RST_BUS_OHCI3>; phys = <&usbphy 3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [5/6] ARM: dts: sunxi: h3/h5: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm/boot/dts/sunxi-h3-h5.dtsi | 6 ------ 1 file changed, 6 deletions(-) diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 3b18fd71efc1..84977d4eb97a 100644 --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi @@ -304,7 +304,6 @@ clocks = <&ccu CLK_BUS_EHCI1>, <&ccu CLK_BUS_OHCI1>; resets = <&ccu RST_BUS_EHCI1>, <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -316,7 +315,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_EHCI1>, <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -327,7 +325,6 @@ clocks = <&ccu CLK_BUS_EHCI2>, <&ccu CLK_BUS_OHCI2>; resets = <&ccu RST_BUS_EHCI2>, <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -339,7 +336,6 @@ <&ccu CLK_USB_OHCI2>; resets = <&ccu RST_BUS_EHCI2>, <&ccu RST_BUS_OHCI2>; phys = <&usbphy 2>; - phy-names = "usb"; status = "disabled"; }; @@ -350,7 +346,6 @@ clocks = <&ccu CLK_BUS_EHCI3>, <&ccu CLK_BUS_OHCI3>; resets = <&ccu RST_BUS_EHCI3>, <&ccu RST_BUS_OHCI3>; phys = <&usbphy 3>; - phy-names = "usb"; status = "disabled"; }; @@ -362,7 +357,6 @@ <&ccu CLK_USB_OHCI3>; resets = <&ccu RST_BUS_EHCI3>, <&ccu RST_BUS_OHCI3>; phys = <&usbphy 3>; - phy-names = "usb"; status = "disabled"; }; ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts | 2 -- arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 2 -- arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 2 -- 3 files changed, 6 deletions(-) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts index 78c82a665c84..2b6345db7dc0 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts @@ -104,7 +104,6 @@ &ehci0 { phys = <&usbphy 0>; - phy-names = "usb"; status = "okay"; }; @@ -151,7 +150,6 @@ &ohci0 { phys = <&usbphy 0>; - phy-names = "usb"; status = "okay"; }; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi index 6f27eb082429..666f32b876c4 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi @@ -553,7 +553,6 @@ resets = <&ccu RST_BUS_OHCI1>, <&ccu RST_BUS_EHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -565,7 +564,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi index e0dc4a05c1ba..3e0ec6e92869 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi @@ -445,7 +445,6 @@ resets = <&ccu RST_BUS_OHCI3>, <&ccu RST_BUS_EHCI3>; phys = <&usb2phy 3>; - phy-names = "usb"; status = "disabled"; }; @@ -457,7 +456,6 @@ <&ccu CLK_USB_OHCI3>; resets = <&ccu RST_BUS_OHCI3>; phys = <&usb2phy 3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, devicetree, Maxime Ripard, linux-usb, Chen-Yu Tsai, Rob Herring, Frank Rowand, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts | 2 -- arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 2 -- arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 2 -- 3 files changed, 6 deletions(-) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts index 78c82a665c84..2b6345db7dc0 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts @@ -104,7 +104,6 @@ &ehci0 { phys = <&usbphy 0>; - phy-names = "usb"; status = "okay"; }; @@ -151,7 +150,6 @@ &ohci0 { phys = <&usbphy 0>; - phy-names = "usb"; status = "okay"; }; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi index 6f27eb082429..666f32b876c4 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi @@ -553,7 +553,6 @@ resets = <&ccu RST_BUS_OHCI1>, <&ccu RST_BUS_EHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -565,7 +564,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi index e0dc4a05c1ba..3e0ec6e92869 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi @@ -445,7 +445,6 @@ resets = <&ccu RST_BUS_OHCI3>, <&ccu RST_BUS_EHCI3>; phys = <&usb2phy 3>; - phy-names = "usb"; status = "disabled"; }; @@ -457,7 +456,6 @@ <&ccu CLK_USB_OHCI3>; resets = <&ccu RST_BUS_OHCI3>; phys = <&usb2phy 3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [PATCH 6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts | 2 -- arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 2 -- arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 2 -- 3 files changed, 6 deletions(-) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts index 78c82a665c84..2b6345db7dc0 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts @@ -104,7 +104,6 @@ &ehci0 { phys = <&usbphy 0>; - phy-names = "usb"; status = "okay"; }; @@ -151,7 +150,6 @@ &ohci0 { phys = <&usbphy 0>; - phy-names = "usb"; status = "okay"; }; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi index 6f27eb082429..666f32b876c4 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi @@ -553,7 +553,6 @@ resets = <&ccu RST_BUS_OHCI1>, <&ccu RST_BUS_EHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -565,7 +564,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi index e0dc4a05c1ba..3e0ec6e92869 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi @@ -445,7 +445,6 @@ resets = <&ccu RST_BUS_OHCI3>, <&ccu RST_BUS_EHCI3>; phys = <&usb2phy 3>; - phy-names = "usb"; status = "disabled"; }; @@ -457,7 +456,6 @@ <&ccu CLK_USB_OHCI3>; resets = <&ccu RST_BUS_OHCI3>; phys = <&usb2phy 3>; - phy-names = "usb"; status = "disabled"; }; -- git-series 0.9.1 ^ permalink raw reply related [flat|nested] 34+ messages in thread
* [6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:28 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-16 8:28 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Mark Rutland, Rob Herring, Frank Rowand, linux-usb, Chen-Yu Tsai, Maxime Ripard, devicetree, linux-arm-kernel Neither the OHCI or EHCI bindings are using the phy-names property, so we can just drop it. Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> --- arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts | 2 -- arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 2 -- arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 2 -- 3 files changed, 6 deletions(-) diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts index 78c82a665c84..2b6345db7dc0 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-pinebook.dts @@ -104,7 +104,6 @@ &ehci0 { phys = <&usbphy 0>; - phy-names = "usb"; status = "okay"; }; @@ -151,7 +150,6 @@ &ohci0 { phys = <&usbphy 0>; - phy-names = "usb"; status = "okay"; }; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi index 6f27eb082429..666f32b876c4 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi @@ -553,7 +553,6 @@ resets = <&ccu RST_BUS_OHCI1>, <&ccu RST_BUS_EHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; @@ -565,7 +564,6 @@ <&ccu CLK_USB_OHCI1>; resets = <&ccu RST_BUS_OHCI1>; phys = <&usbphy 1>; - phy-names = "usb"; status = "disabled"; }; diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi index e0dc4a05c1ba..3e0ec6e92869 100644 --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi @@ -445,7 +445,6 @@ resets = <&ccu RST_BUS_OHCI3>, <&ccu RST_BUS_EHCI3>; phys = <&usb2phy 3>; - phy-names = "usb"; status = "disabled"; }; @@ -457,7 +456,6 @@ <&ccu CLK_USB_OHCI3>; resets = <&ccu RST_BUS_OHCI3>; phys = <&usb2phy 3>; - phy-names = "usb"; status = "disabled"; }; ^ permalink raw reply related [flat|nested] 34+ messages in thread
* Re: [PATCH 6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:43 ` Chen-Yu Tsai 0 siblings, 0 replies; 34+ messages in thread From: Chen-Yu Tsai @ 2019-04-16 8:43 UTC (permalink / raw) To: Maxime Ripard Cc: Mark Rutland, devicetree, Greg Kroah-Hartman, linux-usb, Rob Herring, Frank Rowand, linux-arm-kernel On Tue, Apr 16, 2019 at 4:28 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > Neither the OHCI or EHCI bindings are using the phy-names property, so we > can just drop it. > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Acked-by: Chen-Yu Tsai <wens@csie.org> for all the DT patches in this series. ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH 6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:43 ` Chen-Yu Tsai 0 siblings, 0 replies; 34+ messages in thread From: Chen-Yu Tsai @ 2019-04-16 8:43 UTC (permalink / raw) To: Maxime Ripard Cc: Mark Rutland, devicetree, Greg Kroah-Hartman, linux-usb, Rob Herring, Frank Rowand, linux-arm-kernel On Tue, Apr 16, 2019 at 4:28 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > Neither the OHCI or EHCI bindings are using the phy-names property, so we > can just drop it. > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Acked-by: Chen-Yu Tsai <wens@csie.org> for all the DT patches in this series. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH 6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:43 ` Chen-Yu Tsai 0 siblings, 0 replies; 34+ messages in thread From: Chen-Yu Tsai @ 2019-04-16 8:43 UTC (permalink / raw) To: Maxime Ripard Cc: Greg Kroah-Hartman, Mark Rutland, Rob Herring, Frank Rowand, linux-usb, devicetree, linux-arm-kernel On Tue, Apr 16, 2019 at 4:28 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > Neither the OHCI or EHCI bindings are using the phy-names property, so we > can just drop it. > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Acked-by: Chen-Yu Tsai <wens@csie.org> for all the DT patches in this series. ^ permalink raw reply [flat|nested] 34+ messages in thread
* [6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-16 8:43 ` Chen-Yu Tsai 0 siblings, 0 replies; 34+ messages in thread From: Chen-Yu Tsai @ 2019-04-16 8:43 UTC (permalink / raw) To: Maxime Ripard Cc: Greg Kroah-Hartman, Mark Rutland, Rob Herring, Frank Rowand, linux-usb, devicetree, linux-arm-kernel On Tue, Apr 16, 2019 at 4:28 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > Neither the OHCI or EHCI bindings are using the phy-names property, so we > can just drop it. > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> Acked-by: Chen-Yu Tsai <wens@csie.org> for all the DT patches in this series. ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH 6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-17 7:58 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-17 7:58 UTC (permalink / raw) To: Chen-Yu Tsai Cc: Mark Rutland, devicetree, Greg Kroah-Hartman, linux-usb, Rob Herring, Frank Rowand, linux-arm-kernel [-- Attachment #1.1: Type: text/plain, Size: 521 bytes --] On Tue, Apr 16, 2019 at 04:43:48PM +0800, Chen-Yu Tsai wrote: > On Tue, Apr 16, 2019 at 4:28 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > > > Neither the OHCI or EHCI bindings are using the phy-names property, so we > > can just drop it. > > > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > > Acked-by: Chen-Yu Tsai <wens@csie.org> > > for all the DT patches in this series. Applied 4, 5 and 6, thanks! Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 176 bytes --] _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 34+ messages in thread
* Re: [PATCH 6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-17 7:58 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-17 7:58 UTC (permalink / raw) To: Chen-Yu Tsai Cc: Greg Kroah-Hartman, Mark Rutland, Rob Herring, Frank Rowand, linux-usb, devicetree, linux-arm-kernel [-- Attachment #1: Type: text/plain, Size: 521 bytes --] On Tue, Apr 16, 2019 at 04:43:48PM +0800, Chen-Yu Tsai wrote: > On Tue, Apr 16, 2019 at 4:28 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > > > Neither the OHCI or EHCI bindings are using the phy-names property, so we > > can just drop it. > > > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > > Acked-by: Chen-Yu Tsai <wens@csie.org> > > for all the DT patches in this series. Applied 4, 5 and 6, thanks! Maxime -- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 34+ messages in thread
* [6/6] arm64: dts: allwinner: Remove useless phy-names from EHCI and OHCI @ 2019-04-17 7:58 ` Maxime Ripard 0 siblings, 0 replies; 34+ messages in thread From: Maxime Ripard @ 2019-04-17 7:58 UTC (permalink / raw) To: Chen-Yu Tsai Cc: Greg Kroah-Hartman, Mark Rutland, Rob Herring, Frank Rowand, linux-usb, devicetree, linux-arm-kernel On Tue, Apr 16, 2019 at 04:43:48PM +0800, Chen-Yu Tsai wrote: > On Tue, Apr 16, 2019 at 4:28 PM Maxime Ripard <maxime.ripard@bootlin.com> wrote: > > > > Neither the OHCI or EHCI bindings are using the phy-names property, so we > > can just drop it. > > > > Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com> > > Acked-by: Chen-Yu Tsai <wens@csie.org> > > for all the DT patches in this series. Applied 4, 5 and 6, thanks! Maxime --- Maxime Ripard, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 34+ messages in thread
end of thread, other threads:[~2019-10-01 20:49 UTC | newest] Thread overview: 34+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-04-16 8:27 [PATCH 1/6] dt-bindings: usb: Convert USB HCD generic binding to YAML Maxime Ripard 2019-04-16 8:27 ` Maxime Ripard 2019-04-16 8:27 ` Maxime Ripard 2019-04-16 8:27 ` [1/6] " Maxime Ripard 2019-04-16 8:27 ` [PATCH 2/6] dt-bindings: usb: Convert the generic OHCI " Maxime Ripard 2019-04-16 8:27 ` Maxime Ripard 2019-04-16 8:27 ` Maxime Ripard 2019-04-16 8:27 ` [2/6] " Maxime Ripard 2019-04-16 8:27 ` [PATCH 3/6] dt-bindings: usb: Convert the generic EHCI " Maxime Ripard 2019-04-16 8:27 ` Maxime Ripard 2019-04-16 8:27 ` Maxime Ripard 2019-04-16 8:27 ` [3/6] " Maxime Ripard 2019-04-16 8:28 ` [PATCH 4/6] ARM: dts: sunxi: Remove useless phy-names from EHCI and OHCI Maxime Ripard 2019-04-16 8:28 ` Maxime Ripard 2019-04-16 8:28 ` Maxime Ripard 2019-04-16 8:28 ` [4/6] " Maxime Ripard 2019-10-01 20:42 ` [PATCH 4/6] " Emmanuel Vadot 2019-10-01 20:42 ` Emmanuel Vadot 2019-10-01 20:42 ` Emmanuel Vadot 2019-04-16 8:28 ` [PATCH 5/6] ARM: dts: sunxi: h3/h5: " Maxime Ripard 2019-04-16 8:28 ` Maxime Ripard 2019-04-16 8:28 ` Maxime Ripard 2019-04-16 8:28 ` [5/6] " Maxime Ripard 2019-04-16 8:28 ` [PATCH 6/6] arm64: dts: allwinner: " Maxime Ripard 2019-04-16 8:28 ` Maxime Ripard 2019-04-16 8:28 ` Maxime Ripard 2019-04-16 8:28 ` [6/6] " Maxime Ripard 2019-04-16 8:43 ` [PATCH 6/6] " Chen-Yu Tsai 2019-04-16 8:43 ` Chen-Yu Tsai 2019-04-16 8:43 ` Chen-Yu Tsai 2019-04-16 8:43 ` [6/6] " Chen-Yu Tsai 2019-04-17 7:58 ` [PATCH 6/6] " Maxime Ripard 2019-04-17 7:58 ` Maxime Ripard 2019-04-17 7:58 ` [6/6] " Maxime Ripard
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.