lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Wed, 14 Oct 2020 20:56:01 +0200
From:   Paul Cercueil <paul@...pouillou.net>
To:     Serge Semin <Sergey.Semin@...kalelectronics.ru>
Cc:     Mathias Nyman <mathias.nyman@...el.com>,
        Felipe Balbi <balbi@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Rob Herring <robh+dt@...nel.org>,
        Alexey Brodkin <abrodkin@...opsys.com>,
        Vineet Gupta <vgupta@...opsys.com>,
        Hauke Mehrtens <hauke@...ke-m.de>,
        Rafał Miłecki <zajec5@...il.com>,
        bcm-kernel-feedback-list@...adcom.com,
        Wei Xu <xuwei5@...ilicon.com>,
        Vladimir Zapolskiy <vz@...ia.com>,
        Maxime Coquelin <mcoquelin.stm32@...il.com>,
        Alexandre Torgue <alexandre.torgue@...com>,
        Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
        Matthias Brugger <matthias.bgg@...il.com>,
        Michael Ellerman <mpe@...erman.id.au>,
        Benjamin Herrenschmidt <benh@...nel.crashing.org>,
        Paul Mackerras <paulus@...ba.org>,
        Serge Semin <fancer.lancer@...il.com>,
        Alexey Malahov <Alexey.Malahov@...kalelectronics.ru>,
        Pavel Parkhomenko <Pavel.Parkhomenko@...kalelectronics.ru>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Manu Gautam <mgautam@...eaurora.org>,
        Roger Quadros <rogerq@...com>,
        Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>,
        Yoshihiro Shimoda <yoshihiro.shimoda.uh@...esas.com>,
        Neil Armstrong <narmstrong@...libre.com>,
        Kevin Hilman <khilman@...libre.com>,
        linux-arm-kernel@...ts.infradead.org,
        linux-snps-arc@...ts.infradead.org, linux-mips@...r.kernel.org,
        linuxppc-dev@...ts.ozlabs.org, linux-usb@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-stm32@...md-mailman.stormreply.com,
        linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH 18/20] arch: dts: Fix EHCI/OHCI DT nodes name



Le mer. 14 oct. 2020 à 13:14, Serge Semin 
<Sergey.Semin@...kalelectronics.ru> a écrit :
> In accordance with the Generic EHCI/OHCI bindings the corresponding 
> node
> name is suppose to comply with the Generic USB HCD DT schema, which
> requires the USB nodes to have the name acceptable by the regexp:
> "^usb(@.*)?" . Let's fix the DTS files, which have the nodes defined 
> with
> incompatible names.
> 
> Signed-off-by: Serge Semin <Sergey.Semin@...kalelectronics.ru>
> 
> ---
> 
> Please, test the patch out to make sure it doesn't brake the 
> dependent DTS
> files. I did only a manual grepping of the possible nodes 
> dependencies.
> ---
>  arch/arc/boot/dts/axs10x_mb.dtsi               | 4 ++--
>  arch/arc/boot/dts/hsdk.dts                     | 4 ++--
>  arch/arc/boot/dts/vdk_axs10x_mb.dtsi           | 2 +-
>  arch/arm/boot/dts/bcm5301x.dtsi                | 4 ++--
>  arch/arm/boot/dts/bcm53573.dtsi                | 4 ++--
>  arch/arm/boot/dts/hisi-x5hd2.dtsi              | 4 ++--
>  arch/arm/boot/dts/lpc18xx.dtsi                 | 4 ++--
>  arch/arm/boot/dts/stm32mp151.dtsi              | 4 ++--
>  arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi | 4 ++--
>  arch/arm64/boot/dts/hisilicon/hip06.dtsi       | 4 ++--
>  arch/arm64/boot/dts/hisilicon/hip07.dtsi       | 4 ++--
>  arch/mips/boot/dts/ingenic/jz4740.dtsi         | 2 +-
>  arch/mips/boot/dts/ingenic/jz4770.dtsi         | 2 +-

