[MERGED] libosmocore[master]: l1sap: Add fields for higher-precision timing offset values

Harald Welte gerrit-no-reply at lists.osmocom.org
Tue Feb 27 19:32:40 UTC 2018


Harald Welte has submitted this change and it was merged.

Change subject: l1sap: Add fields for higher-precision timing offset values
......................................................................


l1sap: Add fields for higher-precision timing offset values

So far, we used quarter-bits across the L1SAP between the hardware/PHY
specific part of OsmoBTS and the common part.  In order to increase
the resolution, let's add fields/members for 1/256th bit.

In order to keep ABI and API compatibility, we use a union around the
old and new values, so old code will still compile + work withe new
libosmocore.

Change-Id: Ibb58113c2819fe2d6d23ecbcfb8b3fce4055025d
---
M include/osmocom/gsm/l1sap.h
1 file changed, 8 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/include/osmocom/gsm/l1sap.h b/include/osmocom/gsm/l1sap.h
index 18ad7d2..9a9c008 100644
--- a/include/osmocom/gsm/l1sap.h
+++ b/include/osmocom/gsm/l1sap.h
@@ -77,7 +77,10 @@
 	uint32_t fn;		/*!< GSM Frame Number */
 	int8_t rssi;		/*!< RSSI of receivedindication */
 	uint16_t ber10k;	/*!< BER in units of 0.01% */
-	int16_t ta_offs_qbits;	/* !< Burst TA Offset in quarter bits */
+	union {
+		int16_t ta_offs_qbits;	/* !< Burst TA Offset in quarter bits */
+		int16_t ta_offs_256bits;/*!< timing advance offset (in 1/256th bits) */
+	};
 	int16_t lqual_cb;	/* !< Link quality in centiBel */
 	enum osmo_ph_pres_info_type pdch_presence_info; /*!< Info regarding presence/validity of header and data parts */
 };
@@ -107,7 +110,10 @@
 	uint8_t chan_nr;	/*!< Channel Number (Like RSL) */
 	uint32_t fn;		/*!< GSM Frame Number */
 	uint16_t ber10k;	/*!< BER in units of 0.01% */
-	int16_t ta_offs_qbits;	/*!< timing advance offset (in qbits) */
+	union {
+		int16_t ta_offs_qbits;	/*!< timing advance offset (in qbits) */
+		int16_t ta_offs_256bits;/*!< timing advance offset (in 1/256th bits) */
+	};
 	int16_t c_i_cb;		/*!< C/I ratio in 0.1 dB */
 	uint8_t is_sub:1;	/*!< flags */
 	uint8_t inv_rssi;	/*!< RSSI in dBm * -1 */

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibb58113c2819fe2d6d23ecbcfb8b3fce4055025d
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder


More information about the gerrit-log mailing list