+-- On Fri, 26 Oct 2018, Peter Maydell wrote --+ | > === | > diff --git a/hw/arm/strongarm.c b/hw/arm/strongarm.c | > index ec2627374d..dd8c4b1f2e 100644 | > --- a/hw/arm/strongarm.c | > +++ b/hw/arm/strongarm.c | > @@ -587,12 +587,12 @@ static void strongarm_gpio_write(void *opaque, hwaddr | > offset, | > | > switch (offset) { | > case GPDR: /* GPIO Pin-Direction registers */ | > - s->dir = value; | > + s->dir = value & 0x3fffff; | > strongarm_gpio_handler_update(s); | > break; | > | > case GPSR: /* GPIO Pin-Output Set registers */ | > - s->olevel |= value; | > + s->olevel |= value & 0x3fffff; | > strongarm_gpio_handler_update(s); | > break; | > === | > | > does this seem okay? | | Yes, that's what I had in mind.
Cool, patch sent. Thank you. -- Prasad J Pandit / Red Hat Product Security Team 47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F