On Tue, Jul 18, 2017 at 7:23 AM, Alastair D'Silva <alast...@d-silva.org> wrote: > I'm not sure this makes sense - we're dealing with time here.
Indeed, thanks for your review! >> --- a/hw/timer/ds1338.c >> +++ b/hw/timer/ds1338.c >> @@ -65,7 +65,7 @@ static void capture_current_time(DS1338State *s) >> s->nvram[1] = to_bcd(now.tm_min); >> if (s->nvram[2] & HOURS_12) { >> int tmp = now.tm_hour; >> - if (tmp % 12 == 0) { >> + if (QEMU_IS_ALIGNED(tmp, 12)) { >> tmp += 12; >> } >> if (tmp <= 12) { >> @@ -154,7 +154,7 @@ static int ds1338_send(I2CSlave *i2c, uint8_t data) >> if (data & HOURS_PM) { >> tmp += 12; >> } >> - if (tmp % 12 == 0) { >> + if (QEMU_IS_ALIGNED(tmp, 12)) { >> tmp -= 12; >> } >> now.tm_hour = tmp;