diff options
-rw-r--r-- | drivers/mailbox/mailbox.c | 11 | ||||
-rw-r--r-- | drivers/mailbox/omap-mailbox.c | 6 |
2 files changed, 11 insertions, 6 deletions
diff --git a/drivers/mailbox/mailbox.c b/drivers/mailbox/mailbox.c index 19b491d2964f..c3c42d42d017 100644 --- a/drivers/mailbox/mailbox.c +++ b/drivers/mailbox/mailbox.c | |||
@@ -318,7 +318,7 @@ struct mbox_chan *mbox_request_channel(struct mbox_client *cl, int index) | |||
318 | return ERR_PTR(-ENODEV); | 318 | return ERR_PTR(-ENODEV); |
319 | } | 319 | } |
320 | 320 | ||
321 | chan = NULL; | 321 | chan = ERR_PTR(-EPROBE_DEFER); |
322 | list_for_each_entry(mbox, &mbox_cons, node) | 322 | list_for_each_entry(mbox, &mbox_cons, node) |
323 | if (mbox->dev->of_node == spec.np) { | 323 | if (mbox->dev->of_node == spec.np) { |
324 | chan = mbox->of_xlate(mbox, &spec); | 324 | chan = mbox->of_xlate(mbox, &spec); |
@@ -327,7 +327,12 @@ struct mbox_chan *mbox_request_channel(struct mbox_client *cl, int index) | |||
327 | 327 | ||
328 | of_node_put(spec.np); | 328 | of_node_put(spec.np); |
329 | 329 | ||
330 | if (!chan || chan->cl || !try_module_get(mbox->dev->driver->owner)) { | 330 | if (IS_ERR(chan)) { |
331 | mutex_unlock(&con_mutex); | ||
332 | return chan; | ||
333 | } | ||
334 | |||
335 | if (chan->cl || !try_module_get(mbox->dev->driver->owner)) { | ||
331 | dev_dbg(dev, "%s: mailbox not free\n", __func__); | 336 | dev_dbg(dev, "%s: mailbox not free\n", __func__); |
332 | mutex_unlock(&con_mutex); | 337 | mutex_unlock(&con_mutex); |
333 | return ERR_PTR(-EBUSY); | 338 | return ERR_PTR(-EBUSY); |
@@ -390,7 +395,7 @@ of_mbox_index_xlate(struct mbox_controller *mbox, | |||
390 | int ind = sp->args[0]; | 395 | int ind = sp->args[0]; |
391 | 396 | ||
392 | if (ind >= mbox->num_chans) | 397 | if (ind >= mbox->num_chans) |
393 | return NULL; | 398 | return ERR_PTR(-EINVAL); |
394 | 399 | ||
395 | return &mbox->chans[ind]; | 400 | return &mbox->chans[ind]; |
396 | } | 401 | } |
diff --git a/drivers/mailbox/omap-mailbox.c b/drivers/mailbox/omap-mailbox.c index 03f8545ba037..a3dbfd9c6479 100644 --- a/drivers/mailbox/omap-mailbox.c +++ b/drivers/mailbox/omap-mailbox.c | |||
@@ -639,18 +639,18 @@ static struct mbox_chan *omap_mbox_of_xlate(struct mbox_controller *controller, | |||
639 | 639 | ||
640 | mdev = container_of(controller, struct omap_mbox_device, controller); | 640 | mdev = container_of(controller, struct omap_mbox_device, controller); |
641 | if (WARN_ON(!mdev)) | 641 | if (WARN_ON(!mdev)) |
642 | return NULL; | 642 | return ERR_PTR(-EINVAL); |
643 | 643 | ||
644 | node = of_find_node_by_phandle(phandle); | 644 | node = of_find_node_by_phandle(phandle); |
645 | if (!node) { | 645 | if (!node) { |
646 | pr_err("%s: could not find node phandle 0x%x\n", | 646 | pr_err("%s: could not find node phandle 0x%x\n", |
647 | __func__, phandle); | 647 | __func__, phandle); |
648 | return NULL; | 648 | return ERR_PTR(-ENODEV); |
649 | } | 649 | } |
650 | 650 | ||
651 | mbox = omap_mbox_device_find(mdev, node->name); | 651 | mbox = omap_mbox_device_find(mdev, node->name); |
652 | of_node_put(node); | 652 | of_node_put(node); |
653 | return mbox ? mbox->chan : NULL; | 653 | return mbox ? mbox->chan : ERR_PTR(-ENOENT); |
654 | } | 654 | } |
655 | 655 | ||
656 | static int omap_mbox_probe(struct platform_device *pdev) | 656 | static int omap_mbox_probe(struct platform_device *pdev) |