diff options
author | Anton Salikhmetov <alexo@tuxera.com> | 2010-12-16 11:08:41 -0500 |
---|---|---|
committer | Christoph Hellwig <hch@tuxera.com> | 2010-12-16 12:08:46 -0500 |
commit | b2837fcf4994e699a4def002e26f274d95b387c1 (patch) | |
tree | fcb2f4e91b11746972e9f1abd9e4b70e7573286c | |
parent | 20b7643d8ee44254fc972d42655bace81e7ab50a (diff) |
hfsplus: %L-to-%ll, macro correction, and remove unneeded braces
Clean-up based on checkpatch.pl report against unnecessary braces
(`{' and `}'), non-standard format option %Lu (%llu recommended)
as well as one trailing statement in a macro definition which
should have been on the next line.
Signed-off-by: Anton Salikhmetov <alexo@tuxera.com>
Signed-off-by: Christoph Hellwig <hch@tuxera.com>
-rw-r--r-- | fs/hfsplus/bnode.c | 9 | ||||
-rw-r--r-- | fs/hfsplus/extents.c | 16 | ||||
-rw-r--r-- | fs/hfsplus/hfsplus_fs.h | 3 |
3 files changed, 14 insertions, 14 deletions
diff --git a/fs/hfsplus/bnode.c b/fs/hfsplus/bnode.c index c8f4ea4bb5ed..1c42cc5b899f 100644 --- a/fs/hfsplus/bnode.c +++ b/fs/hfsplus/bnode.c | |||
@@ -363,9 +363,8 @@ void hfs_bnode_unlink(struct hfs_bnode *node) | |||
363 | tree->leaf_tail = node->prev; | 363 | tree->leaf_tail = node->prev; |
364 | 364 | ||
365 | /* move down? */ | 365 | /* move down? */ |
366 | if (!node->prev && !node->next) { | 366 | if (!node->prev && !node->next) |
367 | dprint(DBG_BNODE_MOD, "hfs_btree_del_level\n"); | 367 | dprint(DBG_BNODE_MOD, "hfs_btree_del_level\n"); |
368 | } | ||
369 | if (!node->parent) { | 368 | if (!node->parent) { |
370 | tree->root = 0; | 369 | tree->root = 0; |
371 | tree->depth = 0; | 370 | tree->depth = 0; |
@@ -392,11 +391,9 @@ struct hfs_bnode *hfs_bnode_findhash(struct hfs_btree *tree, u32 cnid) | |||
392 | } | 391 | } |
393 | 392 | ||
394 | for (node = tree->node_hash[hfs_bnode_hash(cnid)]; | 393 | for (node = tree->node_hash[hfs_bnode_hash(cnid)]; |
395 | node; node = node->next_hash) { | 394 | node; node = node->next_hash) |
396 | if (node->this == cnid) { | 395 | if (node->this == cnid) |
397 | return node; | 396 | return node; |
398 | } | ||
399 | } | ||
400 | return NULL; | 397 | return NULL; |
401 | } | 398 | } |
402 | 399 | ||
diff --git a/fs/hfsplus/extents.c b/fs/hfsplus/extents.c index 4b693b5b6230..52a0bcaa7b6d 100644 --- a/fs/hfsplus/extents.c +++ b/fs/hfsplus/extents.c | |||
@@ -397,12 +397,13 @@ int hfsplus_file_extend(struct inode *inode) | |||
397 | u32 start, len, goal; | 397 | u32 start, len, goal; |
398 | int res; | 398 | int res; |
399 | 399 | ||
400 | if (sbi->alloc_file->i_size * 8 < | 400 | if (sbi->total_blocks - sbi->free_blocks + 8 > |
401 | sbi->total_blocks - sbi->free_blocks + 8) { | 401 | sbi->alloc_file->i_size * 8) { |
402 | /* extend alloc file */ | 402 | /* extend alloc file */ |
403 | printk(KERN_ERR "hfs: extend alloc file! (%Lu,%u,%u)\n", | 403 | printk(KERN_ERR "hfs: extend alloc file! " |
404 | sbi->alloc_file->i_size * 8, | 404 | "(%llu,%u,%u)\n", |
405 | sbi->total_blocks, sbi->free_blocks); | 405 | sbi->alloc_file->i_size * 8, |
406 | sbi->total_blocks, sbi->free_blocks); | ||
406 | return -ENOSPC; | 407 | return -ENOSPC; |
407 | } | 408 | } |
408 | 409 | ||
@@ -490,8 +491,9 @@ void hfsplus_file_truncate(struct inode *inode) | |||
490 | u32 alloc_cnt, blk_cnt, start; | 491 | u32 alloc_cnt, blk_cnt, start; |
491 | int res; | 492 | int res; |
492 | 493 | ||
493 | dprint(DBG_INODE, "truncate: %lu, %Lu -> %Lu\n", | 494 | dprint(DBG_INODE, "truncate: %lu, %llu -> %llu\n", |
494 | inode->i_ino, (long long)hip->phys_size, inode->i_size); | 495 | inode->i_ino, (long long)hip->phys_size, |
496 | inode->i_size); | ||
495 | 497 | ||
496 | if (inode->i_size > hip->phys_size) { | 498 | if (inode->i_size > hip->phys_size) { |
497 | struct address_space *mapping = inode->i_mapping; | 499 | struct address_space *mapping = inode->i_mapping; |
diff --git a/fs/hfsplus/hfsplus_fs.h b/fs/hfsplus/hfsplus_fs.h index a15e1c11e820..f7cbdf89ac9b 100644 --- a/fs/hfsplus/hfsplus_fs.h +++ b/fs/hfsplus/hfsplus_fs.h | |||
@@ -31,7 +31,8 @@ | |||
31 | #define DBG_MASK (0) | 31 | #define DBG_MASK (0) |
32 | 32 | ||
33 | #define dprint(flg, fmt, args...) \ | 33 | #define dprint(flg, fmt, args...) \ |
34 | if (flg & DBG_MASK) printk(fmt , ## args) | 34 | if (flg & DBG_MASK) \ |
35 | printk(fmt , ## args) | ||
35 | 36 | ||
36 | /* Runtime config options */ | 37 | /* Runtime config options */ |
37 | #define HFSPLUS_DEF_CR_TYPE 0x3F3F3F3F /* '????' */ | 38 | #define HFSPLUS_DEF_CR_TYPE 0x3F3F3F3F /* '????' */ |