diff options
author | Alan Stern <stern@rowland.harvard.edu> | 2012-01-24 13:35:24 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2012-01-24 19:00:35 -0500 |
commit | f3ff9247088a0af0c192a28908dab76ff3d8871f (patch) | |
tree | 4ff2c822bcf82213d14b2195d4f00fe470876b1d /lib | |
parent | 9f30ea950edfaefa51221dd26a065b3442599778 (diff) |
Remove useless get_driver()/put_driver() calls
As part of the removal of get_driver()/put_driver(), this patch
(as1512) gets rid of various useless and unnecessary calls in several
drivers. In some cases it may be desirable to pin the driver by
calling try_module_get(), but that can be done later.
Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
CC: "David S. Miller" <davem@davemloft.net>
CC: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
CC: Michael Buesch <m@bues.ch>
CC: Joerg Roedel <joerg.roedel@amd.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/dma-debug.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/dma-debug.c b/lib/dma-debug.c index fea790a2b176..13ef2338be41 100644 --- a/lib/dma-debug.c +++ b/lib/dma-debug.c | |||
@@ -170,7 +170,7 @@ static bool driver_filter(struct device *dev) | |||
170 | return false; | 170 | return false; |
171 | 171 | ||
172 | /* driver filter on but not yet initialized */ | 172 | /* driver filter on but not yet initialized */ |
173 | drv = get_driver(dev->driver); | 173 | drv = dev->driver; |
174 | if (!drv) | 174 | if (!drv) |
175 | return false; | 175 | return false; |
176 | 176 | ||
@@ -185,7 +185,6 @@ static bool driver_filter(struct device *dev) | |||
185 | } | 185 | } |
186 | 186 | ||
187 | read_unlock_irqrestore(&driver_name_lock, flags); | 187 | read_unlock_irqrestore(&driver_name_lock, flags); |
188 | put_driver(drv); | ||
189 | 188 | ||
190 | return ret; | 189 | return ret; |
191 | } | 190 | } |