diff options
| author |   Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2015-02-10 19:06:07 +0200 |
|---|---|---|
| committer |   Wolfram Sang <wsa@the-dreams.de> | 2015-03-07 01:07:27 +0100 |
| commit | 259aada436e13ec75a8b0f252a78e6577879008e (patch) | |
| tree | 388a83a11aacfaeaef7f88ae214b12059f7f313a | |
| parent | i2c: designware-baytrail: describe magic numbers (diff) | |
| download | linux-dev-259aada436e13ec75a8b0f252a78e6577879008e.tar.xz linux-dev-259aada436e13ec75a8b0f252a78e6577879008e.zip | |
i2c: designware-baytrail: fix typo in error path
It seems we have same message for different return values in get_sem() and
baytrail_i2c_acquire(). I suspect this is just a typo, so this patch fixes it.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: David E. Box <david.e.box@linux.intel.com>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
| -rw-r--r-- | drivers/i2c/busses/i2c-designware-baytrail.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/i2c/busses/i2c-designware-baytrail.c b/drivers/i2c/busses/i2c-designware-baytrail.c index e9cb3555dc79..9b6765554c70 100644 --- a/drivers/i2c/busses/i2c-designware-baytrail.c +++ b/drivers/i2c/busses/i2c-designware-baytrail.c @@ -99,8 +99,8 @@ int baytrail_i2c_acquire(struct dw_i2c_dev *dev) reset_semaphore(dev->dev); ret = iosf_mbi_read(BT_MBI_UNIT_PMC, BT_MBI_BUNIT_READ, - PUNIT_SEMAPHORE, &sem); - if (!ret) + PUNIT_SEMAPHORE, &sem); + if (ret) dev_err(dev->dev, "iosf failed to read punit semaphore\n"); else dev_err(dev->dev, "PUNIT SEM: %d\n", sem); |