lynxis lazus has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42160?usp=email ) Change subject: ccid_common: iso_handle_fsm_events: add missing return ...................................................................... ccid_common: iso_handle_fsm_events: add missing return Fix warning about control reaches end of non-void function. Change-Id: I4066c8a1ab1eadb3d8f8cbaf5aa0afbbb2e5dd80 --- M ccid_common/ccid_slot_fsm.c 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve lynxis lazus: Looks good to me, approved diff --git a/ccid_common/ccid_slot_fsm.c b/ccid_common/ccid_slot_fsm.c index 0c56275..c34ad72 100644 --- a/ccid_common/ccid_slot_fsm.c +++ b/ccid_common/ccid_slot_fsm.c @@ -199,7 +199,8 @@ } /* do not free msgbs passed from the fsms, they are statically allocated! */ -static int iso_handle_fsm_events(struct ccid_slot *cs, bool enable){ +static int iso_handle_fsm_events(struct ccid_slot *cs, bool enable) +{ struct iso_fsm_slot *ss = ccid_slot2iso_fsm_slot(cs); struct msgb *tpdu, *resp; volatile uint32_t event = cs->event; @@ -338,6 +339,8 @@ __func__, event, data); break; } + + return 0; } static int iso_fsm_slot_xfr_block_async(struct ccid_slot *cs, struct msgb *msg, -- To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42160?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-ccid-firmware Gerrit-Branch: master Gerrit-Change-Id: I4066c8a1ab1eadb3d8f8cbaf5aa0afbbb2e5dd80 Gerrit-Change-Number: 42160 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <lynxis@fe80.eu> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu> Gerrit-Reviewer: pespin <pespin@sysmocom.de>