2008-03-28 12:58:35

by David Fries

[permalink] [raw]
Subject: [PATCH 18/35] W1: w1_therm consistent mutex access code cleanup

w1_therm.c 1.7
sl->master->mutex and dev->mutex refer to the same mutex variable, but
be consistent and use the same set of pointers for the lock and unlock
calls. It is less confusing (and one less pointer dereference this
way).

Signed-off-by: David Fries <[email protected]>
---
drivers/w1/slaves/w1_therm.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/w1/slaves/w1_therm.c b/drivers/w1/slaves/w1_therm.c
index ca47421..284acac 100644
--- a/drivers/w1/slaves/w1_therm.c
+++ b/drivers/w1/slaves/w1_therm.c
@@ -163,7 +163,7 @@ static ssize_t w1_therm_read(struct device *device,
int i, max_trying = 10;
ssize_t c=PAGE_SIZE;

- mutex_lock(&sl->master->mutex);
+ mutex_lock(&dev->mutex);

memset(rom, 0, sizeof(rom));

--
1.4.4.4


Attachments:
(No filename) (776.00 B)
(No filename) (189.00 B)
Download all attachments