evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

staging: et131x: Remove unnecessary i2c_wack variable

Browse source 
i2c_wack is only used to implement a while(1) loop, so let's remove it.

Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Mark Einon 2014-08-21 16:02:32 +01:00 • committed by Greg Kroah-Hartman
parent becce4a47e
commit 5a5835ea5f
1 changed files with 1 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
drivers/staging/et131x/et131x.c
View file

@ -573,7 +573,6 @@ static int eeprom_write(struct et131x_adapter *adapter, u32 addr, u8 data)
int index = 0; int index = 0;
int retries; int retries;
int err = 0; int err = 0;
int i2c_wack = 0;
int writeok = 0; int writeok = 0;
u32 status; u32 status;
u32 val = 0; u32 val = 0;
@ -599,8 +598,6 @@ static int eeprom_write(struct et131x_adapter *adapter, u32 addr, u8 data)
LBCIF_CONTROL_LBCIF_ENABLE | LBCIF_CONTROL_I2C_WRITE)) LBCIF_CONTROL_LBCIF_ENABLE | LBCIF_CONTROL_I2C_WRITE))
return -EIO; return -EIO;
i2c_wack = 1;
/* Prepare EEPROM address for Step 3 */ /* Prepare EEPROM address for Step 3 */
for (retries = 0; retries < MAX_NUM_WRITE_RETRIES; retries++) { for (retries = 0; retries < MAX_NUM_WRITE_RETRIES; retries++) {
@ -656,7 +653,7 @@ static int eeprom_write(struct et131x_adapter *adapter, u32 addr, u8 data)
*/ */
udelay(10); udelay(10);
while (i2c_wack) { while (1) {
if (pci_write_config_byte(pdev, LBCIF_CONTROL_REGISTER, if (pci_write_config_byte(pdev, LBCIF_CONTROL_REGISTER,
LBCIF_CONTROL_LBCIF_ENABLE)) LBCIF_CONTROL_LBCIF_ENABLE))
writeok = 0; writeok = 0;