Harald Welte has posted comments on this change. ( https://gerrit.osmocom.org/10214 )
Change subject: cardem: show detailed reset cause ...................................................................... Patch Set 1: (1 comment) https://gerrit.osmocom.org/#/c/10214/1/firmware/apps/cardem/main.c File firmware/apps/cardem/main.c: https://gerrit.osmocom.org/#/c/10214/1/firmware/apps/cardem/main.c@166 PS1, Line 166: static const char* reset_causes[] = { are you sure only one reset cause is reported here? I might be thinking of STM32, but AFAIR multiple causes can exist in parallel (like a flag-mask?) Sorry for not checking SAM3 data sheet myself :/ -- To view, visit https://gerrit.osmocom.org/10214 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: simtrace2 Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2d59c2f2c8fe9e559eddfafacf25879263ef81ff Gerrit-Change-Number: 10214 Gerrit-PatchSet: 1 Gerrit-Owner: Kévin Redon <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: Harald Welte <[email protected]> Gerrit-Comment-Date: Sat, 28 Jul 2018 21:23:01 +0000 Gerrit-HasComments: Yes Gerrit-HasLabels: No
