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/display | |
| parent | 93198b6cad8af03996373584284a1673ad6000cb (diff) | |
| download | qemu-2ac4c5f4d2415116d3f417a32311d437791dcfce.zip qemu-2ac4c5f4d2415116d3f417a32311d437791dcfce.tar.gz qemu-2ac4c5f4d2415116d3f417a32311d437791dcfce.tar.bz2 | |
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/display')
| -rw-r--r-- | hw/display/sii9022.c | 2 | ||||
| -rw-r--r-- | hw/display/ssd0303.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/hw/display/sii9022.c b/hw/display/sii9022.c index eaf11a6..9994385 100644 --- a/hw/display/sii9022.c +++ b/hw/display/sii9022.c @@ -79,7 +79,7 @@ static int sii9022_event(I2CSlave *i2c, enum i2c_event event) return 0; } -static int sii9022_rx(I2CSlave *i2c) +static uint8_t sii9022_rx(I2CSlave *i2c) { sii9022_state *s = SII9022(i2c); uint8_t res = 0x00; diff --git a/hw/display/ssd0303.c b/hw/display/ssd0303.c index eb90ba2..8edf349 100644 --- a/hw/display/ssd0303.c +++ b/hw/display/ssd0303.c @@ -62,10 +62,10 @@ typedef struct { uint8_t framebuffer[132*8]; } ssd0303_state; -static int ssd0303_recv(I2CSlave *i2c) +static uint8_t ssd0303_recv(I2CSlave *i2c) { BADF("Reads not implemented\n"); - return -1; + return 0xff; } static int ssd0303_send(I2CSlave *i2c, uint8_t data) |