udf:super.c Fix warning: variable 'sbi' set but not used
in [Kernel]
|
Prev: 68328serial: check return value of copy_*_user() instead of access_ok()
Next: usb: fsl_udc_core: check return value of create_proc_read_entry()
From: Justin P. Mattock on 31 Jul 2010 15:10 any ideas on the status of this? > This fixes this warning when building the kernel: > CC fs/udf/super.o > fs/udf/super.c: In function 'udf_load_sequence': > fs/udf/super.c:1582:22: warning: variable 'sbi' set but not used > Please have a look, when you have time and let me know. > > Signed-off-by: Justin P. Mattock<justinmattock(a)gmail.com> > > --- > fs/udf/super.c | 2 -- > 1 files changed, 0 insertions(+), 2 deletions(-) > > diff --git a/fs/udf/super.c b/fs/udf/super.c > index 612d1e2..12bb651 100644 > --- a/fs/udf/super.c > +++ b/fs/udf/super.c > @@ -1579,9 +1579,7 @@ static int udf_load_sequence(struct super_block *sb, struct buffer_head *bh, > { > struct anchorVolDescPtr *anchor; > long main_s, main_e, reserve_s, reserve_e; > - struct udf_sb_info *sbi; > > - sbi = UDF_SB(sb); > anchor = (struct anchorVolDescPtr *)bh->b_data; > > /* Locate the main sequence */ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Jan Kara on 2 Aug 2010 09:00 On Sat 31-07-10 12:03:03, Justin P. Mattock wrote: > any ideas on the status of this? I have it in my tree and will push it to Linus in this merge window. Honza > > >This fixes this warning when building the kernel: > > CC fs/udf/super.o > >fs/udf/super.c: In function 'udf_load_sequence': > >fs/udf/super.c:1582:22: warning: variable 'sbi' set but not used > >Please have a look, when you have time and let me know. > > > >Signed-off-by: Justin P. Mattock<justinmattock(a)gmail.com> > > > >--- > > fs/udf/super.c | 2 -- > > 1 files changed, 0 insertions(+), 2 deletions(-) > > > >diff --git a/fs/udf/super.c b/fs/udf/super.c > >index 612d1e2..12bb651 100644 > >--- a/fs/udf/super.c > >+++ b/fs/udf/super.c > >@@ -1579,9 +1579,7 @@ static int udf_load_sequence(struct super_block *sb, struct buffer_head *bh, > > { > > struct anchorVolDescPtr *anchor; > > long main_s, main_e, reserve_s, reserve_e; > >- struct udf_sb_info *sbi; > > > >- sbi = UDF_SB(sb); > > anchor = (struct anchorVolDescPtr *)bh->b_data; > > > > /* Locate the main sequence */ > -- Jan Kara <jack(a)suse.cz> SUSE Labs, CR -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Justin P. Mattock on 2 Aug 2010 09:40
On 08/02/2010 05:51 AM, Jan Kara wrote: > On Sat 31-07-10 12:03:03, Justin P. Mattock wrote: >> any ideas on the status of this? > I have it in my tree and will push it to Linus in this merge window. > > Honza o.k. cool.. >> >>> This fixes this warning when building the kernel: >>> CC fs/udf/super.o >>> fs/udf/super.c: In function 'udf_load_sequence': >>> fs/udf/super.c:1582:22: warning: variable 'sbi' set but not used >>> Please have a look, when you have time and let me know. >>> >>> Signed-off-by: Justin P. Mattock<justinmattock(a)gmail.com> >>> >>> --- >>> fs/udf/super.c | 2 -- >>> 1 files changed, 0 insertions(+), 2 deletions(-) >>> >>> diff --git a/fs/udf/super.c b/fs/udf/super.c >>> index 612d1e2..12bb651 100644 >>> --- a/fs/udf/super.c >>> +++ b/fs/udf/super.c >>> @@ -1579,9 +1579,7 @@ static int udf_load_sequence(struct super_block *sb, struct buffer_head *bh, >>> { >>> struct anchorVolDescPtr *anchor; >>> long main_s, main_e, reserve_s, reserve_e; >>> - struct udf_sb_info *sbi; >>> >>> - sbi = UDF_SB(sb); >>> anchor = (struct anchorVolDescPtr *)bh->b_data; >>> >>> /* Locate the main sequence */ >> Justin P. Mattock -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |