diff options
author | Corey Minyard <cminyard@mvista.com> | 2018-11-14 11:50:50 -0600 |
---|---|---|
committer | Corey Minyard <cminyard@mvista.com> | 2019-02-27 21:06:08 -0600 |
commit | 2ac4c5f4d2415116d3f417a32311d437791dcfce (patch) | |
tree | 5abf8e22939c5c1c030643ffe2fcdfe7bb16344c /hw/timer/twl92230.c | |
parent | 93198b6cad8af03996373584284a1673ad6000cb (diff) |
i2c: have I2C receive operation return uint8_t
It is never supposed to fail and cannot return an error, so just
have it return the proper type. Have it return 0xff on nothing
available, since that's what would happen on a real bus.
Signed-off-by: Corey Minyard <cminyard@mvista.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Diffstat (limited to 'hw/timer/twl92230.c')
-rw-r--r-- | hw/timer/twl92230.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/timer/twl92230.c b/hw/timer/twl92230.c index 51ec355f3f..c83d803dd8 100644 --- a/hw/timer/twl92230.c +++ b/hw/timer/twl92230.c @@ -737,7 +737,7 @@ static int menelaus_tx(I2CSlave *i2c, uint8_t data) return 0; } -static int menelaus_rx(I2CSlave *i2c) +static uint8_t menelaus_rx(I2CSlave *i2c) { MenelausState *s = TWL92230(i2c); |