xref: /linux/drivers/hwspinlock/hwspinlock_core.c (revision b85d45947951d23cb22d90caecf4c1eb81342c96)
1 /*
2  * Hardware spinlock framework
3  *
4  * Copyright (C) 2010 Texas Instruments Incorporated - http://www.ti.com
5  *
6  * Contact: Ohad Ben-Cohen <ohad@wizery.com>
7  *
8  * This program is free software; you can redistribute it and/or modify it
9  * under the terms of the GNU General Public License version 2 as published
10  * by the Free Software Foundation.
11  *
12  * This program is distributed in the hope that it will be useful,
13  * but WITHOUT ANY WARRANTY; without even the implied warranty of
14  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
15  * GNU General Public License for more details.
16  */
17 
18 #define pr_fmt(fmt)    "%s: " fmt, __func__
19 
20 #include <linux/kernel.h>
21 #include <linux/module.h>
22 #include <linux/spinlock.h>
23 #include <linux/types.h>
24 #include <linux/err.h>
25 #include <linux/jiffies.h>
26 #include <linux/radix-tree.h>
27 #include <linux/hwspinlock.h>
28 #include <linux/pm_runtime.h>
29 #include <linux/mutex.h>
30 #include <linux/of.h>
31 
32 #include "hwspinlock_internal.h"
33 
34 /* radix tree tags */
35 #define HWSPINLOCK_UNUSED	(0) /* tags an hwspinlock as unused */
36 
37 /*
38  * A radix tree is used to maintain the available hwspinlock instances.
39  * The tree associates hwspinlock pointers with their integer key id,
40  * and provides easy-to-use API which makes the hwspinlock core code simple
41  * and easy to read.
42  *
43  * Radix trees are quick on lookups, and reasonably efficient in terms of
44  * storage, especially with high density usages such as this framework
45  * requires (a continuous range of integer keys, beginning with zero, is
46  * used as the ID's of the hwspinlock instances).
47  *
48  * The radix tree API supports tagging items in the tree, which this
49  * framework uses to mark unused hwspinlock instances (see the
50  * HWSPINLOCK_UNUSED tag above). As a result, the process of querying the
51  * tree, looking for an unused hwspinlock instance, is now reduced to a
52  * single radix tree API call.
53  */
54 static RADIX_TREE(hwspinlock_tree, GFP_KERNEL);
55 
56 /*
57  * Synchronization of access to the tree is achieved using this mutex,
58  * as the radix-tree API requires that users provide all synchronisation.
59  * A mutex is needed because we're using non-atomic radix tree allocations.
60  */
61 static DEFINE_MUTEX(hwspinlock_tree_lock);
62 
63 
64 /**
65  * __hwspin_trylock() - attempt to lock a specific hwspinlock
66  * @hwlock: an hwspinlock which we want to trylock
67  * @mode: controls whether local interrupts are disabled or not
68  * @flags: a pointer where the caller's interrupt state will be saved at (if
69  *         requested)
70  *
71  * This function attempts to lock an hwspinlock, and will immediately
72  * fail if the hwspinlock is already taken.
73  *
74  * Upon a successful return from this function, preemption (and possibly
75  * interrupts) is disabled, so the caller must not sleep, and is advised to
76  * release the hwspinlock as soon as possible. This is required in order to
77  * minimize remote cores polling on the hardware interconnect.
78  *
79  * The user decides whether local interrupts are disabled or not, and if yes,
80  * whether he wants their previous state to be saved. It is up to the user
81  * to choose the appropriate @mode of operation, exactly the same way users
82  * should decide between spin_trylock, spin_trylock_irq and
83  * spin_trylock_irqsave.
84  *
85  * Returns 0 if we successfully locked the hwspinlock or -EBUSY if
86  * the hwspinlock was already taken.
87  * This function will never sleep.
88  */
89 int __hwspin_trylock(struct hwspinlock *hwlock, int mode, unsigned long *flags)
90 {
91 	int ret;
92 
93 	BUG_ON(!hwlock);
94 	BUG_ON(!flags && mode == HWLOCK_IRQSTATE);
95 
96 	/*
97 	 * This spin_lock{_irq, _irqsave} serves three purposes:
98 	 *
99 	 * 1. Disable preemption, in order to minimize the period of time
100 	 *    in which the hwspinlock is taken. This is important in order
101 	 *    to minimize the possible polling on the hardware interconnect
102 	 *    by a remote user of this lock.
103 	 * 2. Make the hwspinlock SMP-safe (so we can take it from
104 	 *    additional contexts on the local host).
105 	 * 3. Ensure that in_atomic/might_sleep checks catch potential
106 	 *    problems with hwspinlock usage (e.g. scheduler checks like
107 	 *    'scheduling while atomic' etc.)
108 	 */
109 	if (mode == HWLOCK_IRQSTATE)
110 		ret = spin_trylock_irqsave(&hwlock->lock, *flags);
111 	else if (mode == HWLOCK_IRQ)
112 		ret = spin_trylock_irq(&hwlock->lock);
113 	else
114 		ret = spin_trylock(&hwlock->lock);
115 
116 	/* is lock already taken by another context on the local cpu ? */
117 	if (!ret)
118 		return -EBUSY;
119 
120 	/* try to take the hwspinlock device */
121 	ret = hwlock->bank->ops->trylock(hwlock);
122 
123 	/* if hwlock is already taken, undo spin_trylock_* and exit */
124 	if (!ret) {
125 		if (mode == HWLOCK_IRQSTATE)
126 			spin_unlock_irqrestore(&hwlock->lock, *flags);
127 		else if (mode == HWLOCK_IRQ)
128 			spin_unlock_irq(&hwlock->lock);
129 		else
130 			spin_unlock(&hwlock->lock);
131 
132 		return -EBUSY;
133 	}
134 
135 	/*
136 	 * We can be sure the other core's memory operations
137 	 * are observable to us only _after_ we successfully take
138 	 * the hwspinlock, and we must make sure that subsequent memory
139 	 * operations (both reads and writes) will not be reordered before
140 	 * we actually took the hwspinlock.
141 	 *
142 	 * Note: the implicit memory barrier of the spinlock above is too
143 	 * early, so we need this additional explicit memory barrier.
144 	 */
145 	mb();
146 
147 	return 0;
148 }
149 EXPORT_SYMBOL_GPL(__hwspin_trylock);
150 
151 /**
152  * __hwspin_lock_timeout() - lock an hwspinlock with timeout limit
153  * @hwlock: the hwspinlock to be locked
154  * @timeout: timeout value in msecs
155  * @mode: mode which controls whether local interrupts are disabled or not
156  * @flags: a pointer to where the caller's interrupt state will be saved at (if
157  *         requested)
158  *
159  * This function locks the given @hwlock. If the @hwlock
160  * is already taken, the function will busy loop waiting for it to
161  * be released, but give up after @timeout msecs have elapsed.
162  *
163  * Upon a successful return from this function, preemption is disabled
164  * (and possibly local interrupts, too), so the caller must not sleep,
165  * and is advised to release the hwspinlock as soon as possible.
166  * This is required in order to minimize remote cores polling on the
167  * hardware interconnect.
168  *
169  * The user decides whether local interrupts are disabled or not, and if yes,
170  * whether he wants their previous state to be saved. It is up to the user
171  * to choose the appropriate @mode of operation, exactly the same way users
172  * should decide between spin_lock, spin_lock_irq and spin_lock_irqsave.
173  *
174  * Returns 0 when the @hwlock was successfully taken, and an appropriate
175  * error code otherwise (most notably -ETIMEDOUT if the @hwlock is still
176  * busy after @timeout msecs). The function will never sleep.
177  */
178 int __hwspin_lock_timeout(struct hwspinlock *hwlock, unsigned int to,
179 					int mode, unsigned long *flags)
180 {
181 	int ret;
182 	unsigned long expire;
183 
184 	expire = msecs_to_jiffies(to) + jiffies;
185 
186 	for (;;) {
187 		/* Try to take the hwspinlock */
188 		ret = __hwspin_trylock(hwlock, mode, flags);
189 		if (ret != -EBUSY)
190 			break;
191 
192 		/*
193 		 * The lock is already taken, let's check if the user wants
194 		 * us to try again
195 		 */
196 		if (time_is_before_eq_jiffies(expire))
197 			return -ETIMEDOUT;
198 
199 		/*
200 		 * Allow platform-specific relax handlers to prevent
201 		 * hogging the interconnect (no sleeping, though)
202 		 */
203 		if (hwlock->bank->ops->relax)
204 			hwlock->bank->ops->relax(hwlock);
205 	}
206 
207 	return ret;
208 }
209 EXPORT_SYMBOL_GPL(__hwspin_lock_timeout);
210 
211 /**
212  * __hwspin_unlock() - unlock a specific hwspinlock
213  * @hwlock: a previously-acquired hwspinlock which we want to unlock
214  * @mode: controls whether local interrupts needs to be restored or not
215  * @flags: previous caller's interrupt state to restore (if requested)
216  *
217  * This function will unlock a specific hwspinlock, enable preemption and
218  * (possibly) enable interrupts or restore their previous state.
219  * @hwlock must be already locked before calling this function: it is a bug
220  * to call unlock on a @hwlock that is already unlocked.
221  *
222  * The user decides whether local interrupts should be enabled or not, and
223  * if yes, whether he wants their previous state to be restored. It is up
224  * to the user to choose the appropriate @mode of operation, exactly the
225  * same way users decide between spin_unlock, spin_unlock_irq and
226  * spin_unlock_irqrestore.
227  *
228  * The function will never sleep.
229  */
230 void __hwspin_unlock(struct hwspinlock *hwlock, int mode, unsigned long *flags)
231 {
232 	BUG_ON(!hwlock);
233 	BUG_ON(!flags && mode == HWLOCK_IRQSTATE);
234 
235 	/*
236 	 * We must make sure that memory operations (both reads and writes),
237 	 * done before unlocking the hwspinlock, will not be reordered
238 	 * after the lock is released.
239 	 *
240 	 * That's the purpose of this explicit memory barrier.
241 	 *
242 	 * Note: the memory barrier induced by the spin_unlock below is too
243 	 * late; the other core is going to access memory soon after it will
244 	 * take the hwspinlock, and by then we want to be sure our memory
245 	 * operations are already observable.
246 	 */
247 	mb();
248 
249 	hwlock->bank->ops->unlock(hwlock);
250 
251 	/* Undo the spin_trylock{_irq, _irqsave} called while locking */
252 	if (mode == HWLOCK_IRQSTATE)
253 		spin_unlock_irqrestore(&hwlock->lock, *flags);
254 	else if (mode == HWLOCK_IRQ)
255 		spin_unlock_irq(&hwlock->lock);
256 	else
257 		spin_unlock(&hwlock->lock);
258 }
259 EXPORT_SYMBOL_GPL(__hwspin_unlock);
260 
261 /**
262  * of_hwspin_lock_simple_xlate - translate hwlock_spec to return a lock id
263  * @bank: the hwspinlock device bank
264  * @hwlock_spec: hwlock specifier as found in the device tree
265  *
266  * This is a simple translation function, suitable for hwspinlock platform
267  * drivers that only has a lock specifier length of 1.
268  *
269  * Returns a relative index of the lock within a specified bank on success,
270  * or -EINVAL on invalid specifier cell count.
271  */
272 static inline int
273 of_hwspin_lock_simple_xlate(const struct of_phandle_args *hwlock_spec)
274 {
275 	if (WARN_ON(hwlock_spec->args_count != 1))
276 		return -EINVAL;
277 
278 	return hwlock_spec->args[0];
279 }
280 
281 /**
282  * of_hwspin_lock_get_id() - get lock id for an OF phandle-based specific lock
283  * @np: device node from which to request the specific hwlock
284  * @index: index of the hwlock in the list of values
285  *
286  * This function provides a means for DT users of the hwspinlock module to
287  * get the global lock id of a specific hwspinlock using the phandle of the
288  * hwspinlock device, so that it can be requested using the normal
289  * hwspin_lock_request_specific() API.
290  *
291  * Returns the global lock id number on success, -EPROBE_DEFER if the hwspinlock
292  * device is not yet registered, -EINVAL on invalid args specifier value or an
293  * appropriate error as returned from the OF parsing of the DT client node.
294  */
295 int of_hwspin_lock_get_id(struct device_node *np, int index)
296 {
297 	struct of_phandle_args args;
298 	struct hwspinlock *hwlock;
299 	struct radix_tree_iter iter;
300 	void **slot;
301 	int id;
302 	int ret;
303 
304 	ret = of_parse_phandle_with_args(np, "hwlocks", "#hwlock-cells", index,
305 					 &args);
306 	if (ret)
307 		return ret;
308 
309 	/* Find the hwspinlock device: we need its base_id */
310 	ret = -EPROBE_DEFER;
311 	rcu_read_lock();
312 	radix_tree_for_each_slot(slot, &hwspinlock_tree, &iter, 0) {
313 		hwlock = radix_tree_deref_slot(slot);
314 		if (unlikely(!hwlock))
315 			continue;
316 
317 		if (hwlock->bank->dev->of_node == args.np) {
318 			ret = 0;
319 			break;
320 		}
321 	}
322 	rcu_read_unlock();
323 	if (ret < 0)
324 		goto out;
325 
326 	id = of_hwspin_lock_simple_xlate(&args);
327 	if (id < 0 || id >= hwlock->bank->num_locks) {
328 		ret = -EINVAL;
329 		goto out;
330 	}
331 	id += hwlock->bank->base_id;
332 
333 out:
334 	of_node_put(args.np);
335 	return ret ? ret : id;
336 }
337 EXPORT_SYMBOL_GPL(of_hwspin_lock_get_id);
338 
339 static int hwspin_lock_register_single(struct hwspinlock *hwlock, int id)
340 {
341 	struct hwspinlock *tmp;
342 	int ret;
343 
344 	mutex_lock(&hwspinlock_tree_lock);
345 
346 	ret = radix_tree_insert(&hwspinlock_tree, id, hwlock);
347 	if (ret) {
348 		if (ret == -EEXIST)
349 			pr_err("hwspinlock id %d already exists!\n", id);
350 		goto out;
351 	}
352 
353 	/* mark this hwspinlock as available */
354 	tmp = radix_tree_tag_set(&hwspinlock_tree, id, HWSPINLOCK_UNUSED);
355 
356 	/* self-sanity check which should never fail */
357 	WARN_ON(tmp != hwlock);
358 
359 out:
360 	mutex_unlock(&hwspinlock_tree_lock);
361 	return 0;
362 }
363 
364 static struct hwspinlock *hwspin_lock_unregister_single(unsigned int id)
365 {
366 	struct hwspinlock *hwlock = NULL;
367 	int ret;
368 
369 	mutex_lock(&hwspinlock_tree_lock);
370 
371 	/* make sure the hwspinlock is not in use (tag is set) */
372 	ret = radix_tree_tag_get(&hwspinlock_tree, id, HWSPINLOCK_UNUSED);
373 	if (ret == 0) {
374 		pr_err("hwspinlock %d still in use (or not present)\n", id);
375 		goto out;
376 	}
377 
378 	hwlock = radix_tree_delete(&hwspinlock_tree, id);
379 	if (!hwlock) {
380 		pr_err("failed to delete hwspinlock %d\n", id);
381 		goto out;
382 	}
383 
384 out:
385 	mutex_unlock(&hwspinlock_tree_lock);
386 	return hwlock;
387 }
388 
389 /**
390  * hwspin_lock_register() - register a new hw spinlock device
391  * @bank: the hwspinlock device, which usually provides numerous hw locks
392  * @dev: the backing device
393  * @ops: hwspinlock handlers for this device
394  * @base_id: id of the first hardware spinlock in this bank
395  * @num_locks: number of hwspinlocks provided by this device
396  *
397  * This function should be called from the underlying platform-specific
398  * implementation, to register a new hwspinlock device instance.
399  *
400  * Should be called from a process context (might sleep)
401  *
402  * Returns 0 on success, or an appropriate error code on failure
403  */
404 int hwspin_lock_register(struct hwspinlock_device *bank, struct device *dev,
405 		const struct hwspinlock_ops *ops, int base_id, int num_locks)
406 {
407 	struct hwspinlock *hwlock;
408 	int ret = 0, i;
409 
410 	if (!bank || !ops || !dev || !num_locks || !ops->trylock ||
411 							!ops->unlock) {
412 		pr_err("invalid parameters\n");
413 		return -EINVAL;
414 	}
415 
416 	bank->dev = dev;
417 	bank->ops = ops;
418 	bank->base_id = base_id;
419 	bank->num_locks = num_locks;
420 
421 	for (i = 0; i < num_locks; i++) {
422 		hwlock = &bank->lock[i];
423 
424 		spin_lock_init(&hwlock->lock);
425 		hwlock->bank = bank;
426 
427 		ret = hwspin_lock_register_single(hwlock, base_id + i);
428 		if (ret)
429 			goto reg_failed;
430 	}
431 
432 	return 0;
433 
434 reg_failed:
435 	while (--i >= 0)
436 		hwspin_lock_unregister_single(base_id + i);
437 	return ret;
438 }
439 EXPORT_SYMBOL_GPL(hwspin_lock_register);
440 
441 /**
442  * hwspin_lock_unregister() - unregister an hw spinlock device
443  * @bank: the hwspinlock device, which usually provides numerous hw locks
444  *
445  * This function should be called from the underlying platform-specific
446  * implementation, to unregister an existing (and unused) hwspinlock.
447  *
448  * Should be called from a process context (might sleep)
449  *
450  * Returns 0 on success, or an appropriate error code on failure
451  */
452 int hwspin_lock_unregister(struct hwspinlock_device *bank)
453 {
454 	struct hwspinlock *hwlock, *tmp;
455 	int i;
456 
457 	for (i = 0; i < bank->num_locks; i++) {
458 		hwlock = &bank->lock[i];
459 
460 		tmp = hwspin_lock_unregister_single(bank->base_id + i);
461 		if (!tmp)
462 			return -EBUSY;
463 
464 		/* self-sanity check that should never fail */
465 		WARN_ON(tmp != hwlock);
466 	}
467 
468 	return 0;
469 }
470 EXPORT_SYMBOL_GPL(hwspin_lock_unregister);
471 
472 /**
473  * __hwspin_lock_request() - tag an hwspinlock as used and power it up
474  *
475  * This is an internal function that prepares an hwspinlock instance
476  * before it is given to the user. The function assumes that
477  * hwspinlock_tree_lock is taken.
478  *
479  * Returns 0 or positive to indicate success, and a negative value to
480  * indicate an error (with the appropriate error code)
481  */
482 static int __hwspin_lock_request(struct hwspinlock *hwlock)
483 {
484 	struct device *dev = hwlock->bank->dev;
485 	struct hwspinlock *tmp;
486 	int ret;
487 
488 	/* prevent underlying implementation from being removed */
489 	if (!try_module_get(dev->driver->owner)) {
490 		dev_err(dev, "%s: can't get owner\n", __func__);
491 		return -EINVAL;
492 	}
493 
494 	/* notify PM core that power is now needed */
495 	ret = pm_runtime_get_sync(dev);
496 	if (ret < 0) {
497 		dev_err(dev, "%s: can't power on device\n", __func__);
498 		pm_runtime_put_noidle(dev);
499 		module_put(dev->driver->owner);
500 		return ret;
501 	}
502 
503 	/* mark hwspinlock as used, should not fail */
504 	tmp = radix_tree_tag_clear(&hwspinlock_tree, hwlock_to_id(hwlock),
505 							HWSPINLOCK_UNUSED);
506 
507 	/* self-sanity check that should never fail */
508 	WARN_ON(tmp != hwlock);
509 
510 	return ret;
511 }
512 
513 /**
514  * hwspin_lock_get_id() - retrieve id number of a given hwspinlock
515  * @hwlock: a valid hwspinlock instance
516  *
517  * Returns the id number of a given @hwlock, or -EINVAL if @hwlock is invalid.
518  */
519 int hwspin_lock_get_id(struct hwspinlock *hwlock)
520 {
521 	if (!hwlock) {
522 		pr_err("invalid hwlock\n");
523 		return -EINVAL;
524 	}
525 
526 	return hwlock_to_id(hwlock);
527 }
528 EXPORT_SYMBOL_GPL(hwspin_lock_get_id);
529 
530 /**
531  * hwspin_lock_request() - request an hwspinlock
532  *
533  * This function should be called by users of the hwspinlock device,
534  * in order to dynamically assign them an unused hwspinlock.
535  * Usually the user of this lock will then have to communicate the lock's id
536  * to the remote core before it can be used for synchronization (to get the
537  * id of a given hwlock, use hwspin_lock_get_id()).
538  *
539  * Should be called from a process context (might sleep)
540  *
541  * Returns the address of the assigned hwspinlock, or NULL on error
542  */
543 struct hwspinlock *hwspin_lock_request(void)
544 {
545 	struct hwspinlock *hwlock;
546 	int ret;
547 
548 	mutex_lock(&hwspinlock_tree_lock);
549 
550 	/* look for an unused lock */
551 	ret = radix_tree_gang_lookup_tag(&hwspinlock_tree, (void **)&hwlock,
552 						0, 1, HWSPINLOCK_UNUSED);
553 	if (ret == 0) {
554 		pr_warn("a free hwspinlock is not available\n");
555 		hwlock = NULL;
556 		goto out;
557 	}
558 
559 	/* sanity check that should never fail */
560 	WARN_ON(ret > 1);
561 
562 	/* mark as used and power up */
563 	ret = __hwspin_lock_request(hwlock);
564 	if (ret < 0)
565 		hwlock = NULL;
566 
567 out:
568 	mutex_unlock(&hwspinlock_tree_lock);
569 	return hwlock;
570 }
571 EXPORT_SYMBOL_GPL(hwspin_lock_request);
572 
573 /**
574  * hwspin_lock_request_specific() - request for a specific hwspinlock
575  * @id: index of the specific hwspinlock that is requested
576  *
577  * This function should be called by users of the hwspinlock module,
578  * in order to assign them a specific hwspinlock.
579  * Usually early board code will be calling this function in order to
580  * reserve specific hwspinlock ids for predefined purposes.
581  *
582  * Should be called from a process context (might sleep)
583  *
584  * Returns the address of the assigned hwspinlock, or NULL on error
585  */
586 struct hwspinlock *hwspin_lock_request_specific(unsigned int id)
587 {
588 	struct hwspinlock *hwlock;
589 	int ret;
590 
591 	mutex_lock(&hwspinlock_tree_lock);
592 
593 	/* make sure this hwspinlock exists */
594 	hwlock = radix_tree_lookup(&hwspinlock_tree, id);
595 	if (!hwlock) {
596 		pr_warn("hwspinlock %u does not exist\n", id);
597 		goto out;
598 	}
599 
600 	/* sanity check (this shouldn't happen) */
601 	WARN_ON(hwlock_to_id(hwlock) != id);
602 
603 	/* make sure this hwspinlock is unused */
604 	ret = radix_tree_tag_get(&hwspinlock_tree, id, HWSPINLOCK_UNUSED);
605 	if (ret == 0) {
606 		pr_warn("hwspinlock %u is already in use\n", id);
607 		hwlock = NULL;
608 		goto out;
609 	}
610 
611 	/* mark as used and power up */
612 	ret = __hwspin_lock_request(hwlock);
613 	if (ret < 0)
614 		hwlock = NULL;
615 
616 out:
617 	mutex_unlock(&hwspinlock_tree_lock);
618 	return hwlock;
619 }
620 EXPORT_SYMBOL_GPL(hwspin_lock_request_specific);
621 
622 /**
623  * hwspin_lock_free() - free a specific hwspinlock
624  * @hwlock: the specific hwspinlock to free
625  *
626  * This function mark @hwlock as free again.
627  * Should only be called with an @hwlock that was retrieved from
628  * an earlier call to omap_hwspin_lock_request{_specific}.
629  *
630  * Should be called from a process context (might sleep)
631  *
632  * Returns 0 on success, or an appropriate error code on failure
633  */
634 int hwspin_lock_free(struct hwspinlock *hwlock)
635 {
636 	struct device *dev;
637 	struct hwspinlock *tmp;
638 	int ret;
639 
640 	if (!hwlock) {
641 		pr_err("invalid hwlock\n");
642 		return -EINVAL;
643 	}
644 
645 	dev = hwlock->bank->dev;
646 	mutex_lock(&hwspinlock_tree_lock);
647 
648 	/* make sure the hwspinlock is used */
649 	ret = radix_tree_tag_get(&hwspinlock_tree, hwlock_to_id(hwlock),
650 							HWSPINLOCK_UNUSED);
651 	if (ret == 1) {
652 		dev_err(dev, "%s: hwlock is already free\n", __func__);
653 		dump_stack();
654 		ret = -EINVAL;
655 		goto out;
656 	}
657 
658 	/* notify the underlying device that power is not needed */
659 	ret = pm_runtime_put(dev);
660 	if (ret < 0)
661 		goto out;
662 
663 	/* mark this hwspinlock as available */
664 	tmp = radix_tree_tag_set(&hwspinlock_tree, hwlock_to_id(hwlock),
665 							HWSPINLOCK_UNUSED);
666 
667 	/* sanity check (this shouldn't happen) */
668 	WARN_ON(tmp != hwlock);
669 
670 	module_put(dev->driver->owner);
671 
672 out:
673 	mutex_unlock(&hwspinlock_tree_lock);
674 	return ret;
675 }
676 EXPORT_SYMBOL_GPL(hwspin_lock_free);
677 
678 MODULE_LICENSE("GPL v2");
679 MODULE_DESCRIPTION("Hardware spinlock interface");
680 MODULE_AUTHOR("Ohad Ben-Cohen <ohad@wizery.com>");
681