Mercurial > public > ostc4
comparison Discovery/Src/logbook.c @ 617:4eba86129d35
Development bugfix: Low battery warning was interpretated as marker:
The previous version checked event bit 1 and 2 and returned a marker in case both were set. The indication for a battery low warning is that the 3 LSBit of the event byte are set => not intended marker in case of low battery situation. To fix this an additional check for bit0 has been added.
| author | Ideenmodellierer |
|---|---|
| date | Wed, 27 Jan 2021 22:03:11 +0100 |
| parents | ae7f8333c900 |
| children | 1b995079c045 |
comparison
equal
deleted
inserted
replaced
| 616:a594a778c883 | 617:4eba86129d35 |
|---|---|
| 737 bytesRead ++; | 737 bytesRead ++; |
| 738 | 738 |
| 739 length--; | 739 length--; |
| 740 | 740 |
| 741 /* marker */ | 741 /* marker */ |
| 742 if(eventByte1.ub.bit1 && eventByte1.ub.bit2 && event != NULL) | 742 if((eventByte1.ub.bit1 && eventByte1.ub.bit2 && !eventByte1.ub.bit0) && (event != NULL)) /* 3 lsb low bit means battery low */ |
| 743 { | 743 { |
| 744 *event = 1; | 744 *event = 1; |
| 745 } | 745 } |
| 746 | 746 |
| 747 //second event byte | 747 //second event byte |