2008-06-20 12:52:57

by Andre Haupt

[permalink] [raw]
Subject: [PATCH 0/2] parport sparse fixes

The following patches fix some warnings sparse gave me, when running
over drivers/parport.

Andre Haupt (2):
parport: remove superfluous local variable.
parport: make parport_cs_release static.

drivers/parport/ieee1284.c | 2 --
drivers/parport/parport_cs.c | 2 +-
2 files changed, 1 insertions(+), 3 deletions(-)


2008-06-20 12:52:30

by Andre Haupt

[permalink] [raw]
Subject: [PATCH 1/2] parport: remove superfluous local variable.

Signed-off-by: Andre Haupt <[email protected]>
---
drivers/parport/ieee1284.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/drivers/parport/ieee1284.c b/drivers/parport/ieee1284.c
index 0338b09..e970594 100644
--- a/drivers/parport/ieee1284.c
+++ b/drivers/parport/ieee1284.c
@@ -199,8 +199,6 @@ int parport_wait_peripheral(struct parport *port,
/* 40ms of slow polling. */
deadline = jiffies + msecs_to_jiffies(40);
while (time_before (jiffies, deadline)) {
- int ret;
-
if (signal_pending (current))
return -EINTR;

--
1.5.5.GIT

2008-06-20 12:52:42

by Andre Haupt

[permalink] [raw]
Subject: [PATCH 2/2] parport: make parport_cs_release static.

Signed-off-by: Andre Haupt <[email protected]>
---
drivers/parport/parport_cs.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/parport/parport_cs.c b/drivers/parport/parport_cs.c
index 802a81d..00e1d96 100644
--- a/drivers/parport/parport_cs.c
+++ b/drivers/parport/parport_cs.c
@@ -235,7 +235,7 @@ failed:

======================================================================*/

-void parport_cs_release(struct pcmcia_device *link)
+static void parport_cs_release(struct pcmcia_device *link)
{
parport_info_t *info = link->priv;

--
1.5.5.GIT