From: "Jörn Engel" <joern@wohnheim.fh-wedel.de>
To: "Gareth Bult (Encryptec)" <Gareth@Encryptec.net>
Cc: Linux MTD <linux-mtd@lists.infradead.org>
Subject: [PATCH 7/22] Lindent
Date: Tue, 21 Dec 2004 14:48:18 +0100 [thread overview]
Message-ID: <20041221134818.GM22636@wohnheim.fh-wedel.de> (raw)
In-Reply-To: <20041221134724.GL22636@wohnheim.fh-wedel.de>
Ran some functions through Lindent. Mostly mindless stuff, some small
manual changes as well.
Signed-off-by: Jörn Engel <joern@wohnheim.fh-wedel.de>
---
blockmtd.c | 158 +++++++++++++++++++++++++++++++------------------------------
1 files changed, 82 insertions(+), 76 deletions(-)
--- linux-2.6.9cow/drivers/mtd/devices/blockmtd.c~blockmtd_lindent 2004-12-20 19:03:50.000000000 +0100
+++ linux-2.6.9cow/drivers/mtd/devices/blockmtd.c 2004-12-20 19:10:28.000000000 +0100
@@ -79,8 +79,8 @@
void cache_readahead(struct address_space *mapping, int index)
{
- filler_t *filler = (filler_t*)(mapping->a_ops->readpage);
- int i,pagei;
+ filler_t *filler = (void*)mapping->a_ops->readpage;
+ int i, pagei;
unsigned ret = 0;
unsigned long end_index;
struct page *page;
@@ -88,7 +88,7 @@
struct inode *inode = mapping->host;
loff_t isize = i_size_read(inode);
- if(!isize) {
+ if (!isize) {
printk(KERN_INFO "iSize=0 in cache_readahead\n");
return;
}
@@ -96,25 +96,28 @@
end_index = ((isize - 1) >> PAGE_CACHE_SHIFT);
spin_lock_irq(&mapping->tree_lock);
- for(i=0;i<PAGE_READAHEAD;i++) {
- pagei=index+i;
+ for (i = 0; i < PAGE_READAHEAD; i++) {
+ pagei = index + i;
if (pagei > end_index) {
printk(KERN_INFO "Overrun end of disk in cache readahead\n");
break;
}
page = radix_tree_lookup(&mapping->page_tree, pagei);
- if(page && (!i)) break;
- if(page) continue;
+ if (page && (!i))
+ break;
+ if (page)
+ continue;
spin_unlock_irq(&mapping->tree_lock);
page = page_cache_alloc_cold(mapping);
spin_lock_irq(&mapping->tree_lock);
- if(!page) break;
- page->index=pagei;
+ if (!page)
+ break;
+ page->index = pagei;
list_add(&page->lru, &page_pool);
ret++;
}
spin_unlock_irq(&mapping->tree_lock);
- if(ret) {
+ if (ret) {
read_cache_pages(mapping, &page_pool, filler, NULL);
//printk(KERN_INFO "Readahead [%d/%d] pages from [%d]\n",ret,PAGE_READAHEAD,index);
}
@@ -256,28 +259,26 @@
len = instr->len;
/* check erase region has valid start and length */
- while(numregions) {
- if(from >= einfo->offset
- && from < einfo->offset + (einfo->erasesize * einfo->numblocks)) {
- if(len == einfo->erasesize
- && ( (from - einfo->offset) % einfo->erasesize == 0))
+ while (numregions) {
+ if (from >= einfo->offset && from < einfo->offset + (einfo->erasesize * einfo->numblocks)) {
+ if (len == einfo->erasesize && ((from - einfo->offset) % einfo->erasesize == 0))
break;
}
numregions--;
einfo++;
}
- if(!numregions) {
+ if (!numregions) {
/* Not a valid erase block */
err("erase: invalid erase request 0x%lX @ 0x%08zX", len, from);
instr->state = MTD_ERASE_FAILED;
err = -EIO;
}
- if(instr->state != MTD_ERASE_FAILED) {
+ if (instr->state != MTD_ERASE_FAILED) {
/* do the erase */
err = write_pages(dev, NULL, from, len, &retlen);
- if(err || retlen != len) {
+ if (err || retlen != len) {
err("erase failed err = %d", err);
instr->state = MTD_ERASE_FAILED;
} else {
@@ -294,35 +295,41 @@
struct blkmtd_dev *dev = mtd->priv;
struct page *page;
//
- int index = from >> PAGE_SHIFT;
- int offset = from - (index << PAGE_SHIFT);
+ int index = from >> PAGE_SHIFT;
+ int offset = from - (index << PAGE_SHIFT);
int cpylen;
- if(from > mtd->size) return -EINVAL;
- if(from + len > mtd->size) len = mtd->size - from;
+ if (from > mtd->size)
+ return -EINVAL;
+ if (from + len > mtd->size)
+ len = mtd->size - from;
- if(retlen) *retlen = 0;
+ if (retlen)
+ *retlen = 0;
- while(len) {
- if((offset+len) > PAGE_SIZE)
+ while (len) {
+ if ((offset + len) > PAGE_SIZE)
cpylen = PAGE_SIZE - offset; // multiple pages
- else cpylen = len; // this page
+ else
+ cpylen = len; // this page
len = len - cpylen;
- //
- // Get page
- //
- page = page_readahead(dev->blkdev->bd_inode->i_mapping,index);
- if(!page) return -ENOMEM;
- if(IS_ERR(page)) return -EIO;
- //
+
+ // Get page
+ page = page_readahead(dev->blkdev->bd_inode->i_mapping, index);
+ if (!page)
+ return -ENOMEM;
+ if (IS_ERR(page))
+ return -EIO;
+
memcpy(buf, page_address(page) + offset, cpylen);
page_cache_release(page);
- //
- if(retlen) *retlen += cpylen;
- buf+=cpylen;
+
+ if (retlen)
+ *retlen += cpylen;
+ buf += cpylen;
offset = 0;
index++;
- //
+
}
return 0;
}
@@ -335,12 +342,16 @@
struct blkmtd_dev *dev = mtd->priv;
int err;
- if(!len) return 0;
- if(to >= mtd->size) return -ENOSPC;
- if(to + len > mtd->size) len = mtd->size - to;
+ if (!len)
+ return 0;
+ if (to >= mtd->size)
+ return -ENOSPC;
+ if (to + len > mtd->size)
+ len = mtd->size - to;
err = write_pages(dev, buf, to, len, retlen);
- if(err > 0) err = 0;
+ if (err > 0)
+ err = 0;
return err;
}
@@ -354,13 +365,13 @@
static void free_device(struct blkmtd_dev *dev)
{
- if(dev) {
- if(dev->mtd_info.eraseregions)
+ if (dev) {
+ if (dev->mtd_info.eraseregions)
kfree(dev->mtd_info.eraseregions);
- if(dev->mtd_info.name)
+ if (dev->mtd_info.name)
kfree(dev->mtd_info.name);
- if(dev->blkdev) {
+ if (dev->blkdev) {
invalidate_inode_pages(dev->blkdev->bd_inode->i_mapping);
close_bdev_excl(dev->blkdev);
}
@@ -381,14 +392,14 @@
/* Make any user specified erasesize be a power of 2
and at least PAGE_SIZE */
- if(erase_size) {
+ if (erase_size) {
int es = erase_size;
erase_size = 1;
- while(es != 1) {
+ while (es != 1) {
es >>= 1;
erase_size <<= 1;
}
- if(erase_size < PAGE_SIZE)
+ if (erase_size < PAGE_SIZE)
erase_size = PAGE_SIZE;
} else {
erase_size = CONFIG_MTD_BLKDEV_ERASESIZE;
@@ -401,28 +412,28 @@
int er_size = erase_size;
int count = 0, offset = 0, regcnt = 0;
- while(tot_size) {
+ while (tot_size) {
count = tot_size / er_size;
- if(count) {
+ if (count) {
tot_size = tot_size % er_size;
- if(info) {
- (info+regcnt)->offset = offset;
- (info+regcnt)->erasesize = er_size;
- (info+regcnt)->numblocks = count;
+ if (info) {
+ (info + regcnt)->offset = offset;
+ (info + regcnt)->erasesize = er_size;
+ (info + regcnt)->numblocks = count;
(*regions)++;
}
regcnt++;
offset += (count * er_size);
}
- while(er_size > tot_size)
+ while (er_size > tot_size)
er_size >>= 1;
}
- if(info == NULL) {
+ if (info == NULL) {
info = kmalloc(regcnt * sizeof(struct mtd_erase_region_info), GFP_KERNEL);
- if(!info)
+ if (!info)
break;
}
- } while(!(*regions));
+ } while (!(*regions));
return info;
}
@@ -436,12 +447,11 @@
int mode;
struct blkmtd_dev *dev;
- if(!devname)
+ if (!devname)
return NULL;
/* Get a handle on the device */
-
#ifdef MODULE
mode = O_RDWR;
bdev = open_bdev_excl(devname, mode, NULL);
@@ -449,19 +459,19 @@
mode = FMODE_WRITE;
bdev = open_by_devnum(name_to_dev_t(devname), mode);
#endif
- if(IS_ERR(bdev)) {
+ if (IS_ERR(bdev)) {
err("error: cannot open device %s", devname);
return NULL;
}
- if(MAJOR(bdev->bd_dev) == MTD_BLOCK_MAJOR) {
+ if (MAJOR(bdev->bd_dev) == MTD_BLOCK_MAJOR) {
err("attempting to use an MTD device as a block device");
blkdev_put(bdev);
return NULL;
}
dev = kmalloc(sizeof(struct blkmtd_dev), GFP_KERNEL);
- if(dev == NULL) {
+ if (dev == NULL) {
blkdev_put(bdev);
return NULL;
}
@@ -476,13 +486,12 @@
/* Setup the MTD structure */
/* make the name contain the block device in */
dev->mtd_info.name = kmalloc(sizeof("blkmtd: ") + strlen(devname), GFP_KERNEL);
- if(dev->mtd_info.name == NULL)
+ if (dev->mtd_info.name == NULL)
goto devinit_err;
sprintf(dev->mtd_info.name, "blkmtd: %s", devname);
- dev->mtd_info.eraseregions = calc_erase_regions(erase_size, dev->mtd_info.size,
- &dev->mtd_info.numeraseregions);
- if(dev->mtd_info.eraseregions == NULL)
+ dev->mtd_info.eraseregions = calc_erase_regions(erase_size, dev->mtd_info.size, &dev->mtd_info.numeraseregions);
+ if (dev->mtd_info.eraseregions == NULL)
goto devinit_err;
dev->mtd_info.erasesize = dev->mtd_info.eraseregions->erasesize;
@@ -503,14 +512,11 @@
list_del(&dev->list);
goto devinit_err;
} else {
- info("mtd%d: [%s] erase_size = %dKiB [%ld]",
- dev->mtd_info.index, dev->mtd_info.name + strlen("blkmtd: "),
- dev->mtd_info.erasesize >> 10,
- PAGE_SIZE);
+ info("mtd%d: [%s] erase_size = %dKiB [%ld]", dev->mtd_info.index, dev->mtd_info.name + strlen("blkmtd: "), dev->mtd_info.erasesize >> 10, PAGE_SIZE);
}
return dev;
- devinit_err:
+devinit_err:
free_device(dev);
return NULL;
}
@@ -524,7 +530,7 @@
{
int i;
- for(i = 0; i < MAX_DEVICES; i++) {
+ for (i = 0; i < MAX_DEVICES; i++) {
device[i] = str;
strsep(&str, ",");
}
@@ -535,9 +541,9 @@
static int __init param_blkmtd_erasesz(char *str)
{
int i;
- for(i = 0; i < MAX_DEVICES; i++) {
+ for (i = 0; i < MAX_DEVICES; i++) {
char *val = strsep(&str, ",");
- if(val)
+ if (val)
erasesz[i] = simple_strtoul(val, NULL, 0);
}
@@ -547,7 +553,7 @@
static int __init param_blkmtd_sync(char *str)
{
- if(str[0] == '1')
+ if (str[0] == '1')
sync = 1;
return 1;
}
next prev parent reply other threads:[~2004-12-21 13:48 UTC|newest]
Thread overview: 70+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-12-15 23:19 JFFS2 mount time Gareth Bult (Encryptec)
2004-12-16 0:15 ` Josh Boyer
2004-12-16 1:02 ` Gareth Bult (Encryptec)
2004-12-16 12:53 ` Josh Boyer
2004-12-16 21:22 ` Gareth Bult (Encryptec)
2004-12-16 21:28 ` Josh Boyer
2004-12-16 21:47 ` Gareth Bult (Encryptec)
2004-12-17 12:54 ` Josh Boyer
2004-12-17 15:33 ` Gareth Bult (Encryptec)
2004-12-17 16:02 ` Josh Boyer
2004-12-17 16:46 ` Gareth Bult (Encryptec)
2004-12-17 17:08 ` Artem B. Bityuckiy
2004-12-17 17:10 ` Josh Boyer
2004-12-17 17:26 ` Gareth Bult (Encryptec)
2004-12-17 17:35 ` Josh Boyer
2004-12-17 18:09 ` Gareth Bult (Encryptec)
2004-12-17 19:14 ` jasmine
2004-12-17 20:55 ` Gareth Bult (Encryptec)
2004-12-18 16:02 ` Jörn Engel
2004-12-20 16:34 ` Josh Boyer
2004-12-20 17:12 ` Gareth Bult (Encryptec)
2004-12-21 13:09 ` Jörn Engel
2004-12-21 13:24 ` Gareth Bult (Encryptec)
2004-12-21 13:34 ` Jörn Engel
2004-12-18 16:19 ` Jörn Engel
2004-12-18 17:32 ` Gareth Bult (Encryptec)
2004-12-18 17:52 ` Jörn Engel
2004-12-18 18:11 ` Jörn Engel
2004-12-18 20:48 ` Gareth Bult (Encryptec)
2004-12-19 2:44 ` Jörn Engel
2004-12-21 13:30 ` Jörn Engel
2004-12-21 13:39 ` [PATCH 1/22] Add drivers/mtd/devices/blockmtd.c Jörn Engel
2004-12-21 13:41 ` [PATCH 2/22] Add copyrights Jörn Engel
2004-12-21 13:42 ` [PATCH 3/22] Remove read-only option Jörn Engel
2004-12-21 13:44 ` [PATCH 4/22] Change init/exit functions Jörn Engel
2004-12-21 13:45 ` [PATCH 5/22] Remove gcc warnings Jörn Engel
2004-12-21 13:47 ` [PATCH 6/22] Remove debug macros Jörn Engel
2004-12-21 13:48 ` Jörn Engel [this message]
2004-12-21 13:49 ` [PATCH 8/22] Remove sync interface Jörn Engel
2004-12-21 13:51 ` [PATCH 9/22] Change parameter interface to phram-style Jörn Engel
2004-12-21 13:53 ` [PATCH 10/22] Cleanup macro usage Jörn Engel
2004-12-21 13:54 ` [PATCH 11/22] kfree simplifications Jörn Engel
2004-12-21 13:55 ` [PATCH 12/22] change blockmtd_sync Jörn Engel
2004-12-21 13:57 ` [PATCH 13/22] remove erase regions Jörn Engel
2004-12-21 14:01 ` [PATCH 14/22] Change add_device Jörn Engel
2004-12-21 14:02 ` [PATCH 15/22] Rename unreadable mutex Jörn Engel
2004-12-21 14:03 ` [PATCH 16/22] list changes Jörn Engel
2004-12-21 14:04 ` [PATCH 17/22] Rename central struct Jörn Engel
2004-12-21 14:06 ` [PATCH 18/22] Function renaming Jörn Engel
2004-12-21 14:09 ` [PATCH 19/22] Fold various erase functions Jörn Engel
2004-12-21 14:10 ` [PATCH 20/22] Fold various write functions Jörn Engel
2004-12-21 14:11 ` [PATCH 21/22] Default erase size Jörn Engel
2004-12-21 14:13 ` [PATCH 22/22] Readahead Jörn Engel
2004-12-21 18:42 ` [PATCH 13/22] remove erase regions Christopher Hoover
2004-12-21 18:49 ` Jörn Engel
2004-12-21 21:09 ` Christopher Hoover
2004-12-22 2:47 ` Eric W. Biederman
2004-12-22 8:59 ` Jörn Engel
2004-12-22 10:05 ` Eric W. Biederman
2004-12-22 10:41 ` Jörn Engel
2004-12-21 13:42 ` [PATCH 1/22] Add drivers/mtd/devices/blockmtd.c Gareth Bult (Encryptec)
2004-12-21 14:15 ` Jörn Engel
2004-12-21 13:40 ` JFFS2 mount time Gareth Bult (Encryptec)
2004-12-21 15:00 ` David Woodhouse
[not found] ` <1103644945.10792.175.camel@squizzey.bult.co.uk>
2004-12-21 16:04 ` Jörn Engel
2004-12-16 13:43 ` Ferenc Havasi
2004-12-20 16:01 ` Gareth Bult (Encryptec)
2004-12-20 16:09 ` Ferenc Havasi
2004-12-20 16:39 ` Gareth Bult (Encryptec)
2004-12-20 17:48 ` Gareth Bult (Encryptec)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20041221134818.GM22636@wohnheim.fh-wedel.de \
--to=joern@wohnheim.fh-wedel.de \
--cc=Gareth@Encryptec.net \
--cc=linux-mtd@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox