* [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations
@ 2023-01-05 9:28 Macpaul Lin
2023-01-05 9:28 ` [PATCH 2/3] arm64: dts: mediatek: switch MT8195 boards into different " Macpaul Lin
` (3 more replies)
0 siblings, 4 replies; 14+ messages in thread
From: Macpaul Lin @ 2023-01-05 9:28 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Matthias Brugger,
linux-arm-kernel, linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Macpaul Lin, Miles Chen,
linux-usb, Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai
Introduce the split MT8195 laptop and iot USB configurations.
The hardware specifications for LAPTOP devices is different from IOT
devices. The major differences include some hardware constrains for
dual-role switch for USB controllers in different configurations,
especially for power management and other control flows as well.
Here are some hardware specifiction differences listed:
1. LAPTOP (Cherry Tomato boards) don't support USB gadget (device mode).
2. IOT devices must support multiple gadget devices and host mode.
3. Dual-role switch is not fully supported. Only USB PORT0 support
dual-role switch.
4. Power management is designed in primary and secondary dominator.
For a dual-role port, the device controller is the primary controller
for power management; while the host controller is the secondary.
LAPTOP devices should remove device nodes for avoiding abnormal
behavior.
This modifcation is to add USB configurations "mt8195-laptop-usb.dtsi"
for LAPTOP devices, and add "mt8195-iot-usb.dtsi" for IOT devices.
To remove common USB configurations for mt8195.dtsi and switch includes
dtsi these new files for the boards will come in next patch.
Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
---
.../boot/dts/mediatek/mt8195-iot-usb.dtsi | 122 ++++++++++++++++++
.../boot/dts/mediatek/mt8195-laptop-usb.dtsi | 102 +++++++++++++++
2 files changed, 224 insertions(+)
create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-laptop-usb.dtsi
diff --git a/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi b/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
new file mode 100644
index 000000000000..f9bd79542044
--- /dev/null
+++ b/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
@@ -0,0 +1,122 @@
+// SPDX-License-Identifier: (GPL-2.0 OR MIT)
+/*
+ * Copyright (C) 2022 MediaTek Inc.
+ */
+
+#include "mt8195.dtsi"
+
+/ {
+ soc {
+ ssusb: ssusb@11200000 {
+ compatible ="mediatek,mt8183-mtu3", "mediatek,mtu3";
+ reg = <0 0x11201000 0 0x2dff>,
+ <0 0x11203e00 0 0x0100>;
+ reg-names = "mac", "ippc";
+ interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH 0>;
+ phys = <&u2port0 PHY_TYPE_USB2>,
+ <&u3port0 PHY_TYPE_USB3>;
+ clocks = <&infracfg_ao CLK_INFRA_AO_SSUSB>,
+ <&topckgen CLK_TOP_SSUSB_REF>,
+ <&infracfg_ao CLK_INFRA_AO_SSUSB_XHCI>;
+ clock-names = "sys_ck", "ref_ck", "mcu_ck";
+ #address-cells = <2>;
+ #size-cells = <2>;
+ ranges;
+ status = "disabled";
+
+ xhci0: xhci@11200000 {
+ compatible = "mediatek,mt8195-xhci",
+ "mediatek,mtk-xhci";
+ reg = <0 0x11200000 0 0x1000>;
+ reg-names = "mac";
+ interrupts = <GIC_SPI 129 IRQ_TYPE_LEVEL_HIGH 0>;
+ assigned-clocks = <&topckgen CLK_TOP_USB_TOP>,
+ <&topckgen CLK_TOP_SSUSB_XHCI>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
+ <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&infracfg_ao CLK_INFRA_AO_SSUSB>,
+ <&infracfg_ao CLK_INFRA_AO_SSUSB_XHCI>,
+ <&topckgen CLK_TOP_SSUSB_REF>,
+ <&apmixedsys CLK_APMIXED_USB1PLL>;
+ clock-names = "sys_ck", "xhci_ck", "ref_ck", "mcu_ck";
+ status = "disabled";
+ };
+ };
+
+ xhci1: xhci1@11290000 {
+ compatible = "mediatek,mt8195-xhci",
+ "mediatek,mtk-xhci";
+ reg = <0 0x11290000 0 0x1000>,
+ <0 0x11293e00 0 0x0100>;
+ reg-names = "mac", "ippc";
+ interrupts = <GIC_SPI 530 IRQ_TYPE_LEVEL_HIGH 0>;
+ phys = <&u2port1 PHY_TYPE_USB2>,
+ <&u3port1 PHY_TYPE_USB3>;
+ assigned-clocks = <&topckgen CLK_TOP_USB_TOP_1P>,
+ <&topckgen CLK_TOP_SSUSB_XHCI_1P>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
+ <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_1P_BUS>,
+ <&pericfg_ao CLK_PERI_AO_SSUSB_1P_XHCI>,
+ <&topckgen CLK_TOP_SSUSB_P1_REF>,
+ <&apmixedsys CLK_APMIXED_USB1PLL>;
+ clock-names = "sys_ck", "xhci_ck", "ref_ck", "mcu_ck";
+ status = "disabled";
+ };
+
+ ssusb1: usb1@112a1000 {
+ compatible = "mediatek,mt8183-mtu3", "mediatek,mtu3";
+ reg = <0 0x112a1000 0 0x2dff>,
+ <0 0x112a3e00 0 0x0100>;
+ reg-names = "mac", "ippc";
+ interrupts = <GIC_SPI 532 IRQ_TYPE_LEVEL_HIGH 0>;
+ phys = <&u2port2 PHY_TYPE_USB2>;
+ assigned-clocks = <&topckgen CLK_TOP_USB_TOP_2P>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_2P_BUS>,
+ <&topckgen CLK_TOP_SSUSB_P2_REF>,
+ <&pericfg_ao CLK_PERI_AO_SSUSB_2P_XHCI>;
+ clock-names = "sys_ck", "ref_ck", "mcu_ck";
+ mediatek,syscon-wakeup = <&pericfg 0x400 4>;
+ #address-cells = <2>;
+ #size-cells = <2>;
+ ranges;
+ status = "disabled";
+
+ xhci2: xhci2@112a0000 {
+ compatible = "mediatek,mt8195-xhci","mediatek,mtk-xhci";
+ reg = <0 0x112a0000 0 0x1000>;
+ reg-names = "mac";
+ interrupts-extended = <&gic GIC_SPI 533 IRQ_TYPE_LEVEL_HIGH 0>,
+ <&pio 220 IRQ_TYPE_EDGE_FALLING>;
+ assigned-clocks = <&topckgen CLK_TOP_SSUSB_XHCI_2P>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_2P_XHCI>;
+ clock-names = "sys_ck";
+ status = "disabled";
+ };
+ };
+
+ xhci3: xhci3@112b0000 {
+ compatible = "mediatek,mt8195-xhci",
+ "mediatek,mtk-xhci";
+ reg = <0 0x112b0000 0 0x1000>,
+ <0 0x112b3e00 0 0x0100>;
+ reg-names = "mac", "ippc";
+ interrupts = <GIC_SPI 536 IRQ_TYPE_LEVEL_HIGH 0>;
+ phys = <&u2port3 PHY_TYPE_USB2>;
+ assigned-clocks = <&topckgen CLK_TOP_USB_TOP_3P>,
+ <&topckgen CLK_TOP_SSUSB_XHCI_3P>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
+ <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_3P_BUS>,
+ <&pericfg_ao CLK_PERI_AO_SSUSB_3P_XHCI>,
+ <&topckgen CLK_TOP_SSUSB_P3_REF>;
+ clock-names = "sys_ck", "xhci_ck", "ref_ck";
+ mediatek,syscon-wakeup = <&pericfg 0x400 106>;
+ wakeup-source;
+ usb2-lpm-disable;
+ status = "disabled";
+ };
+ };
+};
diff --git a/arch/arm64/boot/dts/mediatek/mt8195-laptop-usb.dtsi b/arch/arm64/boot/dts/mediatek/mt8195-laptop-usb.dtsi
new file mode 100644
index 000000000000..5e6999bfd83c
--- /dev/null
+++ b/arch/arm64/boot/dts/mediatek/mt8195-laptop-usb.dtsi
@@ -0,0 +1,102 @@
+// SPDX-License-Identifier: (GPL-2.0 OR MIT)
+/*
+ * Copyright (C) 2022 MediaTek Inc.
+ */
+
+#include "mt8195.dtsi"
+
+/ {
+ soc {
+ xhci0: usb@11200000 {
+ compatible = "mediatek,mt8195-xhci",
+ "mediatek,mtk-xhci";
+ reg = <0 0x11200000 0 0x1000>,
+ <0 0x11203e00 0 0x0100>;
+ reg-names = "mac", "ippc";
+ interrupts = <GIC_SPI 129 IRQ_TYPE_LEVEL_HIGH 0>;
+ phys = <&u2port0 PHY_TYPE_USB2>,
+ <&u3port0 PHY_TYPE_USB3>;
+ assigned-clocks = <&topckgen CLK_TOP_USB_TOP>,
+ <&topckgen CLK_TOP_SSUSB_XHCI>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
+ <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&infracfg_ao CLK_INFRA_AO_SSUSB>,
+ <&topckgen CLK_TOP_SSUSB_REF>,
+ <&apmixedsys CLK_APMIXED_USB1PLL>,
+ <&clk26m>,
+ <&infracfg_ao CLK_INFRA_AO_SSUSB_XHCI>;
+ clock-names = "sys_ck", "ref_ck", "mcu_ck", "dma_ck",
+ "xhci_ck";
+ mediatek,syscon-wakeup = <&pericfg 0x400 103>;
+ wakeup-source;
+ status = "disabled";
+ };
+
+ xhci1: usb@11290000 {
+ compatible = "mediatek,mt8195-xhci",
+ "mediatek,mtk-xhci";
+ reg = <0 0x11290000 0 0x1000>,
+ <0 0x11293e00 0 0x0100>;
+ reg-names = "mac", "ippc";
+ interrupts = <GIC_SPI 530 IRQ_TYPE_LEVEL_HIGH 0>;
+ phys = <&u2port1 PHY_TYPE_USB2>;
+ assigned-clocks = <&topckgen CLK_TOP_USB_TOP_1P>,
+ <&topckgen CLK_TOP_SSUSB_XHCI_1P>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
+ <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_1P_BUS>,
+ <&topckgen CLK_TOP_SSUSB_P1_REF>,
+ <&apmixedsys CLK_APMIXED_USB1PLL>,
+ <&clk26m>,
+ <&pericfg_ao CLK_PERI_AO_SSUSB_1P_XHCI>;
+ clock-names = "sys_ck", "ref_ck", "mcu_ck", "dma_ck",
+ "xhci_ck";
+ mediatek,syscon-wakeup = <&pericfg 0x400 104>;
+ wakeup-source;
+ status = "disabled";
+ };
+
+ xhci2: usb@112a0000 {
+ compatible = "mediatek,mt8195-xhci",
+ "mediatek,mtk-xhci";
+ reg = <0 0x112a0000 0 0x1000>,
+ <0 0x112a3e00 0 0x0100>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
+ <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_2P_BUS>,
+ <&topckgen CLK_TOP_SSUSB_P2_REF>,
+ <&clk26m>,
+ <&clk26m>,
+ <&pericfg_ao CLK_PERI_AO_SSUSB_2P_XHCI>;
+ clock-names = "sys_ck", "ref_ck", "mcu_ck", "dma_ck",
+ "xhci_ck";
+ mediatek,syscon-wakeup = <&pericfg 0x400 105>;
+ wakeup-source;
+ status = "disabled";
+ };
+
+ xhci3: usb@112b0000 {
+ compatible = "mediatek,mt8195-xhci",
+ "mediatek,mtk-xhci";
+ reg = <0 0x112b0000 0 0x1000>,
+ <0 0x112b3e00 0 0x0100>;
+ reg-names = "mac", "ippc";
+ interrupts = <GIC_SPI 536 IRQ_TYPE_LEVEL_HIGH 0>;
+ phys = <&u2port3 PHY_TYPE_USB2>;
+ assigned-clocks = <&topckgen CLK_TOP_USB_TOP_3P>,
+ <&topckgen CLK_TOP_SSUSB_XHCI_3P>;
+ assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
+ <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
+ clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_3P_BUS>,
+ <&topckgen CLK_TOP_SSUSB_P3_REF>,
+ <&clk26m>,
+ <&clk26m>,
+ <&pericfg_ao CLK_PERI_AO_SSUSB_3P_XHCI>;
+ clock-names = "sys_ck", "ref_ck", "mcu_ck", "dma_ck",
+ "xhci_ck";
+ mediatek,syscon-wakeup = <&pericfg 0x400 106>;
+ wakeup-source;
+ status = "disabled";
+ };
+ };
+};
--
2.18.0
^ permalink raw reply related [flat|nested] 14+ messages in thread* [PATCH 2/3] arm64: dts: mediatek: switch MT8195 boards into different USB configurations
2023-01-05 9:28 [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Macpaul Lin
@ 2023-01-05 9:28 ` Macpaul Lin
2023-01-05 9:28 ` [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board Macpaul Lin
` (2 subsequent siblings)
3 siblings, 0 replies; 14+ messages in thread
From: Macpaul Lin @ 2023-01-05 9:28 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Matthias Brugger,
linux-arm-kernel, linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Macpaul Lin, Miles Chen,
linux-usb, Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai
This modification includes
1. Remove common USB configurations in mt8195.dtsi.
2. Switch include dtsi "mt8195-laptop-usb.dtsi" for LAPTOP devices.
- mt8195-cherry.dtsi
- mt8195-evb.dts
3. Switch include dtsi "mt8195-iot-usb.dtsi" for IOT devices.
- mt8195-demo.dts
Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
---
.../boot/dts/mediatek/mt8195-cherry.dtsi | 2 +-
arch/arm64/boot/dts/mediatek/mt8195-demo.dts | 2 +-
arch/arm64/boot/dts/mediatek/mt8195-evb.dts | 2 +-
arch/arm64/boot/dts/mediatek/mt8195.dtsi | 97 -------------------
4 files changed, 3 insertions(+), 100 deletions(-)
diff --git a/arch/arm64/boot/dts/mediatek/mt8195-cherry.dtsi b/arch/arm64/boot/dts/mediatek/mt8195-cherry.dtsi
index 560103e29017..a28da16d1d74 100644
--- a/arch/arm64/boot/dts/mediatek/mt8195-cherry.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8195-cherry.dtsi
@@ -5,7 +5,7 @@
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/spmi/spmi.h>
-#include "mt8195.dtsi"
+#include "mt8195-laptop-usb.dtsi"
#include "mt6359.dtsi"
/ {
diff --git a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
index dec85d254838..07a864cb8b54 100644
--- a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
+++ b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
@@ -5,7 +5,7 @@
*/
/dts-v1/;
-#include "mt8195.dtsi"
+#include "mt8195-iot-usb.dtsi"
#include "mt6359.dtsi"
#include <dt-bindings/gpio/gpio.h>
diff --git a/arch/arm64/boot/dts/mediatek/mt8195-evb.dts b/arch/arm64/boot/dts/mediatek/mt8195-evb.dts
index 690dc7717f2c..c7026c0f2ca9 100644
--- a/arch/arm64/boot/dts/mediatek/mt8195-evb.dts
+++ b/arch/arm64/boot/dts/mediatek/mt8195-evb.dts
@@ -4,7 +4,7 @@
* Author: Seiya Wang <seiya.wang@mediatek.com>
*/
/dts-v1/;
-#include "mt8195.dtsi"
+#include "mt8195-laptop-usb.dtsi"
/ {
model = "MediaTek MT8195 evaluation board";
diff --git a/arch/arm64/boot/dts/mediatek/mt8195.dtsi b/arch/arm64/boot/dts/mediatek/mt8195.dtsi
index 5d31536f4c48..ac953a6e3ca7 100644
--- a/arch/arm64/boot/dts/mediatek/mt8195.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8195.dtsi
@@ -1046,31 +1046,6 @@
status = "disabled";
};
- xhci0: usb@11200000 {
- compatible = "mediatek,mt8195-xhci",
- "mediatek,mtk-xhci";
- reg = <0 0x11200000 0 0x1000>,
- <0 0x11203e00 0 0x0100>;
- reg-names = "mac", "ippc";
- interrupts = <GIC_SPI 129 IRQ_TYPE_LEVEL_HIGH 0>;
- phys = <&u2port0 PHY_TYPE_USB2>,
- <&u3port0 PHY_TYPE_USB3>;
- assigned-clocks = <&topckgen CLK_TOP_USB_TOP>,
- <&topckgen CLK_TOP_SSUSB_XHCI>;
- assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
- <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
- clocks = <&infracfg_ao CLK_INFRA_AO_SSUSB>,
- <&topckgen CLK_TOP_SSUSB_REF>,
- <&apmixedsys CLK_APMIXED_USB1PLL>,
- <&clk26m>,
- <&infracfg_ao CLK_INFRA_AO_SSUSB_XHCI>;
- clock-names = "sys_ck", "ref_ck", "mcu_ck", "dma_ck",
- "xhci_ck";
- mediatek,syscon-wakeup = <&pericfg 0x400 103>;
- wakeup-source;
- status = "disabled";
- };
-
mmc0: mmc@11230000 {
compatible = "mediatek,mt8195-mmc",
"mediatek,mt8183-mmc";
@@ -1114,78 +1089,6 @@
status = "disabled";
};
- xhci1: usb@11290000 {
- compatible = "mediatek,mt8195-xhci",
- "mediatek,mtk-xhci";
- reg = <0 0x11290000 0 0x1000>,
- <0 0x11293e00 0 0x0100>;
- reg-names = "mac", "ippc";
- interrupts = <GIC_SPI 530 IRQ_TYPE_LEVEL_HIGH 0>;
- phys = <&u2port1 PHY_TYPE_USB2>;
- assigned-clocks = <&topckgen CLK_TOP_USB_TOP_1P>,
- <&topckgen CLK_TOP_SSUSB_XHCI_1P>;
- assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
- <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
- clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_1P_BUS>,
- <&topckgen CLK_TOP_SSUSB_P1_REF>,
- <&apmixedsys CLK_APMIXED_USB1PLL>,
- <&clk26m>,
- <&pericfg_ao CLK_PERI_AO_SSUSB_1P_XHCI>;
- clock-names = "sys_ck", "ref_ck", "mcu_ck", "dma_ck",
- "xhci_ck";
- mediatek,syscon-wakeup = <&pericfg 0x400 104>;
- wakeup-source;
- status = "disabled";
- };
-
- xhci2: usb@112a0000 {
- compatible = "mediatek,mt8195-xhci",
- "mediatek,mtk-xhci";
- reg = <0 0x112a0000 0 0x1000>,
- <0 0x112a3e00 0 0x0100>;
- reg-names = "mac", "ippc";
- interrupts = <GIC_SPI 533 IRQ_TYPE_LEVEL_HIGH 0>;
- phys = <&u2port2 PHY_TYPE_USB2>;
- assigned-clocks = <&topckgen CLK_TOP_USB_TOP_2P>,
- <&topckgen CLK_TOP_SSUSB_XHCI_2P>;
- assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
- <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
- clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_2P_BUS>,
- <&topckgen CLK_TOP_SSUSB_P2_REF>,
- <&clk26m>,
- <&clk26m>,
- <&pericfg_ao CLK_PERI_AO_SSUSB_2P_XHCI>;
- clock-names = "sys_ck", "ref_ck", "mcu_ck", "dma_ck",
- "xhci_ck";
- mediatek,syscon-wakeup = <&pericfg 0x400 105>;
- wakeup-source;
- status = "disabled";
- };
-
- xhci3: usb@112b0000 {
- compatible = "mediatek,mt8195-xhci",
- "mediatek,mtk-xhci";
- reg = <0 0x112b0000 0 0x1000>,
- <0 0x112b3e00 0 0x0100>;
- reg-names = "mac", "ippc";
- interrupts = <GIC_SPI 536 IRQ_TYPE_LEVEL_HIGH 0>;
- phys = <&u2port3 PHY_TYPE_USB2>;
- assigned-clocks = <&topckgen CLK_TOP_USB_TOP_3P>,
- <&topckgen CLK_TOP_SSUSB_XHCI_3P>;
- assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
- <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
- clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_3P_BUS>,
- <&topckgen CLK_TOP_SSUSB_P3_REF>,
- <&clk26m>,
- <&clk26m>,
- <&pericfg_ao CLK_PERI_AO_SSUSB_3P_XHCI>;
- clock-names = "sys_ck", "ref_ck", "mcu_ck", "dma_ck",
- "xhci_ck";
- mediatek,syscon-wakeup = <&pericfg 0x400 106>;
- wakeup-source;
- status = "disabled";
- };
-
pcie0: pcie@112f0000 {
compatible = "mediatek,mt8195-pcie",
"mediatek,mt8192-pcie";
--
2.18.0
^ permalink raw reply related [flat|nested] 14+ messages in thread* [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board
2023-01-05 9:28 [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Macpaul Lin
2023-01-05 9:28 ` [PATCH 2/3] arm64: dts: mediatek: switch MT8195 boards into different " Macpaul Lin
@ 2023-01-05 9:28 ` Macpaul Lin
2023-01-06 12:56 ` Krzysztof Kozlowski
2023-02-01 12:24 ` Matthias Brugger
2023-01-06 12:56 ` [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Krzysztof Kozlowski
2023-01-09 15:13 ` AngeloGioacchino Del Regno
3 siblings, 2 replies; 14+ messages in thread
From: Macpaul Lin @ 2023-01-05 9:28 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Matthias Brugger,
linux-arm-kernel, linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Macpaul Lin, Miles Chen,
linux-usb, Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai
1. Enable USB device port (USB port0).
2. Enable u2 speed for other USB HOST (USB port1/2/3).
3. Configure mt6360 (TYPEC) related pinctrls.
Note: Full dual-role switch capability requires TYPEC MUX driver and dts
update will be send in separate patches.
Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
---
arch/arm64/boot/dts/mediatek/mt8195-demo.dts | 51 ++++++++++++++++++++
1 file changed, 51 insertions(+)
diff --git a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
index 07a864cb8b54..207bb5f4c58f 100644
--- a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
+++ b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
@@ -12,6 +12,7 @@
#include <dt-bindings/input/input.h>
#include <dt-bindings/pinctrl/mt8195-pinfunc.h>
#include <dt-bindings/regulator/mediatek,mt6360-regulator.h>
+#include <dt-bindings/usb/pd.h>
/ {
model = "MediaTek MT8195 demo board";
@@ -380,6 +381,22 @@
};
};
+ mt6360_pins: mt6360-pins {
+ pins {
+ pinmux = <PINMUX_GPIO100__FUNC_GPIO100>,
+ <PINMUX_GPIO101__FUNC_GPIO101>;
+ input-enable;
+ bias-pull-up;
+ };
+ };
+
+ u3_p0_vbus: u3_p0vbusdefault {
+ pins_cmd_dat {
+ pinmux = <PINMUX_GPIO63__FUNC_VBUSVALID>;
+ input-enable;
+ };
+ };
+
uart0_pins: uart0-pins {
pins {
pinmux = <PINMUX_GPIO98__FUNC_UTXD0>,
@@ -393,6 +410,14 @@
<PINMUX_GPIO103__FUNC_URXD1>;
};
};
+
+ usb_otg_vbus: usb-otg-vbus-regulator {
+ compatible = "regulator-fixed";
+ regulator-name = "otg-vbus";
+ regulator-min-microvolt = <5000000>;
+ regulator-max-microvolt = <5000000>;
+ enable-active-high;
+ };
};
@@ -412,6 +437,22 @@
status = "okay";
};
+&u3port0 {
+ status = "okay";
+};
+
+&u2port1 {
+ status = "okay";
+};
+
+&u2port2 {
+ status = "okay";
+};
+
+&u2port3 {
+ status = "okay";
+};
+
&u3phy0 {
status = "okay";
};
@@ -428,6 +469,16 @@
status = "okay";
};
+&ssusb {
+ pinctrl-names = "default";
+ pinctrl-0 = <&u3_p0_vbus>;
+ vusb33-supply = <&mt6359_vusb_ldo_reg>;
+ dr_mode = "otg";
+ mediatek,usb3-drd;
+ usb-role-switch;
+ status = "okay";
+};
+
&xhci0 {
vusb33-supply = <&mt6359_vusb_ldo_reg>;
vbus-supply = <&otg_vbus_regulator>;
--
2.18.0
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board
2023-01-05 9:28 ` [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board Macpaul Lin
@ 2023-01-06 12:56 ` Krzysztof Kozlowski
2023-01-11 5:41 ` Macpaul Lin
2023-02-01 12:24 ` Matthias Brugger
1 sibling, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-06 12:56 UTC (permalink / raw)
To: Macpaul Lin, Rob Herring, Krzysztof Kozlowski, Matthias Brugger,
linux-arm-kernel, linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Miles Chen, linux-usb,
Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai
On 05/01/2023 10:28, Macpaul Lin wrote:
> 1. Enable USB device port (USB port0).
> 2. Enable u2 speed for other USB HOST (USB port1/2/3).
> 3. Configure mt6360 (TYPEC) related pinctrls.
>
> Note: Full dual-role switch capability requires TYPEC MUX driver and dts
> update will be send in separate patches.
>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> ---
> arch/arm64/boot/dts/mediatek/mt8195-demo.dts | 51 ++++++++++++++++++++
> 1 file changed, 51 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> index 07a864cb8b54..207bb5f4c58f 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> @@ -12,6 +12,7 @@
> #include <dt-bindings/input/input.h>
> #include <dt-bindings/pinctrl/mt8195-pinfunc.h>
> #include <dt-bindings/regulator/mediatek,mt6360-regulator.h>
> +#include <dt-bindings/usb/pd.h>
>
> / {
> model = "MediaTek MT8195 demo board";
> @@ -380,6 +381,22 @@
> };
> };
>
> + mt6360_pins: mt6360-pins {
> + pins {
> + pinmux = <PINMUX_GPIO100__FUNC_GPIO100>,
> + <PINMUX_GPIO101__FUNC_GPIO101>;
> + input-enable;
> + bias-pull-up;
> + };
> + };
> +
> + u3_p0_vbus: u3_p0vbusdefault {
No underscores in node names. Also does not look correct naming. Please
run `make dtbs_check` (see
Documentation/devicetree/bindings/writing-schema.rst for instructions).
> + pins_cmd_dat {
Same problem.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board
2023-01-06 12:56 ` Krzysztof Kozlowski
@ 2023-01-11 5:41 ` Macpaul Lin
0 siblings, 0 replies; 14+ messages in thread
From: Macpaul Lin @ 2023-01-11 5:41 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski,
Matthias Brugger, linux-arm-kernel, linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Miles Chen, linux-usb,
Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai,
yow-shin.liou, ben.lok
On 1/6/23 20:56, Krzysztof Kozlowski wrote:
> On 05/01/2023 10:28, Macpaul Lin wrote:
>> 1. Enable USB device port (USB port0).
>> 2. Enable u2 speed for other USB HOST (USB port1/2/3).
>> 3. Configure mt6360 (TYPEC) related pinctrls.
>>
>> Note: Full dual-role switch capability requires TYPEC MUX driver and dts
>> update will be send in separate patches.
>>
>> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
>> ---
>> arch/arm64/boot/dts/mediatek/mt8195-demo.dts | 51 ++++++++++++++++++++
>> 1 file changed, 51 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
>> index 07a864cb8b54..207bb5f4c58f 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
>> @@ -12,6 +12,7 @@
>> #include <dt-bindings/input/input.h>
>> #include <dt-bindings/pinctrl/mt8195-pinfunc.h>
>> #include <dt-bindings/regulator/mediatek,mt6360-regulator.h>
>> +#include <dt-bindings/usb/pd.h>
>>
>> / {
>> model = "MediaTek MT8195 demo board";
>> @@ -380,6 +381,22 @@
>> };
>> };
>>
>> + mt6360_pins: mt6360-pins {
>> + pins {
>> + pinmux = <PINMUX_GPIO100__FUNC_GPIO100>,
>> + <PINMUX_GPIO101__FUNC_GPIO101>;
>> + input-enable;
>> + bias-pull-up;
>> + };
>> + };
>> +
>> + u3_p0_vbus: u3_p0vbusdefault {
>
> No underscores in node names. Also does not look correct naming. Please
> run `make dtbs_check` (see
> Documentation/devicetree/bindings/writing-schema.rst for instructions).
Thanks for the suggest.
I'm trying to request IT to setup for the environment for supporting
"make dtbs_check". This should be fix in next patch.
>> + pins_cmd_dat {
>
> Same problem.
>
Got it.
>
> Best regards,
> Krzysztof
>
Thanks!
Macpaul Lin
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board
2023-01-05 9:28 ` [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board Macpaul Lin
2023-01-06 12:56 ` Krzysztof Kozlowski
@ 2023-02-01 12:24 ` Matthias Brugger
2023-02-07 9:36 ` Chunfeng Yun (云春峰)
1 sibling, 1 reply; 14+ messages in thread
From: Matthias Brugger @ 2023-02-01 12:24 UTC (permalink / raw)
To: Macpaul Lin, Rob Herring, Krzysztof Kozlowski, linux-arm-kernel,
linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Miles Chen, linux-usb,
Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai
On 05/01/2023 10:28, Macpaul Lin wrote:
> 1. Enable USB device port (USB port0).
> 2. Enable u2 speed for other USB HOST (USB port1/2/3).
> 3. Configure mt6360 (TYPEC) related pinctrls.
>
> Note: Full dual-role switch capability requires TYPEC MUX driver and dts
> update will be send in separate patches.
>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> ---
> arch/arm64/boot/dts/mediatek/mt8195-demo.dts | 51 ++++++++++++++++++++
> 1 file changed, 51 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> index 07a864cb8b54..207bb5f4c58f 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> @@ -12,6 +12,7 @@
> #include <dt-bindings/input/input.h>
> #include <dt-bindings/pinctrl/mt8195-pinfunc.h>
> #include <dt-bindings/regulator/mediatek,mt6360-regulator.h>
> +#include <dt-bindings/usb/pd.h>
>
> / {
> model = "MediaTek MT8195 demo board";
> @@ -380,6 +381,22 @@
> };
> };
>
> + mt6360_pins: mt6360-pins {
> + pins {
> + pinmux = <PINMUX_GPIO100__FUNC_GPIO100>,
> + <PINMUX_GPIO101__FUNC_GPIO101>;
> + input-enable;
> + bias-pull-up;
> + };
> + };
> +
> + u3_p0_vbus: u3_p0vbusdefault {
> + pins_cmd_dat {
> + pinmux = <PINMUX_GPIO63__FUNC_VBUSVALID>;
> + input-enable;
> + };
> + };
> +
> uart0_pins: uart0-pins {
> pins {
> pinmux = <PINMUX_GPIO98__FUNC_UTXD0>,
> @@ -393,6 +410,14 @@
> <PINMUX_GPIO103__FUNC_URXD1>;
> };
> };
> +
> + usb_otg_vbus: usb-otg-vbus-regulator {
> + compatible = "regulator-fixed";
> + regulator-name = "otg-vbus";
> + regulator-min-microvolt = <5000000>;
> + regulator-max-microvolt = <5000000>;
> + enable-active-high;
> + };
> };
>
>
> @@ -412,6 +437,22 @@
> status = "okay";
> };
>
> +&u3port0 {
> + status = "okay";
> +};
> +
> +&u2port1 {
> + status = "okay";
> +};
> +
> +&u2port2 {
> + status = "okay";
> +};
> +
> +&u2port3 {
> + status = "okay";
> +};
> +
> &u3phy0 {
> status = "okay";
> };
> @@ -428,6 +469,16 @@
> status = "okay";
> };
>
> +&ssusb {
> + pinctrl-names = "default";
> + pinctrl-0 = <&u3_p0_vbus>;
> + vusb33-supply = <&mt6359_vusb_ldo_reg>;
> + dr_mode = "otg";
> + mediatek,usb3-drd;
What it that used for?
git grep didn't gave me any hit.
Regards,
Matthias
> + usb-role-switch;
> + status = "okay";
> +};
> +
> &xhci0 {
> vusb33-supply = <&mt6359_vusb_ldo_reg>;
> vbus-supply = <&otg_vbus_regulator>;
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board
2023-02-01 12:24 ` Matthias Brugger
@ 2023-02-07 9:36 ` Chunfeng Yun (云春峰)
0 siblings, 0 replies; 14+ messages in thread
From: Chunfeng Yun (云春峰) @ 2023-02-07 9:36 UTC (permalink / raw)
To: linux-arm-kernel@lists.infradead.org, matthias.bgg@gmail.com,
linux-mediatek@lists.infradead.org,
Macpaul Lin (林智斌), robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, devicetree@vger.kernel.org
Cc: Pablo Sun (孫毓翔), linux-usb@vger.kernel.org,
Bear Wang (萩原惟德), wenst@chromium.org,
macpaul@gmail.com, Miles Chen (陳民樺),
Rex-BC Chen (陳柏辰),
TingHan Shen (沈廷翰)
On Wed, 2023-02-01 at 13:24 +0100, Matthias Brugger wrote:
>
> On 05/01/2023 10:28, Macpaul Lin wrote:
> > 1. Enable USB device port (USB port0).
> > 2. Enable u2 speed for other USB HOST (USB port1/2/3).
> > 3. Configure mt6360 (TYPEC) related pinctrls.
> >
> > Note: Full dual-role switch capability requires TYPEC MUX driver
> > and dts
> > update will be send in separate patches.
> >
> > Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> > ---
> > arch/arm64/boot/dts/mediatek/mt8195-demo.dts | 51
> > ++++++++++++++++++++
> > 1 file changed, 51 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> > b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> > index 07a864cb8b54..207bb5f4c58f 100644
> > --- a/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> > +++ b/arch/arm64/boot/dts/mediatek/mt8195-demo.dts
> > @@ -12,6 +12,7 @@
> > #include <dt-bindings/input/input.h>
> > #include <dt-bindings/pinctrl/mt8195-pinfunc.h>
> > #include <dt-bindings/regulator/mediatek,mt6360-regulator.h>
> > +#include <dt-bindings/usb/pd.h>
> >
> > / {
> > model = "MediaTek MT8195 demo board";
> > @@ -380,6 +381,22 @@
> > };
> > };
> >
> > + mt6360_pins: mt6360-pins {
> > + pins {
> > + pinmux = <PINMUX_GPIO100__FUNC_GPIO100>,
> > + <PINMUX_GPIO101__FUNC_GPIO101>;
> > + input-enable;
> > + bias-pull-up;
> > + };
> > + };
> > +
> > + u3_p0_vbus: u3_p0vbusdefault {
> > + pins_cmd_dat {
> > + pinmux = <PINMUX_GPIO63__FUNC_VBUSVALID>;
> > + input-enable;
> > + };
> > + };
> > +
> > uart0_pins: uart0-pins {
> > pins {
> > pinmux = <PINMUX_GPIO98__FUNC_UTXD0>,
> > @@ -393,6 +410,14 @@
> > <PINMUX_GPIO103__FUNC_URXD1>;
> > };
> > };
> > +
> > + usb_otg_vbus: usb-otg-vbus-regulator {
> > + compatible = "regulator-fixed";
> > + regulator-name = "otg-vbus";
> > + regulator-min-microvolt = <5000000>;
> > + regulator-max-microvolt = <5000000>;
> > + enable-active-high;
> > + };
> > };
> >
> >
> > @@ -412,6 +437,22 @@
> > status = "okay";
> > };
> >
> > +&u3port0 {
> > + status = "okay";
> > +};
> > +
> > +&u2port1 {
> > + status = "okay";
> > +};
> > +
> > +&u2port2 {
> > + status = "okay";
> > +};
> > +
> > +&u2port3 {
> > + status = "okay";
> > +};
> > +
> > &u3phy0 {
> > status = "okay";
> > };
> > @@ -428,6 +469,16 @@
> > status = "okay";
> > };
> >
> > +&ssusb {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&u3_p0_vbus>;
> > + vusb33-supply = <&mt6359_vusb_ldo_reg>;
> > + dr_mode = "otg";
> > + mediatek,usb3-drd;
>
> What it that used for?
> git grep didn't gave me any hit.
"mediatek,usb3-drd;" can be now removed,
the patch sent out on Jul 8 2022:
"683ff6e485f1 usb: mtu3: check capability of usb3 dual role"
already removed the property.
Thanks a lot
>
> Regards,
> Matthias
>
> > + usb-role-switch;
> > + status = "okay";
> > +};
> > +
> > &xhci0 {
> > vusb33-supply = <&mt6359_vusb_ldo_reg>;
> > vbus-supply = <&otg_vbus_regulator>;
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations
2023-01-05 9:28 [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Macpaul Lin
2023-01-05 9:28 ` [PATCH 2/3] arm64: dts: mediatek: switch MT8195 boards into different " Macpaul Lin
2023-01-05 9:28 ` [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board Macpaul Lin
@ 2023-01-06 12:56 ` Krzysztof Kozlowski
2023-01-11 2:19 ` Chunfeng Yun (云春峰)
2023-01-09 15:13 ` AngeloGioacchino Del Regno
3 siblings, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2023-01-06 12:56 UTC (permalink / raw)
To: Macpaul Lin, Rob Herring, Krzysztof Kozlowski, Matthias Brugger,
linux-arm-kernel, linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Miles Chen, linux-usb,
Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai
On 05/01/2023 10:28, Macpaul Lin wrote:
> Introduce the split MT8195 laptop and iot USB configurations.
> The hardware specifications for LAPTOP devices is different from IOT
> devices. The major differences include some hardware constrains for
> dual-role switch for USB controllers in different configurations,
> especially for power management and other control flows as well.
>
> Here are some hardware specifiction differences listed:
> 1. LAPTOP (Cherry Tomato boards) don't support USB gadget (device mode).
> 2. IOT devices must support multiple gadget devices and host mode.
> 3. Dual-role switch is not fully supported. Only USB PORT0 support
> dual-role switch.
> 4. Power management is designed in primary and secondary dominator.
> For a dual-role port, the device controller is the primary controller
> for power management; while the host controller is the secondary.
> LAPTOP devices should remove device nodes for avoiding abnormal
> behavior.
>
> This modifcation is to add USB configurations "mt8195-laptop-usb.dtsi"
> for LAPTOP devices, and add "mt8195-iot-usb.dtsi" for IOT devices.
>
> To remove common USB configurations for mt8195.dtsi and switch includes
> dtsi these new files for the boards will come in next patch.
>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> ---
> .../boot/dts/mediatek/mt8195-iot-usb.dtsi | 122 ++++++++++++++++++
> .../boot/dts/mediatek/mt8195-laptop-usb.dtsi | 102 +++++++++++++++
> 2 files changed, 224 insertions(+)
> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-laptop-usb.dtsi
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi b/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
> new file mode 100644
> index 000000000000..f9bd79542044
> --- /dev/null
> +++ b/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
> @@ -0,0 +1,122 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +/*
> + * Copyright (C) 2022 MediaTek Inc.
> + */
> +
> +#include "mt8195.dtsi"
> +
> +/ {
> + soc {
> + ssusb: ssusb@11200000 {
Node name: usb
> + compatible ="mediatek,mt8183-mtu3", "mediatek,mtu3";
> + reg = <0 0x11201000 0 0x2dff>,
> + <0 0x11203e00 0 0x0100>;
> + reg-names = "mac", "ippc";
> + interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH 0>;
> + phys = <&u2port0 PHY_TYPE_USB2>,
> + <&u3port0 PHY_TYPE_USB3>;
> + clocks = <&infracfg_ao CLK_INFRA_AO_SSUSB>,
> + <&topckgen CLK_TOP_SSUSB_REF>,
> + <&infracfg_ao CLK_INFRA_AO_SSUSB_XHCI>;
> + clock-names = "sys_ck", "ref_ck", "mcu_ck";
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> + status = "disabled";
> +
> + xhci0: xhci@11200000 {
> + compatible = "mediatek,mt8195-xhci",
> + "mediatek,mtk-xhci";
> + reg = <0 0x11200000 0 0x1000>;
> + reg-names = "mac";
> + interrupts = <GIC_SPI 129 IRQ_TYPE_LEVEL_HIGH 0>;
> + assigned-clocks = <&topckgen CLK_TOP_USB_TOP>,
> + <&topckgen CLK_TOP_SSUSB_XHCI>;
> + assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
> + <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
> + clocks = <&infracfg_ao CLK_INFRA_AO_SSUSB>,
> + <&infracfg_ao CLK_INFRA_AO_SSUSB_XHCI>,
> + <&topckgen CLK_TOP_SSUSB_REF>,
> + <&apmixedsys CLK_APMIXED_USB1PLL>;
> + clock-names = "sys_ck", "xhci_ck", "ref_ck", "mcu_ck";
> + status = "disabled";
> + };
> + };
> +
> + xhci1: xhci1@11290000 {
Node names should be generic, so just "xhci"
https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation
> + compatible = "mediatek,mt8195-xhci",
> + "mediatek,mtk-xhci";
> + reg = <0 0x11290000 0 0x1000>,
> + <0 0x11293e00 0 0x0100>;
> + reg-names = "mac", "ippc";
> + interrupts = <GIC_SPI 530 IRQ_TYPE_LEVEL_HIGH 0>;
> + phys = <&u2port1 PHY_TYPE_USB2>,
> + <&u3port1 PHY_TYPE_USB3>;
> + assigned-clocks = <&topckgen CLK_TOP_USB_TOP_1P>,
> + <&topckgen CLK_TOP_SSUSB_XHCI_1P>;
> + assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
> + <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
> + clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_1P_BUS>,
> + <&pericfg_ao CLK_PERI_AO_SSUSB_1P_XHCI>,
> + <&topckgen CLK_TOP_SSUSB_P1_REF>,
> + <&apmixedsys CLK_APMIXED_USB1PLL>;
> + clock-names = "sys_ck", "xhci_ck", "ref_ck", "mcu_ck";
> + status = "disabled";
> + };
> +
> + ssusb1: usb1@112a1000 {
usb
> + compatible = "mediatek,mt8183-mtu3", "mediatek,mtu3";
> + reg = <0 0x112a1000 0 0x2dff>,
> + <0 0x112a3e00 0 0x0100>;
> + reg-names = "mac", "ippc";
> + interrupts = <GIC_SPI 532 IRQ_TYPE_LEVEL_HIGH 0>;
> + phys = <&u2port2 PHY_TYPE_USB2>;
> + assigned-clocks = <&topckgen CLK_TOP_USB_TOP_2P>;
> + assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
> + clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_2P_BUS>,
> + <&topckgen CLK_TOP_SSUSB_P2_REF>,
> + <&pericfg_ao CLK_PERI_AO_SSUSB_2P_XHCI>;
> + clock-names = "sys_ck", "ref_ck", "mcu_ck";
> + mediatek,syscon-wakeup = <&pericfg 0x400 4>;
> + #address-cells = <2>;
> + #size-cells = <2>;
> + ranges;
> + status = "disabled";
> +
> + xhci2: xhci2@112a0000 {
xhci
> + compatible = "mediatek,mt8195-xhci","mediatek,mtk-xhci";
> + reg = <0 0x112a0000 0 0x1000>;
> + reg-names = "mac";
> + interrupts-extended = <&gic GIC_SPI 533 IRQ_TYPE_LEVEL_HIGH 0>,
> + <&pio 220 IRQ_TYPE_EDGE_FALLING>;
> + assigned-clocks = <&topckgen CLK_TOP_SSUSB_XHCI_2P>;
> + assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
> + clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_2P_XHCI>;
> + clock-names = "sys_ck";
> + status = "disabled";
> + };
> + };
> +
> + xhci3: xhci3@112b0000 {
xhci
> + compatible = "mediatek,mt8195-xhci",
> + "mediatek,mtk-xhci";
> + reg = <0 0x112b0000 0 0x1000>,
> + <0 0x112b3e00 0 0x0100>;
> + reg-names = "mac", "ippc";
> + interrupts = <GIC_SPI 536 IRQ_TYPE_LEVEL_HIGH 0>;
> + phys = <&u2port3 PHY_TYPE_USB2>;
> + assigned-clocks = <&topckgen CLK_TOP_USB_TOP_3P>,
> + <&topckgen CLK_TOP_SSUSB_XHCI_3P>;
> + assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D5_D4>,
> + <&topckgen CLK_TOP_UNIVPLL_D5_D4>;
> + clocks = <&pericfg_ao CLK_PERI_AO_SSUSB_3P_BUS>,
> + <&pericfg_ao CLK_PERI_AO_SSUSB_3P_XHCI>,
> + <&topckgen CLK_TOP_SSUSB_P3_REF>;
> + clock-names = "sys_ck", "xhci_ck", "ref_ck";
> + mediatek,syscon-wakeup = <&pericfg 0x400 106>;
> + wakeup-source;
> + usb2-lpm-disable;
> + status = "disabled";
> + };
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations
2023-01-06 12:56 ` [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Krzysztof Kozlowski
@ 2023-01-11 2:19 ` Chunfeng Yun (云春峰)
2023-01-11 3:53 ` Macpaul Lin
0 siblings, 1 reply; 14+ messages in thread
From: Chunfeng Yun (云春峰) @ 2023-01-11 2:19 UTC (permalink / raw)
To: linux-mediatek@lists.infradead.org, robh+dt@kernel.org,
devicetree@vger.kernel.org, krzysztof.kozlowski@linaro.org,
Macpaul Lin (林智斌),
linux-arm-kernel@lists.infradead.org,
krzysztof.kozlowski+dt@linaro.org, matthias.bgg@gmail.com
Cc: Pablo Sun (孫毓翔), linux-usb@vger.kernel.org,
Bear Wang (萩原惟德), wenst@chromium.org,
macpaul@gmail.com, Miles Chen (陳民樺),
Rex-BC Chen (陳柏辰),
TingHan Shen (沈廷翰)
On Fri, 2023-01-06 at 13:56 +0100, Krzysztof Kozlowski wrote:
> On 05/01/2023 10:28, Macpaul Lin wrote:
> > Introduce the split MT8195 laptop and iot USB configurations.
> > The hardware specifications for LAPTOP devices is different from
> > IOT
> > devices. The major differences include some hardware constrains for
> > dual-role switch for USB controllers in different configurations,
> > especially for power management and other control flows as well.
> >
> > Here are some hardware specifiction differences listed:
> > 1. LAPTOP (Cherry Tomato boards) don't support USB gadget (device
> > mode).
> > 2. IOT devices must support multiple gadget devices and host
> > mode.
> > 3. Dual-role switch is not fully supported. Only USB PORT0
> > support
> > dual-role switch.
> > 4. Power management is designed in primary and secondary
> > dominator.
> > For a dual-role port, the device controller is the primary
> > controller
> > for power management; while the host controller is the
> > secondary.
> > LAPTOP devices should remove device nodes for avoiding
> > abnormal
> > behavior.
> >
> > This modifcation is to add USB configurations "mt8195-laptop-
> > usb.dtsi"
> > for LAPTOP devices, and add "mt8195-iot-usb.dtsi" for IOT devices.
> >
> > To remove common USB configurations for mt8195.dtsi and switch
> > includes
> > dtsi these new files for the boards will come in next patch.
> >
> > Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> > ---
> > .../boot/dts/mediatek/mt8195-iot-usb.dtsi | 122
> > ++++++++++++++++++
> > .../boot/dts/mediatek/mt8195-laptop-usb.dtsi | 102
> > +++++++++++++++
> > 2 files changed, 224 insertions(+)
> > create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-iot-
> > usb.dtsi
> > create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-laptop-
> > usb.dtsi
> >
> > diff --git a/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
> > b/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
> > new file mode 100644
> > index 000000000000..f9bd79542044
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
> > @@ -0,0 +1,122 @@
> > +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> > +/*
> > + * Copyright (C) 2022 MediaTek Inc.
> > + */
> > +
> > +#include "mt8195.dtsi"
> > +
> > +/ {
> > + soc {
> > + ssusb: ssusb@11200000 {
>
> Node name: usb
>
> > + compatible ="mediatek,mt8183-mtu3",
> > "mediatek,mtu3";
> > + reg = <0 0x11201000 0 0x2dff>,
> > + <0 0x11203e00 0 0x0100>;
> > + reg-names = "mac", "ippc";
> > + interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH
> > 0>;
> > + phys = <&u2port0 PHY_TYPE_USB2>,
> > + <&u3port0 PHY_TYPE_USB3>;
> > + clocks = <&infracfg_ao CLK_INFRA_AO_SSUSB>,
> > + <&topckgen CLK_TOP_SSUSB_REF>,
> > + <&infracfg_ao
> > CLK_INFRA_AO_SSUSB_XHCI>;
> > + clock-names = "sys_ck", "ref_ck", "mcu_ck";
> > + #address-cells = <2>;
> > + #size-cells = <2>;
> > + ranges;
> > + status = "disabled";
> > +
> > + xhci0: xhci@11200000 {
s/xhci/usb
> > + compatible = "mediatek,mt8195-xhci",
> > + "mediatek,mtk-xhci";
> > + reg = <0 0x11200000 0 0x1000>;
> > + reg-names = "mac";
> > + interrupts = <GIC_SPI 129
> > IRQ_TYPE_LEVEL_HIGH 0>;
> > + assigned-clocks = <&topckgen
> > CLK_TOP_USB_TOP>,
> > + <&topckgen
> > CLK_TOP_SSUSB_XHCI>;
> > + assigned-clock-parents = <&topckgen
> > CLK_TOP_UNIVPLL_D5_D4>,
> > + <&topckgen
> > CLK_TOP_UNIVPLL_D5_D4>;
> > + clocks = <&infracfg_ao
> > CLK_INFRA_AO_SSUSB>,
> > + <&infracfg_ao
> > CLK_INFRA_AO_SSUSB_XHCI>,
> > + <&topckgen CLK_TOP_SSUSB_REF>,
> > + <&apmixedsys
> > CLK_APMIXED_USB1PLL>;
> > + clock-names = "sys_ck", "xhci_ck",
> > "ref_ck", "mcu_ck";
> > + status = "disabled";
> > + };
> > + };
> > +
> > + xhci1: xhci1@11290000 {
>
> Node names should be generic, so just "xhci"
also use "usb", no generic name "xhci" in the following spec
>
https://urldefense.com/v3/__https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html*generic-names-recommendation__;Iw!!CTRNKA9wMg0ARbw!n5NxtuBHI5f9tS659pqDgRDZNAFxnXriet0G48KA64n1CEWYYsL0vXY0I-Mw99H8a8UDW5KR_79k7IvYg8Er5gNvmkXE9HZH$
>
>
> > + compatible = "mediatek,mt8195-xhci",
> > + "mediatek,mtk-xhci";
> > + reg = <0 0x11290000 0 0x1000>,
> > + <0 0x11293e00 0 0x0100>;
> > + reg-names = "mac", "ippc";
> > + interrupts = <GIC_SPI 530 IRQ_TYPE_LEVEL_HIGH
> > 0>;
> > + phys = <&u2port1 PHY_TYPE_USB2>,
> > + <&u3port1 PHY_TYPE_USB3>;
> > + assigned-clocks = <&topckgen
> > CLK_TOP_USB_TOP_1P>,
> > + <&topckgen
> > CLK_TOP_SSUSB_XHCI_1P>;
> > + assigned-clock-parents = <&topckgen
> > CLK_TOP_UNIVPLL_D5_D4>,
> > + <&topckgen
> > CLK_TOP_UNIVPLL_D5_D4>;
> > + clocks = <&pericfg_ao
> > CLK_PERI_AO_SSUSB_1P_BUS>,
> > + <&pericfg_ao
> > CLK_PERI_AO_SSUSB_1P_XHCI>,
> > + <&topckgen CLK_TOP_SSUSB_P1_REF>,
> > + <&apmixedsys CLK_APMIXED_USB1PLL>;
> > + clock-names = "sys_ck", "xhci_ck", "ref_ck",
> > "mcu_ck";
> > + status = "disabled";
> > + };
> > +
> > + ssusb1: usb1@112a1000 {
>
> usb
>
> > + compatible = "mediatek,mt8183-mtu3",
> > "mediatek,mtu3";
> > + reg = <0 0x112a1000 0 0x2dff>,
> > + <0 0x112a3e00 0 0x0100>;
> > + reg-names = "mac", "ippc";
> > + interrupts = <GIC_SPI 532 IRQ_TYPE_LEVEL_HIGH
> > 0>;
> > + phys = <&u2port2 PHY_TYPE_USB2>;
> > + assigned-clocks = <&topckgen
> > CLK_TOP_USB_TOP_2P>;
> > + assigned-clock-parents = <&topckgen
> > CLK_TOP_UNIVPLL_D5_D4>;
> > + clocks = <&pericfg_ao
> > CLK_PERI_AO_SSUSB_2P_BUS>,
> > + <&topckgen CLK_TOP_SSUSB_P2_REF>,
> > + <&pericfg_ao
> > CLK_PERI_AO_SSUSB_2P_XHCI>;
> > + clock-names = "sys_ck", "ref_ck", "mcu_ck";
> > + mediatek,syscon-wakeup = <&pericfg 0x400 4>;
> > + #address-cells = <2>;
> > + #size-cells = <2>;
> > + ranges;
> > + status = "disabled";
> > +
> > + xhci2: xhci2@112a0000 {
>
> xhci
use "usb", no need change it when move from mt8195.dtsi
>
> > + compatible = "mediatek,mt8195-
> > xhci","mediatek,mtk-xhci";
> > + reg = <0 0x112a0000 0 0x1000>;
> > + reg-names = "mac";
> > + interrupts-extended = <&gic GIC_SPI 533
> > IRQ_TYPE_LEVEL_HIGH 0>,
> > + <&pio 220
> > IRQ_TYPE_EDGE_FALLING>;
> > + assigned-clocks = <&topckgen
> > CLK_TOP_SSUSB_XHCI_2P>;
> > + assigned-clock-parents = <&topckgen
> > CLK_TOP_UNIVPLL_D5_D4>;
> > + clocks = <&pericfg_ao
> > CLK_PERI_AO_SSUSB_2P_XHCI>;
> > + clock-names = "sys_ck";
> > + status = "disabled";
> > + };
> > + };
> > +
> > + xhci3: xhci3@112b0000 {
>
> xhci
usb
> > + compatible = "mediatek,mt8195-xhci",
> > + "mediatek,mtk-xhci";
> > + reg = <0 0x112b0000 0 0x1000>,
> > + <0 0x112b3e00 0 0x0100>;
> > + reg-names = "mac", "ippc";
> > + interrupts = <GIC_SPI 536 IRQ_TYPE_LEVEL_HIGH
> > 0>;
> > + phys = <&u2port3 PHY_TYPE_USB2>;
> > + assigned-clocks = <&topckgen
> > CLK_TOP_USB_TOP_3P>,
> > + <&topckgen
> > CLK_TOP_SSUSB_XHCI_3P>;
> > + assigned-clock-parents = <&topckgen
> > CLK_TOP_UNIVPLL_D5_D4>,
> > + <&topckgen
> > CLK_TOP_UNIVPLL_D5_D4>;
> > + clocks = <&pericfg_ao
> > CLK_PERI_AO_SSUSB_3P_BUS>,
> > + <&pericfg_ao
> > CLK_PERI_AO_SSUSB_3P_XHCI>,
> > + <&topckgen CLK_TOP_SSUSB_P3_REF>;
> > + clock-names = "sys_ck", "xhci_ck", "ref_ck";
> > + mediatek,syscon-wakeup = <&pericfg 0x400 106>;
> > + wakeup-source;
> > + usb2-lpm-disable;
> > + status = "disabled";
> > + };
>
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations
2023-01-11 2:19 ` Chunfeng Yun (云春峰)
@ 2023-01-11 3:53 ` Macpaul Lin
0 siblings, 0 replies; 14+ messages in thread
From: Macpaul Lin @ 2023-01-11 3:53 UTC (permalink / raw)
To: Chunfeng Yun (云春峰),
linux-mediatek@lists.infradead.org, robh+dt@kernel.org,
devicetree@vger.kernel.org, krzysztof.kozlowski@linaro.org,
linux-arm-kernel@lists.infradead.org,
krzysztof.kozlowski+dt@linaro.org, matthias.bgg@gmail.com
Cc: Pablo Sun (孫毓翔), linux-usb@vger.kernel.org,
Bear Wang (萩原惟德), wenst@chromium.org,
macpaul@gmail.com, Miles Chen (陳民樺),
Rex-BC Chen (陳柏辰),
TingHan Shen (沈廷翰), yow-shin.liou, ben.lok
On 1/11/23 10:19, Chunfeng Yun (云春峰) and Krzysztof Kozlowski wrote:
> On Fri, 2023-01-06 at 13:56 +0100, Krzysztof Kozlowski wrote:
>> On 05/01/2023 10:28, Macpaul Lin wrote:
>>> Introduce the split MT8195 laptop and iot USB configurations.
>>> The hardware specifications for LAPTOP devices is different from
>>> IOT
>>> devices. The major differences include some hardware constrains for
>>> dual-role switch for USB controllers in different configurations,
>>> especially for power management and other control flows as well.
>>>
>>> Here are some hardware specifiction differences listed:
>>> 1. LAPTOP (Cherry Tomato boards) don't support USB gadget (device
>>> mode).
>>> 2. IOT devices must support multiple gadget devices and host
>>> mode.
>>> 3. Dual-role switch is not fully supported. Only USB PORT0
>>> support
>>> dual-role switch.
>>> 4. Power management is designed in primary and secondary
>>> dominator.
>>> For a dual-role port, the device controller is the primary
>>> controller
>>> for power management; while the host controller is the
>>> secondary.
>>> LAPTOP devices should remove device nodes for avoiding
>>> abnormal
>>> behavior.
>>>
>>> This modifcation is to add USB configurations "mt8195-laptop-
>>> usb.dtsi"
>>> for LAPTOP devices, and add "mt8195-iot-usb.dtsi" for IOT devices.
>>>
>>> To remove common USB configurations for mt8195.dtsi and switch
>>> includes
>>> dtsi these new files for the boards will come in next patch.
>>>
>>> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
>>> ---
>>> .../boot/dts/mediatek/mt8195-iot-usb.dtsi | 122
>>> ++++++++++++++++++
>>> .../boot/dts/mediatek/mt8195-laptop-usb.dtsi | 102
>>> +++++++++++++++
>>> 2 files changed, 224 insertions(+)
>>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-iot-
>>> usb.dtsi
>>> create mode 100644 arch/arm64/boot/dts/mediatek/mt8195-laptop-
>>> usb.dtsi
>>>
>>> diff --git a/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
>>> b/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
>>> new file mode 100644
>>> index 000000000000..f9bd79542044
>>> --- /dev/null
>>> +++ b/arch/arm64/boot/dts/mediatek/mt8195-iot-usb.dtsi
>>> @@ -0,0 +1,122 @@
>>> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
>>> +/*
>>> + * Copyright (C) 2022 MediaTek Inc.
>>> + */
>>> +
>>> +#include "mt8195.dtsi"
>>> +
>>> +/ {
>>> + soc {
>>> + ssusb: ssusb@11200000 {
>>
>> Node name: usb
Got it.
>>
>>> + compatible ="mediatek,mt8183-mtu3",
>>> "mediatek,mtu3";
>>> + reg = <0 0x11201000 0 0x2dff>,
>>> + <0 0x11203e00 0 0x0100>;
>>> + reg-names = "mac", "ippc";
>>> + interrupts = <GIC_SPI 128 IRQ_TYPE_LEVEL_HIGH
>>> 0>;
>>> + phys = <&u2port0 PHY_TYPE_USB2>,
>>> + <&u3port0 PHY_TYPE_USB3>;
>>> + clocks = <&infracfg_ao CLK_INFRA_AO_SSUSB>,
>>> + <&topckgen CLK_TOP_SSUSB_REF>,
>>> + <&infracfg_ao
>>> CLK_INFRA_AO_SSUSB_XHCI>;
>>> + clock-names = "sys_ck", "ref_ck", "mcu_ck";
>>> + #address-cells = <2>;
>>> + #size-cells = <2>;
>>> + ranges;
>>> + status = "disabled";
>>> +
>>> + xhci0: xhci@11200000 {
> s/xhci/usb
>>> + compatible = "mediatek,mt8195-xhci",
>>> + "mediatek,mtk-xhci";
>>> + reg = <0 0x11200000 0 0x1000>;
>>> + reg-names = "mac";
>>> + interrupts = <GIC_SPI 129
>>> IRQ_TYPE_LEVEL_HIGH 0>;
>>> + assigned-clocks = <&topckgen
>>> CLK_TOP_USB_TOP>,
>>> + <&topckgen
>>> CLK_TOP_SSUSB_XHCI>;
>>> + assigned-clock-parents = <&topckgen
>>> CLK_TOP_UNIVPLL_D5_D4>,
>>> + <&topckgen
>>> CLK_TOP_UNIVPLL_D5_D4>;
>>> + clocks = <&infracfg_ao
>>> CLK_INFRA_AO_SSUSB>,
>>> + <&infracfg_ao
>>> CLK_INFRA_AO_SSUSB_XHCI>,
>>> + <&topckgen CLK_TOP_SSUSB_REF>,
>>> + <&apmixedsys
>>> CLK_APMIXED_USB1PLL>;
>>> + clock-names = "sys_ck", "xhci_ck",
>>> "ref_ck", "mcu_ck";
>>> + status = "disabled";
>>> + };
>>> + };
>>> +
>>> + xhci1: xhci1@11290000 {
>>
>> Node names should be generic, so just "xhci"
> also use "usb", no generic name "xhci" in the following spec
>
Thanks for the reminding. I'll align the naming assignment in origin
mt8195.dtsi and in the following specification.
>>
> https://urldefense.com/v3/__https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html*generic-names-recommendation__;Iw!!CTRNKA9wMg0ARbw!n5NxtuBHI5f9tS659pqDgRDZNAFxnXriet0G48KA64n1CEWYYsL0vXY0I-Mw99H8a8UDW5KR_79k7IvYg8Er5gNvmkXE9HZH$
>>
>>
>>> + compatible = "mediatek,mt8195-xhci",
>>> + "mediatek,mtk-xhci";
>>> + reg = <0 0x11290000 0 0x1000>,
>>> + <0 0x11293e00 0 0x0100>;
>>> + reg-names = "mac", "ippc";
>>> + interrupts = <GIC_SPI 530 IRQ_TYPE_LEVEL_HIGH
>>> 0>;
>>> + phys = <&u2port1 PHY_TYPE_USB2>,
>>> + <&u3port1 PHY_TYPE_USB3>;
>>> + assigned-clocks = <&topckgen
>>> CLK_TOP_USB_TOP_1P>,
>>> + <&topckgen
>>> CLK_TOP_SSUSB_XHCI_1P>;
>>> + assigned-clock-parents = <&topckgen
>>> CLK_TOP_UNIVPLL_D5_D4>,
>>> + <&topckgen
>>> CLK_TOP_UNIVPLL_D5_D4>;
>>> + clocks = <&pericfg_ao
>>> CLK_PERI_AO_SSUSB_1P_BUS>,
>>> + <&pericfg_ao
>>> CLK_PERI_AO_SSUSB_1P_XHCI>,
>>> + <&topckgen CLK_TOP_SSUSB_P1_REF>,
>>> + <&apmixedsys CLK_APMIXED_USB1PLL>;
>>> + clock-names = "sys_ck", "xhci_ck", "ref_ck",
>>> "mcu_ck";
>>> + status = "disabled";
>>> + };
>>> +
>>> + ssusb1: usb1@112a1000 {
>>
>> usb
>>
Got it.
>>> + compatible = "mediatek,mt8183-mtu3",
>>> "mediatek,mtu3";
>>> + reg = <0 0x112a1000 0 0x2dff>,
>>> + <0 0x112a3e00 0 0x0100>;
>>> + reg-names = "mac", "ippc";
>>> + interrupts = <GIC_SPI 532 IRQ_TYPE_LEVEL_HIGH
>>> 0>;
>>> + phys = <&u2port2 PHY_TYPE_USB2>;
>>> + assigned-clocks = <&topckgen
>>> CLK_TOP_USB_TOP_2P>;
>>> + assigned-clock-parents = <&topckgen
>>> CLK_TOP_UNIVPLL_D5_D4>;
>>> + clocks = <&pericfg_ao
>>> CLK_PERI_AO_SSUSB_2P_BUS>,
>>> + <&topckgen CLK_TOP_SSUSB_P2_REF>,
>>> + <&pericfg_ao
>>> CLK_PERI_AO_SSUSB_2P_XHCI>;
>>> + clock-names = "sys_ck", "ref_ck", "mcu_ck";
>>> + mediatek,syscon-wakeup = <&pericfg 0x400 4>;
>>> + #address-cells = <2>;
>>> + #size-cells = <2>;
>>> + ranges;
>>> + status = "disabled";
>>> +
>>> + xhci2: xhci2@112a0000 {
>>
>> xhci
> use "usb", no need change it when move from mt8195.dtsi
>
Got it.
>>
>>> + compatible = "mediatek,mt8195-
>>> xhci","mediatek,mtk-xhci";
>>> + reg = <0 0x112a0000 0 0x1000>;
>>> + reg-names = "mac";
>>> + interrupts-extended = <&gic GIC_SPI 533
>>> IRQ_TYPE_LEVEL_HIGH 0>,
>>> + <&pio 220
>>> IRQ_TYPE_EDGE_FALLING>;
>>> + assigned-clocks = <&topckgen
>>> CLK_TOP_SSUSB_XHCI_2P>;
>>> + assigned-clock-parents = <&topckgen
>>> CLK_TOP_UNIVPLL_D5_D4>;
>>> + clocks = <&pericfg_ao
>>> CLK_PERI_AO_SSUSB_2P_XHCI>;
>>> + clock-names = "sys_ck";
>>> + status = "disabled";
>>> + };
>>> + };
>>> +
>>> + xhci3: xhci3@112b0000 {
>>
>> xhci
> usb
Got it.
>>> + compatible = "mediatek,mt8195-xhci",
>>> + "mediatek,mtk-xhci";
>>> + reg = <0 0x112b0000 0 0x1000>,
>>> + <0 0x112b3e00 0 0x0100>;
>>> + reg-names = "mac", "ippc";
>>> + interrupts = <GIC_SPI 536 IRQ_TYPE_LEVEL_HIGH
>>> 0>;
>>> + phys = <&u2port3 PHY_TYPE_USB2>;
>>> + assigned-clocks = <&topckgen
>>> CLK_TOP_USB_TOP_3P>,
>>> + <&topckgen
>>> CLK_TOP_SSUSB_XHCI_3P>;
>>> + assigned-clock-parents = <&topckgen
>>> CLK_TOP_UNIVPLL_D5_D4>,
>>> + <&topckgen
>>> CLK_TOP_UNIVPLL_D5_D4>;
>>> + clocks = <&pericfg_ao
>>> CLK_PERI_AO_SSUSB_3P_BUS>,
>>> + <&pericfg_ao
>>> CLK_PERI_AO_SSUSB_3P_XHCI>,
>>> + <&topckgen CLK_TOP_SSUSB_P3_REF>;
>>> + clock-names = "sys_ck", "xhci_ck", "ref_ck";
>>> + mediatek,syscon-wakeup = <&pericfg 0x400 106>;
>>> + wakeup-source;
>>> + usb2-lpm-disable;
>>> + status = "disabled";
>>> + };
>>
>>
>> Best regards,
>> Krzysztof
>>
Thanks
Macpaul Lin
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations
2023-01-05 9:28 [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Macpaul Lin
` (2 preceding siblings ...)
2023-01-06 12:56 ` [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Krzysztof Kozlowski
@ 2023-01-09 15:13 ` AngeloGioacchino Del Regno
2023-01-11 5:37 ` Macpaul Lin
3 siblings, 1 reply; 14+ messages in thread
From: AngeloGioacchino Del Regno @ 2023-01-09 15:13 UTC (permalink / raw)
To: Macpaul Lin, Rob Herring, Krzysztof Kozlowski, Matthias Brugger,
linux-arm-kernel, linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Miles Chen, linux-usb,
Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai
Il 05/01/23 10:28, Macpaul Lin ha scritto:
> Introduce the split MT8195 laptop and iot USB configurations.
> The hardware specifications for LAPTOP devices is different from IOT
> devices. The major differences include some hardware constrains for
> dual-role switch for USB controllers in different configurations,
> especially for power management and other control flows as well.
>
> Here are some hardware specifiction differences listed:
> 1. LAPTOP (Cherry Tomato boards) don't support USB gadget (device mode).
> 2. IOT devices must support multiple gadget devices and host mode.
> 3. Dual-role switch is not fully supported. Only USB PORT0 support
> dual-role switch.
> 4. Power management is designed in primary and secondary dominator.
> For a dual-role port, the device controller is the primary controller
> for power management; while the host controller is the secondary.
> LAPTOP devices should remove device nodes for avoiding abnormal
> behavior.
>
> This modifcation is to add USB configurations "mt8195-laptop-usb.dtsi"
> for LAPTOP devices, and add "mt8195-iot-usb.dtsi" for IOT devices.
>
> To remove common USB configurations for mt8195.dtsi and switch includes
> dtsi these new files for the boards will come in next patch.
>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
I'm mostly sure that there's no reason to split the two configurations.
I agree in that Tomato doesn't support gadget mode on the Type-A port and I
honestly don't currently know (and I'll test that later!) if it would be possible
to act as gadget on any of the two Type-C ports.
Of course I agree on the fact that a laptop acting as a gadget may not be useful,
but that's not something that I want to judge, as someone may find a usecase.
In any case, even if Tomato does *not* support gadget mode on *any* port at all,
I wonder why we wouldn't be able to probe MTU3 (and correctly describe the SoC)
on Chromebooks but only on MT8195-based IoT boards...
...and in case there's any real issue, we can always force host mode (with a
generic devicetree property!) on the MTU3 on Tomato.
Finally, if we're able to add MTU3 to Tomato boards, this means that we won't be
seeing these two DTSI files and that USB nodes are still going to all lie in the
main `mt8195.dtsi` file, without all this duplication that I'm seeing here.
What do you think?
Regards,
Angelo
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations
2023-01-09 15:13 ` AngeloGioacchino Del Regno
@ 2023-01-11 5:37 ` Macpaul Lin
2023-02-01 12:52 ` Matthias Brugger
0 siblings, 1 reply; 14+ messages in thread
From: Macpaul Lin @ 2023-01-11 5:37 UTC (permalink / raw)
To: AngeloGioacchino Del Regno, Rob Herring, Krzysztof Kozlowski,
Matthias Brugger, linux-arm-kernel, linux-mediatek, devicetree
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Miles Chen, linux-usb,
Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai,
yow-shin.liou, ben.lok
On 1/9/23 23:13, AngeloGioacchino Del Regno wrote:
> Il 05/01/23 10:28, Macpaul Lin ha scritto:
>> Introduce the split MT8195 laptop and iot USB configurations.
>> The hardware specifications for LAPTOP devices is different from IOT
>> devices. The major differences include some hardware constrains for
>> dual-role switch for USB controllers in different configurations,
>> especially for power management and other control flows as well.
>>
>> Here are some hardware specifiction differences listed:
>> 1. LAPTOP (Cherry Tomato boards) don't support USB gadget (device
>> mode).
>> 2. IOT devices must support multiple gadget devices and host mode.
>> 3. Dual-role switch is not fully supported. Only USB PORT0 support
>> dual-role switch.
>> 4. Power management is designed in primary and secondary dominator.
>> For a dual-role port, the device controller is the primary
>> controller
>> for power management; while the host controller is the secondary.
>> LAPTOP devices should remove device nodes for avoiding abnormal
>> behavior.
>>
>> This modifcation is to add USB configurations "mt8195-laptop-usb.dtsi"
>> for LAPTOP devices, and add "mt8195-iot-usb.dtsi" for IOT devices.
>>
>> To remove common USB configurations for mt8195.dtsi and switch includes
>> dtsi these new files for the boards will come in next patch.
>>
>> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
>
> I'm mostly sure that there's no reason to split the two configurations.
>
> I agree in that Tomato doesn't support gadget mode on the Type-A port and I
> honestly don't currently know (and I'll test that later!) if it would be
> possible
> to act as gadget on any of the two Type-C ports.
> Of course I agree on the fact that a laptop acting as a gadget may not
> be useful,
> but that's not something that I want to judge, as someone may find a
> usecase.
>
> In any case, even if Tomato does *not* support gadget mode on *any* port
> at all,
> I wonder why we wouldn't be able to probe MTU3 (and correctly describe
> the SoC)
> on Chromebooks but only on MT8195-based IoT boards...
> ...and in case there's any real issue, we can always force host mode
> (with a
> generic devicetree property!) on the MTU3 on Tomato.
We are sorry it cannot be achieved by even setting "force host mode" to
usb device node. At least, it cannot be done on MT8195.
The basic reason is the power requirements for USB host on a LAPTOP are
different from those on an IoT device.
The main cause is low power management. The hardware of each device port
is different on MT8195. Even the bit fields definition in registers were
different.
Some details such as sequence need to be coordinated with the SPM
firmware. When a device hardware is involved in runtime PM, function
like remote wakeup and other suspend/resume behavior will be abnormal
for a LAPTOP device. If we split the dtsi for different devices, people
can choose different configuration in SPM firmware in coreboot or in
TF-A to meet the requirement. Hence we'd better not to get more messy
code in Linux driver.
> Finally, if we're able to add MTU3 to Tomato boards, this means that we
> won't be
> seeing these two DTSI files and that USB nodes are still going to all
> lie in the
> main `mt8195.dtsi` file, without all this duplication that I'm seeing here.
>
> What do you think?
>
> Regards,
> Angelo
>
Thanks for the suggestion, we hope the next platform in the future could
avoid this issue and reduce some duplicate dts.
Macpaul Lin
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations
2023-01-11 5:37 ` Macpaul Lin
@ 2023-02-01 12:52 ` Matthias Brugger
2023-02-09 10:44 ` Macpaul Lin
0 siblings, 1 reply; 14+ messages in thread
From: Matthias Brugger @ 2023-02-01 12:52 UTC (permalink / raw)
To: Macpaul Lin, AngeloGioacchino Del Regno, Rob Herring,
Krzysztof Kozlowski, linux-arm-kernel, linux-mediatek, devicetree,
Greg Kroah-Hartman
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Miles Chen, linux-usb,
Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai,
yow-shin.liou, ben.lok
Hi all,
On 11/01/2023 06:37, Macpaul Lin wrote:
>
>
> On 1/9/23 23:13, AngeloGioacchino Del Regno wrote:
>> Il 05/01/23 10:28, Macpaul Lin ha scritto:
>>> Introduce the split MT8195 laptop and iot USB configurations.
>>> The hardware specifications for LAPTOP devices is different from IOT
>>> devices. The major differences include some hardware constrains for
>>> dual-role switch for USB controllers in different configurations,
>>> especially for power management and other control flows as well.
>>>
>>> Here are some hardware specifiction differences listed:
>>> 1. LAPTOP (Cherry Tomato boards) don't support USB gadget (device mode).
>>> 2. IOT devices must support multiple gadget devices and host mode.
>>> 3. Dual-role switch is not fully supported. Only USB PORT0 support
>>> dual-role switch.
>>> 4. Power management is designed in primary and secondary dominator.
>>> For a dual-role port, the device controller is the primary controller
>>> for power management; while the host controller is the secondary.
>>> LAPTOP devices should remove device nodes for avoiding abnormal
>>> behavior.
>>>
>>> This modifcation is to add USB configurations "mt8195-laptop-usb.dtsi"
>>> for LAPTOP devices, and add "mt8195-iot-usb.dtsi" for IOT devices.
>>>
>>> To remove common USB configurations for mt8195.dtsi and switch includes
>>> dtsi these new files for the boards will come in next patch.
>>>
>>> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
>>
>> I'm mostly sure that there's no reason to split the two configurations.
>>
>> I agree in that Tomato doesn't support gadget mode on the Type-A port and I
>> honestly don't currently know (and I'll test that later!) if it would be possible
>> to act as gadget on any of the two Type-C ports.
>> Of course I agree on the fact that a laptop acting as a gadget may not be useful,
>> but that's not something that I want to judge, as someone may find a usecase.
>>
>> In any case, even if Tomato does *not* support gadget mode on *any* port at all,
>> I wonder why we wouldn't be able to probe MTU3 (and correctly describe the SoC)
>> on Chromebooks but only on MT8195-based IoT boards...
>> ...and in case there's any real issue, we can always force host mode (with a
>> generic devicetree property!) on the MTU3 on Tomato.
>
> We are sorry it cannot be achieved by even setting "force host mode" to usb
> device node. At least, it cannot be done on MT8195.
>
> The basic reason is the power requirements for USB host on a LAPTOP are
> different from those on an IoT device.
>
> The main cause is low power management. The hardware of each device port is
> different on MT8195. Even the bit fields definition in registers were different.
>
> Some details such as sequence need to be coordinated with the SPM firmware. When
> a device hardware is involved in runtime PM, function like remote wakeup and
> other suspend/resume behavior will be abnormal for a LAPTOP device. If we split
> the dtsi for different devices, people can choose different configuration in SPM
> firmware in coreboot or in TF-A to meet the requirement. Hence we'd better not
> to get more messy code in Linux driver.
>
I'm not sure I understand everything here. If the XHCI device is a child of the
mtu3 node then we have problems with some SPM firmware that is not coordinated
with the runtime PM functions of the kernel?
Fixing that in the device tree sounds wrong here. I think the real fix would be,
to fix the SPM firmware, so that it can deal with that.
Or is there more to it? If so what? In that case can we try to ignore the
runtime PM in the MTU3 kernel driver?
I'm not an USB expert but to me it looks very strange that we can have the XHCI
devices nodes as 'standalone' or as children of mtu3. We should try to describe
the HW as it is in DT.
Regards,
Matthias
>> Finally, if we're able to add MTU3 to Tomato boards, this means that we won't be
>> seeing these two DTSI files and that USB nodes are still going to all lie in the
>> main `mt8195.dtsi` file, without all this duplication that I'm seeing here.
>>
>> What do you think?
>>
>> Regards,
>> Angelo
>>
>
> Thanks for the suggestion, we hope the next platform in the future could avoid
> this issue and reduce some duplicate dts.
>
> Macpaul Lin
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations
2023-02-01 12:52 ` Matthias Brugger
@ 2023-02-09 10:44 ` Macpaul Lin
0 siblings, 0 replies; 14+ messages in thread
From: Macpaul Lin @ 2023-02-09 10:44 UTC (permalink / raw)
To: Matthias Brugger, AngeloGioacchino Del Regno, Rob Herring,
Krzysztof Kozlowski, linux-arm-kernel, linux-mediatek, devicetree,
Greg Kroah-Hartman
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Miles Chen, linux-usb,
Chunfeng Yun, Tinghan Shen, Rex-BC Chen, Chen-Yu Tsai,
yow-shin.liou, ben.lok
On 2/1/23 20:52, Matthias Brugger wrote:
> Hi all,
>
> On 11/01/2023 06:37, Macpaul Lin wrote:
>>
>>
>> On 1/9/23 23:13, AngeloGioacchino Del Regno wrote:
>>> Il 05/01/23 10:28, Macpaul Lin ha scritto:
>>>> Introduce the split MT8195 laptop and iot USB configurations.
>>>> The hardware specifications for LAPTOP devices is different from IOT
>>>> devices. The major differences include some hardware constrains for
>>>> dual-role switch for USB controllers in different configurations,
>>>> especially for power management and other control flows as well.
>>>>
>>>> Here are some hardware specifiction differences listed:
>>>> 1. LAPTOP (Cherry Tomato boards) don't support USB gadget (device
>>>> mode).
>>>> 2. IOT devices must support multiple gadget devices and host mode.
>>>> 3. Dual-role switch is not fully supported. Only USB PORT0 support
>>>> dual-role switch.
>>>> 4. Power management is designed in primary and secondary dominator.
>>>> For a dual-role port, the device controller is the primary
>>>> controller
>>>> for power management; while the host controller is the secondary.
>>>> LAPTOP devices should remove device nodes for avoiding abnormal
>>>> behavior.
>>>>
>>>> This modifcation is to add USB configurations "mt8195-laptop-usb.dtsi"
>>>> for LAPTOP devices, and add "mt8195-iot-usb.dtsi" for IOT devices.
>>>>
>>>> To remove common USB configurations for mt8195.dtsi and switch includes
>>>> dtsi these new files for the boards will come in next patch.
>>>>
>>>> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
>>>
>>> I'm mostly sure that there's no reason to split the two configurations.
>>>
>>> I agree in that Tomato doesn't support gadget mode on the Type-A port
>>> and I
>>> honestly don't currently know (and I'll test that later!) if it would
>>> be possible
>>> to act as gadget on any of the two Type-C ports.
>>> Of course I agree on the fact that a laptop acting as a gadget may
>>> not be useful,
>>> but that's not something that I want to judge, as someone may find a
>>> usecase.
>>>
>>> In any case, even if Tomato does *not* support gadget mode on *any*
>>> port at all,
>>> I wonder why we wouldn't be able to probe MTU3 (and correctly
>>> describe the SoC)
>>> on Chromebooks but only on MT8195-based IoT boards...
>>> ...and in case there's any real issue, we can always force host mode
>>> (with a
>>> generic devicetree property!) on the MTU3 on Tomato.
>>
>> We are sorry it cannot be achieved by even setting "force host mode"
>> to usb device node. At least, it cannot be done on MT8195.
>>
>> The basic reason is the power requirements for USB host on a LAPTOP
>> are different from those on an IoT device.
>>
>> The main cause is low power management. The hardware of each device
>> port is different on MT8195. Even the bit fields definition in
>> registers were different.
>>
>> Some details such as sequence need to be coordinated with the SPM
>> firmware. When a device hardware is involved in runtime PM, function
>> like remote wakeup and other suspend/resume behavior will be abnormal
>> for a LAPTOP device. If we split the dtsi for different devices,
>> people can choose different configuration in SPM firmware in coreboot
>> or in TF-A to meet the requirement. Hence we'd better not to get more
>> messy code in Linux driver.
>>
>
> I'm not sure I understand everything here. If the XHCI device is a child
> of the mtu3 node then we have problems with some SPM firmware that is
> not coordinated with the runtime PM functions of the kernel?
The behavior of runtime PM function, especially the behavior of USB
remote wake up will be different if mtu3 node is involved, for MT8195.
> Fixing that in the device tree sounds wrong here. I think the real fix
> would be, to fix the SPM firmware, so that it can deal with that.
>
> Or is there more to it? If so what? In that case can we try to ignore
> the runtime PM in the MTU3 kernel driver?
>
> I'm not an USB expert but to me it looks very strange that we can have
> the XHCI devices nodes as 'standalone' or as children of mtu3. We should
> try to describe the HW as it is in DT.
After a discussion with MTU3 maintainer Chunfeng, we think there might
be a way to give it a try to refactor the mtu3 driver.
That is to create an extra USB platform device to handle mtu3 and
xhci-mtk as the children at the same level. Both mtu3 platform device
xhci-mtk platform device will become the children of a common USB
platform device. However, we are not sure if this could work and solved the
dependencies. It requires some development time and MediaTek need
to allocate some developer's resource to verify this approach.
> Regards,
> Matthias
>
>>> Finally, if we're able to add MTU3 to Tomato boards, this means that
>>> we won't be
>>> seeing these two DTSI files and that USB nodes are still going to all
>>> lie in the
>>> main `mt8195.dtsi` file, without all this duplication that I'm seeing
>>> here.
>>>
>>> What do you think?
>>>
>>> Regards,
>>> Angelo
>>>
>>
>> Thanks for the suggestion, we hope the next platform in the future
>> could avoid this issue and reduce some duplicate dts.
>>
>> Macpaul Lin
Thanks
Macpaul Lin
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2023-02-09 10:46 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-01-05 9:28 [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Macpaul Lin
2023-01-05 9:28 ` [PATCH 2/3] arm64: dts: mediatek: switch MT8195 boards into different " Macpaul Lin
2023-01-05 9:28 ` [PATCH 3/3] arm64: dts: mediatek: enable USB device port for mt8195-demo board Macpaul Lin
2023-01-06 12:56 ` Krzysztof Kozlowski
2023-01-11 5:41 ` Macpaul Lin
2023-02-01 12:24 ` Matthias Brugger
2023-02-07 9:36 ` Chunfeng Yun (云春峰)
2023-01-06 12:56 ` [PATCH 1/3] arm64: dts: mediatek: Introduce MT8195 LAPTOP and IOT's USB configurations Krzysztof Kozlowski
2023-01-11 2:19 ` Chunfeng Yun (云春峰)
2023-01-11 3:53 ` Macpaul Lin
2023-01-09 15:13 ` AngeloGioacchino Del Regno
2023-01-11 5:37 ` Macpaul Lin
2023-02-01 12:52 ` Matthias Brugger
2023-02-09 10:44 ` Macpaul Lin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).