diff mbox series

[2/2] mvebu: enable 4K sectors config option

Message ID 20240701073943.3886539-2-mrkiko.rs@gmail.com
State Superseded
Headers show
Series [1/2] mvebu: GL-MV1000: add custom boot script | expand

Commit Message

Enrico Mioso July 1, 2024, 7:37 a.m. UTC
Enable the CONFIG_MTD_SPI_NOR_USE_4K_SECTORS kernel option to allow for
U-Boot environment writing. This might be hiding a problem somewhere else,
since the w25q128fw chip supports 32K erases, still this change makes it
much easier to switch the GL-MV1000 boot media without an UART cable
connection.

Thanks to @robimarko for the precious hints.

Signed-off-by: Enrico Mioso <mrkiko.rs@gmail.com>
---
 target/linux/mvebu/config-6.6 | 1 +
 1 file changed, 1 insertion(+)

Comments

Thibaut VARÈNE July 1, 2024, 9:52 a.m. UTC | #1
Hi,

Disclaimer: I know nothing about this arch or this hw, however this CONFIG will make the entire flash storage significantly slower. Have you considered CONFIG_MTD_SPI_NOR_USE_VARIABLE_ERASE ? It allows writing to partitions that do not fit a 64K EB by selectively using 4K EB.

Cheers,
T

> Le 1 juil. 2024 à 09:37, Enrico Mioso <mrkiko.rs@gmail.com> a écrit :
> 
> Enable the CONFIG_MTD_SPI_NOR_USE_4K_SECTORS kernel option to allow for
> U-Boot environment writing. This might be hiding a problem somewhere else,
> since the w25q128fw chip supports 32K erases, still this change makes it
> much easier to switch the GL-MV1000 boot media without an UART cable
> connection.
> 
> Thanks to @robimarko for the precious hints.
> 
> Signed-off-by: Enrico Mioso <mrkiko.rs@gmail.com>
> ---
> target/linux/mvebu/config-6.6 | 1 +
> 1 file changed, 1 insertion(+)
> 
> diff --git a/target/linux/mvebu/config-6.6 b/target/linux/mvebu/config-6.6
> index 88e5fff4d9..b8302eb798 100644
> --- a/target/linux/mvebu/config-6.6
> +++ b/target/linux/mvebu/config-6.6
> @@ -270,6 +270,7 @@ CONFIG_MTD_NAND_ECC_SW_HAMMING=y
> CONFIG_MTD_NAND_MARVELL=y
> CONFIG_MTD_RAW_NAND=y
> CONFIG_MTD_SPI_NOR=y
> +CONFIG_MTD_SPI_NOR_USE_4K_SECTORS=y
> CONFIG_MTD_SPLIT_FIRMWARE=y
> CONFIG_MTD_UBI=y
> CONFIG_MTD_UBI_BEB_LIMIT=20
> -- 
> 2.45.2
> 
> 
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/mailman/listinfo/openwrt-devel
Enrico Mioso July 1, 2024, 12:30 p.m. UTC | #2
On Mon, Jul 01, 2024 at 11:52:05AM +0200, Thibaut wrote:
> Hi,
> 
> Disclaimer: I know nothing about this arch or this hw, however this CONFIG will make the entire flash storage significantly slower. Have you considered CONFIG_MTD_SPI_NOR_USE_VARIABLE_ERASE ? It allows writing to partitions that do not fit a 64K EB by selectively using 4K EB.
> 
> Cheers,
> T

Hi!

I didn't know about this option. I will try that out and come back with result.

Thanks a lot,

Enrico



> 
> > Le 1 juil. 2024 à 09:37, Enrico Mioso <mrkiko.rs@gmail.com> a écrit :
> > 
> > Enable the CONFIG_MTD_SPI_NOR_USE_4K_SECTORS kernel option to allow for
> > U-Boot environment writing. This might be hiding a problem somewhere else,
> > since the w25q128fw chip supports 32K erases, still this change makes it
> > much easier to switch the GL-MV1000 boot media without an UART cable
> > connection.
> > 
> > Thanks to @robimarko for the precious hints.
> > 
> > Signed-off-by: Enrico Mioso <mrkiko.rs@gmail.com>
> > ---
> > target/linux/mvebu/config-6.6 | 1 +
> > 1 file changed, 1 insertion(+)
> > 
> > diff --git a/target/linux/mvebu/config-6.6 b/target/linux/mvebu/config-6.6
> > index 88e5fff4d9..b8302eb798 100644
> > --- a/target/linux/mvebu/config-6.6
> > +++ b/target/linux/mvebu/config-6.6
> > @@ -270,6 +270,7 @@ CONFIG_MTD_NAND_ECC_SW_HAMMING=y
> > CONFIG_MTD_NAND_MARVELL=y
> > CONFIG_MTD_RAW_NAND=y
> > CONFIG_MTD_SPI_NOR=y
> > +CONFIG_MTD_SPI_NOR_USE_4K_SECTORS=y
> > CONFIG_MTD_SPLIT_FIRMWARE=y
> > CONFIG_MTD_UBI=y
> > CONFIG_MTD_UBI_BEB_LIMIT=20
> > -- 
> > 2.45.2
> > 
> > 
> > _______________________________________________
> > openwrt-devel mailing list
> > openwrt-devel@lists.openwrt.org
> > https://lists.openwrt.org/mailman/listinfo/openwrt-devel
>
diff mbox series

Patch

diff --git a/target/linux/mvebu/config-6.6 b/target/linux/mvebu/config-6.6
index 88e5fff4d9..b8302eb798 100644
--- a/target/linux/mvebu/config-6.6
+++ b/target/linux/mvebu/config-6.6
@@ -270,6 +270,7 @@  CONFIG_MTD_NAND_ECC_SW_HAMMING=y
 CONFIG_MTD_NAND_MARVELL=y
 CONFIG_MTD_RAW_NAND=y
 CONFIG_MTD_SPI_NOR=y
+CONFIG_MTD_SPI_NOR_USE_4K_SECTORS=y
 CONFIG_MTD_SPLIT_FIRMWARE=y
 CONFIG_MTD_UBI=y
 CONFIG_MTD_UBI_BEB_LIMIT=20