Signed-off-by: Rafał Miłecki <zajec5@gmail.com> SVN-Revision: 45942master
parent
4734c4459b
commit
b273bb1896
@ -0,0 +1,136 @@ |
||||
From f229d75f1472c4cd30f464e4a0f94f410046bd80 Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
|
||||
Date: Sat, 6 Jun 2015 23:16:23 +0200
|
||||
Subject: [PATCH] MIPS: BCM47xx: Add helper variable for storing NVRAM length
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
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>
|
||||
---
|
||||
arch/mips/bcm47xx/nvram.c | 37 ++++++++++++++++---------------------
|
||||
1 file 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;
|
||||
--
|
||||
1.8.4.5
|
||||
|
@ -0,0 +1,35 @@ |
||||
From 5521bb0c510ed5c1881636524badfb9bc951f6ac Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
|
||||
Date: Sun, 7 Jun 2015 13:26:44 +0200
|
||||
Subject: [PATCH] MIPS: BCM47xx: Don't select BCMA_HOST_PCI
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
SoC may have non-Broadcom PCI device attached or one may want to use
|
||||
totally different PCI driver.
|
||||
|
||||
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
|
||||
Cc: linux-mips@linux-mips.org
|
||||
Cc: Hauke Mehrtens <hauke@hauke-m.de>
|
||||
Patchwork: https://patchwork.linux-mips.org/patch/10537/
|
||||
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
||||
---
|
||||
arch/mips/bcm47xx/Kconfig | 1 -
|
||||
1 file changed, 1 deletion(-)
|
||||
|
||||
diff --git a/arch/mips/bcm47xx/Kconfig b/arch/mips/bcm47xx/Kconfig
|
||||
index fc21d36..51ed599 100644
|
||||
--- a/arch/mips/bcm47xx/Kconfig
|
||||
+++ b/arch/mips/bcm47xx/Kconfig
|
||||
@@ -25,7 +25,6 @@ config BCM47XX_BCMA
|
||||
select BCMA
|
||||
select BCMA_HOST_SOC
|
||||
select BCMA_DRIVER_MIPS
|
||||
- select BCMA_HOST_PCI if PCI
|
||||
select BCMA_DRIVER_PCI_HOSTMODE if PCI
|
||||
select BCMA_DRIVER_GPIO
|
||||
default y
|
||||
--
|
||||
1.8.4.5
|
||||
|
@ -0,0 +1,136 @@ |
||||
From f229d75f1472c4cd30f464e4a0f94f410046bd80 Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
|
||||
Date: Sat, 6 Jun 2015 23:16:23 +0200
|
||||
Subject: [PATCH] MIPS: BCM47xx: Add helper variable for storing NVRAM length
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
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>
|
||||
---
|
||||
arch/mips/bcm47xx/nvram.c | 37 ++++++++++++++++---------------------
|
||||
1 file 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;
|
||||
--
|
||||
1.8.4.5
|
||||
|
@ -0,0 +1,35 @@ |
||||
From 5521bb0c510ed5c1881636524badfb9bc951f6ac Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <zajec5@gmail.com>
|
||||
Date: Sun, 7 Jun 2015 13:26:44 +0200
|
||||
Subject: [PATCH] MIPS: BCM47xx: Don't select BCMA_HOST_PCI
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
SoC may have non-Broadcom PCI device attached or one may want to use
|
||||
totally different PCI driver.
|
||||
|
||||
Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
|
||||
Cc: linux-mips@linux-mips.org
|
||||
Cc: Hauke Mehrtens <hauke@hauke-m.de>
|
||||
Patchwork: https://patchwork.linux-mips.org/patch/10537/
|
||||
Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
|
||||
---
|
||||
arch/mips/bcm47xx/Kconfig | 1 -
|
||||
1 file changed, 1 deletion(-)
|
||||
|
||||
diff --git a/arch/mips/bcm47xx/Kconfig b/arch/mips/bcm47xx/Kconfig
|
||||
index fc21d36..51ed599 100644
|
||||
--- a/arch/mips/bcm47xx/Kconfig
|
||||
+++ b/arch/mips/bcm47xx/Kconfig
|
||||
@@ -25,7 +25,6 @@ config BCM47XX_BCMA
|
||||
select BCMA
|
||||
select BCMA_HOST_SOC
|
||||
select BCMA_DRIVER_MIPS
|
||||
- select BCMA_HOST_PCI if PCI
|
||||
select BCMA_DRIVER_PCI_HOSTMODE if PCI
|
||||
select BCMA_DRIVER_GPIO
|
||||
default y
|
||||
--
|
||||
1.8.4.5
|
||||
|
Loading…
Reference in new issue