diff options
author | Rafał Miłecki <zajec5@gmail.com> | 2015-06-06 21:16:23 (GMT) |
---|---|---|
committer | Ralf Baechle <ralf@linux-mips.org> | 2015-06-21 19:54:32 (GMT) |
commit | 94a0535baf4d78246f5e35c347d484baef8aeb2e (patch) | |
tree | 1cc72f38cf2338b2d36404c688661982ef92d41b /arch/mips/bcm47xx | |
parent | cb2224d7c40e3d2dfc6f4a1676cd817acc79f012 (diff) | |
download | linux-94a0535baf4d78246f5e35c347d484baef8aeb2e.tar.xz |
MIPS: BCM47xx: Add helper variable for storing NVRAM length
This simplifies code just a bit (also maybe makes it a bit more
intuitive?) and will allow us to stop storing header. Right now we copy
whole NVRAM including its header to the internal buffer. It is not
needed to store a header as we don't access all these details like CRC,
flags, etc. The next improvement that should follow is copying only the
real contents.
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
Acked-by: Hauke Mehrtens <hauke@hauke-m.de>
Cc: linux-mips@linux-mips.org
Cc: Arend van Spriel <arend@broadcom.com>
Cc: Hante Meuleman <meuleman@broadcom.com>
Patchwork: https://patchwork.linux-mips.org/patch/10535/
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
Diffstat (limited to 'arch/mips/bcm47xx')
-rw-r--r-- | arch/mips/bcm47xx/nvram.c | 37 |
1 files changed, 16 insertions, 21 deletions
diff --git a/arch/mips/bcm47xx/nvram.c b/arch/mips/bcm47xx/nvram.c index 2ed762e..9ccdce8 100644 --- a/arch/mips/bcm47xx/nvram.c +++ b/arch/mips/bcm47xx/nvram.c @@ -35,6 +35,7 @@ struct nvram_header { }; static char nvram_buf[NVRAM_SPACE]; +static size_t nvram_len; static const u32 nvram_sizes[] = {0x8000, 0xF000, 0x10000}; static u32 find_nvram_size(void __iomem *end) @@ -60,7 +61,7 @@ static int nvram_find_and_copy(void __iomem *iobase, u32 lim) u32 *src, *dst; u32 size; - if (nvram_buf[0]) { + if (nvram_len) { pr_warn("nvram already initialized\n"); return -EEXIST; } @@ -99,17 +100,18 @@ found: for (i = 0; i < sizeof(struct nvram_header); i += 4) *dst++ = __raw_readl(src++); header = (struct nvram_header *)nvram_buf; - if (header->len > size) { + nvram_len = header->len; + if (nvram_len > size) { pr_err("The nvram size according to the header seems to be bigger than the partition on flash\n"); - header->len = size; + nvram_len = size; } - if (header->len >= NVRAM_SPACE) { + if (nvram_len >= NVRAM_SPACE) { pr_err("nvram on flash (%i bytes) is bigger than the reserved space in memory, will just copy the first %i bytes\n", header->len, NVRAM_SPACE - 1); - header->len = NVRAM_SPACE - 1; + nvram_len = NVRAM_SPACE - 1; } /* proceed reading data after header */ - for (; i < header->len; i += 4) + for (; i < nvram_len; i += 4) *dst++ = readl(src++); nvram_buf[NVRAM_SPACE - 1] = '\0'; @@ -144,7 +146,6 @@ static int nvram_init(void) #ifdef CONFIG_MTD struct mtd_info *mtd; struct nvram_header header; - struct nvram_header *pheader; size_t bytes_read; int err; @@ -155,20 +156,16 @@ static int nvram_init(void) err = mtd_read(mtd, 0, sizeof(header), &bytes_read, (uint8_t *)&header); if (!err && header.magic == NVRAM_MAGIC && header.len > sizeof(header)) { - if (header.len >= NVRAM_SPACE) { + nvram_len = header.len; + if (nvram_len >= NVRAM_SPACE) { pr_err("nvram on flash (%i bytes) is bigger than the reserved space in memory, will just copy the first %i bytes\n", header.len, NVRAM_SPACE); - header.len = NVRAM_SPACE - 1; + nvram_len = NVRAM_SPACE - 1; } - err = mtd_read(mtd, 0, header.len, &bytes_read, + err = mtd_read(mtd, 0, nvram_len, &nvram_len, (u8 *)nvram_buf); - if (err) - return err; - - pheader = (struct nvram_header *)nvram_buf; - pheader->len = header.len; - return 0; + return err; } #endif @@ -183,7 +180,7 @@ int bcm47xx_nvram_getenv(const char *name, char *val, size_t val_len) if (!name) return -EINVAL; - if (!nvram_buf[0]) { + if (!nvram_len) { err = nvram_init(); if (err) return err; @@ -231,16 +228,14 @@ char *bcm47xx_nvram_get_contents(size_t *nvram_size) { int err; char *nvram; - struct nvram_header *header; - if (!nvram_buf[0]) { + if (!nvram_len) { err = nvram_init(); if (err) return NULL; } - header = (struct nvram_header *)nvram_buf; - *nvram_size = header->len - sizeof(struct nvram_header); + *nvram_size = nvram_len - sizeof(struct nvram_header); nvram = vmalloc(*nvram_size); if (!nvram) return NULL; |