* [PATCH v2 2/2] arm64: dts: mediatek: mt8390-genio-700-evk: add keys and USB HUB
2024-10-07 9:02 [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names Macpaul Lin
@ 2024-10-07 9:02 ` Macpaul Lin
2024-10-08 8:20 ` AngeloGioacchino Del Regno
2024-10-07 18:44 ` [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names Rob Herring (Arm)
` (4 subsequent siblings)
5 siblings, 1 reply; 9+ messages in thread
From: Macpaul Lin @ 2024-10-07 9:02 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Matthias Brugger,
AngeloGioacchino Del Regno, devicetree, linux-kernel,
linux-arm-kernel, linux-mediatek, Alexandre Mergnat
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Macpaul Lin, Sen Chu,
Chris-qj chen, MediaTek Chromebook Upstream, Chen-Yu Tsai
1. Add i2c, mmc to aliases.
4. Add PMIC_KEY setting.
5. Add USB HUB TUSB8020 to xhci1.
6. Re-order spi2 node.
Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
---
.../dts/mediatek/mt8390-genio-700-evk.dts | 57 +++++++++++++++----
1 file changed, 47 insertions(+), 10 deletions(-)
Changes for v2:
- Fix order of spi2.
- Update pinctrl in i2c4 and rt1715.
- Drop IT5205 and RT1715 nodes since the DTS are not completed yet.
- Add #address-cells and #size-cells to xhci1 for supporting USB hubs.
diff --git a/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts b/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
index 96b272567cb1..3e77f59f2c74 100644
--- a/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
+++ b/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
@@ -23,6 +23,15 @@ / {
"mediatek,mt8188";
aliases {
+ i2c0 = &i2c0;
+ i2c1 = &i2c1;
+ i2c2 = &i2c2;
+ i2c3 = &i2c3;
+ i2c4 = &i2c4;
+ i2c5 = &i2c5;
+ i2c6 = &i2c6;
+ mmc0 = &mmc0;
+ mmc1 = &mmc1;
serial0 = &uart0;
};
@@ -249,7 +258,6 @@ &i2c3 {
&i2c4 {
pinctrl-names = "default";
pinctrl-0 = <&i2c4_pins>;
- pinctrl-1 = <&rt1715_int_pins>;
clock-frequency = <1000000>;
status = "okay";
};
@@ -867,6 +875,17 @@ pins-wifi-enable {
&pmic {
interrupt-parent = <&pio>;
interrupts = <222 IRQ_TYPE_LEVEL_HIGH>;
+
+ mt6359keys: keys {
+ compatible = "mediatek,mt6359-keys";
+ mediatek,long-press-mode = <1>;
+ power-off-time-sec = <0>;
+
+ power-key {
+ linux,keycodes = <KEY_POWER>;
+ wakeup-source;
+ };
+ };
};
&scp {
@@ -874,6 +893,15 @@ &scp {
status = "okay";
};
+&spi2 {
+ pinctrl-0 = <&spi2_pins>;
+ pinctrl-names = "default";
+ mediatek,pad-select = <0>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+ status = "okay";
+};
+
&uart0 {
pinctrl-0 = <&uart0_pins>;
pinctrl-names = "default";
@@ -892,15 +920,6 @@ &uart2 {
status = "okay";
};
-&spi2 {
- pinctrl-0 = <&spi2_pins>;
- pinctrl-names = "default";
- mediatek,pad-select = <0>;
- #address-cells = <1>;
- #size-cells = <0>;
- status = "okay";
-};
-
&u3phy0 {
status = "okay";
};
@@ -921,6 +940,24 @@ &xhci0 {
&xhci1 {
status = "okay";
vusb33-supply = <&mt6359_vusb_ldo_reg>;
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ hub_2_0: hub@1 {
+ compatible = "usb451,8025";
+ reg = <1>;
+ peer-hub = <&hub_3_0>;
+ reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
+ vdd-supply = <&usb_hub_fixed_3v3>;
+ };
+
+ hub_3_0: hub@2 {
+ compatible = "usb451,8027";
+ reg = <2>;
+ peer-hub = <&hub_2_0>;
+ reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
+ vdd-supply = <&usb_hub_fixed_3v3>;
+ };
};
&xhci2 {
--
2.45.2
^ permalink raw reply related [flat|nested] 9+ messages in thread* Re: [PATCH v2 2/2] arm64: dts: mediatek: mt8390-genio-700-evk: add keys and USB HUB
2024-10-07 9:02 ` [PATCH v2 2/2] arm64: dts: mediatek: mt8390-genio-700-evk: add keys and USB HUB Macpaul Lin
@ 2024-10-08 8:20 ` AngeloGioacchino Del Regno
2024-10-09 8:39 ` Macpaul Lin
0 siblings, 1 reply; 9+ messages in thread
From: AngeloGioacchino Del Regno @ 2024-10-08 8:20 UTC (permalink / raw)
To: Macpaul Lin, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Matthias Brugger, devicetree, linux-kernel, linux-arm-kernel,
linux-mediatek, Alexandre Mergnat
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Sen Chu, Chris-qj chen,
MediaTek Chromebook Upstream, Chen-Yu Tsai
Il 07/10/24 11:02, Macpaul Lin ha scritto:
> 1. Add i2c, mmc to aliases.
> 4. Add PMIC_KEY setting.
> 5. Add USB HUB TUSB8020 to xhci1.
> 6. Re-order spi2 node.
Please either add the aliases in a different commit, or add that to the title.
arm64: dts: mediatek: mt8390-genio-700-evk: Add aliases, keys and USB HUB
Additionally, I'd really like to see a "conversation-like" description instead
of a kind-of-checkbox list.
Something like..
"
Add aliases for the I2C and MMC/SD controllers to keep the numbering consistent and
describe the TUSB8020 hub present on the USB XHCI1 controller instance to enable
resetting it with its specific reset GPIO.
While at it, also move the spi2 node to keep nodes alphabetically ordered.
"
>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> ---
> .../dts/mediatek/mt8390-genio-700-evk.dts | 57 +++++++++++++++----
> 1 file changed, 47 insertions(+), 10 deletions(-)
>
> Changes for v2:
> - Fix order of spi2.
> - Update pinctrl in i2c4 and rt1715.
> - Drop IT5205 and RT1715 nodes since the DTS are not completed yet.
> - Add #address-cells and #size-cells to xhci1 for supporting USB hubs.
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts b/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
> index 96b272567cb1..3e77f59f2c74 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
> +++ b/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
> @@ -23,6 +23,15 @@ / {
> "mediatek,mt8188";
>
> aliases {
> + i2c0 = &i2c0;
> + i2c1 = &i2c1;
> + i2c2 = &i2c2;
> + i2c3 = &i2c3;
> + i2c4 = &i2c4;
> + i2c5 = &i2c5;
> + i2c6 = &i2c6;
> + mmc0 = &mmc0;
> + mmc1 = &mmc1;
> serial0 = &uart0;
> };
>
> @@ -249,7 +258,6 @@ &i2c3 {
> &i2c4 {
> pinctrl-names = "default";
> pinctrl-0 = <&i2c4_pins>;
> - pinctrl-1 = <&rt1715_int_pins>;
What is this extra change?
Please describe it in the commit description.
> clock-frequency = <1000000>;
> status = "okay";
> };
> @@ -867,6 +875,17 @@ pins-wifi-enable {
> &pmic {
> interrupt-parent = <&pio>;
> interrupts = <222 IRQ_TYPE_LEVEL_HIGH>;
> +
> + mt6359keys: keys {
Is there any reason why we can't just put this node in mt6359.dtsi?
Cheers,
Angelo
> + compatible = "mediatek,mt6359-keys";
> + mediatek,long-press-mode = <1>;
> + power-off-time-sec = <0>;
> +
> + power-key {
> + linux,keycodes = <KEY_POWER>;
> + wakeup-source;
> + };
> + };
> };
>
> &scp {
> @@ -874,6 +893,15 @@ &scp {
> status = "okay";
> };
>
> +&spi2 {
> + pinctrl-0 = <&spi2_pins>;
> + pinctrl-names = "default";
> + mediatek,pad-select = <0>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> + status = "okay";
> +};
> +
> &uart0 {
> pinctrl-0 = <&uart0_pins>;
> pinctrl-names = "default";
> @@ -892,15 +920,6 @@ &uart2 {
> status = "okay";
> };
>
> -&spi2 {
> - pinctrl-0 = <&spi2_pins>;
> - pinctrl-names = "default";
> - mediatek,pad-select = <0>;
> - #address-cells = <1>;
> - #size-cells = <0>;
> - status = "okay";
> -};
> -
> &u3phy0 {
> status = "okay";
> };
> @@ -921,6 +940,24 @@ &xhci0 {
> &xhci1 {
> status = "okay";
> vusb33-supply = <&mt6359_vusb_ldo_reg>;
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + hub_2_0: hub@1 {
> + compatible = "usb451,8025";
> + reg = <1>;
> + peer-hub = <&hub_3_0>;
> + reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
> + vdd-supply = <&usb_hub_fixed_3v3>;
> + };
> +
> + hub_3_0: hub@2 {
> + compatible = "usb451,8027";
> + reg = <2>;
> + peer-hub = <&hub_2_0>;
> + reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
> + vdd-supply = <&usb_hub_fixed_3v3>;
> + };
> };
>
> &xhci2 {
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH v2 2/2] arm64: dts: mediatek: mt8390-genio-700-evk: add keys and USB HUB
2024-10-08 8:20 ` AngeloGioacchino Del Regno
@ 2024-10-09 8:39 ` Macpaul Lin
0 siblings, 0 replies; 9+ messages in thread
From: Macpaul Lin @ 2024-10-09 8:39 UTC (permalink / raw)
To: AngeloGioacchino Del Regno, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Matthias Brugger, devicetree, linux-kernel,
linux-arm-kernel, linux-mediatek, Alexandre Mergnat
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Sen Chu, Chris-qj chen,
MediaTek Chromebook Upstream, Chen-Yu Tsai
On 10/8/24 16:20, AngeloGioacchino Del Regno wrote:
> Il 07/10/24 11:02, Macpaul Lin ha scritto:
>> 1. Add i2c, mmc to aliases.
>> 4. Add PMIC_KEY setting.
>> 5. Add USB HUB TUSB8020 to xhci1.
>> 6. Re-order spi2 node.
>
> Please either add the aliases in a different commit, or add that to the
> title.
>
> arm64: dts: mediatek: mt8390-genio-700-evk: Add aliases, keys and USB HUB
>
> Additionally, I'd really like to see a "conversation-like" description
> instead
> of a kind-of-checkbox list.
>
> Something like..
>
> "
> Add aliases for the I2C and MMC/SD controllers to keep the numbering
> consistent and
> describe the TUSB8020 hub present on the USB XHCI1 controller instance
> to enable
> resetting it with its specific reset GPIO.
>
> While at it, also move the spi2 node to keep nodes alphabetically ordered.
> "
Okay, Sorry for providing incorrect order of modified and added items.
I'll update in the next patch.
>>
>> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
>> ---
>> .../dts/mediatek/mt8390-genio-700-evk.dts | 57 +++++++++++++++----
>> 1 file changed, 47 insertions(+), 10 deletions(-)
>>
>> Changes for v2:
>> - Fix order of spi2.
>> - Update pinctrl in i2c4 and rt1715.
>> - Drop IT5205 and RT1715 nodes since the DTS are not completed yet.
>> - Add #address-cells and #size-cells to xhci1 for supporting USB hubs.
>>
>> diff --git a/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
>> b/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
>> index 96b272567cb1..3e77f59f2c74 100644
>> --- a/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
>> +++ b/arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dts
>> @@ -23,6 +23,15 @@ / {
>> "mediatek,mt8188";
>> aliases {
>> + i2c0 = &i2c0;
>> + i2c1 = &i2c1;
>> + i2c2 = &i2c2;
>> + i2c3 = &i2c3;
>> + i2c4 = &i2c4;
>> + i2c5 = &i2c5;
>> + i2c6 = &i2c6;
>> + mmc0 = &mmc0;
>> + mmc1 = &mmc1;
>> serial0 = &uart0;
>> };
>> @@ -249,7 +258,6 @@ &i2c3 {
>> &i2c4 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&i2c4_pins>;
>> - pinctrl-1 = <&rt1715_int_pins>;
>
> What is this extra change?
> Please describe it in the commit description.
okay, I'll fix this.
>> clock-frequency = <1000000>;
>> status = "okay";
>> };
>> @@ -867,6 +875,17 @@ pins-wifi-enable {
>> &pmic {
>> interrupt-parent = <&pio>;
>> interrupts = <222 IRQ_TYPE_LEVEL_HIGH>;
>> +
>> + mt6359keys: keys {
>
> Is there any reason why we can't just put this node in mt6359.dtsi?
That's a good point.
I think I can try to add this in a separate patch for mt6359.dtsi.
> Cheers,
> Angelo
>
>> + compatible = "mediatek,mt6359-keys";
>> + mediatek,long-press-mode = <1>;
>> + power-off-time-sec = <0>;
>> +
>> + power-key {
>> + linux,keycodes = <KEY_POWER>;
>> + wakeup-source;
>> + };
>> + };
>> };
>> &scp {
>> @@ -874,6 +893,15 @@ &scp {
>> status = "okay";
>> };
>> +&spi2 {
>> + pinctrl-0 = <&spi2_pins>;
>> + pinctrl-names = "default";
>> + mediatek,pad-select = <0>;
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> + status = "okay";
>> +};
>> +
>> &uart0 {
>> pinctrl-0 = <&uart0_pins>;
>> pinctrl-names = "default";
>> @@ -892,15 +920,6 @@ &uart2 {
>> status = "okay";
>> };
>> -&spi2 {
>> - pinctrl-0 = <&spi2_pins>;
>> - pinctrl-names = "default";
>> - mediatek,pad-select = <0>;
>> - #address-cells = <1>;
>> - #size-cells = <0>;
>> - status = "okay";
>> -};
>> -
>> &u3phy0 {
>> status = "okay";
>> };
>> @@ -921,6 +940,24 @@ &xhci0 {
>> &xhci1 {
>> status = "okay";
>> vusb33-supply = <&mt6359_vusb_ldo_reg>;
>> + #address-cells = <1>;
>> + #size-cells = <0>;
>> +
>> + hub_2_0: hub@1 {
>> + compatible = "usb451,8025";
>> + reg = <1>;
>> + peer-hub = <&hub_3_0>;
>> + reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
>> + vdd-supply = <&usb_hub_fixed_3v3>;
>> + };
>> +
>> + hub_3_0: hub@2 {
>> + compatible = "usb451,8027";
>> + reg = <2>;
>> + peer-hub = <&hub_2_0>;
>> + reset-gpios = <&pio 7 GPIO_ACTIVE_HIGH>;
>> + vdd-supply = <&usb_hub_fixed_3v3>;
>> + };
>> };
>> &xhci2 {
>
Thanks
Macpaul Lin
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names
2024-10-07 9:02 [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names Macpaul Lin
2024-10-07 9:02 ` [PATCH v2 2/2] arm64: dts: mediatek: mt8390-genio-700-evk: add keys and USB HUB Macpaul Lin
@ 2024-10-07 18:44 ` Rob Herring (Arm)
2024-10-08 9:16 ` AngeloGioacchino Del Regno
` (3 subsequent siblings)
5 siblings, 0 replies; 9+ messages in thread
From: Rob Herring (Arm) @ 2024-10-07 18:44 UTC (permalink / raw)
To: Macpaul Lin
Cc: Chen-Yu Tsai, Sen Chu, Krzysztof Kozlowski,
AngeloGioacchino Del Regno, Macpaul Lin, linux-arm-kernel,
Pablo Sun, Chris-qj chen, Conor Dooley, Bear Wang, devicetree,
MediaTek Chromebook Upstream, Alexandre Mergnat, linux-kernel,
Matthias Brugger, linux-mediatek
On Mon, 07 Oct 2024 17:02:43 +0800, Macpaul Lin wrote:
> Update regulator names to match schematics, replacing generic terms.
> 1. Add system wide 'reg_vsys' node for 4.2V power rail.
> 2. Add 'reg_vsys' node as 'vin-supply' for the following nodes
> - common_fixed_5v, edp_panel_fixed_3v3, gpio_fixed_3v3, sdio_fixed_3v3,
> touch0_fixed_3v3, usb_hub_fixed_3v3, usb_p0_vbus, and usb_p1_vbus.
> 3. Update regulator names according to the stable output name on
> schematics.
> - vdd_5v, vedp_3v3, ext_3v3, vio18_conn, wifi_3v3, vio33_tp1, vhub_3v3,
> vbus_p0, vbus_p1.
> - vcn18_pmu, vcn33_2_pmu, dvdd_proc_l, dvdd_core, vpa_pmu, dvdd_adsp,
> va12_abb2_pmu, vsim1_pmu, vufs18_pmu.
> 4. Remove usb_hub_reset_1v8. Use 'hub' node to probe USB HUB
> in subsequent patches.
>
> Suggested-by: Chen-Yu Tsai <wenst@chromium.org>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
> ---
> .../dts/mediatek/mt8390-genio-700-evk.dts | 65 ++++++++++++-------
> 1 file changed, 42 insertions(+), 23 deletions(-)
>
> Changes for v2:
> - Add Suggested-by: tag.
> - Rebase on mediatek/dts64 branch (v6.12-rc1)
>
My bot found new DTB warnings on the .dts files added or changed in this
series.
Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.
If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:
pip3 install dtschema --upgrade
New warnings running 'make CHECK_DTBS=y mediatek/mt8390-genio-700-evk.dtb' for 20241007090244.1731-1-macpaul.lin@mediatek.com:
arch/arm64/boot/dts/mediatek/mt8390-genio-700-evk.dtb: /soc/pwrap@10024000/pmic/keys: failed to match any schema with compatible: ['mediatek,mt6359-keys']
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names
2024-10-07 9:02 [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names Macpaul Lin
2024-10-07 9:02 ` [PATCH v2 2/2] arm64: dts: mediatek: mt8390-genio-700-evk: add keys and USB HUB Macpaul Lin
2024-10-07 18:44 ` [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names Rob Herring (Arm)
@ 2024-10-08 9:16 ` AngeloGioacchino Del Regno
2024-10-14 10:43 ` AngeloGioacchino Del Regno
` (2 subsequent siblings)
5 siblings, 0 replies; 9+ messages in thread
From: AngeloGioacchino Del Regno @ 2024-10-08 9:16 UTC (permalink / raw)
To: Macpaul Lin, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Matthias Brugger, devicetree, linux-kernel, linux-arm-kernel,
linux-mediatek, Alexandre Mergnat
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Sen Chu, Chris-qj chen,
MediaTek Chromebook Upstream, Chen-Yu Tsai
Il 07/10/24 11:02, Macpaul Lin ha scritto:
> Update regulator names to match schematics, replacing generic terms.
> 1. Add system wide 'reg_vsys' node for 4.2V power rail.
> 2. Add 'reg_vsys' node as 'vin-supply' for the following nodes
> - common_fixed_5v, edp_panel_fixed_3v3, gpio_fixed_3v3, sdio_fixed_3v3,
> touch0_fixed_3v3, usb_hub_fixed_3v3, usb_p0_vbus, and usb_p1_vbus.
> 3. Update regulator names according to the stable output name on
> schematics.
> - vdd_5v, vedp_3v3, ext_3v3, vio18_conn, wifi_3v3, vio33_tp1, vhub_3v3,
> vbus_p0, vbus_p1.
> - vcn18_pmu, vcn33_2_pmu, dvdd_proc_l, dvdd_core, vpa_pmu, dvdd_adsp,
> va12_abb2_pmu, vsim1_pmu, vufs18_pmu.
> 4. Remove usb_hub_reset_1v8. Use 'hub' node to probe USB HUB
> in subsequent patches.
>
> Suggested-by: Chen-Yu Tsai <wenst@chromium.org>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names
2024-10-07 9:02 [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names Macpaul Lin
` (2 preceding siblings ...)
2024-10-08 9:16 ` AngeloGioacchino Del Regno
@ 2024-10-14 10:43 ` AngeloGioacchino Del Regno
2024-10-14 10:43 ` AngeloGioacchino Del Regno
2024-10-14 10:45 ` AngeloGioacchino Del Regno
5 siblings, 0 replies; 9+ messages in thread
From: AngeloGioacchino Del Regno @ 2024-10-14 10:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Matthias Brugger,
devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Alexandre Mergnat, Macpaul Lin
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Sen Chu, Chris-qj chen,
MediaTek Chromebook Upstream, Chen-Yu Tsai
On Mon, 07 Oct 2024 17:02:43 +0800, Macpaul Lin wrote:
> Update regulator names to match schematics, replacing generic terms.
> 1. Add system wide 'reg_vsys' node for 4.2V power rail.
> 2. Add 'reg_vsys' node as 'vin-supply' for the following nodes
> - common_fixed_5v, edp_panel_fixed_3v3, gpio_fixed_3v3, sdio_fixed_3v3,
> touch0_fixed_3v3, usb_hub_fixed_3v3, usb_p0_vbus, and usb_p1_vbus.
> 3. Update regulator names according to the stable output name on
> schematics.
> - vdd_5v, vedp_3v3, ext_3v3, vio18_conn, wifi_3v3, vio33_tp1, vhub_3v3,
> vbus_p0, vbus_p1.
> - vcn18_pmu, vcn33_2_pmu, dvdd_proc_l, dvdd_core, vpa_pmu, dvdd_adsp,
> va12_abb2_pmu, vsim1_pmu, vufs18_pmu.
> 4. Remove usb_hub_reset_1v8. Use 'hub' node to probe USB HUB
> in subsequent patches.
>
> [...]
Applied to mediatek-next, thanks!
[1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names
commit: 54a1168b1a0adf9ee033b29bf32affca6059be7e
[2/2] arm64: dts: mediatek: mt8390-genio-700-evk: add keys and USB HUB
commit: 46c2803425543deff4cb9be01da0cbd4bd6045a7
Cheers,
Angelo
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names
2024-10-07 9:02 [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names Macpaul Lin
` (3 preceding siblings ...)
2024-10-14 10:43 ` AngeloGioacchino Del Regno
@ 2024-10-14 10:43 ` AngeloGioacchino Del Regno
2024-10-14 10:45 ` AngeloGioacchino Del Regno
5 siblings, 0 replies; 9+ messages in thread
From: AngeloGioacchino Del Regno @ 2024-10-14 10:43 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Matthias Brugger,
devicetree, linux-kernel, linux-arm-kernel, linux-mediatek,
Alexandre Mergnat, Macpaul Lin
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Sen Chu, Chris-qj chen,
MediaTek Chromebook Upstream, Chen-Yu Tsai
On Mon, 07 Oct 2024 17:02:43 +0800, Macpaul Lin wrote:
> Update regulator names to match schematics, replacing generic terms.
> 1. Add system wide 'reg_vsys' node for 4.2V power rail.
> 2. Add 'reg_vsys' node as 'vin-supply' for the following nodes
> - common_fixed_5v, edp_panel_fixed_3v3, gpio_fixed_3v3, sdio_fixed_3v3,
> touch0_fixed_3v3, usb_hub_fixed_3v3, usb_p0_vbus, and usb_p1_vbus.
> 3. Update regulator names according to the stable output name on
> schematics.
> - vdd_5v, vedp_3v3, ext_3v3, vio18_conn, wifi_3v3, vio33_tp1, vhub_3v3,
> vbus_p0, vbus_p1.
> - vcn18_pmu, vcn33_2_pmu, dvdd_proc_l, dvdd_core, vpa_pmu, dvdd_adsp,
> va12_abb2_pmu, vsim1_pmu, vufs18_pmu.
> 4. Remove usb_hub_reset_1v8. Use 'hub' node to probe USB HUB
> in subsequent patches.
>
> [...]
Applied to v6.12-next/dts64, thanks!
[1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names
commit: 4fcf4347c2a74bbc297b6f626de4ad0ca8fee96b
[2/2] arm64: dts: mediatek: mt8390-genio-700-evk: add keys and USB HUB
commit: 13799dcd86813793349bf2136937bca91ae73253
Cheers,
Angelo
^ permalink raw reply [flat|nested] 9+ messages in thread* Re: [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names
2024-10-07 9:02 [PATCH v2 1/2] arm64: dts: mediatek: mt8390-genio-700-evk: update regulator names Macpaul Lin
` (4 preceding siblings ...)
2024-10-14 10:43 ` AngeloGioacchino Del Regno
@ 2024-10-14 10:45 ` AngeloGioacchino Del Regno
5 siblings, 0 replies; 9+ messages in thread
From: AngeloGioacchino Del Regno @ 2024-10-14 10:45 UTC (permalink / raw)
To: Macpaul Lin, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Matthias Brugger, devicetree, linux-kernel, linux-arm-kernel,
linux-mediatek, Alexandre Mergnat
Cc: Bear Wang, Pablo Sun, Macpaul Lin, Sen Chu, Chris-qj chen,
MediaTek Chromebook Upstream, Chen-Yu Tsai
Il 07/10/24 11:02, Macpaul Lin ha scritto:
> Update regulator names to match schematics, replacing generic terms.
> 1. Add system wide 'reg_vsys' node for 4.2V power rail.
> 2. Add 'reg_vsys' node as 'vin-supply' for the following nodes
> - common_fixed_5v, edp_panel_fixed_3v3, gpio_fixed_3v3, sdio_fixed_3v3,
> touch0_fixed_3v3, usb_hub_fixed_3v3, usb_p0_vbus, and usb_p1_vbus.
> 3. Update regulator names according to the stable output name on
> schematics.
> - vdd_5v, vedp_3v3, ext_3v3, vio18_conn, wifi_3v3, vio33_tp1, vhub_3v3,
> vbus_p0, vbus_p1.
> - vcn18_pmu, vcn33_2_pmu, dvdd_proc_l, dvdd_core, vpa_pmu, dvdd_adsp,
> va12_abb2_pmu, vsim1_pmu, vufs18_pmu.
> 4. Remove usb_hub_reset_1v8. Use 'hub' node to probe USB HUB
> in subsequent patches.
>
> Suggested-by: Chen-Yu Tsai <wenst@chromium.org>
> Signed-off-by: Macpaul Lin <macpaul.lin@mediatek.com>
Please disregard the first "applied" email, as that was on the wrong kernel tree.
I confirm the series was applied to v6.12-next/dts64 and that this is the
right tree and right branch.
Cheers,
Angelo
^ permalink raw reply [flat|nested] 9+ messages in thread