* [RFC: 2.6 patch] chelsio/espi.c:tricn_init(): remove dead code
@ 2006-03-09 23:06 Adrian Bunk
2006-03-09 23:34 ` Scott Bardone
0 siblings, 1 reply; 2+ messages in thread
From: Adrian Bunk @ 2006-03-09 23:06 UTC (permalink / raw)
To: maintainers; +Cc: jgarzik, netdev, linux-kernel
The Coverity checker spotted these two unused variables.
Please check whether this patch is correct or whether they should be
used.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
drivers/net/chelsio/espi.c | 14 +++-----------
1 file changed, 3 insertions(+), 11 deletions(-)
--- linux-2.6.16-rc5-mm3-full/drivers/net/chelsio/espi.c.old 2006-03-09 23:19:54.000000000 +0100
+++ linux-2.6.16-rc5-mm3-full/drivers/net/chelsio/espi.c 2006-03-09 23:20:35.000000000 +0100
@@ -87,15 +87,9 @@
static int tricn_init(adapter_t *adapter)
{
int i = 0;
- int sme = 1;
int stat = 0;
int timeout = 0;
int is_ready = 0;
- int dynamic_deskew = 0;
-
- if (dynamic_deskew)
- sme = 0;
-
/* 1 */
timeout=1000;
@@ -113,11 +107,9 @@
}
/* 2 */
- if (sme) {
- tricn_write(adapter, 0, 0, 0, TRICN_CNFG, 0x81);
- tricn_write(adapter, 0, 1, 0, TRICN_CNFG, 0x81);
- tricn_write(adapter, 0, 2, 0, TRICN_CNFG, 0x81);
- }
+ tricn_write(adapter, 0, 0, 0, TRICN_CNFG, 0x81);
+ tricn_write(adapter, 0, 1, 0, TRICN_CNFG, 0x81);
+ tricn_write(adapter, 0, 2, 0, TRICN_CNFG, 0x81);
for (i=1; i<= 8; i++) tricn_write(adapter, 0, 0, i, TRICN_CNFG, 0xf1);
for (i=1; i<= 2; i++) tricn_write(adapter, 0, 1, i, TRICN_CNFG, 0xf1);
for (i=1; i<= 3; i++) tricn_write(adapter, 0, 2, i, TRICN_CNFG, 0xe1);
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [RFC: 2.6 patch] chelsio/espi.c:tricn_init(): remove dead code
2006-03-09 23:06 [RFC: 2.6 patch] chelsio/espi.c:tricn_init(): remove dead code Adrian Bunk
@ 2006-03-09 23:34 ` Scott Bardone
0 siblings, 0 replies; 2+ messages in thread
From: Scott Bardone @ 2006-03-09 23:34 UTC (permalink / raw)
To: Adrian Bunk; +Cc: maintainers, jgarzik, netdev, linux-kernel
This patch is correct, these two variables are unused in this driver. Thanks for
catching this!
Signed-off-by: Scott Bardone <sbardone@chelsio.com>
Adrian Bunk wrote:
> The Coverity checker spotted these two unused variables.
>
> Please check whether this patch is correct or whether they should be
> used.
>
>
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
>
> ---
>
> drivers/net/chelsio/espi.c | 14 +++-----------
> 1 file changed, 3 insertions(+), 11 deletions(-)
>
> --- linux-2.6.16-rc5-mm3-full/drivers/net/chelsio/espi.c.old 2006-03-09 23:19:54.000000000 +0100
> +++ linux-2.6.16-rc5-mm3-full/drivers/net/chelsio/espi.c 2006-03-09 23:20:35.000000000 +0100
> @@ -87,15 +87,9 @@
> static int tricn_init(adapter_t *adapter)
> {
> int i = 0;
> - int sme = 1;
> int stat = 0;
> int timeout = 0;
> int is_ready = 0;
> - int dynamic_deskew = 0;
> -
> - if (dynamic_deskew)
> - sme = 0;
> -
>
> /* 1 */
> timeout=1000;
> @@ -113,11 +107,9 @@
> }
>
> /* 2 */
> - if (sme) {
> - tricn_write(adapter, 0, 0, 0, TRICN_CNFG, 0x81);
> - tricn_write(adapter, 0, 1, 0, TRICN_CNFG, 0x81);
> - tricn_write(adapter, 0, 2, 0, TRICN_CNFG, 0x81);
> - }
> + tricn_write(adapter, 0, 0, 0, TRICN_CNFG, 0x81);
> + tricn_write(adapter, 0, 1, 0, TRICN_CNFG, 0x81);
> + tricn_write(adapter, 0, 2, 0, TRICN_CNFG, 0x81);
> for (i=1; i<= 8; i++) tricn_write(adapter, 0, 0, i, TRICN_CNFG, 0xf1);
> for (i=1; i<= 2; i++) tricn_write(adapter, 0, 1, i, TRICN_CNFG, 0xf1);
> for (i=1; i<= 3; i++) tricn_write(adapter, 0, 2, i, TRICN_CNFG, 0xe1);
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2006-03-09 23:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-03-09 23:06 [RFC: 2.6 patch] chelsio/espi.c:tricn_init(): remove dead code Adrian Bunk
2006-03-09 23:34 ` Scott Bardone
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).