2011-04-10 14:11:33

by Wanlong Gao

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

From: Wanlong Gao <[email protected]>

removed the unused variable "unsigned long tmp".

Signed-off-by: Wanlong Gao <[email protected]>
---
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


2011-04-10 14:12:13

by Wanlong Gao

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

From: Wanlong Gao <[email protected]>

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

Signed-off-by: Wanlong Gao <[email protected]>
---
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

2011-04-10 14:12:34

by Wanlong Gao

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

From: Wanlong Gao <[email protected]>

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

Signed-off-by: Wanlong Gao <[email protected]>
---
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

2011-04-10 14:12:56

by Wanlong Gao

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

From: Wanlong Gao <[email protected]>

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

Signed-off-by: Wanlong Gao <[email protected]>
---
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

2011-04-10 14:13:18

by Wanlong Gao

[permalink] [raw]
Subject: [PATCH 5/5] xfs:removed the unused variable

From: Wanlong Gao <[email protected]>

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

Signed-off-by: Wanlong Gao <[email protected]>
---
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

2011-04-11 04:05:51

by Dave Chinner

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

On Sun, Apr 10, 2011 at 10:10:29PM +0800, [email protected] wrote:
> From: Wanlong Gao <[email protected]>
>
> removed the unused variable "bdi" of xfs_buf.c .
>
> Signed-off-by: Wanlong Gao <[email protected]>
> ---
> 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
[email protected]

2011-04-11 12:19:30

by Wanlong Gao

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

On 4/11/11, Dave Chinner <[email protected]> wrote:
> On Sun, Apr 10, 2011 at 10:10:29PM +0800, [email protected] wrote:
>> From: Wanlong Gao <[email protected]>
>>
>> removed the unused variable "bdi" of xfs_buf.c .
>>
>> Signed-off-by: Wanlong Gao <[email protected]>
>> ---
>> 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
> [email protected]
>

2011-04-11 19:51:50

by Samuel Ortiz

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

Hi Wanlong,

On Sun, Apr 10, 2011 at 10:10:26PM +0800, [email protected] wrote:
> From: Wanlong Gao <[email protected]>
>
> removed the unused variable "u16 val" of wl1273-core.c.
Patch applied, thanks.

Cheers,
Samuel.

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

2011-04-12 07:57:11

by Jesper Nilsson

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

On Sun, Apr 10, 2011 at 04:10:25PM +0200, [email protected] wrote:
> From: Wanlong Gao <[email protected]>
>
> removed the unused variable "unsigned long tmp".
>
> Signed-off-by: Wanlong Gao <[email protected]>

Applied, thanks!

/^JN - Jesper Nilsson
--
Jesper Nilsson -- [email protected]

2011-04-12 07:57:37

by Jesper Nilsson

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

On Sun, Apr 10, 2011 at 04:10:27PM +0200, [email protected] wrote:
> From: Wanlong Gao <[email protected]>
>
> removed the unused variable "np" of eth_v10.c.
>
> Signed-off-by: Wanlong Gao <[email protected]>

Applied, thanks!

/^JN - Jesper Nilsson
--
Jesper Nilsson -- [email protected]

2011-04-12 15:07:10

by Wanlong Gao

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

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

Best regards .
> --
> Jesper Nilsson -- [email protected]
>

2011-04-12 15:15:47

by Wanlong Gao

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

On 4/12/11, Samuel Ortiz <[email protected]> wrote:
> Hi Wanlong,
>
> On Sun, Apr 10, 2011 at 10:10:26PM +0800, [email protected] wrote:
>> From: Wanlong Gao <[email protected]>
>>
>> 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/
>