9
0
Fork 0

mtd: fix bad block ioctls

Since this commit we interpret the argument to the bad block ioctls
as a pointer to a 64bit number:

|commit e71c343668
|Author: Sascha Hauer <s.hauer@pengutronix.de>
|Date:   Fri Oct 14 11:57:55 2011 +0200
|
|    mtd: fix arguments to bad block ioctls
|
|    In the Kernel the mtd ioctls expect a pointer to the offset, whereas
|    barebox interprets the pointer itself as an offset. Since we want
|    to add 64bit support for file sizes a pointer may not be sufficient,
|    so align with the kernel and convert it to a pointer to the offset.
|
|    Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>

This missed some places, fix them aswell.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
This commit is contained in:
Sascha Hauer 2012-07-08 22:29:39 +02:00
parent 75821bdef5
commit 4987f61ee2
3 changed files with 6 additions and 5 deletions

View File

@ -79,6 +79,7 @@ static int do_nand(int argc, char *argv[])
if (command & NAND_MARKBAD) {
if (optind < argc) {
int ret = 0, fd;
loff_t __badblock = badblock;
printf("marking block at 0x%08x on %s as bad\n", badblock, argv[optind]);
@ -88,7 +89,7 @@ static int do_nand(int argc, char *argv[])
return 1;
}
ret = ioctl(fd, MEMSETBADBLOCK, (void *)badblock);
ret = ioctl(fd, MEMSETBADBLOCK, &__badblock);
if (ret)
perror("ioctl");

View File

@ -307,11 +307,11 @@ static int do_nandtest(int argc, char *argv[])
for (test_ofs = flash_offset;
test_ofs < flash_offset+length;
test_ofs += meminfo.erasesize) {
loff_t __test_ofs = test_ofs;
srand(seed);
seed = rand();
if (ioctl(fd, MEMGETBADBLOCK, (void *)test_ofs)) {
if (ioctl(fd, MEMGETBADBLOCK, &__test_ofs)) {
printf("\rBad block at 0x%08x\n",
(unsigned)(test_ofs +
memregion.offset));

View File

@ -216,7 +216,7 @@ static int nand_bb_calc_size(struct nand_bb *bb)
static loff_t nand_bb_lseek(struct cdev *cdev, loff_t __offset)
{
struct nand_bb *bb = cdev->priv;
unsigned long raw_pos = 0;
loff_t raw_pos = 0;
uint32_t offset = __offset;
int ret;
@ -226,7 +226,7 @@ static loff_t nand_bb_lseek(struct cdev *cdev, loff_t __offset)
while (raw_pos < bb->raw_size) {
off_t now = min(offset, bb->info.erasesize);
ret = cdev_ioctl(bb->cdev_parent, MEMGETBADBLOCK, (void *)raw_pos);
ret = cdev_ioctl(bb->cdev_parent, MEMGETBADBLOCK, &raw_pos);
if (ret < 0)
return ret;
if (!ret) {