Commit a5c330fe authored by Martyn Welch's avatar Martyn Welch Committed by Greg Kroah-Hartman

staging: vme: Fix mutex locking

Fix incorrect use of mutex_trylock().
Signed-off-by: default avatarMartyn Welch <martyn.welch@gefanuc.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 58e50798
...@@ -879,7 +879,7 @@ int vme_dma_list_add(struct vme_dma_list *list, struct vme_dma_attr *src, ...@@ -879,7 +879,7 @@ int vme_dma_list_add(struct vme_dma_list *list, struct vme_dma_attr *src,
return -EINVAL; return -EINVAL;
} }
if (mutex_trylock(&(list->mtx))) { if (!mutex_trylock(&(list->mtx))) {
printk("Link List already submitted\n"); printk("Link List already submitted\n");
return -EINVAL; return -EINVAL;
} }
...@@ -922,7 +922,7 @@ int vme_dma_list_free(struct vme_dma_list *list) ...@@ -922,7 +922,7 @@ int vme_dma_list_free(struct vme_dma_list *list)
return -EINVAL; return -EINVAL;
} }
if (mutex_trylock(&(list->mtx))) { if (!mutex_trylock(&(list->mtx))) {
printk("Link List in use\n"); printk("Link List in use\n");
return -EINVAL; return -EINVAL;
} }
...@@ -955,7 +955,7 @@ int vme_dma_free(struct vme_resource *resource) ...@@ -955,7 +955,7 @@ int vme_dma_free(struct vme_resource *resource)
ctrlr = list_entry(resource->entry, struct vme_dma_resource, list); ctrlr = list_entry(resource->entry, struct vme_dma_resource, list);
if (mutex_trylock(&(ctrlr->mtx))) { if (!mutex_trylock(&(ctrlr->mtx))) {
printk("Resource busy, can't free\n"); printk("Resource busy, can't free\n");
return -EBUSY; return -EBUSY;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment