diff mbox series

[v2,06/22] hw/pvrdma: Make function reset_device return void

Message ID 20181108160818.5485-7-yuval.shaia@oracle.com (mailing list archive)
State New, archived
Headers show
Series Add support for RDMA MAD | expand

Commit Message

Yuval Shaia Nov. 8, 2018, 4:08 p.m. UTC
This function cannot fail - fix it to return void

Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
---
 hw/rdma/vmw/pvrdma_main.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Marcel Apfelbaum Nov. 10, 2018, 6:17 p.m. UTC | #1
On 11/8/18 6:08 PM, Yuval Shaia wrote:
> This function cannot fail - fix it to return void
>
> Signed-off-by: Yuval Shaia <yuval.shaia@oracle.com>
> ---
>   hw/rdma/vmw/pvrdma_main.c | 4 +---
>   1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/hw/rdma/vmw/pvrdma_main.c b/hw/rdma/vmw/pvrdma_main.c
> index 6c8c0154fa..fc2abd34af 100644
> --- a/hw/rdma/vmw/pvrdma_main.c
> +++ b/hw/rdma/vmw/pvrdma_main.c
> @@ -369,13 +369,11 @@ static int unquiesce_device(PVRDMADev *dev)
>       return 0;
>   }
>   
> -static int reset_device(PVRDMADev *dev)
> +static void reset_device(PVRDMADev *dev)
>   {
>       pvrdma_stop(dev);
>   
>       pr_dbg("Device reset complete\n");
> -
> -    return 0;
>
>   
>   static uint64_t regs_read(void *opaque, hwaddr addr, unsigned size)

Reviewed-by: Marcel Apfelbaum<marcel.apfelbaum@gmail.com>

Thanks,
Marcel
diff mbox series

Patch

diff --git a/hw/rdma/vmw/pvrdma_main.c b/hw/rdma/vmw/pvrdma_main.c
index 6c8c0154fa..fc2abd34af 100644
--- a/hw/rdma/vmw/pvrdma_main.c
+++ b/hw/rdma/vmw/pvrdma_main.c
@@ -369,13 +369,11 @@  static int unquiesce_device(PVRDMADev *dev)
     return 0;
 }
 
-static int reset_device(PVRDMADev *dev)
+static void reset_device(PVRDMADev *dev)
 {
     pvrdma_stop(dev);
 
     pr_dbg("Device reset complete\n");
-
-    return 0;
 }
 
 static uint64_t regs_read(void *opaque, hwaddr addr, unsigned size)