For jz4740.dtsi and jz4770.dtsi:
Acked-by: Paul Cercueil <paul@...pouillou.net>

Cheers,
-Paul

>  arch/mips/boot/dts/mti/sead3.dts               | 2 +-
>  arch/mips/boot/dts/ralink/mt7628a.dtsi         | 2 +-
>  arch/powerpc/boot/dts/akebono.dts              | 6 +++---
>  16 files changed, 28 insertions(+), 28 deletions(-)
> 
> diff --git a/arch/arc/boot/dts/axs10x_mb.dtsi 
> b/arch/arc/boot/dts/axs10x_mb.dtsi
> index 99d3e7175bf7..b64435385304 100644
> --- a/arch/arc/boot/dts/axs10x_mb.dtsi
> +++ b/arch/arc/boot/dts/axs10x_mb.dtsi
> @@ -87,13 +87,13 @@ gmac: ethernet@...00 {
>  			mac-address = [00 00 00 00 00 00]; /* Filled in by U-Boot */
>  		};
> 
> -		ehci@...00 {
> +		usb@...00 {
>  			compatible = "generic-ehci";
>  			reg = < 0x40000 0x100 >;
>  			interrupts = < 8 >;
>  		};
> 
> -		ohci@...00 {
> +		usb@...00 {
>  			compatible = "generic-ohci";
>  			reg = < 0x60000 0x100 >;
>  			interrupts = < 8 >;
> diff --git a/arch/arc/boot/dts/hsdk.dts b/arch/arc/boot/dts/hsdk.dts
> index dcaa44e408ac..fdd4f7f635d3 100644
> --- a/arch/arc/boot/dts/hsdk.dts
> +++ b/arch/arc/boot/dts/hsdk.dts
> @@ -234,7 +234,7 @@ phy0: ethernet-phy@0 { /* Micrel KSZ9031 */
>  			};
>  		};
> 
> -		ohci@...00 {
> +		usb@...00 {
>  			compatible = "snps,hsdk-v1.0-ohci", "generic-ohci";
>  			reg = <0x60000 0x100>;
>  			interrupts = <15>;
> @@ -242,7 +242,7 @@ ohci@...00 {
>  			dma-coherent;
>  		};
> 
> -		ehci@...00 {
> +		usb@...00 {
>  			compatible = "snps,hsdk-v1.0-ehci", "generic-ehci";
>  			reg = <0x40000 0x100>;
>  			interrupts = <15>;
> diff --git a/arch/arc/boot/dts/vdk_axs10x_mb.dtsi 
> b/arch/arc/boot/dts/vdk_axs10x_mb.dtsi
> index cbb179770293..90a412026e64 100644
> --- a/arch/arc/boot/dts/vdk_axs10x_mb.dtsi
> +++ b/arch/arc/boot/dts/vdk_axs10x_mb.dtsi
> @@ -46,7 +46,7 @@ ethernet@...00 {
>  			clock-names = "stmmaceth";
>  		};
> 
> -		ehci@...00 {
> +		usb@...00 {
>  			compatible = "generic-ehci";
>  			reg = < 0x40000 0x100 >;
>  			interrupts = < 8 >;
> diff --git a/arch/arm/boot/dts/bcm5301x.dtsi 
> b/arch/arm/boot/dts/bcm5301x.dtsi
> index 0016720ce530..bf5656d79a55 100644
> --- a/arch/arm/boot/dts/bcm5301x.dtsi
> +++ b/arch/arm/boot/dts/bcm5301x.dtsi
> @@ -261,7 +261,7 @@ usb2: usb2@...00 {
> 
>  			interrupt-parent = <&gic>;
> 
> -			ehci: ehci@...00 {
> +			ehci: usb@...00 {
>  				#usb-cells = <0>;
> 
>  				compatible = "generic-ehci";
> @@ -283,7 +283,7 @@ ehci_port2: port@2 {
>  				};
>  			};
> 
> -			ohci: ohci@...00 {
> +			ohci: usb@...00 {
>  				#usb-cells = <0>;
> 
>  				compatible = "generic-ohci";
> diff --git a/arch/arm/boot/dts/bcm53573.dtsi 
> b/arch/arm/boot/dts/bcm53573.dtsi
> index 4af8e3293cff..51546fccc616 100644
> --- a/arch/arm/boot/dts/bcm53573.dtsi
> +++ b/arch/arm/boot/dts/bcm53573.dtsi
> @@ -135,7 +135,7 @@ usb2: usb2@...0 {
>  			#address-cells = <1>;
>  			#size-cells = <1>;
> 
> -			ehci: ehci@...0 {
> +			ehci: usb@...0 {
>  				compatible = "generic-ehci";
>  				reg = <0x4000 0x1000>;
>  				interrupt-parent = <&gic>;
> @@ -155,7 +155,7 @@ ehci_port2: port@2 {
>  				};
>  			};
> 
> -			ohci: ohci@...0 {
> +			ohci: usb@...0 {
>  				#usb-cells = <0>;
> 
>  				compatible = "generic-ohci";
> diff --git a/arch/arm/boot/dts/hisi-x5hd2.dtsi 
> b/arch/arm/boot/dts/hisi-x5hd2.dtsi
> index 3ee7967c202d..693b85b2cc7d 100644
> --- a/arch/arm/boot/dts/hisi-x5hd2.dtsi
> +++ b/arch/arm/boot/dts/hisi-x5hd2.dtsi
> @@ -452,14 +452,14 @@ gmac1: ethernet@...1000 {
>  			status = "disabled";
>  		};
> 
> -		usb0: ehci@...0000 {
> +		usb0: usb@...0000 {
>  			compatible = "generic-ehci";
>  			reg = <0x1890000 0x1000>;
>  			interrupts = <0 66 4>;
>  			clocks = <&clock HIX5HD2_USB_CLK>;
>  		};
> 
> -		usb1: ohci@...0000 {
> +		usb1: usb@...0000 {
>  			compatible = "generic-ohci";
>  			reg = <0x1880000 0x1000>;
>  			interrupts = <0 67 4>;
> diff --git a/arch/arm/boot/dts/lpc18xx.dtsi 
> b/arch/arm/boot/dts/lpc18xx.dtsi
> index 10b8249b8ab6..82ffd7b0ad8a 100644
> --- a/arch/arm/boot/dts/lpc18xx.dtsi
> +++ b/arch/arm/boot/dts/lpc18xx.dtsi
> @@ -121,7 +121,7 @@ mmcsd: mmcsd@...04000 {
>  			status = "disabled";
>  		};
> 
> -		usb0: ehci@...06100 {
> +		usb0: usb@...06100 {
>  			compatible = "nxp,lpc1850-ehci", "generic-ehci";
>  			reg = <0x40006100 0x100>;
>  			interrupts = <8>;
> @@ -133,7 +133,7 @@ usb0: ehci@...06100 {
>  			status = "disabled";
>  		};
> 
> -		usb1: ehci@...07100 {
> +		usb1: usb@...07100 {
>  			compatible = "nxp,lpc1850-ehci", "generic-ehci";
>  			reg = <0x40007100 0x100>;
>  			interrupts = <9>;
> diff --git a/arch/arm/boot/dts/stm32mp151.dtsi 
> b/arch/arm/boot/dts/stm32mp151.dtsi
> index bfe29023fbd5..576f7da564c5 100644
> --- a/arch/arm/boot/dts/stm32mp151.dtsi
> +++ b/arch/arm/boot/dts/stm32mp151.dtsi
> @@ -1404,7 +1404,7 @@ ethernet0: ethernet@...0a000 {
>  			status = "disabled";
>  		};
> 
> -		usbh_ohci: usbh-ohci@...0c000 {
> +		usbh_ohci: usb@...0c000 {
>  			compatible = "generic-ohci";
>  			reg = <0x5800c000 0x1000>;
>  			clocks = <&rcc USBH>;
> @@ -1413,7 +1413,7 @@ usbh_ohci: usbh-ohci@...0c000 {
>  			status = "disabled";
>  		};
> 
> -		usbh_ehci: usbh-ehci@...0d000 {
> +		usbh_ehci: usb@...0d000 {
>  			compatible = "generic-ehci";
>  			reg = <0x5800d000 0x1000>;
>  			clocks = <&rcc USBH>;
> diff --git a/arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi 
> b/arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi
> index 12bc1d3ed424..a4acecb75c89 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi
> +++ b/arch/arm64/boot/dts/hisilicon/hi3798cv200.dtsi
> @@ -585,7 +585,7 @@ pcie: pcie@...0000 {
>  			status = "disabled";
>  		};
> 
> -		ohci: ohci@...0000 {
> +		ohci: usb@...0000 {
>  			compatible = "generic-ohci";
>  			reg = <0x9880000 0x10000>;
>  			interrupts = <GIC_SPI 67 IRQ_TYPE_LEVEL_HIGH>;
> @@ -600,7 +600,7 @@ ohci: ohci@...0000 {
>  			status = "disabled";
>  		};
> 
> -		ehci: ehci@...0000 {
> +		ehci: usb@...0000 {
>  			compatible = "generic-ehci";
>  			reg = <0x9890000 0x10000>;
>  			interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
> diff --git a/arch/arm64/boot/dts/hisilicon/hip06.dtsi 
> b/arch/arm64/boot/dts/hisilicon/hip06.dtsi
> index 50ceaa959bdc..1226440d54ad 100644
> --- a/arch/arm64/boot/dts/hisilicon/hip06.dtsi
> +++ b/arch/arm64/boot/dts/hisilicon/hip06.dtsi
> @@ -373,7 +373,7 @@ refclk: refclk {
>  			#clock-cells = <0>;
>  		};
> 
> -		usb_ohci: ohci@...30000 {
> +		usb_ohci: usb@...30000 {
>  			compatible = "generic-ohci";
>  			reg = <0x0 0xa7030000 0x0 0x10000>;
>  			interrupt-parent = <&mbigen_usb>;
> @@ -382,7 +382,7 @@ usb_ohci: ohci@...30000 {
>  			status = "disabled";
>  		};
> 
> -		usb_ehci: ehci@...20000 {
> +		usb_ehci: usb@...20000 {
>  			compatible = "generic-ehci";
>  			reg = <0x0 0xa7020000 0x0 0x10000>;
>  			interrupt-parent = <&mbigen_usb>;
> diff --git a/arch/arm64/boot/dts/hisilicon/hip07.dtsi 
> b/arch/arm64/boot/dts/hisilicon/hip07.dtsi
> index 4773a533fce5..93f99a5255ac 100644
> --- a/arch/arm64/boot/dts/hisilicon/hip07.dtsi
> +++ b/arch/arm64/boot/dts/hisilicon/hip07.dtsi
> @@ -1253,7 +1253,7 @@ uart0: uart@...b0000 {
>  			status = "disabled";
>  		};
> 
> -		usb_ohci: ohci@...30000 {
> +		usb_ohci: usb@...30000 {
>  			compatible = "generic-ohci";
>  			reg = <0x0 0xa7030000 0x0 0x10000>;
>  			interrupt-parent = <&mbigen_usb>;
> @@ -1262,7 +1262,7 @@ usb_ohci: ohci@...30000 {
>  			status = "disabled";
>  		};
> 
> -		usb_ehci: ehci@...20000 {
> +		usb_ehci: usb@...20000 {
>  			compatible = "generic-ehci";
>  			reg = <0x0 0xa7020000 0x0 0x10000>;
>  			interrupt-parent = <&mbigen_usb>;
> diff --git a/arch/mips/boot/dts/ingenic/jz4740.dtsi 
> b/arch/mips/boot/dts/ingenic/jz4740.dtsi
> index 1520585c235c..b989ff62ffbc 100644
> --- a/arch/mips/boot/dts/ingenic/jz4740.dtsi
> +++ b/arch/mips/boot/dts/ingenic/jz4740.dtsi
> @@ -281,7 +281,7 @@ dmac: dma-controller@...20000 {
>  		clocks = <&cgu JZ4740_CLK_DMA>;
>  	};
> 
> -	uhc: uhc@...30000 {
> +	uhc: usb@...30000 {
>  		compatible = "ingenic,jz4740-ohci", "generic-ohci";
>  		reg = <0x13030000 0x1000>;
> 
> diff --git a/arch/mips/boot/dts/ingenic/jz4770.dtsi 
> b/arch/mips/boot/dts/ingenic/jz4770.dtsi
> index fa11ac950499..e45c03038826 100644
> --- a/arch/mips/boot/dts/ingenic/jz4770.dtsi
> +++ b/arch/mips/boot/dts/ingenic/jz4770.dtsi
> @@ -417,7 +417,7 @@ dmac1: dma-controller@...20100 {
>  		interrupts = <23>;
>  	};
> 
> -	uhc: uhc@...30000 {
> +	uhc: usb@...30000 {
>  		compatible = "generic-ohci";
>  		reg = <0x13430000 0x1000>;
> 
> diff --git a/arch/mips/boot/dts/mti/sead3.dts 
> b/arch/mips/boot/dts/mti/sead3.dts
> index 192c26ff1d3d..1cf6728af8fe 100644
> --- a/arch/mips/boot/dts/mti/sead3.dts
> +++ b/arch/mips/boot/dts/mti/sead3.dts
> @@ -56,7 +56,7 @@ gic: interrupt-controller@...c0000 {
>  		interrupt-parent = <&cpu_intc>;
>  	};
> 
> -	ehci@...00000 {
> +	usb@...00000 {
>  		compatible = "generic-ehci";
>  		reg = <0x1b200000 0x1000>;
> 
> diff --git a/arch/mips/boot/dts/ralink/mt7628a.dtsi 
> b/arch/mips/boot/dts/ralink/mt7628a.dtsi
> index 892e8ab863c5..45bf96a3d17a 100644
> --- a/arch/mips/boot/dts/ralink/mt7628a.dtsi
> +++ b/arch/mips/boot/dts/ralink/mt7628a.dtsi
> @@ -275,7 +275,7 @@ usb_phy: usb-phy@...20000 {
>  		reset-names = "host", "device";
>  	};
> 
> -	ehci@...c0000 {
> +	usb@...c0000 {
>  		compatible = "generic-ehci";
>  		reg = <0x101c0000 0x1000>;
> 
> diff --git a/arch/powerpc/boot/dts/akebono.dts 
> b/arch/powerpc/boot/dts/akebono.dts
> index df18f8dc4642..343326c30380 100644
> --- a/arch/powerpc/boot/dts/akebono.dts
> +++ b/arch/powerpc/boot/dts/akebono.dts
> @@ -126,7 +126,7 @@ SATA0: sata@...00010000 {
>  			interrupts = <93 2>;
>  		};
> 
> -		EHCI0: ehci@...10000000 {
> +		EHCI0: usb@...10000000 {
>  			compatible = "ibm,476gtr-ehci", "generic-ehci";
>  			reg = <0x300 0x10000000 0x0 0x10000>;
>  			interrupt-parent = <&MPIC>;
> @@ -140,14 +140,14 @@ SD0: sd@...00000000 {
>  			interrupt-parent = <&MPIC>;
>  		};
> 
> -		OHCI0: ohci@...10010000 {
> +		OHCI0: usb@...10010000 {
>  			compatible = "ibm,476gtr-ohci", "generic-ohci";
>  			reg = <0x300 0x10010000 0x0 0x10000>;
>  			interrupt-parent = <&MPIC>;
>  			interrupts = <89 1>;
>  			};
> 
> -		OHCI1: ohci@...10020000 {
> +		OHCI1: usb@...10020000 {
>  			compatible = "ibm,476gtr-ohci", "generic-ohci";
>  			reg = <0x300 0x10020000 0x0 0x10000>;
>  			interrupt-parent = <&MPIC>;
> --
> 2.27.0
> 


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