From: Corey Minyard <cminy...@mvista.com> It can't fail, and now that it returns a uint8_t a 0xff mask is unnecessary.
Signed-off-by: Corey Minyard <cminy...@mvista.com> Suggested-by: Peter Maydell <peter.mayd...@linaro.org> --- hw/arm/stellaris.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/arm/stellaris.c b/hw/arm/stellaris.c index 6c69ce79b2..638b649911 100644 --- a/hw/arm/stellaris.c +++ b/hw/arm/stellaris.c @@ -811,7 +811,7 @@ static void stellaris_i2c_write(void *opaque, hwaddr offset, /* TODO: Handle errors. */ if (s->msa & 1) { /* Recv */ - s->mdr = i2c_recv(s->bus) & 0xff; + s->mdr = i2c_recv(s->bus); } else { /* Send */ i2c_send(s->bus, s->mdr); -- 2.17.1