[v2] riscv: dts: starfive: jh7110-common: drop no-sdio property from mmc1

[PATCH v2] riscv: dts: starfive: jh7110-common: drop no-sdio property from mmc1

Posted by E Shattow 4 months, 2 weeks ago

Relax no-sdio restriction on mmc1 for jh7110 boards. Property was
introduced for StarFive VisionFive2 dts to configure mmc1 for SD Card
but this is not necessary, the restriction is only needed to block use of
commands that would cause a device to malfunction.

Signed-off-by: E Shattow <e@freeshell.de>
---
 arch/riscv/boot/dts/starfive/jh7110-common.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
index 4baeb981d4df..b156f8703016 100644
--- a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
+++ b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
@@ -290,7 +290,6 @@ &mmc1 {
 	assigned-clock-rates = <50000000>;
 	bus-width = <4>;
 	bootph-pre-ram;
-	no-sdio;
 	no-mmc;
 	cd-gpios = <&sysgpio 41 GPIO_ACTIVE_LOW>;
 	disable-wp;

base-commit: 038d61fd642278bab63ee8ef722c50d10ab01e8f
-- 
2.50.0

Re: [PATCH v2] riscv: dts: starfive: jh7110-common: drop no-sdio property from mmc1

Posted by Conor Dooley 3 months, 3 weeks ago

On Tue, Jul 29, 2025 at 07:11:35AM -0700, E Shattow wrote:
> Relax no-sdio restriction on mmc1 for jh7110 boards. Property was
> introduced for StarFive VisionFive2 dts to configure mmc1 for SD Card
> but this is not necessary, the restriction is only needed to block use of
> commands that would cause a device to malfunction.
> 
> Signed-off-by: E Shattow <e@freeshell.de>

I'm going to apply this one to for-next, to give it more of a chance to
soak.

> ---
>  arch/riscv/boot/dts/starfive/jh7110-common.dtsi | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
> index 4baeb981d4df..b156f8703016 100644
> --- a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
> +++ b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
> @@ -290,7 +290,6 @@ &mmc1 {
>  	assigned-clock-rates = <50000000>;
>  	bus-width = <4>;
>  	bootph-pre-ram;
> -	no-sdio;
>  	no-mmc;
>  	cd-gpios = <&sysgpio 41 GPIO_ACTIVE_LOW>;
>  	disable-wp;
> 
> base-commit: 038d61fd642278bab63ee8ef722c50d10ab01e8f
> -- 
> 2.50.0
>

Re: [PATCH v2] riscv: dts: starfive: jh7110-common: drop no-sdio property from mmc1

Posted by ChaosEsque Team 2 months, 1 week ago

E, Conor.
That white boy you used to play paintball with.

On Tue, Aug 19, 2025 at 2:13 PM Conor Dooley <conor@kernel.org> wrote:
>
> On Tue, Jul 29, 2025 at 07:11:35AM -0700, E Shattow wrote:
> > Relax no-sdio restriction on mmc1 for jh7110 boards. Property was
> > introduced for StarFive VisionFive2 dts to configure mmc1 for SD Card
> > but this is not necessary, the restriction is only needed to block use of
> > commands that would cause a device to malfunction.
> >
> > Signed-off-by: E Shattow <e@freeshell.de>
>
> I'm going to apply this one to for-next, to give it more of a chance to
> soak.
>
> > ---
> >  arch/riscv/boot/dts/starfive/jh7110-common.dtsi | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
> > index 4baeb981d4df..b156f8703016 100644
> > --- a/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
> > +++ b/arch/riscv/boot/dts/starfive/jh7110-common.dtsi
> > @@ -290,7 +290,6 @@ &mmc1 {
> >       assigned-clock-rates = <50000000>;
> >       bus-width = <4>;
> >       bootph-pre-ram;
> > -     no-sdio;
> >       no-mmc;
> >       cd-gpios = <&sysgpio 41 GPIO_ACTIVE_LOW>;
> >       disable-wp;
> >
> > base-commit: 038d61fd642278bab63ee8ef722c50d10ab01e8f
> > --
> > 2.50.0
> >