diff options
| author | Stefan Weil <sw@weilnetz.de> | 2012-02-25 14:50:25 +0100 |
|---|---|---|
| committer | Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> | 2012-03-19 10:52:52 +0000 |
| commit | fbac6a7d35d119a52606c175aface9bcec805f09 (patch) | |
| tree | a034d3c8a2355abfbc62b57f959a2241c681e330 | |
| parent | ee032ca146e932ce616aff7762596a10bf24ecac (diff) | |
ds1338: Add missing break statement
Without the break statement, case 5 sets month and year from the same
data. This does not look correct.
The missing break was reported by splint.
Signed-off-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Andreas Färber <afaerber@suse.de>
Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
| -rw-r--r-- | hw/ds1338.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/ds1338.c b/hw/ds1338.c index 6397f0aa6f..d590d9c007 100644 --- a/hw/ds1338.c +++ b/hw/ds1338.c @@ -100,6 +100,7 @@ static int ds1338_send(I2CSlave *i2c, uint8_t data) break; case 5: s->now.tm_mon = from_bcd(data & 0x1f) - 1; + break; case 6: s->now.tm_year = from_bcd(data) + 100; break; |