From: Alexander Dahl <ada@thorsis.com>
Date: Fri, 28 Jun 2019 12:41:23 +0000 (+0200)
Subject: fpga: altera: cyclon2: Check function pointer before calling
X-Git-Tag: v2025.01-rc5-pxa1908~2854^2~2
X-Git-Url: http://git.dujemihanovic.xyz/%22http:/kyber.dk/phpMyBuilder/static/%7B%7B%20%28.OutputFormats.Get?a=commitdiff_plain;h=b283d6ba67a7b4c5914949ef1d756bf88cb3a0e4;p=u-boot.git

fpga: altera: cyclon2: Check function pointer before calling

As already done for the 'pre' function, a check is added to not follow a
NULL pointer, if somebody has not assigned a 'post' function.

Signed-off-by: Alexander Dahl <ada@thorsis.com>
---

diff --git a/drivers/fpga/cyclon2.c b/drivers/fpga/cyclon2.c
index ab979b017a..c929cd2cc5 100644
--- a/drivers/fpga/cyclon2.c
+++ b/drivers/fpga/cyclon2.c
@@ -183,8 +183,12 @@ static int CYC2_ps_load(Altera_desc *desc, const void *buf, size_t bsize)
 		else
 			puts("Fail.\n");
 #endif
-		(*fn->post) (cookie);
 
+		/*
+		 * Run the post configuration function if there is one.
+		 */
+		if (*fn->post)
+			(*fn->post) (cookie);
 	} else {
 		printf("%s: NULL Interface function table!\n", __func__);
 	}