| Message ID | 20210210172850.20849-1-Sergey.Semin@baikalelectronics.ru (mailing list archive) |
|---|---|
| Headers | show |
| Series | dt-bindings: usb: Harmonize xHCI/EHCI/OHCI/DWC3 nodes name | expand |
On 2/10/21 9:28 AM, Serge Semin wrote: > As the subject states this series is an attempt to harmonize the xHCI, > EHCI, OHCI and DWC USB3 DT nodes with the DT schema introduced in the > framework of the patchset [1]. > > Firstly as Krzysztof suggested we've deprecated a support of DWC USB3 > controllers with "synopsys,"-vendor prefix compatible string in favor of > the ones with valid "snps,"-prefix. It's done in all the DTS files, > which have been unfortunate to define such nodes. > > Secondly we suggest to fix the snps,quirk-frame-length-adjustment property > declaration in the Amlogic meson-g12-common.dtsi DTS file, since it has > been erroneously declared as boolean while having uint32 type. Neil said > it was ok to init that property with 0x20 value. > > Thirdly the main part of the patchset concern fixing the xHCI, EHCI/OHCI > and DWC USB3 DT nodes name as in accordance with their DT schema 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(@.*)?". Such requirement had been applicable even before we > introduced the new DT schema in [1], but as we can see it hasn't been > strictly implemented for a lot the DTS files. Since DT schema is now > available the automated DTS validation shall make sure that the rule isn't > violated. > > Note most of these patches have been a part of the last three patches of > [1]. But since there is no way to have them merged in in a combined > manner, I had to move them to the dedicated series and split them up so to > be accepted by the corresponding subsystem maintainers one-by-one. > > [1] Link: https://lore.kernel.org/linux-usb/20201014101402.18271-1-Sergey.Semin@baikalelectronics.ru/ > Changelog v1: > - As Krzysztof suggested I've created a script which checked whether the > node names had been also updated in all the depended dts files. As a > result I found two more files which should have been also modified: > arch/arc/boot/dts/{axc003.dtsi,axc003_idu.dtsi} > - Correct the USB DWC3 nodes name found in > arch/arm64/boot/dts/apm/{apm-storm.dtsi,apm-shadowcat.dtsi} too. > > Link: https://lore.kernel.org/linux-usb/20201020115959.2658-1-Sergey.Semin@baikalelectronics.ru > Changelog v2: > - Drop the patch: > [PATCH 01/29] usb: dwc3: Discard synopsys,dwc3 compatibility string > and get back the one which marks the "synopsys,dwc3" compatible string > as deprecated into the DT schema related series. > - Drop the patches: > [PATCH 03/29] arm: dts: am437x: Correct DWC USB3 compatible string > [PATCH 04/29] arm: dts: exynos: Correct DWC USB3 compatible string > [PATCH 07/29] arm: dts: bcm53x: Harmonize EHCI/OHCI DT nodes name > [PATCH 08/29] arm: dts: stm32: Harmonize EHCI/OHCI DT nodes name > [PATCH 16/29] arm: dts: bcm5301x: Harmonize xHCI DT nodes name > [PATCH 19/29] arm: dts: exynos: Harmonize DWC USB3 DT nodes name > [PATCH 21/29] arm: dts: ls1021a: Harmonize DWC USB3 DT nodes name > [PATCH 22/29] arm: dts: omap5: Harmonize DWC USB3 DT nodes name > [PATCH 24/29] arm64: dts: allwinner: h6: Harmonize DWC USB3 DT nodes name > [PATCH 26/29] arm64: dts: exynos: Harmonize DWC USB3 DT nodes name > [PATCH 27/29] arm64: dts: layerscape: Harmonize DWC USB3 DT nodes name > since they have been applied to the corresponding maintainers repos. > - Fix drivers/usb/dwc3/dwc3-qcom.c to be looking for the "usb@"-prefixed > sub-node and falling back to the "dwc3@"-prefixed one on failure. > > Link: https://lore.kernel.org/linux-usb/20201111091552.15593-1-Sergey.Semin@baikalelectronics.ru > Changelog v3: > - Drop the patches: > [PATCH v2 04/18] arm: dts: hisi-x5hd2: Harmonize EHCI/OHCI DT nodes name > [PATCH v2 06/18] arm64: dts: hisi: Harmonize EHCI/OHCI DT nodes name > [PATCH v2 07/18] mips: dts: jz47x: Harmonize EHCI/OHCI DT nodes name > [PATCH v2 08/18] mips: dts: sead3: Harmonize EHCI/OHCI DT nodes name > [PATCH v2 09/18] mips: dts: ralink: mt7628a: Harmonize EHCI/OHCI DT nodes name > [PATCH v2 11/18] arm64: dts: marvell: cp11x: Harmonize xHCI DT nodes name > [PATCH v2 12/18] arm: dts: marvell: armada-375: Harmonize DWC USB3 DT nodes name > [PATCH v2 16/18] arm64: dts: hi3660: Harmonize DWC USB3 DT nodes name > since they have been applied to the corresponding maintainers repos. > > Link: https://lore.kernel.org/linux-usb/20201205155621.3045-1-Sergey.Semin@baikalelectronics.ru > Changelog v4: > - Just resend. > > Link: https://lore.kernel.org/linux-usb/20201210091756.18057-1-Sergey.Semin@baikalelectronics.ru/ > Changelog v5: > - Drop the patch: > [PATCH v4 02/10] arm64: dts: amlogic: meson-g12: Set FL-adj property value > since it has been applied to the corresponding maintainers repos. > - Get back the patch: > [PATCH 21/29] arm: dts: ls1021a: Harmonize DWC USB3 DT nodes name > as it has been missing in the kernel 5.11-rc7 > - Rebase onto the kernel 5.11-rc7 > > Link: https://lore.kernel.org/lkml/20210208135154.6645-1-Sergey.Semin@baikalelectronics.ru/ > Changelog v6: > - Just resend and add linux-usb.vger.kernel.org to the list of Ccecipients. If this needs to go on, can you drop the people who already took your patches (trying to lower my email amount to something manageable). Thank you.
On Wed, Feb 10, 2021 at 10:21:47AM -0800, Florian Fainelli wrote: > On 2/10/21 9:28 AM, Serge Semin wrote: > > As the subject states this series is an attempt to harmonize the xHCI, > > EHCI, OHCI and DWC USB3 DT nodes with the DT schema introduced in the > > framework of the patchset [1]. > > > > Firstly as Krzysztof suggested we've deprecated a support of DWC USB3 > > controllers with "synopsys,"-vendor prefix compatible string in favor of > > the ones with valid "snps,"-prefix. It's done in all the DTS files, > > which have been unfortunate to define such nodes. > > > > Secondly we suggest to fix the snps,quirk-frame-length-adjustment property > > declaration in the Amlogic meson-g12-common.dtsi DTS file, since it has > > been erroneously declared as boolean while having uint32 type. Neil said > > it was ok to init that property with 0x20 value. > > > > Thirdly the main part of the patchset concern fixing the xHCI, EHCI/OHCI > > and DWC USB3 DT nodes name as in accordance with their DT schema 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(@.*)?". Such requirement had been applicable even before we > > introduced the new DT schema in [1], but as we can see it hasn't been > > strictly implemented for a lot the DTS files. Since DT schema is now > > available the automated DTS validation shall make sure that the rule isn't > > violated. > > > > Note most of these patches have been a part of the last three patches of > > [1]. But since there is no way to have them merged in in a combined > > manner, I had to move them to the dedicated series and split them up so to > > be accepted by the corresponding subsystem maintainers one-by-one. > > > > [1] Link: https://lore.kernel.org/linux-usb/20201014101402.18271-1-Sergey.Semin@baikalelectronics.ru/ > > Changelog v1: > > - As Krzysztof suggested I've created a script which checked whether the > > node names had been also updated in all the depended dts files. As a > > result I found two more files which should have been also modified: > > arch/arc/boot/dts/{axc003.dtsi,axc003_idu.dtsi} > > - Correct the USB DWC3 nodes name found in > > arch/arm64/boot/dts/apm/{apm-storm.dtsi,apm-shadowcat.dtsi} too. > > > > Link: https://lore.kernel.org/linux-usb/20201020115959.2658-1-Sergey.Semin@baikalelectronics.ru > > Changelog v2: > > - Drop the patch: > > [PATCH 01/29] usb: dwc3: Discard synopsys,dwc3 compatibility string > > and get back the one which marks the "synopsys,dwc3" compatible string > > as deprecated into the DT schema related series. > > - Drop the patches: > > [PATCH 03/29] arm: dts: am437x: Correct DWC USB3 compatible string > > [PATCH 04/29] arm: dts: exynos: Correct DWC USB3 compatible string > > [PATCH 07/29] arm: dts: bcm53x: Harmonize EHCI/OHCI DT nodes name > > [PATCH 08/29] arm: dts: stm32: Harmonize EHCI/OHCI DT nodes name > > [PATCH 16/29] arm: dts: bcm5301x: Harmonize xHCI DT nodes name > > [PATCH 19/29] arm: dts: exynos: Harmonize DWC USB3 DT nodes name > > [PATCH 21/29] arm: dts: ls1021a: Harmonize DWC USB3 DT nodes name > > [PATCH 22/29] arm: dts: omap5: Harmonize DWC USB3 DT nodes name > > [PATCH 24/29] arm64: dts: allwinner: h6: Harmonize DWC USB3 DT nodes name > > [PATCH 26/29] arm64: dts: exynos: Harmonize DWC USB3 DT nodes name > > [PATCH 27/29] arm64: dts: layerscape: Harmonize DWC USB3 DT nodes name > > since they have been applied to the corresponding maintainers repos. > > - Fix drivers/usb/dwc3/dwc3-qcom.c to be looking for the "usb@"-prefixed > > sub-node and falling back to the "dwc3@"-prefixed one on failure. > > > > Link: https://lore.kernel.org/linux-usb/20201111091552.15593-1-Sergey.Semin@baikalelectronics.ru > > Changelog v3: > > - Drop the patches: > > [PATCH v2 04/18] arm: dts: hisi-x5hd2: Harmonize EHCI/OHCI DT nodes name > > [PATCH v2 06/18] arm64: dts: hisi: Harmonize EHCI/OHCI DT nodes name > > [PATCH v2 07/18] mips: dts: jz47x: Harmonize EHCI/OHCI DT nodes name > > [PATCH v2 08/18] mips: dts: sead3: Harmonize EHCI/OHCI DT nodes name > > [PATCH v2 09/18] mips: dts: ralink: mt7628a: Harmonize EHCI/OHCI DT nodes name > > [PATCH v2 11/18] arm64: dts: marvell: cp11x: Harmonize xHCI DT nodes name > > [PATCH v2 12/18] arm: dts: marvell: armada-375: Harmonize DWC USB3 DT nodes name > > [PATCH v2 16/18] arm64: dts: hi3660: Harmonize DWC USB3 DT nodes name > > since they have been applied to the corresponding maintainers repos. > > > > Link: https://lore.kernel.org/linux-usb/20201205155621.3045-1-Sergey.Semin@baikalelectronics.ru > > Changelog v4: > > - Just resend. > > > > Link: https://lore.kernel.org/linux-usb/20201210091756.18057-1-Sergey.Semin@baikalelectronics.ru/ > > Changelog v5: > > - Drop the patch: > > [PATCH v4 02/10] arm64: dts: amlogic: meson-g12: Set FL-adj property value > > since it has been applied to the corresponding maintainers repos. > > - Get back the patch: > > [PATCH 21/29] arm: dts: ls1021a: Harmonize DWC USB3 DT nodes name > > as it has been missing in the kernel 5.11-rc7 > > - Rebase onto the kernel 5.11-rc7 > > > > Link: https://lore.kernel.org/lkml/20210208135154.6645-1-Sergey.Semin@baikalelectronics.ru/ > > Changelog v6: > > - Just resend and add linux-usb.vger.kernel.org to the list of Ccecipients. > > > If this needs to go on, can you drop the people who already took your > patches (trying to lower my email amount to something manageable). > Thank you. Ah, sorry for the noise. I'll clean the Cc-list up in the next attempt to have this finally fully accepted. -Sergey > -- > Florian
As the subject states this series is an attempt to harmonize the xHCI, EHCI, OHCI and DWC USB3 DT nodes with the DT schema introduced in the framework of the patchset [1]. Firstly as Krzysztof suggested we've deprecated a support of DWC USB3 controllers with "synopsys,"-vendor prefix compatible string in favor of the ones with valid "snps,"-prefix. It's done in all the DTS files, which have been unfortunate to define such nodes. Secondly we suggest to fix the snps,quirk-frame-length-adjustment property declaration in the Amlogic meson-g12-common.dtsi DTS file, since it has been erroneously declared as boolean while having uint32 type. Neil said it was ok to init that property with 0x20 value. Thirdly the main part of the patchset concern fixing the xHCI, EHCI/OHCI and DWC USB3 DT nodes name as in accordance with their DT schema 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(@.*)?". Such requirement had been applicable even before we introduced the new DT schema in [1], but as we can see it hasn't been strictly implemented for a lot the DTS files. Since DT schema is now available the automated DTS validation shall make sure that the rule isn't violated. Note most of these patches have been a part of the last three patches of [1]. But since there is no way to have them merged in in a combined manner, I had to move them to the dedicated series and split them up so to be accepted by the corresponding subsystem maintainers one-by-one. [1] Link: https://lore.kernel.org/linux-usb/20201014101402.18271-1-Sergey.Semin@baikalelectronics.ru/ Changelog v1: - As Krzysztof suggested I've created a script which checked whether the node names had been also updated in all the depended dts files. As a result I found two more files which should have been also modified: arch/arc/boot/dts/{axc003.dtsi,axc003_idu.dtsi} - Correct the USB DWC3 nodes name found in arch/arm64/boot/dts/apm/{apm-storm.dtsi,apm-shadowcat.dtsi} too. Link: https://lore.kernel.org/linux-usb/20201020115959.2658-1-Sergey.Semin@baikalelectronics.ru Changelog v2: - Drop the patch: [PATCH 01/29] usb: dwc3: Discard synopsys,dwc3 compatibility string and get back the one which marks the "synopsys,dwc3" compatible string as deprecated into the DT schema related series. - Drop the patches: [PATCH 03/29] arm: dts: am437x: Correct DWC USB3 compatible string [PATCH 04/29] arm: dts: exynos: Correct DWC USB3 compatible string [PATCH 07/29] arm: dts: bcm53x: Harmonize EHCI/OHCI DT nodes name [PATCH 08/29] arm: dts: stm32: Harmonize EHCI/OHCI DT nodes name [PATCH 16/29] arm: dts: bcm5301x: Harmonize xHCI DT nodes name [PATCH 19/29] arm: dts: exynos: Harmonize DWC USB3 DT nodes name [PATCH 21/29] arm: dts: ls1021a: Harmonize DWC USB3 DT nodes name [PATCH 22/29] arm: dts: omap5: Harmonize DWC USB3 DT nodes name [PATCH 24/29] arm64: dts: allwinner: h6: Harmonize DWC USB3 DT nodes name [PATCH 26/29] arm64: dts: exynos: Harmonize DWC USB3 DT nodes name [PATCH 27/29] arm64: dts: layerscape: Harmonize DWC USB3 DT nodes name since they have been applied to the corresponding maintainers repos. - Fix drivers/usb/dwc3/dwc3-qcom.c to be looking for the "usb@"-prefixed sub-node and falling back to the "dwc3@"-prefixed one on failure. Link: https://lore.kernel.org/linux-usb/20201111091552.15593-1-Sergey.Semin@baikalelectronics.ru Changelog v3: - Drop the patches: [PATCH v2 04/18] arm: dts: hisi-x5hd2: Harmonize EHCI/OHCI DT nodes name [PATCH v2 06/18] arm64: dts: hisi: Harmonize EHCI/OHCI DT nodes name [PATCH v2 07/18] mips: dts: jz47x: Harmonize EHCI/OHCI DT nodes name [PATCH v2 08/18] mips: dts: sead3: Harmonize EHCI/OHCI DT nodes name [PATCH v2 09/18] mips: dts: ralink: mt7628a: Harmonize EHCI/OHCI DT nodes name [PATCH v2 11/18] arm64: dts: marvell: cp11x: Harmonize xHCI DT nodes name [PATCH v2 12/18] arm: dts: marvell: armada-375: Harmonize DWC USB3 DT nodes name [PATCH v2 16/18] arm64: dts: hi3660: Harmonize DWC USB3 DT nodes name since they have been applied to the corresponding maintainers repos. Link: https://lore.kernel.org/linux-usb/20201205155621.3045-1-Sergey.Semin@baikalelectronics.ru Changelog v4: - Just resend. Link: https://lore.kernel.org/linux-usb/20201210091756.18057-1-Sergey.Semin@baikalelectronics.ru/ Changelog v5: - Drop the patch: [PATCH v4 02/10] arm64: dts: amlogic: meson-g12: Set FL-adj property value since it has been applied to the corresponding maintainers repos. - Get back the patch: [PATCH 21/29] arm: dts: ls1021a: Harmonize DWC USB3 DT nodes name as it has been missing in the kernel 5.11-rc7 - Rebase onto the kernel 5.11-rc7 Link: https://lore.kernel.org/lkml/20210208135154.6645-1-Sergey.Semin@baikalelectronics.ru/ Changelog v6: - Just resend and add linux-usb.vger.kernel.org to the list of Ccecipients. Cc: Vineet Gupta <vgupta@synopsys.com> Cc: Rafal Milecki <zajec5@gmail.com> Cc: Wei Xu <xuwei5@hisilicon.com> Cc: Thomas Bogendoerfer <tsbogend@alpha.franken.de> Cc: Michael Ellerman <mpe@ellerman.id.au> Cc: Jason Cooper <jason@lakedaemon.net> Cc: Santosh Shilimkar <ssantosh@kernel.org> Cc: Shawn Guo <shawnguo@kernel.org> Cc: Benoit Cousson <bcousson@baylibre.com> Cc: Patrice Chotard <patrice.chotard@st.com> Cc: Maxime Ripard <mripard@kernel.org> Cc: Khuong Dinh <khuong@os.amperecomputing.com> Cc: Andy Gross <agross@kernel.org> Cc: Alexey Brodkin <abrodkin@synopsys.com> Cc: Hauke Mehrtens <hauke@hauke-m.de> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com> Cc: Alexandre Torgue <alexandre.torgue@st.com> Cc: Amelie Delaunay <amelie.delaunay@st.com> Cc: Vladimir Zapolskiy <vz@mleia.com> Cc: Paul Cercueil <paul@crapouillou.net> Cc: Matthias Brugger <matthias.bgg@gmail.com> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Andrew Lunn <andrew@lunn.ch> Cc: Gregory Clement <gregory.clement@bootlin.com> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com> Cc: Kukjin Kim <kgene@kernel.org> Cc: Li Yang <leoyang.li@nxp.com> Cc: Tony Lindgren <tony@atomide.com> Cc: Chen-Yu Tsai <wens@csie.org> Cc: Bjorn Andersson <bjorn.andersson@linaro.org> Cc: Jun Li <lijun.kernel@gmail.com> Cc: linux-snps-arc@lists.infradead.org Cc: bcm-kernel-feedback-list@broadcom.com Cc: linux-stm32@st-md-mailman.stormreply.com Cc: linux-arm-kernel@lists.infradead.org Cc: linux-mips@vger.kernel.org Cc: linux-mediatek@lists.infradead.org Cc: linuxppc-dev@lists.ozlabs.org Cc: linux-samsung-soc@vger.kernel.org Cc: linux-omap@vger.kernel.org Cc: linux-arm-msm@vger.kernel.org Cc: devicetree@vger.kernel.org Cc: linux-kernel@vger.kernel.org Serge Semin (10): arm: dts: ls1021a: Harmonize DWC USB3 DT nodes name arm: dts: keystone: Correct DWC USB3 compatible string arc: dts: Harmonize EHCI/OHCI DT nodes name arm: dts: lpc18xx: Harmonize EHCI/OHCI DT nodes name powerpc: dts: akebono: Harmonize EHCI/OHCI DT nodes name arm: dts: keystone: Harmonize DWC USB3 DT nodes name arm: dts: stih407-family: Harmonize DWC USB3 DT nodes name arm64: dts: apm: Harmonize DWC USB3 DT nodes name usb: dwc3: qcom: Detect DWC3 DT-nodes with "usb"-prefixed names arm64: dts: qcom: Harmonize DWC USB3 DT nodes name arch/arc/boot/dts/axc003.dtsi | 4 ++-- arch/arc/boot/dts/axc003_idu.dtsi | 4 ++-- 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/keystone-k2e.dtsi | 6 +++--- arch/arm/boot/dts/keystone.dtsi | 4 ++-- arch/arm/boot/dts/lpc18xx.dtsi | 4 ++-- arch/arm/boot/dts/ls1021a.dtsi | 2 +- arch/arm/boot/dts/stih407-family.dtsi | 2 +- arch/arm64/boot/dts/apm/apm-shadowcat.dtsi | 4 ++-- arch/arm64/boot/dts/apm/apm-storm.dtsi | 6 +++--- arch/arm64/boot/dts/qcom/apq8096-db820c.dtsi | 4 ++-- arch/arm64/boot/dts/qcom/ipq8074.dtsi | 4 ++-- arch/arm64/boot/dts/qcom/msm8996.dtsi | 4 ++-- arch/arm64/boot/dts/qcom/msm8998.dtsi | 2 +- arch/arm64/boot/dts/qcom/qcs404-evb.dtsi | 2 +- arch/arm64/boot/dts/qcom/qcs404.dtsi | 4 ++-- arch/arm64/boot/dts/qcom/sc7180.dtsi | 2 +- arch/arm64/boot/dts/qcom/sdm845.dtsi | 4 ++-- arch/arm64/boot/dts/qcom/sm8150.dtsi | 2 +- arch/powerpc/boot/dts/akebono.dts | 6 +++--- drivers/usb/dwc3/dwc3-qcom.c | 3 ++- 23 files changed, 42 insertions(+), 41 deletions(-)