[PATCH 21/27] bios: mtrr make function multi_types_check static

Colin King colin.king at canonical.com
Wed Aug 15 13:11:23 UTC 2018


From: Colin Ian King <colin.king at canonical.com>

Function multi_types_check is local to the source and does not
need to be in the global namespace so make it static. Also remove
redundant voidification of function return.

Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
 src/bios/mtrr/mtrr.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/bios/mtrr/mtrr.c b/src/bios/mtrr/mtrr.c
index ecd6204e..5dfb4a60 100644
--- a/src/bios/mtrr/mtrr.c
+++ b/src/bios/mtrr/mtrr.c
@@ -360,7 +360,7 @@ static int guess_cache_type(
 	return FWTS_OK;
 }
 
-void multi_types_check(fwts_framework *fw, int *type)
+static void multi_types_check(fwts_framework *fw, int *type)
 {
 	int n_types = 0, i;
 
@@ -463,7 +463,7 @@ static int validate_iomem(fwts_framework *fw)
 
 		type = cache_types(start, end);
 
-		(void)multi_types_check(fw, &type);
+		multi_types_check(fw, &type);
 
 		if (guess_cache_type(fw, c2, &type_must, &type_mustnot, start) != FWTS_OK) {
 			/*  This has failed, give up at this point */
-- 
2.17.1




More information about the fwts-devel mailing list