[PATCH] osmo-bts[master]: l1_tch: remove dead code

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

dexter gerrit-no-reply at lists.osmocom.org
Thu Mar 1 15:08:05 UTC 2018


Review at  https://gerrit.osmocom.org/7029

l1_tch: remove dead code

the file l1_tch.c in the octphy spcific code has its root in the
tch.c file of sysmobts. It contains some sysmobts specific ifdefs
which enclose dead code.

- Remove the sysmobts specif ifdefs and the dead code

Change-Id: I33f3e0fd293ca8387fddf5ed20f642621e9680de
---
M src/osmo-bts-octphy/l1_tch.c
1 file changed, 0 insertions(+), 64 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/29/7029/1

diff --git a/src/osmo-bts-octphy/l1_tch.c b/src/osmo-bts-octphy/l1_tch.c
index eee24aa..df0469d 100644
--- a/src/osmo-bts-octphy/l1_tch.c
+++ b/src/osmo-bts-octphy/l1_tch.c
@@ -47,11 +47,6 @@
 	if (!msg)
 		return NULL;
 
-#ifdef USE_L1_RTP_MODE
-	/* new L1 can deliver bits like we need them */
-	cur = msgb_put(msg, GSM_FR_BYTES);
-	memcpy(cur, l1_payload, GSM_FR_BYTES);
-#else
 	/* step1: reverse the bit-order of each payload byte */
 	osmo_revbytebits_buf(l1_payload, payload_len);
 
@@ -61,7 +56,6 @@
 	osmo_nibble_shift_right(cur, l1_payload, GSM_FR_BITS / 4);
 
 	cur[0] |= 0xD0;
-#endif /* USE_L1_RTP_MODE */
 
 	return msg;
 }
@@ -75,56 +69,12 @@
 int rtppayload_to_l1_fr(uint8_t *l1_payload, const uint8_t *rtp_payload,
 			unsigned int payload_len)
 {
-#ifdef USE_L1_RTP_MODE
-	/* new L1 can deliver bits like we need them */
-	memcpy(l1_payload, rtp_payload, GSM_FR_BYTES);
-#else
 	/* step2: we need to shift the RTP payload left by one nibble */
 	osmo_nibble_shift_left_unal(l1_payload, rtp_payload, GSM_FR_BITS / 4);
 
 	/* step1: reverse the bit-order of each payload byte */
 	osmo_revbytebits_buf(l1_payload, payload_len);
-#endif /* USE_L1_RTP_MODE */
 	return GSM_FR_BYTES;
-}
-
-static struct msgb *l1_to_rtppayload_efr(uint8_t *l1_payload, uint8_t payload_len)
-{
-	struct msgb *msg;
-	uint8_t *cur;
-
-	msg = msgb_alloc_headroom(1024, 128, "L1P-to-RTP");
-	if (!msg)
-		return NULL;
-
-#ifdef USE_L1_RTP_MODE
-	/* new L1 can deliver bits like we need them */
-	cur = msgb_put(msg, GSM_EFR_BYTES);
-	memcpy(cur, l1_payload, GSM_EFR_BYTES);
-#else
-	/* step1: reverse the bit-order of each payload byte */
-	osmo_revbytebits_buf(l1_payload, payload_len);
-
-	cur = msgb_put(msg, GSM_EFR_BYTES);
-
-	/* step 2: we need to shift the entire L1 payload by 4 bits right */
-	osmo_nibble_shift_right(cur, l1_payload, GSM_EFR_BITS/4);
-
-	cur[0] |= 0xC0;
-#endif /* USE_L1_RTP_MODE */
-	return msg;
-}
-
-static int rtppayload_to_l1_efr(uint8_t *l1_payload, const uint8_t *rtp_payload,
-				unsigned int payload_len)
-{
-#ifndef USE_L1_RTP_MODE
-#warning "We don't support EFR with L1 that doesn't support RTP mode!"
-#else
-	memcpy(l1_payload, rtp_payload, payload_len);
-
-#endif
-	return payload_len;
 }
 
 static struct msgb *l1_to_rtppayload_hr(uint8_t *l1_payload, uint8_t payload_len)
@@ -145,10 +95,8 @@
 	cur = msgb_put(msg, GSM_HR_BYTES);
 	memcpy(cur, l1_payload, GSM_HR_BYTES);
 
-#ifndef USE_L1_RTP_MODE
 	/* reverse the bit-order of each payload byte */
 	osmo_revbytebits_buf(cur, GSM_HR_BYTES);
-#endif /* USE_L1_RTP_MODE */
 
 	return msg;
 }
@@ -171,10 +119,8 @@
 
 	memcpy(l1_payload, rtp_payload, GSM_HR_BYTES);
 
-#ifndef USE_L1_RTP_MODE
 	/* reverse the bit-order of each payload byte */
 	osmo_revbytebits_buf(l1_payload, GSM_HR_BYTES);
-#endif /* USE_L1_RTP_MODE */
 
 	return GSM_HR_BYTES;
 }
@@ -315,18 +261,8 @@
 		break;
 	case GSM48_CMODE_SPEECH_EFR:
 		/* Not supported currently */
-#if 0
-		*payload_type = cOCTVC1_GSM_PAYLOAD_TYPE_ENUM_EFR;
-		rc = rtppayload_to_l1_efr(l1_payload, rtp_pl, rtp_pl_len);
-		break;
-#endif
 	case GSM48_CMODE_SPEECH_AMR:
 		/* Not supported currently */
-#if 0
-		*payload_type = cOCTVC1_GSM_PAYLOAD_TYPE_ENUM_AMR;
-		rc = rtppayload_to_l1_amr(l1_payload, rtp_pl, rtp_pl_len);
-		break;
-#endif
 		LOGP(DRTP, LOGL_ERROR, "OctPHY only supports FR!\n");
 	default:
 		/* we don't support CSD modes */

-- 
To view, visit https://gerrit.osmocom.org/7029
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I33f3e0fd293ca8387fddf5ed20f642621e9680de
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>



More information about the gerrit-log mailing list