On Tue, Dec 25, 2007 at 11:54:38AM +, Christoph Hellwig wrote:
Looks generally good, but it would be nice if you could avoid
introducing new overly long lines. Then again this really won't be
noticed in udf anyway and it looks like you plan to clean it up later
anyway.
Yes, that's what
convert byte order of constant instead of variable
it will be done at compile time (vs run time)
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
---
drivers/net/qla3xxx.c |6 ++
1 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/net/qla3xxx.c b/drivers/net/qla3xxx.c
convert byte order of constant instead of variable
it will be done at compile time (vs run time)
ext3/4 bits are #if 0'ed, but someone might copy this code into other places
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
---
fs/ext2/super.c |8 +++-
fs/ext3/super.c |2 +-
fs/ext4
convert byte order of constant instead of variable
it will be done at compile time (vs run time)
remove unused le32_and_cpu
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Mark Fasheh [EMAIL PROTECTED]
CC: Kurt Hackel [EMAIL PROTECTED]
---
fs/ocfs2/endian.h |5 -
fs/ocfs2/inode.c
+ memset to kcalloc
- check if kcalloc failed (partially)
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
CC: Christoph Hellwig [EMAIL PROTECTED]
---
fs/udf/super.c | 25 +++--
fs/udf/udf_sb.h | 13 -
2
fix sparse warnings:
fs/udf/super.c:1431:24: warning: symbol 'bh' shadows an earlier one
fs/udf/super.c:1347:21: originally declared here
fs/udf/super.c:472:6: warning: symbol 'udf_write_super' was not declared.
Should it be static?
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
C
udf_load_logicalvol may fail eg in out of memory conditions - check it
and propagate error further
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
CC: Christoph Hellwig <[EMAIL PROTECTED]>
---
fs/udf/s
convert UDF_SB_ALLOC_BITMAP macro to udf_sb_alloc_bitmap function
convert UDF_SB_FREE_BITMAP macro to udf_sb_free_bitmap function
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
CC: Christoph Hellwig
- convert UDF_SB_ALLOC_PARTMAPS macro to udf_sb_alloc_partition_maps function
- convert kmalloc + memset to kzalloc
- check if kzalloc failed (partially)
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
CC:
warnings, 1872 lines checked
all 164 warnings left are lines longer than 80 characters;
this file has too much indentation with really long expressions
to break all those lines now
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[
to super.c
19 didn't change
22,23 was combined into one and functions moved from udf_sb.h to super.c
24 didn't change
Marcin Slusarz
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.
to super.c
19 didn't change
22,23 was combined into one and functions moved from udf_sb.h to super.c
24 didn't change
Marcin Slusarz
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org
warnings, 1872 lines checked
all 164 warnings left are lines longer than 80 characters;
this file has too much indentation with really long expressions
to break all those lines now
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf
- convert UDF_SB_ALLOC_PARTMAPS macro to udf_sb_alloc_partition_maps function
- convert kmalloc + memset to kzalloc
- check if kzalloc failed (partially)
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
CC: Christoph Hellwig [EMAIL
udf_load_logicalvol may fail eg in out of memory conditions - check it
and propagate error further
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
CC: Christoph Hellwig [EMAIL PROTECTED]
---
fs/udf/super.c |7 ++-
1 files
convert UDF_SB_ALLOC_BITMAP macro to udf_sb_alloc_bitmap function
convert UDF_SB_FREE_BITMAP macro to udf_sb_free_bitmap function
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
CC: Christoph Hellwig [EMAIL PROTECTED]
---
fs/udf
fix sparse warnings:
fs/udf/super.c:1431:24: warning: symbol 'bh' shadows an earlier one
fs/udf/super.c:1347:21: originally declared here
fs/udf/super.c:472:6: warning: symbol 'udf_write_super' was not declared.
Should it be static?
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema
: symbol 'i' shadows an earlier one
fs/udf/super.c:1701:6: originally declared here
fs/udf/super.c:1714:4: warning: symbol 'i' shadows an earlier one
fs/udf/super.c:1701:6: originally declared here
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan
fix warnings:
fs/udf/super.c:1370:24: warning: symbol 'bh' shadows an earlier one
fs/udf/super.c:1288:21: originally declared here
fs/udf/super.c:450:6: warning: symbol 'udf_write_super' was not declared.
Should it be static?
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben F
remove macros:
- UDF_SB_PARTTYPE
- UDF_SB_PARTROOT
- UDF_SB_PARTLEN
- UDF_SB_PARTVSN
- UDF_SB_PARTNUM
- UDF_SB_TYPESPAR
- UDF_SB_TYPEVIRT
- UDF_SB_PARTFUNC
- UDF_SB_PARTFLAGS
and replace all uses
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/super.c |4 ++--
fs/udf/udf_sb.h | 37 -
2 files changed, 22 insertions(+), 19 deletions(-)
diff --git
udf_load_logicalvol may fail eg in out of memory conditions - check it
and propagate error further
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/super.c |7 ++-
1 files changed,
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/ialloc.c |2 +-
fs/udf/udf_sb.h |8 ++--
2 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/fs/udf/ialloc.c b/fs/udf/iall
- change UDF_SB_VAT macro to udf_sb_vat_inode inline function
- rename s_vat field to s_vat_inode
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/partition.c|6 +++---
fs/udf/super.c
- change UDF_SB_ALLOC_PARTMAPS macro to udf_sb_alloc_partition_maps inline
function
- convert kmalloc + memset to kzalloc
- check if kzalloc failed (partially)
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTE
- change UDF_SB_RECORDTIME macro to udf_sb_record_time inline function
- rename s_recordtime field to s_record_time
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/inode.c| 14
- change UDF_SB_SERIALNUM macro to udf_sb_serial_number inline function
- rename s_serialnum field to s_serial_number
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/inode.c|2
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/balloc.c |2 +-
fs/udf/inode.c| 14 +++---
fs/udf/misc.c |2 +-
fs/udf/namei.c|2 +-
fs/udf/super.c| 10
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/ialloc.c | 16
fs/udf/super.c | 34 +-
fs/udf/udf_sb.h | 10 --
3 files changed, 33
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/udf_sb.h |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/fs/udf/udf_sb.h b/fs/udf/udf_sb.h
index a9ff8fb..8683b71 1
- change UDF_SB_LVIDBH macro to udf_sb_lvid_bh inline function
- rename s_lvidbh field to s_lvid_bh
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/balloc.c | 26 +---
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/balloc.c | 24
fs/udf/ialloc.c |2 +-
fs/udf/namei.c |4 ++--
fs/udf/super.c | 40 --
- change UDF_SB_LASTBLOCK macro to udf_sb_last_block inline function
- rename s_lastblock field to s_last_block
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/super.c| 20 ++
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/misc.c |8
fs/udf/super.c | 28 ++--
fs/udf/udf_sb.h |6 +-
3 files changed, 23 insertions(+), 19
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/super.c | 44 ++--
fs/udf/udf_sb.h |6 +-
2 files changed, 27 insertions(+), 23 deletions(-)
d
- change UDF_SB_VOLIDENT macro to udf_sb_volume_ident inline function
- rename s_volident field to s_volume_ident
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/file.c |2
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/partition.c |6 +++---
fs/udf/super.c | 20 ++--
fs/udf/udf_sb.h| 10 +++---
3 files changed, 20 insertion
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/balloc.c | 10
fs/udf/super.c | 68 +++---
fs/udf/udf_sb.h |6 -
3 files chan
macro UDF_SB_PARTMAPS -> function udf_sb_partmaps
macro UDF_SB_FREE -> function udf_sb_free
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/balloc.c | 24 ++--
warnings, 1872 lines checked
all 164 warnings left are lines longer than 80 characters;
this file has too much indentation with really long expressions
to break all those lines
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROT
rename UDF_SB function to udf_sb
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
CC: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/balloc.c | 12 ++--
fs/udf/ialloc.c |4 ++--
fs/udf/inode.c |6 +++---
fs/ud
going to do more cleanups later.
PS: This is the first time I use git-send-email - tell me if I do something
wrong.
Marcin Slusarz
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.
On Fri, Dec 21, 2007 at 04:21:17PM -0800, Andrew Morton wrote:
> On Fri, 21 Dec 2007 16:55:03 +0100
> Marcin Slusarz <[EMAIL PROTECTED]> wrote:
>
> > sparse generated:
> > fs/udf/dir.c:78:5: warning: symbol 'udf_readdir' was not declared. Should
> > it be
On Fri, Dec 21, 2007 at 04:21:17PM -0800, Andrew Morton wrote:
On Fri, 21 Dec 2007 16:55:03 +0100
Marcin Slusarz [EMAIL PROTECTED] wrote:
sparse generated:
fs/udf/dir.c:78:5: warning: symbol 'udf_readdir' was not declared. Should
it be static?
there are 2 different prototypes
going to do more cleanups later.
PS: This is the first time I use git-send-email - tell me if I do something
wrong.
Marcin Slusarz
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org
warnings, 1872 lines checked
all 164 warnings left are lines longer than 80 characters;
this file has too much indentation with really long expressions
to break all those lines
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf
rename UDF_SB function to udf_sb
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/balloc.c | 12 ++--
fs/udf/ialloc.c |4 ++--
fs/udf/inode.c |6 +++---
fs/udf/super.c | 32
macro UDF_SB_PARTMAPS - function udf_sb_partmaps
macro UDF_SB_FREE - function udf_sb_free
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/balloc.c | 24 ++--
fs/udf/super.c | 52
- change UDF_SB_VOLIDENT macro to udf_sb_volume_ident inline function
- rename s_volident field to s_volume_ident
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/file.c |2 +-
fs/udf/super.c
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/partition.c |6 +++---
fs/udf/super.c | 20 ++--
fs/udf/udf_sb.h| 10 +++---
3 files changed, 20 insertions(+), 16 deletions(-)
diff
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/balloc.c | 10
fs/udf/super.c | 68 +++---
fs/udf/udf_sb.h |6 -
3 files changed, 44 insertions(+), 40
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/misc.c |8
fs/udf/super.c | 28 ++--
fs/udf/udf_sb.h |6 +-
3 files changed, 23 insertions(+), 19 deletions(-)
diff --git
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/super.c | 44 ++--
fs/udf/udf_sb.h |6 +-
2 files changed, 27 insertions(+), 23 deletions(-)
diff --git a/fs/udf/super.c
- change UDF_SB_LASTBLOCK macro to udf_sb_last_block inline function
- rename s_lastblock field to s_last_block
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/super.c| 20 ++--
fs/udf
- change UDF_SB_LVIDBH macro to udf_sb_lvid_bh inline function
- rename s_lvidbh field to s_lvid_bh
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/balloc.c | 26 +-
fs/udf/ialloc.c
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/balloc.c | 24
fs/udf/ialloc.c |2 +-
fs/udf/namei.c |4 ++--
fs/udf/super.c | 40
fs/udf
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/ialloc.c | 16
fs/udf/super.c | 34 +-
fs/udf/udf_sb.h | 10 --
3 files changed, 33 insertions(+), 27
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/udf_sb.h |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/fs/udf/udf_sb.h b/fs/udf/udf_sb.h
index a9ff8fb..8683b71 100644
--- a/fs/udf/udf_sb.h
- change UDF_SB_RECORDTIME macro to udf_sb_record_time inline function
- rename s_recordtime field to s_record_time
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/inode.c| 14 +++---
fs/udf/super.c
- change UDF_SB_SERIALNUM macro to udf_sb_serial_number inline function
- rename s_serialnum field to s_serial_number
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/inode.c|2 +-
fs/udf/misc.c
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/balloc.c |2 +-
fs/udf/inode.c| 14 +++---
fs/udf/misc.c |2 +-
fs/udf/namei.c|2 +-
fs/udf/super.c| 10 +-
fs/udf
- change UDF_SB_VAT macro to udf_sb_vat_inode inline function
- rename s_vat field to s_vat_inode
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/partition.c|6 +++---
fs/udf/super.c| 24
- change UDF_SB_ALLOC_PARTMAPS macro to udf_sb_alloc_partition_maps inline
function
- convert kmalloc + memset to kzalloc
- check if kzalloc failed (partially)
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/super.c
udf_load_logicalvol may fail eg in out of memory conditions - check it
and propagate error further
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/super.c |7 ++-
1 files changed, 6 insertions(+), 1 deletions
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/ialloc.c |2 +-
fs/udf/udf_sb.h |8 ++--
2 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/fs/udf/ialloc.c b/fs/udf/ialloc.c
index 269f588..4d1684c
remove macros:
- UDF_SB_PARTTYPE
- UDF_SB_PARTROOT
- UDF_SB_PARTLEN
- UDF_SB_PARTVSN
- UDF_SB_PARTNUM
- UDF_SB_TYPESPAR
- UDF_SB_TYPEVIRT
- UDF_SB_PARTFUNC
- UDF_SB_PARTFLAGS
and replace all uses
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL PROTECTED]
---
fs/udf/super.c |4 ++--
fs/udf/udf_sb.h | 37 -
2 files changed, 22 insertions(+), 19 deletions(-)
diff --git a/fs/udf/super.c b/fs/udf
: symbol 'i' shadows an earlier one
fs/udf/super.c:1701:6: originally declared here
fs/udf/super.c:1714:4: warning: symbol 'i' shadows an earlier one
fs/udf/super.c:1701:6: originally declared here
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
CC: Jan Kara [EMAIL
fix warnings:
fs/udf/super.c:1370:24: warning: symbol 'bh' shadows an earlier one
fs/udf/super.c:1288:21: originally declared here
fs/udf/super.c:450:6: warning: symbol 'udf_write_super' was not declared.
Should it be static?
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL
in argument 3 (different
signedness)
fs/udf/namei.c:1152:78:expected int *offset
fs/udf/namei.c:1152:78:got unsigned int *
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
Acked-by: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/namei.c |2 +-
1 files changed, 1 insertions(+),
e.c: In function 'udf_get_block':
fs/udf/inode.c:299: warning: 'phys' may be used uninitialized in this function
initialize it to 0 (if someday someone will break inode_getblk we will catch it
immediately)
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
fix coding style, checkpatch warned about:
- "braces {} are not necessary for single statement blocks"
- "line over 80 characters"
- "do not use assignment in if condition"
- "use tabs not spaces"
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC:
make reading do_udf_readdir easier by adding new variable
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
Acked-by: Jan Kara <[EMAIL PROTECTED]>
---
fs/udf/dir.c | 15 +--
1 files changed, 9 insertions(+), 6 deletions(-)
d
sparse generated:
fs/udf/dir.c:78:5: warning: symbol 'udf_readdir' was not declared. Should it be
static?
there are 2 different prototypes of udf_readdir - remove them and move
code around to make it still compile
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
Acked-by: Jan Kara &
sparse generated:
fs/udf/dir.c:78:5: warning: symbol 'udf_readdir' was not declared. Should it be
static?
there are 2 different prototypes of udf_readdir - remove them and move
code around to make it still compile
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
Acked-by: Jan Kara [EMAIL
fix coding style, checkpatch warned about:
- braces {} are not necessary for single statement blocks
- line over 80 characters
- do not use assignment in if condition
- use tabs not spaces
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
Acked-by: Jan Kara [EMAIL
to uint32_t
gcc warned:
fs/udf/inode.c: In function 'udf_get_block':
fs/udf/inode.c:299: warning: 'phys' may be used uninitialized in this function
initialize it to 0 (if someday someone will break inode_getblk we will catch it
immediately)
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben
make reading do_udf_readdir easier by adding new variable
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
Acked-by: Jan Kara [EMAIL PROTECTED]
---
fs/udf/dir.c | 15 +--
1 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/fs/udf/dir.c b
type in argument 3 (different
signedness)
fs/udf/namei.c:1152:78:expected int *offset
fs/udf/namei.c:1152:78:got unsigned int *noident
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
Acked-by: Jan Kara [EMAIL PROTECTED]
---
fs/udf/namei.c |2 +-
1 files changed, 1 insertions(+), 1
int *offset
> > fs/udf/namei.c:1147:41:got unsigned int *
> > fs/udf/namei.c:1152:78: warning: incorrect type in argument 3 (different
> > signedness)
> > fs/udf/namei.c:1152:78:expected int *offset
> > fs/udf/namei.c:1152:78:got unsigned int *
> >
:6: originally declared here
> > fs/udf/super.c:1660:4: warning: symbol 'i' shadows an earlier one
> > fs/udf/super.c:1648:6: originally declared here
> > fs/udf/super.c:450:6: warning: symbol 'udf_write_super' was not declared.
> > Should it be static?
> >
> &g
On Mon, Dec 17, 2007 at 10:30:01AM +0100, Takashi Iwai wrote:
> At Sun, 16 Dec 2007 02:58:31 +0100,
> Marcin Slusarz wrote:
> >
> > On Fri, Dec 14, 2007 at 12:02:46PM +0100, Takashi Iwai wrote:
> > > At Sat, 8 Dec 2007 21:50:45 +0100,
> > > Marcin Ślusarz
On Tue, Dec 18, 2007 at 03:36:42PM -0800, Andrew Morton wrote:
> On Sun, 9 Dec 2007 22:23:42 +0100
> Marcin __lusarz <[EMAIL PROTECTED]> wrote:
>
> > usb/storage/initializers.c: fix signedness difference
> >
> > Signed-off-by: Marcin __lusarz <[EMAIL PROTECTED]>
> > ---
> >
On Tue, Dec 18, 2007 at 03:36:42PM -0800, Andrew Morton wrote:
On Sun, 9 Dec 2007 22:23:42 +0100
Marcin __lusarz [EMAIL PROTECTED] wrote:
usb/storage/initializers.c: fix signedness difference
Signed-off-by: Marcin __lusarz [EMAIL PROTECTED]
---
drivers/usb/storage/initializers.c |
On Mon, Dec 17, 2007 at 10:30:01AM +0100, Takashi Iwai wrote:
At Sun, 16 Dec 2007 02:58:31 +0100,
Marcin Slusarz wrote:
On Fri, Dec 14, 2007 at 12:02:46PM +0100, Takashi Iwai wrote:
At Sat, 8 Dec 2007 21:50:45 +0100,
Marcin Ślusarz wrote:
sound/core.h: include sound/driver.h
' shadows an earlier one
fs/udf/super.c:1648:6: originally declared here
fs/udf/super.c:450:6: warning: symbol 'udf_write_super' was not declared.
Should it be static?
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED]
Thanks for the patch. The 'bh' change
:got unsigned int *noident
fs/udf/namei.c:1152:78: warning: incorrect type in argument 3 (different
signedness)
fs/udf/namei.c:1152:78:expected int *offset
fs/udf/namei.c:1152:78:got unsigned int *noident
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
I don't think
comverted -> converted
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
---
include/asm-ia64/compat.h|2 +-
include/asm-mips/compat.h|2 +-
include/asm-parisc/compat.h |2 +-
include/asm-powerpc/compat.h |2 +-
include/asm-s390/compat.h|2 +-
include/as
fix warning:
drivers/usb/host/ehci-hcd.c:832:8: warning: symbol 'status' shadows an earlier
one
drivers/usb/host/ehci-hcd.c:790:71: originally declared here
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Greg Kroah-Hartman <[EMAIL PROTECTED]>
---
drivers/usb/host/ehci-
fix warning:
drivers/usb/core/devio.c:1226:20: warning: incorrect type in assignment
(different address spaces)
drivers/usb/core/devio.c:1226:20:expected void *usercontext
drivers/usb/core/devio.c:1226:20:got void [noderef] *
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC
: incorrect type in argument 5
(different signedness)
drivers/usb/storage/initializers.c:89:26:expected unsigned int *act_len
drivers/usb/storage/initializers.c:89:26:got int *
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Greg Kroah-Hartman <[EMAIL PROTECTED]>
---
/super.c:450:6: warning: symbol 'udf_write_super' was not declared.
Should it be static?
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
---
fs/udf/super.c | 15 +++
1 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/fs/udf/
in argument 3 (different
signedness)
fs/udf/namei.c:1152:78:expected int *offset
fs/udf/namei.c:1152:78:got unsigned int *
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
---
fs/udf/namei.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/udf/namei.c b/
e.c: In function 'udf_get_block':
fs/udf/inode.c:299: warning: 'phys' may be used uninitialized in this function
initialize it to 0 (if someday someone will break inode_getblk we will catch it
immediately)
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]&
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
---
fs/udf/dir.c | 46 +++---
1 files changed, 27 insertions(+), 19 deletions(-)
diff --git a/fs/udf/dir.c b/fs/udf/dir.c
index c26e281..c5e38d6 100644
--- a/fs/udf/dir.c
+++ b/fs/udf/dir.c
@@
make reading do_udf_readdir easier by adding new variable
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema <[EMAIL PROTECTED]>
---
fs/udf/dir.c | 15 +--
1 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/fs/udf/dir.c b/fs/udf/dir.c
i
sparse generated:
fs/udf/dir.c:78:5: warning: symbol 'udf_readdir' was not declared. Should it be
static?
there are 2 different prototypes of udf_readdir - remove them and move
code around to make it still compile
Signed-off-by: Marcin Slusarz <[EMAIL PROTECTED]>
CC: Ben Fennema &
On Fri, Dec 14, 2007 at 12:02:46PM +0100, Takashi Iwai wrote:
> At Sat, 8 Dec 2007 21:50:45 +0100,
> Marcin Ślusarz wrote:
> >
> > sound/core.h: include sound/driver.h
> >
> > include sound/driver.h in sound/core.h because core.h
> > uses SNDRV_CARDS (which is defined in sound/driver.h)
> >
> >
On Fri, Dec 14, 2007 at 12:02:46PM +0100, Takashi Iwai wrote:
At Sat, 8 Dec 2007 21:50:45 +0100,
Marcin Ślusarz wrote:
sound/core.h: include sound/driver.h
include sound/driver.h in sound/core.h because core.h
uses SNDRV_CARDS (which is defined in sound/driver.h)
Signed-off-by:
sparse generated:
fs/udf/dir.c:78:5: warning: symbol 'udf_readdir' was not declared. Should it be
static?
there are 2 different prototypes of udf_readdir - remove them and move
code around to make it still compile
Signed-off-by: Marcin Slusarz [EMAIL PROTECTED]
CC: Ben Fennema [EMAIL PROTECTED
301 - 400 of 411 matches
Mail list logo