diff options
| author | Evgeniy Polyakov <johnpol@2ka.mipt.ru> | 2008-02-06 04:38:07 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-06 13:41:10 -0500 |
| commit | 18a2354db7d7e43da08c0351263fd5b5fc20a38d (patch) | |
| tree | 62d5eac10f24ebce9b4ab1b0dde59f6121064026 /drivers/w1 | |
| parent | e1d42c983ff7ba4b55d8635899186ae2ef2578ad (diff) | |
w1: remove unused and confusing variable.
Remvoe variable which actually is not used (except assigning it a value)
and confusing break out of the family checking loop. Found by Harry Mason.
Signed-off-by: Evgeniy Polyakov <johnpol@2ka.mipt.ru>
Cc: Harry J Mason <hjm03r@ecs.soton.ac.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/w1')
| -rw-r--r-- | drivers/w1/w1.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c index 33e50310e9e0..7293c9b11f91 100644 --- a/drivers/w1/w1.c +++ b/drivers/w1/w1.c | |||
| @@ -675,7 +675,6 @@ static void w1_slave_found(void *data, u64 rn) | |||
| 675 | struct w1_slave *sl; | 675 | struct w1_slave *sl; |
| 676 | struct list_head *ent; | 676 | struct list_head *ent; |
| 677 | struct w1_reg_num *tmp; | 677 | struct w1_reg_num *tmp; |
| 678 | int family_found = 0; | ||
| 679 | struct w1_master *dev; | 678 | struct w1_master *dev; |
| 680 | u64 rn_le = cpu_to_le64(rn); | 679 | u64 rn_le = cpu_to_le64(rn); |
| 681 | 680 | ||
| @@ -698,9 +697,6 @@ static void w1_slave_found(void *data, u64 rn) | |||
| 698 | sl->reg_num.crc == tmp->crc) { | 697 | sl->reg_num.crc == tmp->crc) { |
| 699 | set_bit(W1_SLAVE_ACTIVE, (long *)&sl->flags); | 698 | set_bit(W1_SLAVE_ACTIVE, (long *)&sl->flags); |
| 700 | break; | 699 | break; |
| 701 | } else if (sl->reg_num.family == tmp->family) { | ||
| 702 | family_found = 1; | ||
| 703 | break; | ||
| 704 | } | 700 | } |
| 705 | 701 | ||
| 706 | slave_count++; | 702 | slave_count++; |
