[PATCH 1/5] cris:removed the unused variable

[PATCH 1/5] cris:removed the unused variable

[wanlong.gao]

From: Wanlong Gao <wanlong.gao@gmail.com>

removed the unused variable "unsigned long tmp".

Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
---
 arch/cris/arch-v32/kernel/ptrace.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/arch/cris/arch-v32/kernel/ptrace.c b/arch/cris/arch-v32/kernel/ptrace.c
index 511ece9..42ed6c7 100644
--- a/arch/cris/arch-v32/kernel/ptrace.c
+++ b/arch/cris/arch-v32/kernel/ptrace.c
@@ -115,8 +115,6 @@ void user_disable_single_step(struct task_struct *child)
 void
 ptrace_disable(struct task_struct *child)
 {
-	unsigned long tmp;
-
 	/* Deconfigure SPC and S-bit. */
 	user_disable_single_step(child);
 	put_reg(child, PT_SPC, 0);
-- 
1.7.3

[PATCH 2/5] mfd:removed the unused variable

[wanlong.gao]

From: Wanlong Gao <wanlong.gao@gmail.com>

removed the unused variable "u16 val" of wl1273-core.c.

Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
---
 drivers/mfd/wl1273-core.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/mfd/wl1273-core.c b/drivers/mfd/wl1273-core.c
index 04914f2..37af778 100644
--- a/drivers/mfd/wl1273-core.c
+++ b/drivers/mfd/wl1273-core.c
@@ -153,7 +153,6 @@ out:
  */
 static int wl1273_fm_set_volume(struct wl1273_core *core, unsigned int volume)
 {
-	u16 val;
 	int r;
 
 	if (volume > WL1273_MAX_VOLUME)
-- 
1.7.3

Re: [PATCH 2/5] mfd:removed the unused variable

[Samuel Ortiz]

Hi Wanlong,

On Sun, Apr 10, 2011 at 10:10:26PM +0800, wanlong.gao@gmail.com wrote:
> From: Wanlong Gao <wanlong.gao@gmail.com>
> 
> removed the unused variable "u16 val" of wl1273-core.c.
Patch applied, thanks.

Cheers,
Samuel.

-- 
Intel Open Source Technology Centre
http://oss.intel.com/

Re: [PATCH 2/5] mfd:removed the unused variable

[wanlong gao]

On 4/12/11, Samuel Ortiz <sameo@linux.intel.com> wrote:
> Hi Wanlong,
>
> On Sun, Apr 10, 2011 at 10:10:26PM +0800, wanlong.gao@gmail.com wrote:
>> From: Wanlong Gao <wanlong.gao@gmail.com>
>>
>> removed the unused variable "u16 val" of wl1273-core.c.
> Patch applied, thanks.
>
> Cheers,
> Samuel.
Thank you for your hard work to the  open source .

Best regards .
>
> --
> Intel Open Source Technology Centre
> http://oss.intel.com/
>

[PATCH 3/5] net:removed the unused variable

[wanlong.gao]

From: Wanlong Gao <wanlong.gao@gmail.com>

removed the unused variable "np" of eth_v10.c.

Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
---
 drivers/net/cris/eth_v10.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/net/cris/eth_v10.c b/drivers/net/cris/eth_v10.c
index 9d267d3..f3b7741 100644
--- a/drivers/net/cris/eth_v10.c
+++ b/drivers/net/cris/eth_v10.c
@@ -1132,7 +1132,6 @@ static irqreturn_t
 e100rxtx_interrupt(int irq, void *dev_id)
 {
 	struct net_device *dev = (struct net_device *)dev_id;
-	struct net_local *np = netdev_priv(dev);
 	unsigned long irqbits;
 
 	/*
-- 
1.7.3

Re: [PATCH 3/5] net:removed the unused variable

[Jesper Nilsson]

On Sun, Apr 10, 2011 at 04:10:27PM +0200, wanlong.gao@gmail.com wrote:
> From: Wanlong Gao <wanlong.gao@gmail.com>
> 
> removed the unused variable "np" of eth_v10.c.
> 
> Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>

Applied, thanks!

/^JN - Jesper Nilsson
-- 
               Jesper Nilsson -- jesper.nilsson@axis.com

[PATCH 4/5] scsi:removed the unused variable

[wanlong.gao]

From: Wanlong Gao <wanlong.gao@gmail.com>

removed the unused variable "oldto" of atari_NCR5380.c .

Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
---
 drivers/scsi/atari_NCR5380.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/drivers/scsi/atari_NCR5380.c b/drivers/scsi/atari_NCR5380.c
index ea439f9..24652f1 100644
--- a/drivers/scsi/atari_NCR5380.c
+++ b/drivers/scsi/atari_NCR5380.c
@@ -914,7 +914,6 @@ static int NCR5380_queue_command_lck(Scsi_Cmnd *cmd, void (*done)(Scsi_Cmnd *))
 {
 	SETUP_HOSTDATA(cmd->device->host);
 	Scsi_Cmnd *tmp;
-	int oldto;
 	unsigned long flags;
 
 #if (NDEBUG & NDEBUG_NO_WRITE)
-- 
1.7.3

[PATCH 5/5] xfs:removed the unused variable

[wanlong.gao]

From: Wanlong Gao <wanlong.gao@gmail.com>

removed the unused variable "bdi" of xfs_buf.c .

Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
---
 fs/xfs/linux-2.6/xfs_buf.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/fs/xfs/linux-2.6/xfs_buf.c b/fs/xfs/linux-2.6/xfs_buf.c
index 5ea4020..a125b32 100644
--- a/fs/xfs/linux-2.6/xfs_buf.c
+++ b/fs/xfs/linux-2.6/xfs_buf.c
@@ -657,8 +657,6 @@ xfs_buf_readahead(
 	xfs_off_t		ioff,
 	size_t			isize)
 {
-	struct backing_dev_info *bdi;
-
 	if (bdi_read_congested(target->bt_bdi))
 		return;
 
-- 
1.7.3

Re: [PATCH 5/5] xfs:removed the unused variable

[Dave Chinner]

On Sun, Apr 10, 2011 at 10:10:29PM +0800, wanlong.gao@gmail.com wrote:
> From: Wanlong Gao <wanlong.gao@gmail.com>
> 
> removed the unused variable "bdi" of xfs_buf.c .
> 
> Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
> ---
>  fs/xfs/linux-2.6/xfs_buf.c |    2 --
>  1 files changed, 0 insertions(+), 2 deletions(-)
> 
> diff --git a/fs/xfs/linux-2.6/xfs_buf.c b/fs/xfs/linux-2.6/xfs_buf.c
> index 5ea4020..a125b32 100644
> --- a/fs/xfs/linux-2.6/xfs_buf.c
> +++ b/fs/xfs/linux-2.6/xfs_buf.c
> @@ -657,8 +657,6 @@ xfs_buf_readahead(
>  	xfs_off_t		ioff,
>  	size_t			isize)
>  {
> -	struct backing_dev_info *bdi;
> -
>  	if (bdi_read_congested(target->bt_bdi))
>  		return;

<sigh>

Doesn't anyone look anymore to see if a problem has already been
reported before or fixed in the maintainers tree?

FYI, you're about the 5th person to send this same patch....

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com

Re: [PATCH 5/5] xfs:removed the unused variable

[wanlong gao]

On 4/11/11, Dave Chinner <david@fromorbit.com> wrote:
> On Sun, Apr 10, 2011 at 10:10:29PM +0800, wanlong.gao@gmail.com wrote:
>> From: Wanlong Gao <wanlong.gao@gmail.com>
>>
>> removed the unused variable "bdi" of xfs_buf.c .
>>
>> Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
>> ---
>>  fs/xfs/linux-2.6/xfs_buf.c |    2 --
>>  1 files changed, 0 insertions(+), 2 deletions(-)
>>
>> diff --git a/fs/xfs/linux-2.6/xfs_buf.c b/fs/xfs/linux-2.6/xfs_buf.c
>> index 5ea4020..a125b32 100644
>> --- a/fs/xfs/linux-2.6/xfs_buf.c
>> +++ b/fs/xfs/linux-2.6/xfs_buf.c
>> @@ -657,8 +657,6 @@ xfs_buf_readahead(
>>  	xfs_off_t		ioff,
>>  	size_t			isize)
>>  {
>> -	struct backing_dev_info *bdi;
>> -
>>  	if (bdi_read_congested(target->bt_bdi))
>>  		return;
>
> <sigh>
>
> Doesn't anyone look anymore to see if a problem has already been
> reported before or fixed in the maintainers tree?
>
> FYI, you're about the 5th person to send this same patch....
>
Sorry for the dupe, I'll look anymore next time .
> Cheers,
>
> Dave.
> --
> Dave Chinner
> david@fromorbit.com
>

Re: [PATCH 1/5] cris:removed the unused variable

[Jesper Nilsson]

On Sun, Apr 10, 2011 at 04:10:25PM +0200, wanlong.gao@gmail.com wrote:
> From: Wanlong Gao <wanlong.gao@gmail.com>
> 
> removed the unused variable "unsigned long tmp".
> 
> Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>

Applied, thanks!

/^JN - Jesper Nilsson
-- 
               Jesper Nilsson -- jesper.nilsson@axis.com

Re: [PATCH 1/5] cris:removed the unused variable

[wanlong gao]

On 4/12/11, Jesper Nilsson <jesper.nilsson@axis.com> wrote:
> On Sun, Apr 10, 2011 at 04:10:25PM +0200, wanlong.gao@gmail.com wrote:
>> From: Wanlong Gao <wanlong.gao@gmail.com>
>>
>> removed the unused variable "unsigned long tmp".
>>
>> Signed-off-by: Wanlong Gao <wanlong.gao@gmail.com>
>
> Applied, thanks!
>
> /^JN - Jesper Nilsson
Thank you for your hard work to the  open source .

Best regards .
> --
>                Jesper Nilsson -- jesper.nilsson@axis.com
>