drivers/rpmsg/virtio_rpmsg_bus.c: fix error return code in rpmsg_probe()
authorWei Yongjun <[email protected]>
Mon, 29 Apr 2013 23:17:09 +0000 (16:17 -0700)
committerLinus Torvalds <[email protected]>
Tue, 30 Apr 2013 01:28:13 +0000 (18:28 -0700)
Return a negative error code from the error handling case instead of 0, as
returned elsewhere in this function.

Signed-off-by: Wei Yongjun <[email protected]>
Cc: Ohad Ben-Cohen <[email protected]>
Cc: Tejun Heo <[email protected]>
Cc: Stephen Boyd <[email protected]>
Cc: Rusty Russell <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
drivers/rpmsg/virtio_rpmsg_bus.c

index a59684b5fc68d33dd7a5b36c1be4ac3e34b3c8c8..7861f1119b7d2e07d10b3738109a2d4f61e8daaf 100644 (file)
@@ -951,8 +951,10 @@ static int rpmsg_probe(struct virtio_device *vdev)
        bufs_va = dma_alloc_coherent(vdev->dev.parent->parent,
                                RPMSG_TOTAL_BUF_SPACE,
                                &vrp->bufs_dma, GFP_KERNEL);
-       if (!bufs_va)
+       if (!bufs_va) {
+               err = -ENOMEM;
                goto vqs_del;
+       }
 
        dev_dbg(&vdev->dev, "buffers: va %p, dma 0x%llx\n", bufs_va,
                                        (unsigned long long)vrp->bufs_dma);