summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorStefan Tauner <stefan.tauner@alumni.tuwien.ac.at>2013-03-03 23:49:48 +0000
committerStefan Tauner <stefan.tauner@alumni.tuwien.ac.at>2013-03-03 23:49:48 +0000
commit50d67aaa79441e8043aa1de32bb40ae94fcc51b2 (patch)
treef1b620f41753f30ca126947b6e634156fea996cb
parent352e50b79e084c8f5f768d7b6f2ba6b6bd2ea8ce (diff)
downloadast2050-flashrom-50d67aaa79441e8043aa1de32bb40ae94fcc51b2.zip
ast2050-flashrom-50d67aaa79441e8043aa1de32bb40ae94fcc51b2.tar.gz
Make write granularity a chip attribute
Corresponding to flashrom svn r1651. Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
-rw-r--r--flash.h3
-rw-r--r--flashrom.c3
2 files changed, 3 insertions, 3 deletions
diff --git a/flash.h b/flash.h
index a479286..9c4b0ac 100644
--- a/flash.h
+++ b/flash.h
@@ -68,9 +68,9 @@ enum chipbustype {
* - 256 bytes: If less than 256 bytes are written, the contents of the unwritten bytes are undefined.
*/
enum write_granularity {
+ write_gran_256bytes = 0, /* We assume 256 byte granularity by default. */
write_gran_1bit,
write_gran_1byte,
- write_gran_256bytes,
};
/*
@@ -162,6 +162,7 @@ struct flashchip {
uint16_t min;
uint16_t max;
} voltage;
+ enum write_granularity gran;
};
struct flashctx {
diff --git a/flashrom.c b/flashrom.c
index e1be398..225b6f0 100644
--- a/flashrom.c
+++ b/flashrom.c
@@ -1273,7 +1273,7 @@ static int erase_and_write_block_helper(struct flashctx *flash,
{
unsigned int starthere = 0, lenhere = 0;
int ret = 0, skip = 1, writecount = 0;
- enum write_granularity gran = write_gran_256bytes; /* FIXME */
+ enum write_granularity gran = flash->chip->gran;
/* curcontents and newcontents are opaque to walk_eraseregions, and
* need to be adjusted here to keep the impression of proper abstraction
@@ -1281,7 +1281,6 @@ static int erase_and_write_block_helper(struct flashctx *flash,
curcontents += start;
newcontents += start;
msg_cdbg(":");
- /* FIXME: Assume 256 byte granularity for now to play it safe. */
if (need_erase(curcontents, newcontents, len, gran)) {
msg_cdbg("E");
ret = erasefn(flash, start, len);
OpenPOWER on IntegriCloud