commit 560111e2ce506b75b112f0d10b5f9b99f007bfa5
author Nico Huber <nico.huber@secunet.com> Wed Apr 26 12:27:17 2017 +0200
committer Nico Huber <nico.h@gmx.de> Tue Jun 20 11:48:06 2017 +0200
tree d9a568d1b4c092a80c9252a648f7ea7ce79bff01
parent 512059118e9ff56d2b4f3c324db5e764e288ac68

ichspi: Drop `dev` parameter from init functions

It's never used and has no clear contract (e.g. will the pointer stay
valid beyond the call?).

Change-Id: I0d4e7cc731364e86eff214b9022b842a577f9ef4
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/19460
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>

chipset_enable.c

diff --git a/chipset_enable.c b/chipset_enable.c
index 208cd3a..83c470f 100644
--- a/chipset_enable.c
+++ b/chipset_enable.c
@@ -718,7 +718,7 @@
 	void *spibar = rcrb + spibar_offset;
 
 	/* This adds BUS_SPI */
-	int ret_spi = ich_init_spi(dev, spibar, ich_generation);
+	int ret_spi = ich_init_spi(spibar, ich_generation);
 	if (ret_spi == ERROR_FATAL)
 		return ret_spi;
 	
@@ -846,7 +846,7 @@
 	 */
 	enable_flash_ich_bios_cntl_memmapped(ich_generation, spibar + 0xFC);
 
-	int ret_spi = ich_init_spi(dev, spibar, ich_generation);
+	int ret_spi = ich_init_spi(spibar, ich_generation);
 	if (ret_spi == ERROR_FATAL)
 		return ret_spi;
 
@@ -949,12 +949,12 @@
 			return ERROR_FATAL;
 	}
 
-	return via_init_spi(dev, spi0_mm_base);
+	return via_init_spi(spi0_mm_base);
 }
 
 static int enable_flash_vt8237s_spi(struct pci_dev *dev, const char *name)
 {
-	return via_init_spi(dev, pci_read_long(dev, 0xbc) << 8);
+	return via_init_spi(pci_read_long(dev, 0xbc) << 8);
 }
 
 static int enable_flash_cs5530(struct pci_dev *dev, const char *name)