laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-tetra/+/28595 )
Change subject: added missing switch statements to suppress compiler warnings
......................................................................
added missing switch statements to suppress compiler warnings
Change-Id: I02f7b1875e5501f6977638130c8c5b8c8634fe4e
---
M src/phy/tetra_burst.c
M src/tetra_llc.c
2 files changed, 10 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
laforge: Looks good to me, approved
dexter: Looks good to me, but someone else must approve
diff --git a/src/phy/tetra_burst.c b/src/phy/tetra_burst.c
index 5bede06..c791ee0 100644
--- a/src/phy/tetra_burst.c
+++ b/src/phy/tetra_burst.c
@@ -371,5 +371,9 @@
tp_sap_udata_ind(TPSAP_T_BBK, bbk_buf, NDB_BBK_BITS, priv);
tp_sap_udata_ind(TPSAP_T_SCH_F, ndbf_buf, 2*NDB_BLK_BITS, priv);
break;
+ case TETRA_TRAIN_NORM_3:
+ case TETRA_TRAIN_EXT:
+ /* uplink training sequences, should not be encountered, ignore
*/
+ break;
}
}
diff --git a/src/tetra_llc.c b/src/tetra_llc.c
index 893673e..e0d5cab 100644
--- a/src/tetra_llc.c
+++ b/src/tetra_llc.c
@@ -150,6 +150,12 @@
/* check if the fragment is complete and hand it off*/
tllc_defrag_out(&g_llcs, &lpp);
break;
+
+ case TLLC_PDUT_DEC_UNKNOWN:
+ case TLLC_PDUT_DEC_ALX_ACK:
+ case TLLC_PDUT_DEC_ALX_RNR:
+ /* fixme: unhandled types */
+ break;
}
if (lpp.tl_sdu && lpp.ss == 0) {
--
To view, visit https://gerrit.osmocom.org/c/osmo-tetra/+/28595
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-tetra
Gerrit-Branch: master
Gerrit-Change-Id: I02f7b1875e5501f6977638130c8c5b8c8634fe4e
Gerrit-Change-Number: 28595
Gerrit-PatchSet: 1
Gerrit-Owner: wbokslag <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-MessageType: merged
