From 87eacb6829b53edf4347d8dba88c88eb02394fc6 Mon Sep 17 00:00:00 2001
From: Rami Rosen <ramirose@gmail.com>
Date: Tue, 18 Sep 2018 16:19:12 +0300
Subject: [PATCH] ioat: change the return type of ioat_channel_destruct() to be
 void

The static method ioat_channel_destruct() never fails. This cleanup patch
changes its return type to be void instead of int.

Change-Id: I0db464b6a810eadc3f58550114c4532ca33d03b5
Signed-off-by: Rami Rosen <ramirose@gmail.com>
Reviewed-on: https://review.gerrithub.io/425936
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
---
 lib/ioat/ioat.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/lib/ioat/ioat.c b/lib/ioat/ioat.c
index ee32ca27f..d8c15bf36 100644
--- a/lib/ioat/ioat.c
+++ b/lib/ioat/ioat.c
@@ -350,7 +350,7 @@ ioat_process_channel_events(struct spdk_ioat_chan *ioat)
 	return 0;
 }
 
-static int
+static void
 ioat_channel_destruct(struct spdk_ioat_chan *ioat)
 {
 	ioat_unmap_pci_bar(ioat);
@@ -367,8 +367,6 @@ ioat_channel_destruct(struct spdk_ioat_chan *ioat)
 		spdk_dma_free((void *)ioat->comp_update);
 		ioat->comp_update = NULL;
 	}
-
-	return 0;
 }
 
 static int