[U-Boot] [PATCH] mtd: denali: do not allocate page_buffer in .bss section

Masahiro Yamada yamada.masahiro at socionext.com
Thu Aug 27 11:52:36 CEST 2015


Since commit 2580a2a7e719 ("mtd: nand: Increase max sizes of OOB and
Page size"), three boards (ph1_ld4, ph1_pro4, ph1_sld8) fail to build
with the following error message:
  arm-linux-gnueabi-ld.bfd: SPL image plus BSS too big

They compile drivers/mtd/nand/denali_spl.c and it has a page_buffer
as static data:

    static uint8_t page_buffer[NAND_MAX_PAGESIZE];

This buffer required 8KB in .bss section before that commit and now
it has been increased to 16KB.  Given limited code/memory size for SPL,
it is not a good idea to allocate a page buffer statically.  In the
first place, the load address 'dst' can be used as a page buffer.

Signed-off-by: Masahiro Yamada <yamada.masahiro at socionext.com>
---

 drivers/mtd/nand/denali_spl.c | 31 +++++++++++--------------------
 1 file changed, 11 insertions(+), 20 deletions(-)

diff --git a/drivers/mtd/nand/denali_spl.c b/drivers/mtd/nand/denali_spl.c
index e98f537..04c59b9 100644
--- a/drivers/mtd/nand/denali_spl.c
+++ b/drivers/mtd/nand/denali_spl.c
@@ -22,7 +22,6 @@ static void __iomem *denali_flash_reg =
 			(void __iomem *)CONFIG_SYS_NAND_REGS_BASE;
 
 static const int flash_bank;
-static uint8_t page_buffer[NAND_MAX_PAGESIZE];
 static int page_size, oob_size, pages_per_block;
 
 static void index_addr(uint32_t address, uint32_t data)
@@ -144,15 +143,15 @@ static int nand_read_page(void *buf, int page)
 	return 0;
 }
 
-static int nand_block_isbad(int block)
+static int nand_block_isbad(void *buf, int block)
 {
 	int ret;
 
-	ret = nand_read_oob(page_buffer, block * pages_per_block);
+	ret = nand_read_oob(buf, block * pages_per_block);
 	if (ret < 0)
 		return ret;
 
-	return page_buffer[CONFIG_SYS_NAND_BAD_BLOCK_POS] != 0xff;
+	return *((uint8_t *)buf + CONFIG_SYS_NAND_BAD_BLOCK_POS) != 0xff;
 }
 
 /* nand_init() - initialize data to make nand usable by SPL */
@@ -184,7 +183,7 @@ int nand_spl_load_image(uint32_t offs, unsigned int size, void *dst)
 
 	while (size) {
 		if (force_bad_block_check || page == 0) {
-			ret = nand_block_isbad(block);
+			ret = nand_block_isbad(dst, block);
 			if (ret < 0)
 				return ret;
 
@@ -196,24 +195,16 @@ int nand_spl_load_image(uint32_t offs, unsigned int size, void *dst)
 
 		force_bad_block_check = 0;
 
-		if (unlikely(column || size < page_size)) {
-			/* Partial page read */
-			ret = nand_read_page(page_buffer,
-					     block * pages_per_block + page);
-			if (ret < 0)
-				return ret;
+		ret = nand_read_page(dst, block * pages_per_block + page);
+		if (ret < 0)
+			return ret;
 
-			readlen = min(page_size - column, (int)size);
-			memcpy(dst, page_buffer, readlen);
+		readlen = min(page_size - column, (int)size);
 
+		if (unlikely(column)) {
+			/* Partial page read */
+			memmove(dst, dst + column, readlen);
 			column = 0;
-		} else {
-			ret = nand_read_page(dst,
-					     block * pages_per_block + page);
-			if (ret < 0)
-				return ret;
-
-			readlen = page_size;
 		}
 
 		size -= readlen;
-- 
1.9.1



More information about the U-Boot mailing list