* [PATCH] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes"
@ 2024-06-11 12:06 ` FUKAUMI Naoki
0 siblings, 0 replies; 14+ messages in thread
From: FUKAUMI Naoki @ 2024-06-11 12:06 UTC (permalink / raw)
To: heiko
Cc: robh, krzk+dt, conor+dt, kever.yang, devicetree, linux-rockchip,
FUKAUMI Naoki
This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6.
inserting and removing microSD card is not detected since this commit.
tested on rock-5a and rock-5b.
Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes")
Signed-off-by: FUKAUMI Naoki <naoki@radxa.com>
---
arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 +
arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 +
arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 +
arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 +
4 files changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts
index 1a604429fb26..e74871491ef5 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts
@@ -444,6 +444,7 @@ &sdhci {
&sdmmc {
bus-width = <4>;
cap-sd-highspeed;
+ cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>;
disable-wp;
max-frequency = <150000000>;
no-sdio;
diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
index b4f22d95ac0e..e80caa36f8e4 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts
@@ -435,6 +435,7 @@ &sdhci {
&sdmmc {
bus-width = <4>;
cap-sd-highspeed;
+ cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>;
disable-wp;
max-frequency = <150000000>;
no-sdio;
diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
index 4e2bf4eaef2b..df845929b084 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts
@@ -390,6 +390,7 @@ &sdmmc {
bus-width = <4>;
cap-mmc-highspeed;
cap-sd-highspeed;
+ cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>;
disable-wp;
sd-uhs-sdr104;
vmmc-supply = <&vcc_3v3_s3>;
diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts
index 8e2a07612d17..b070955627be 100644
--- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts
@@ -366,6 +366,7 @@ &sdmmc {
bus-width = <4>;
cap-mmc-highspeed;
cap-sd-highspeed;
+ cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>;
disable-wp;
max-frequency = <150000000>;
no-sdio;
--
2.43.0
_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip
^ permalink raw reply related [flat|nested] 14+ messages in thread* [PATCH] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" @ 2024-06-11 12:06 ` FUKAUMI Naoki 0 siblings, 0 replies; 14+ messages in thread From: FUKAUMI Naoki @ 2024-06-11 12:06 UTC (permalink / raw) To: heiko Cc: robh, krzk+dt, conor+dt, kever.yang, devicetree, linux-rockchip, FUKAUMI Naoki This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. inserting and removing microSD card is not detected since this commit. tested on rock-5a and rock-5b. Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> --- arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + 4 files changed, 4 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts index 1a604429fb26..e74871491ef5 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts @@ -444,6 +444,7 @@ &sdhci { &sdmmc { bus-width = <4>; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts index b4f22d95ac0e..e80caa36f8e4 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts @@ -435,6 +435,7 @@ &sdhci { &sdmmc { bus-width = <4>; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts index 4e2bf4eaef2b..df845929b084 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts @@ -390,6 +390,7 @@ &sdmmc { bus-width = <4>; cap-mmc-highspeed; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; sd-uhs-sdr104; vmmc-supply = <&vcc_3v3_s3>; diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts index 8e2a07612d17..b070955627be 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts @@ -366,6 +366,7 @@ &sdmmc { bus-width = <4>; cap-mmc-highspeed; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; -- 2.43.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" 2024-06-11 12:06 ` FUKAUMI Naoki @ 2024-06-11 12:53 ` Heiko Stübner -1 siblings, 0 replies; 14+ messages in thread From: Heiko Stübner @ 2024-06-11 12:53 UTC (permalink / raw) To: FUKAUMI Naoki Cc: robh, krzk+dt, conor+dt, kever.yang, devicetree, linux-rockchip, FUKAUMI Naoki Am Dienstag, 11. Juni 2024, 14:06:44 CEST schrieb FUKAUMI Naoki: > This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. > > inserting and removing microSD card is not detected since this commit. > tested on rock-5a and rock-5b. > > Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") > > Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> This is probably the same thing as 5 years ago on rk3399 https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ So we'll go back to cd-gpios for now. _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" @ 2024-06-11 12:53 ` Heiko Stübner 0 siblings, 0 replies; 14+ messages in thread From: Heiko Stübner @ 2024-06-11 12:53 UTC (permalink / raw) To: FUKAUMI Naoki Cc: robh, krzk+dt, conor+dt, kever.yang, devicetree, linux-rockchip, FUKAUMI Naoki Am Dienstag, 11. Juni 2024, 14:06:44 CEST schrieb FUKAUMI Naoki: > This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. > > inserting and removing microSD card is not detected since this commit. > tested on rock-5a and rock-5b. > > Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") > > Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> This is probably the same thing as 5 years ago on rk3399 https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ So we'll go back to cd-gpios for now. ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" 2024-06-11 12:06 ` FUKAUMI Naoki @ 2024-06-13 0:17 ` FUKAUMI Naoki -1 siblings, 0 replies; 14+ messages in thread From: FUKAUMI Naoki @ 2024-06-13 0:17 UTC (permalink / raw) To: heiko; +Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip, FUKAUMI Naoki This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. Inserting and removing microSD card is not detected since above commit. Reverting it fixes this problem. This is probably the same thing as 5 years ago on rk3399 https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ So we'll go back to cd-gpios for now. this patch is tested on Radxa ROCK 5A and 5B. Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> Changes in v2: - reword commit message - remove empty line --- arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + 4 files changed, 4 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts index 1a604429fb26..e74871491ef5 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts @@ -444,6 +444,7 @@ &sdhci { &sdmmc { bus-width = <4>; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts index b4f22d95ac0e..e80caa36f8e4 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts @@ -435,6 +435,7 @@ &sdhci { &sdmmc { bus-width = <4>; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts index 4e2bf4eaef2b..df845929b084 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts @@ -390,6 +390,7 @@ &sdmmc { bus-width = <4>; cap-mmc-highspeed; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; sd-uhs-sdr104; vmmc-supply = <&vcc_3v3_s3>; diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts index 8e2a07612d17..b070955627be 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts @@ -366,6 +366,7 @@ &sdmmc { bus-width = <4>; cap-mmc-highspeed; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; -- 2.43.0 _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" @ 2024-06-13 0:17 ` FUKAUMI Naoki 0 siblings, 0 replies; 14+ messages in thread From: FUKAUMI Naoki @ 2024-06-13 0:17 UTC (permalink / raw) To: heiko; +Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip, FUKAUMI Naoki This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. Inserting and removing microSD card is not detected since above commit. Reverting it fixes this problem. This is probably the same thing as 5 years ago on rk3399 https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ So we'll go back to cd-gpios for now. this patch is tested on Radxa ROCK 5A and 5B. Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> Changes in v2: - reword commit message - remove empty line --- arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + 4 files changed, 4 insertions(+) diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts index 1a604429fb26..e74871491ef5 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts @@ -444,6 +444,7 @@ &sdhci { &sdmmc { bus-width = <4>; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts index b4f22d95ac0e..e80caa36f8e4 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts @@ -435,6 +435,7 @@ &sdhci { &sdmmc { bus-width = <4>; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts index 4e2bf4eaef2b..df845929b084 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts @@ -390,6 +390,7 @@ &sdmmc { bus-width = <4>; cap-mmc-highspeed; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; sd-uhs-sdr104; vmmc-supply = <&vcc_3v3_s3>; diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts index 8e2a07612d17..b070955627be 100644 --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts @@ -366,6 +366,7 @@ &sdmmc { bus-width = <4>; cap-mmc-highspeed; cap-sd-highspeed; + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; disable-wp; max-frequency = <150000000>; no-sdio; -- 2.43.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" 2024-06-13 0:17 ` FUKAUMI Naoki @ 2024-06-17 21:16 ` Heiko Stübner -1 siblings, 0 replies; 14+ messages in thread From: Heiko Stübner @ 2024-06-17 21:16 UTC (permalink / raw) To: FUKAUMI Naoki Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip, FUKAUMI Naoki Am Donnerstag, 13. Juni 2024, 02:17:57 CEST schrieb FUKAUMI Naoki: > This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. > > Inserting and removing microSD card is not detected since above commit. > Reverting it fixes this problem. > > This is probably the same thing as 5 years ago on rk3399 > https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ > > So we'll go back to cd-gpios for now. > > this patch is tested on Radxa ROCK 5A and 5B. > > Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") > Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> applied as fix for 6.10 sidenote: please don't send v2 patches as replies to v1 but instead start a new mail thread. Tooling like b4 does not cope that well with multiple patch versions in the same thread. Thanks Heiko _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" @ 2024-06-17 21:16 ` Heiko Stübner 0 siblings, 0 replies; 14+ messages in thread From: Heiko Stübner @ 2024-06-17 21:16 UTC (permalink / raw) To: FUKAUMI Naoki Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip, FUKAUMI Naoki Am Donnerstag, 13. Juni 2024, 02:17:57 CEST schrieb FUKAUMI Naoki: > This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. > > Inserting and removing microSD card is not detected since above commit. > Reverting it fixes this problem. > > This is probably the same thing as 5 years ago on rk3399 > https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ > > So we'll go back to cd-gpios for now. > > this patch is tested on Radxa ROCK 5A and 5B. > > Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") > Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> applied as fix for 6.10 sidenote: please don't send v2 patches as replies to v1 but instead start a new mail thread. Tooling like b4 does not cope that well with multiple patch versions in the same thread. Thanks Heiko ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" 2024-06-13 0:17 ` FUKAUMI Naoki @ 2024-09-12 10:26 ` Kever Yang -1 siblings, 0 replies; 14+ messages in thread From: Kever Yang @ 2024-09-12 10:26 UTC (permalink / raw) To: FUKAUMI Naoki, heiko; +Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip Hi Naoki, Heiko, On 2024/6/13 08:17, FUKAUMI Naoki wrote: > This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. > > Inserting and removing microSD card is not detected since above commit. > Reverting it fixes this problem. > > This is probably the same thing as 5 years ago on rk3399 > https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ Sorry for didn't notice this patch. The SD card hotplug can not work because the controller is getting into runtime low power mode, so we need the fix in mmc driver to make the CD function work as below: https://lkml.org/lkml/2024/9/12/323 Thanks, - Kever > > So we'll go back to cd-gpios for now. > > this patch is tested on Radxa ROCK 5A and 5B. > > Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") > Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> > > Changes in v2: > - reword commit message > - remove empty line > --- > arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + > arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + > arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + > arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + > 4 files changed, 4 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts > index 1a604429fb26..e74871491ef5 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts > @@ -444,6 +444,7 @@ &sdhci { > &sdmmc { > bus-width = <4>; > cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; > disable-wp; > max-frequency = <150000000>; > no-sdio; > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts > index b4f22d95ac0e..e80caa36f8e4 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts > @@ -435,6 +435,7 @@ &sdhci { > &sdmmc { > bus-width = <4>; > cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; > disable-wp; > max-frequency = <150000000>; > no-sdio; > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts > index 4e2bf4eaef2b..df845929b084 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts > @@ -390,6 +390,7 @@ &sdmmc { > bus-width = <4>; > cap-mmc-highspeed; > cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; > disable-wp; > sd-uhs-sdr104; > vmmc-supply = <&vcc_3v3_s3>; > diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts > index 8e2a07612d17..b070955627be 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts > @@ -366,6 +366,7 @@ &sdmmc { > bus-width = <4>; > cap-mmc-highspeed; > cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; > disable-wp; > max-frequency = <150000000>; > no-sdio; _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" @ 2024-09-12 10:26 ` Kever Yang 0 siblings, 0 replies; 14+ messages in thread From: Kever Yang @ 2024-09-12 10:26 UTC (permalink / raw) To: FUKAUMI Naoki, heiko; +Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip Hi Naoki, Heiko, On 2024/6/13 08:17, FUKAUMI Naoki wrote: > This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. > > Inserting and removing microSD card is not detected since above commit. > Reverting it fixes this problem. > > This is probably the same thing as 5 years ago on rk3399 > https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ Sorry for didn't notice this patch. The SD card hotplug can not work because the controller is getting into runtime low power mode, so we need the fix in mmc driver to make the CD function work as below: https://lkml.org/lkml/2024/9/12/323 Thanks, - Kever > > So we'll go back to cd-gpios for now. > > this patch is tested on Radxa ROCK 5A and 5B. > > Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes") > Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> > > Changes in v2: > - reword commit message > - remove empty line > --- > arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + > arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + > arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + > arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + > 4 files changed, 4 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts > index 1a604429fb26..e74871491ef5 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts > @@ -444,6 +444,7 @@ &sdhci { > &sdmmc { > bus-width = <4>; > cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; > disable-wp; > max-frequency = <150000000>; > no-sdio; > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts > index b4f22d95ac0e..e80caa36f8e4 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts > @@ -435,6 +435,7 @@ &sdhci { > &sdmmc { > bus-width = <4>; > cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; > disable-wp; > max-frequency = <150000000>; > no-sdio; > diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts > index 4e2bf4eaef2b..df845929b084 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts > @@ -390,6 +390,7 @@ &sdmmc { > bus-width = <4>; > cap-mmc-highspeed; > cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; > disable-wp; > sd-uhs-sdr104; > vmmc-supply = <&vcc_3v3_s3>; > diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts > index 8e2a07612d17..b070955627be 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts > +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts > @@ -366,6 +366,7 @@ &sdmmc { > bus-width = <4>; > cap-mmc-highspeed; > cap-sd-highspeed; > + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; > disable-wp; > max-frequency = <150000000>; > no-sdio; ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" 2024-09-12 10:26 ` Kever Yang @ 2024-09-12 21:31 ` FUKAUMI Naoki -1 siblings, 0 replies; 14+ messages in thread From: FUKAUMI Naoki @ 2024-09-12 21:31 UTC (permalink / raw) To: Kever Yang, heiko; +Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip hi On 9/12/24 19:26, Kever Yang wrote: > Hi Naoki, Heiko, > > On 2024/6/13 08:17, FUKAUMI Naoki wrote: >> This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. >> >> Inserting and removing microSD card is not detected since above commit. >> Reverting it fixes this problem. >> >> This is probably the same thing as 5 years ago on rk3399 >> https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ > > Sorry for didn't notice this patch. thank you for your information! > The SD card hotplug can not work because the controller is getting into > runtime low power mode, > > so we need the fix in mmc driver to make the CD function work as below: > > https://lkml.org/lkml/2024/9/12/323 Heiko, just FYI, this doesn't fix a problem below https://lore.kernel.org/linux-rockchip/20240903085551.568-1-naoki@radxa.com/t/#u Best regards, -- FUKAUMI Naoki Radxa Computer (Shenzhen) Co., Ltd. > Thanks, > - Kever >> >> So we'll go back to cd-gpios for now. >> >> this patch is tested on Radxa ROCK 5A and 5B. >> >> Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios >> from rk3588 sdmmc nodes") >> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> >> >> Changes in v2: >> - reword commit message >> - remove empty line >> --- >> arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + >> 4 files changed, 4 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> index 1a604429fb26..e74871491ef5 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> @@ -444,6 +444,7 @@ &sdhci { >> &sdmmc { >> bus-width = <4>; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> index b4f22d95ac0e..e80caa36f8e4 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> @@ -435,6 +435,7 @@ &sdhci { >> &sdmmc { >> bus-width = <4>; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> index 4e2bf4eaef2b..df845929b084 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> @@ -390,6 +390,7 @@ &sdmmc { >> bus-width = <4>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> sd-uhs-sdr104; >> vmmc-supply = <&vcc_3v3_s3>; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> index 8e2a07612d17..b070955627be 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> @@ -366,6 +366,7 @@ &sdmmc { >> bus-width = <4>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" @ 2024-09-12 21:31 ` FUKAUMI Naoki 0 siblings, 0 replies; 14+ messages in thread From: FUKAUMI Naoki @ 2024-09-12 21:31 UTC (permalink / raw) To: Kever Yang, heiko; +Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip hi On 9/12/24 19:26, Kever Yang wrote: > Hi Naoki, Heiko, > > On 2024/6/13 08:17, FUKAUMI Naoki wrote: >> This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. >> >> Inserting and removing microSD card is not detected since above commit. >> Reverting it fixes this problem. >> >> This is probably the same thing as 5 years ago on rk3399 >> https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ > > Sorry for didn't notice this patch. thank you for your information! > The SD card hotplug can not work because the controller is getting into > runtime low power mode, > > so we need the fix in mmc driver to make the CD function work as below: > > https://lkml.org/lkml/2024/9/12/323 Heiko, just FYI, this doesn't fix a problem below https://lore.kernel.org/linux-rockchip/20240903085551.568-1-naoki@radxa.com/t/#u Best regards, -- FUKAUMI Naoki Radxa Computer (Shenzhen) Co., Ltd. > Thanks, > - Kever >> >> So we'll go back to cd-gpios for now. >> >> this patch is tested on Radxa ROCK 5A and 5B. >> >> Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios >> from rk3588 sdmmc nodes") >> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> >> >> Changes in v2: >> - reword commit message >> - remove empty line >> --- >> arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + >> 4 files changed, 4 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> index 1a604429fb26..e74871491ef5 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> @@ -444,6 +444,7 @@ &sdhci { >> &sdmmc { >> bus-width = <4>; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> index b4f22d95ac0e..e80caa36f8e4 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> @@ -435,6 +435,7 @@ &sdhci { >> &sdmmc { >> bus-width = <4>; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> index 4e2bf4eaef2b..df845929b084 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> @@ -390,6 +390,7 @@ &sdmmc { >> bus-width = <4>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> sd-uhs-sdr104; >> vmmc-supply = <&vcc_3v3_s3>; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> index 8e2a07612d17..b070955627be 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> @@ -366,6 +366,7 @@ &sdmmc { >> bus-width = <4>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" 2024-09-12 10:26 ` Kever Yang @ 2024-09-12 23:33 ` FUKAUMI Naoki -1 siblings, 0 replies; 14+ messages in thread From: FUKAUMI Naoki @ 2024-09-12 23:33 UTC (permalink / raw) To: Kever Yang, heiko; +Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip hi On 9/12/24 19:26, Kever Yang wrote: > Hi Naoki, Heiko, > > On 2024/6/13 08:17, FUKAUMI Naoki wrote: >> This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. >> >> Inserting and removing microSD card is not detected since above commit. >> Reverting it fixes this problem. >> >> This is probably the same thing as 5 years ago on rk3399 >> https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ > > Sorry for didn't notice this patch. > > The SD card hotplug can not work because the controller is getting into > runtime low power mode, > > so we need the fix in mmc driver to make the CD function work as below: > > https://lkml.org/lkml/2024/9/12/323 I tested this patch on Radxa E54C and ROCK 5A/5B/5C. CD works fine without cd-gpios. revert patch can be reverted :) thank you very much! Best regards, -- FUKAUMI Naoki Radxa Computer (Shenzhen) Co., Ltd. > Thanks, > - Kever >> >> So we'll go back to cd-gpios for now. >> >> this patch is tested on Radxa ROCK 5A and 5B. >> >> Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios >> from rk3588 sdmmc nodes") >> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> >> >> Changes in v2: >> - reword commit message >> - remove empty line >> --- >> arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + >> 4 files changed, 4 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> index 1a604429fb26..e74871491ef5 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> @@ -444,6 +444,7 @@ &sdhci { >> &sdmmc { >> bus-width = <4>; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> index b4f22d95ac0e..e80caa36f8e4 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> @@ -435,6 +435,7 @@ &sdhci { >> &sdmmc { >> bus-width = <4>; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> index 4e2bf4eaef2b..df845929b084 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> @@ -390,6 +390,7 @@ &sdmmc { >> bus-width = <4>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> sd-uhs-sdr104; >> vmmc-supply = <&vcc_3v3_s3>; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> index 8e2a07612d17..b070955627be 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> @@ -366,6 +366,7 @@ &sdmmc { >> bus-width = <4>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; > _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v2] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" @ 2024-09-12 23:33 ` FUKAUMI Naoki 0 siblings, 0 replies; 14+ messages in thread From: FUKAUMI Naoki @ 2024-09-12 23:33 UTC (permalink / raw) To: Kever Yang, heiko; +Cc: robh, krzk+dt, conor+dt, devicetree, linux-rockchip hi On 9/12/24 19:26, Kever Yang wrote: > Hi Naoki, Heiko, > > On 2024/6/13 08:17, FUKAUMI Naoki wrote: >> This reverts commit d859ad305ed19d9a77d8c8ecd22459b73da36ba6. >> >> Inserting and removing microSD card is not detected since above commit. >> Reverting it fixes this problem. >> >> This is probably the same thing as 5 years ago on rk3399 >> https://lore.kernel.org/all/0608599d485117a9d99f5fb274fbb1b55f6ba9f7.1547466003.git.robin.murphy@arm.com/ > > Sorry for didn't notice this patch. > > The SD card hotplug can not work because the controller is getting into > runtime low power mode, > > so we need the fix in mmc driver to make the CD function work as below: > > https://lkml.org/lkml/2024/9/12/323 I tested this patch on Radxa E54C and ROCK 5A/5B/5C. CD works fine without cd-gpios. revert patch can be reverted :) thank you very much! Best regards, -- FUKAUMI Naoki Radxa Computer (Shenzhen) Co., Ltd. > Thanks, > - Kever >> >> So we'll go back to cd-gpios for now. >> >> this patch is tested on Radxa ROCK 5A and 5B. >> >> Fixes: d859ad305ed1 ("arm64: dts: rockchip: remove redundant cd-gpios >> from rk3588 sdmmc nodes") >> Signed-off-by: FUKAUMI Naoki <naoki@radxa.com> >> >> Changes in v2: >> - reword commit message >> - remove empty line >> --- >> arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts | 1 + >> arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts | 1 + >> 4 files changed, 4 insertions(+) >> >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> index 1a604429fb26..e74871491ef5 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-orangepi-5-plus.dts >> @@ -444,6 +444,7 @@ &sdhci { >> &sdmmc { >> bus-width = <4>; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> index b4f22d95ac0e..e80caa36f8e4 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-quartzpro64.dts >> @@ -435,6 +435,7 @@ &sdhci { >> &sdmmc { >> bus-width = <4>; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> index 4e2bf4eaef2b..df845929b084 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588-rock-5b.dts >> @@ -390,6 +390,7 @@ &sdmmc { >> bus-width = <4>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> sd-uhs-sdr104; >> vmmc-supply = <&vcc_3v3_s3>; >> diff --git a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> index 8e2a07612d17..b070955627be 100644 >> --- a/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> +++ b/arch/arm64/boot/dts/rockchip/rk3588s-rock-5a.dts >> @@ -366,6 +366,7 @@ &sdmmc { >> bus-width = <4>; >> cap-mmc-highspeed; >> cap-sd-highspeed; >> + cd-gpios = <&gpio0 RK_PA4 GPIO_ACTIVE_LOW>; >> disable-wp; >> max-frequency = <150000000>; >> no-sdio; > ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2024-09-12 23:33 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-06-11 12:06 [PATCH] Revert "arm64: dts: rockchip: remove redundant cd-gpios from rk3588 sdmmc nodes" FUKAUMI Naoki 2024-06-11 12:06 ` FUKAUMI Naoki 2024-06-11 12:53 ` Heiko Stübner 2024-06-11 12:53 ` Heiko Stübner 2024-06-13 0:17 ` [PATCH v2] " FUKAUMI Naoki 2024-06-13 0:17 ` FUKAUMI Naoki 2024-06-17 21:16 ` Heiko Stübner 2024-06-17 21:16 ` Heiko Stübner 2024-09-12 10:26 ` Kever Yang 2024-09-12 10:26 ` Kever Yang 2024-09-12 21:31 ` FUKAUMI Naoki 2024-09-12 21:31 ` FUKAUMI Naoki 2024-09-12 23:33 ` FUKAUMI Naoki 2024-09-12 23:33 ` FUKAUMI Naoki
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.