From 10201e11e1c05f7b1674669d6405aa27345e7b0c Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Sun, 4 Jun 2017 05:53:04 -0700 Subject: [PATCH] UPSTREAM: spi: Remove unused/unnecessary spi_init function definitions Remove spi_init definitions which: 1. Do nothing 2. Set static global variables to 0 BUG=none BRANCH=none TEST=none Change-Id: I471dc31bf496936968c3a4a55f7ecac517752e5d Signed-off-by: Patrick Georgi Original-Commit-Id: 2dafd897692a257b3727723601ac68f902e746f2 Original-Change-Id: If4c0cdbe2271fc7561becd87ad3b96bd45e77430 Original-Signed-off-by: Furquan Shaikh Original-Reviewed-on: https://review.coreboot.org/20039 Original-Reviewed-by: Aaron Durbin Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Philippe Mathieu-Daud Original-Reviewed-by: Paul Menzel Reviewed-on: https://chromium-review.googlesource.com/528265 Commit-Ready: Patrick Georgi Tested-by: Patrick Georgi Reviewed-by: Patrick Georgi --- src/soc/imgtec/pistachio/spi.c | 7 ------- src/soc/qualcomm/ipq40xx/spi.c | 6 ------ src/soc/qualcomm/ipq806x/spi.c | 6 ------ src/soc/samsung/exynos5420/spi.c | 4 ---- src/southbridge/amd/sb700/spi.c | 5 ----- 5 files changed, 28 deletions(-) diff --git a/src/soc/imgtec/pistachio/spi.c b/src/soc/imgtec/pistachio/spi.c index bfd982cf62..7e1a7a67f5 100644 --- a/src/soc/imgtec/pistachio/spi.c +++ b/src/soc/imgtec/pistachio/spi.c @@ -410,13 +410,6 @@ static int spim_io(const struct spi_slave *slave, struct spim_buffer *first, return SPIM_OK; } -/* Initialization, must be called once on start up */ -void spi_init(void) -{ - /* Clear everything just in case */ - memset(img_spi_slaves, 0, sizeof(img_spi_slaves)); -} - /* Claim the bus and prepare it for communication */ static int spi_ctrlr_claim_bus(const struct spi_slave *slave) { diff --git a/src/soc/qualcomm/ipq40xx/spi.c b/src/soc/qualcomm/ipq40xx/spi.c index 157903cf50..3dc6022b03 100644 --- a/src/soc/qualcomm/ipq40xx/spi.c +++ b/src/soc/qualcomm/ipq40xx/spi.c @@ -201,12 +201,6 @@ static void spi_reset(struct ipq_spi_slave *ds) static struct ipq_spi_slave spi_slave_pool[2]; -void spi_init(void) -{ - /* just in case */ - memset(spi_slave_pool, 0, sizeof(spi_slave_pool)); -} - static struct ipq_spi_slave *to_ipq_spi(const struct spi_slave *slave) { struct ipq_spi_slave *ds; diff --git a/src/soc/qualcomm/ipq806x/spi.c b/src/soc/qualcomm/ipq806x/spi.c index c20ad9a325..f9a89fcfed 100644 --- a/src/soc/qualcomm/ipq806x/spi.c +++ b/src/soc/qualcomm/ipq806x/spi.c @@ -493,12 +493,6 @@ static const struct gsbi_spi spi_reg[] = { }; static struct ipq_spi_slave spi_slave_pool[2]; -void spi_init() -{ - /* just in case */ - memset(spi_slave_pool, 0, sizeof(spi_slave_pool)); -} - static struct ipq_spi_slave *to_ipq_spi(const struct spi_slave *slave) { struct ipq_spi_slave *ds; diff --git a/src/soc/samsung/exynos5420/spi.c b/src/soc/samsung/exynos5420/spi.c index fa53c0f4a0..2023b10130 100644 --- a/src/soc/samsung/exynos5420/spi.c +++ b/src/soc/samsung/exynos5420/spi.c @@ -95,10 +95,6 @@ static void spi_sw_reset(struct exynos_spi *regs, int word) setbits_le32(®s->ch_cfg, SPI_RX_CH_ON | SPI_TX_CH_ON); } -void spi_init(void) -{ -} - static void exynos_spi_init(struct exynos_spi *regs) { // Set FB_CLK_SEL. diff --git a/src/southbridge/amd/sb700/spi.c b/src/southbridge/amd/sb700/spi.c index df44c0475d..caddfc88c0 100644 --- a/src/southbridge/amd/sb700/spi.c +++ b/src/southbridge/amd/sb700/spi.c @@ -35,11 +35,6 @@ static uint32_t get_spi_bar(void) return pci_read_config32(dev, 0xa0) & ~0x1f; } -void spi_init(void) -{ - /* Not needed */ -} - static void reset_internal_fifo_pointer(void) { uint32_t spibar = get_spi_bar();