From: Michael Brown Date: Wed, 1 May 2013 08:15:51 +0000 (+0100) Subject: [realtek] Ensure EEPROM writes reach chip before starting udelay() X-Git-Tag: v1.20.1~1518 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=9b1ec3132fed94c644c77a5e59a8ba2112110455;p=thirdparty%2Fipxe.git [realtek] Ensure EEPROM writes reach chip before starting udelay() On some systems, it appears to be possible for writes to the EEPROM registers to be delayed for long enough that the EEPROM's setup and hold times are violated, resulting in invalid data being read from the EEPROM. Fix by inserting a PCI read cycle immediately after writes to RTL_9346CR, to ensure that the write has completed before starting the udelay() used to time the SPI bus transitions. Reported-by: Gelip Tested-by: Gelip Signed-off-by: Michael Brown --- diff --git a/src/drivers/net/realtek.c b/src/drivers/net/realtek.c index 70c0ac1c6..f7b5ec588 100644 --- a/src/drivers/net/realtek.c +++ b/src/drivers/net/realtek.c @@ -74,6 +74,7 @@ static void realtek_spi_open_bit ( struct bit_basher *basher ) { /* Enable EEPROM access */ writeb ( RTL_9346CR_EEM_EEPROM, rtl->regs + RTL_9346CR ); + readb ( rtl->regs + RTL_9346CR ); /* Ensure write reaches chip */ } /** @@ -87,6 +88,7 @@ static void realtek_spi_close_bit ( struct bit_basher *basher ) { /* Disable EEPROM access */ writeb ( RTL_9346CR_EEM_NORMAL, rtl->regs + RTL_9346CR ); + readb ( rtl->regs + RTL_9346CR ); /* Ensure write reaches chip */ } /** @@ -129,6 +131,7 @@ static void realtek_spi_write_bit ( struct bit_basher *basher, reg &= ~mask; reg |= ( data & mask ); writeb ( reg, rtl->regs + RTL_9346CR ); + readb ( rtl->regs + RTL_9346CR ); /* Ensure write reaches chip */ DBG_ENABLE ( DBGLVL_IO ); }