diff options
author | Gregory Nutt <gnutt@nuttx.org> | 2013-10-31 10:40:10 -0600 |
---|---|---|
committer | Gregory Nutt <gnutt@nuttx.org> | 2013-10-31 10:40:10 -0600 |
commit | 069e297bc0c07b58594efaa2eab1ded52a307f82 (patch) | |
tree | b5ef7cfc07ee6a458795f6482fbe59380f617820 | |
parent | e9310cadab97326be44c8a8be6ca7259cec4cc80 (diff) | |
download | nuttx-069e297bc0c07b58594efaa2eab1ded52a307f82.tar.gz nuttx-069e297bc0c07b58594efaa2eab1ded52a307f82.tar.bz2 nuttx-069e297bc0c07b58594efaa2eab1ded52a307f82.zip |
Fix erase block vs page block confusion in MTD partition logic. From Ken Pettit
-rw-r--r-- | nuttx/ChangeLog | 3 | ||||
-rw-r--r-- | nuttx/drivers/mtd/mtd_partition.c | 13 |
2 files changed, 10 insertions, 6 deletions
diff --git a/nuttx/ChangeLog b/nuttx/ChangeLog index 83502eee6..941b02e41 100644 --- a/nuttx/ChangeLog +++ b/nuttx/ChangeLog @@ -5917,3 +5917,6 @@ header file (2013-10-30). * arch/arm/src/sama5/chip/sam_pwm.h: PWM register definition header file (2013-10-31). + * drivers/mtd/mtd_partition.c: Fix erase block vs page block confusion. + From Ken Pettit (2013-10-31). + diff --git a/nuttx/drivers/mtd/mtd_partition.c b/nuttx/drivers/mtd/mtd_partition.c index 699a2fa5a..91ccdc74f 100644 --- a/nuttx/drivers/mtd/mtd_partition.c +++ b/nuttx/drivers/mtd/mtd_partition.c @@ -162,9 +162,9 @@ static int part_erase(FAR struct mtd_dev_s *dev, off_t startblock, /* Make sure that erase would not extend past the end of the partition */ - if (!part_blockcheck(priv, startblock + nblocks - 1)) + if (!part_blockcheck(priv, (startblock + nblocks - 1) * priv->blkpererase)) { - fdbg("ERROR: Read beyond the end of the partition\n"); + fdbg("ERROR: Erase beyond the end of the partition\n"); return -ENXIO; } @@ -379,8 +379,9 @@ static int part_ioctl(FAR struct mtd_dev_s *dev, int cmd, unsigned long arg) { /* Erase the entire partition */ - ret = priv->parent->erase(priv->parent, priv->firstblock, - priv->neraseblocks * priv->blkpererase); + ret = priv->parent->erase(priv->parent, + priv->firstblock / priv->blkpererase, + priv->neraseblocks); } break; @@ -461,7 +462,7 @@ FAR struct mtd_dev_s *mtd_partition(FAR struct mtd_dev_s *mtd, off_t firstblock, fdbg("ERROR: sub-region too small\n"); return NULL; } - + /* Verify that the sub-region is valid for this geometry */ devblocks = blkpererase * geo.neraseblocks; @@ -477,7 +478,7 @@ FAR struct mtd_dev_s *mtd_partition(FAR struct mtd_dev_s *mtd, off_t firstblock, if (!part) { fdbg("ERROR: Failed to allocate memory for the partition device\n"); - return NULL; + return NULL; } /* Initialize the partition device structure. (unsupported methods were |