i2c: nforce2: Do PCI error check on own line

Instead of a if condition with a line split, use the usual error
handling pattern with a separate variable to improve readability.

No functional changes intended.

Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Reviewed-by: Andi Shyti <andi.shyti@kernel.org>
Signed-off-by: Wolfram Sang <wsa@kernel.org>

diff --git a/drivers/i2c/busses/i2c-nforce2.c b/drivers/i2c/busses/i2c-nforce2.c
index 777278386f58eb1904a042c3f896ce1e7a6ba32b..38d203d93eeec4bde521ddb034a1e4bd1fc88da0 100644 (file)
--- a/drivers/i2c/busses/i2c-nforce2.c
+++ b/drivers/i2c/busses/i2c-nforce2.c
@@ -327,8 +327,8 @@ static int nforce2_probe_smb(struct pci_dev *dev, int bar, int alt_reg,
                /* Older incarnations of the device used non-standard BARs */
                u16 iobase;
 
-               if (pci_read_config_word(dev, alt_reg, &iobase)
-                   != PCIBIOS_SUCCESSFUL) {
+               error = pci_read_config_word(dev, alt_reg, &iobase);
+               if (error != PCIBIOS_SUCCESSFUL) {
                        dev_err(&dev->dev, "Error reading PCI config for %s\n",
                                name);
                        return -EIO;