Hoernchen has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42545?usp=email )


Change subject: firmware: werror on missing return
......................................................................

firmware: werror on missing return

Silently ignoring a missing return just broke all timers,
so prevent similar mistakes in the future.

Change-Id: I2e25884077af6334c9e9ddace3900b04061fae04
---
M sysmoOCTSIM/gcc/Makefile
1 file changed, 4 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-ccid-firmware 
refs/changes/45/42545/1

diff --git a/sysmoOCTSIM/gcc/Makefile b/sysmoOCTSIM/gcc/Makefile
index 461b313..445324e 100644
--- a/sysmoOCTSIM/gcc/Makefile
+++ b/sysmoOCTSIM/gcc/Makefile
@@ -11,7 +11,10 @@

 CFLAGS_CPU=-D__SAME54N19A__ -mcpu=cortex-m4 -mfloat-abi=softfp 
-mfpu=fpv4-sp-d16
 CFLAGS=-x c -mthumb -DDEBUG -Os -ffunction-sections -fdata-sections 
-mlong-calls \
-       -fno-omit-frame-pointer -ggdb3 -Wall -c -std=gnu99 $(CFLAGS_CPU) 
-DOCTSIMFWBUILD -DDISABLE_DFU_DETACH=$(DISABLE_DFU_DETACH) 
-Wno-discarded-qualifiers -Werror=strict-prototypes -fno-common 
-Wno-unused-variable -Wno-unused-function -Wlarger-than=512 -Wstack-usage=255
+       -fno-omit-frame-pointer -ggdb3 -Wall -c -std=gnu99 $(CFLAGS_CPU) 
-DOCTSIMFWBUILD \
+       -DDISABLE_DFU_DETACH=$(DISABLE_DFU_DETACH) -Wno-discarded-qualifiers 
-Werror=strict-prototypes \
+       -fno-common -Wno-unused-variable -Wno-unused-function -Wlarger-than=512 
-Wstack-usage=255 \
+       -Werror=return-type


 CFLAGS += -DLIBOSMOCORE_NO_LOGGING

--
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42545?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: newchange
Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: I2e25884077af6334c9e9ddace3900b04061fae04
Gerrit-Change-Number: 42545
Gerrit-PatchSet: 1
Gerrit-Owner: Hoernchen <[email protected]>

Reply via email to