[coreboot-gerrit] New patch to review for coreboot: soc/mediatek/mt8173: Do not initialize static variables to 0

Furquan Shaikh (furquan@google.com) gerrit at coreboot.org
Wed Dec 7 16:36:31 CET 2016


Furquan Shaikh (furquan at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17765

-gerrit

commit 5a319999508bfb7621f0798bcea4fadb18095f44
Author: Furquan Shaikh <furquan at chromium.org>
Date:   Wed Dec 7 07:35:01 2016 -0800

    soc/mediatek/mt8173: Do not initialize static variables to 0
    
    Change-Id: Ibf0bd772bfdb3bbf6482a0ec9ff90a5c0a8945d2
    Signed-off-by: Furquan Shaikh <furquan at chromium.org>
---
 src/soc/mediatek/mt8173/flash_controller.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/soc/mediatek/mt8173/flash_controller.c b/src/soc/mediatek/mt8173/flash_controller.c
index 2aee668..f6f6e2a 100644
--- a/src/soc/mediatek/mt8173/flash_controller.c
+++ b/src/soc/mediatek/mt8173/flash_controller.c
@@ -235,8 +235,8 @@ static int nor_erase(const struct spi_flash *flash, u32 offset, size_t len)
 
 struct spi_flash *spi_flash_programmer_probe(struct spi_slave *spi, int force)
 {
-	static struct spi_flash flash = {0};
-	static int done = 0;
+	static struct spi_flash flash;
+	static int done;
 
 	if (done)
 		return &flash;



More information about the coreboot-gerrit mailing list