aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--drivers/mailbox/mailbox.c11
-rw-r--r--drivers/mailbox/omap-mailbox.c6
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
656static int omap_mbox_probe(struct platform_device *pdev) 656static int omap_mbox_probe(struct platform_device *pdev)