media: atomisp: Deduplicate return ret in gmin_i2c_write()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Fri, 26 Jun 2020 12:19:25 +0000 (14:19 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Sat, 18 Jul 2020 05:22:44 +0000 (07:22 +0200)
Deduplicate return ret in gmin_i2c_write().

While here, replace 'Kernel' by 'kernel' in the message and
reduce amount of LOCs.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/atomisp/pci/atomisp_gmin_platform.c

index 56a2684..997c3ec 100644 (file)
@@ -413,16 +413,12 @@ static int gmin_i2c_write(struct device *dev, u16 i2c_addr, u8 reg,
                "I2C write, addr: 0x%02x, reg: 0x%02x, value: 0x%02x, mask: 0x%02x\n",
                i2c_addr, reg, value, mask);
 
-       ret = intel_soc_pmic_exec_mipi_pmic_seq_element(i2c_addr, reg,
-                                                       value, mask);
-
-       if (ret == -EOPNOTSUPP) {
+       ret = intel_soc_pmic_exec_mipi_pmic_seq_element(i2c_addr, reg, value, mask);
+       if (ret == -EOPNOTSUPP)
                dev_err(dev,
                        "ACPI didn't mapped the OpRegion needed to access I2C address 0x%02x.\n"
-                       "Need to compile the Kernel using CONFIG_*_PMIC_OPREGION settings\n",
+                       "Need to compile the kernel using CONFIG_*_PMIC_OPREGION settings\n",
                        i2c_addr);
-               return ret;
-       }
 
        return ret;
 }