On 26/11/18 21:04, [email protected] wrote: > From: Corey Minyard <[email protected]> > > It can't fail, and now that it returns a uint8_t a 0xff mask > is unnecessary. > > Signed-off-by: Corey Minyard <[email protected]> > Suggested-by: Peter Maydell <[email protected]> > --- > 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); >
This could be squashed in the previous patch. Reviewed-by: Philippe Mathieu-Daudé <[email protected]> Tested-by: Philippe Mathieu-Daudé <[email protected]>
