From a037fe1ac20c14c8addd0a186efda142578c081e Mon Sep 17 00:00:00 2001 From: Caveh Jalali Date: Fri, 11 Jun 2021 18:21:26 -0700 Subject: bb_retimer: Explicitly check EC status codes This replaces inverted boolean logic with explicit comparison with EC_SUCCESS for improved readability. BRANCH=none BUG=b:181743576,b:188826559 TEST=compare_builds.sh matches Signed-off-by: Caveh Jalali Change-Id: Id5aaa167b2e9071e5757f561d788da49af03f9d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2962214 Reviewed-by: Boris Mittelberg Commit-Queue: Boris Mittelberg --- driver/retimer/bb_retimer.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'driver/retimer') diff --git a/driver/retimer/bb_retimer.c b/driver/retimer/bb_retimer.c index 2d23827f91..2f98b4b185 100644 --- a/driver/retimer/bb_retimer.c +++ b/driver/retimer/bb_retimer.c @@ -62,7 +62,7 @@ static int bb_retimer_read(const struct usb_mux *me, rv = i2c_xfer(me->i2c_port, me->i2c_addr_flags, &offset, 1, buf, BB_RETIMER_READ_SIZE); - if (!rv) + if (rv == EC_SUCCESS) break; if (++retry >= BB_RETIMER_I2C_RETRY) { @@ -110,7 +110,7 @@ static int bb_retimer_write(const struct usb_mux *me, rv = i2c_xfer(me->i2c_port, me->i2c_addr_flags, buf, BB_RETIMER_WRITE_SIZE, NULL, 0); - if (!rv) + if (rv == EC_SUCCESS) break; if (++retry >= BB_RETIMER_I2C_RETRY) { @@ -496,14 +496,14 @@ static int retimer_init(const struct usb_mux *me) return rv; rv = bb_retimer_read(me, BB_RETIMER_REG_VENDOR_ID, &data); - if (rv) + if (rv != EC_SUCCESS) return rv; if ((data != BB_RETIMER_VENDOR_ID_1) && data != BB_RETIMER_VENDOR_ID_2) return EC_ERROR_INVAL; rv = bb_retimer_read(me, BB_RETIMER_REG_DEVICE_ID, &data); - if (rv) + if (rv != EC_SUCCESS) return rv; if (data != BB_RETIMER_DEVICE_ID) return EC_ERROR_INVAL; -- cgit v1.2.1