]> git.ipfire.org Git - thirdparty/u-boot.git/commitdiff
fw_env: move "erasesize" calculation up in a code
authorRafał Miłecki <rafal@milecki.pl>
Sat, 21 Feb 2026 10:00:10 +0000 (11:00 +0100)
committerTom Rini <trini@konsulko.com>
Fri, 6 Mar 2026 16:32:41 +0000 (10:32 -0600)
Calculate "erasesize" in the same place where other offsets and lengths
/ sizes are calculated. It makes code more consistent and will allow
further cleanups.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
tools/env/fw_env.c

index 8ca6a3e5478eecbb2a4a3a9b2f1ec3c9efb5ff92..de3a33a380d1cfa92f9f8d9315308950561c59b6 100644 (file)
@@ -1007,6 +1007,7 @@ static int flash_write_buf(int dev, int fd, void *buf, size_t count)
        if (DEVTYPE(dev) == MTD_ABSENT) {
                blocklen = count;
                erase_len = blocklen;
+               erasesize = erase_len;
                block_seek = 0;
                write_total = blocklen;
        } else {
@@ -1016,6 +1017,15 @@ static int flash_write_buf(int dev, int fd, void *buf, size_t count)
 
                /* Maximum area we may use */
                erase_len = environment_end(dev) - erase_offset;
+               if (DEVTYPE(dev) == MTD_NANDFLASH) {
+                       /*
+                        * NAND: calculate which blocks we are writing. We have
+                        * to write one block at a time to skip bad blocks.
+                        */
+                       erasesize = blocklen;
+               } else {
+                       erasesize = erase_len;
+               }
 
                /* Offset inside a block */
                block_seek = DEVOFFSET(dev) - erase_offset;
@@ -1069,16 +1079,6 @@ static int flash_write_buf(int dev, int fd, void *buf, size_t count)
                data = buf;
        }
 
-       if (DEVTYPE(dev) == MTD_NANDFLASH) {
-               /*
-                * NAND: calculate which blocks we are writing. We have
-                * to write one block at a time to skip bad blocks.
-                */
-               erasesize = blocklen;
-       } else {
-               erasesize = erase_len;
-       }
-
        erase.length = erasesize;
 
        /* This only runs once on NOR flash and SPI-dataflash */