Change in osmocom-bb[master]: trxcon/trx_if: send NOPE indications if there is no burst

fixeria gerrit-no-reply at lists.osmocom.org
Mon Jun 14 21:56:32 UTC 2021


fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/24662 )


Change subject: trxcon/trx_if: send NOPE indications if there is no burst
......................................................................

trxcon/trx_if: send NOPE indications if there is no burst

In a typical setup operating on the real radio interface, it's
the duty of the transceiver (e.g. osmo-trx) to send NOPE.ind to
the L1 implementation (e.g. osmo-bts-trx).  However, in a
virtual environment for ttcn3-bts-test we use a fake transceiver,
which due to its simplicity cannot send NOPE indications itself.

The lack of queues and buffering does not allow us to implement
NOPE indications in fake_trx.py, so the easiest approach is to
generate them from trxcon.  Send TRXD PDUs without the burst bits,
and fake_trx.py will tranform them info NOPE.ind for us.

Change-Id: I1c7f1315b8ef44f651efd6a22fb5b854f65c0946
Related: SYS#5313, OS#1569
---
M src/host/trxcon/trx_if.c
1 file changed, 4 insertions(+), 5 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/62/24662/1

diff --git a/src/host/trxcon/trx_if.c b/src/host/trxcon/trx_if.c
index c4561ce..8dbbd12 100644
--- a/src/host/trxcon/trx_if.c
+++ b/src/host/trxcon/trx_if.c
@@ -635,9 +635,6 @@
 	uint8_t buf[TRXD_BUF_SIZE];
 	size_t length;
 
-	if (br->burst_len == 0)
-		return 0;
-
 	/**
 	 * We must be sure that we have clock,
 	 * and we have sent all control data
@@ -662,8 +659,10 @@
 	length = 6;
 
 	/* Copy ubits {0,1} */
-	memcpy(buf + 6, br->burst, br->burst_len);
-	length += br->burst_len;
+	if (br->burst_len != 0) {
+		memcpy(buf + 6, br->burst, br->burst_len);
+		length += br->burst_len;
+	}
 
 	/* Send data to transceiver */
 	send(trx->trx_ofd_data.fd, buf, length, 0);

-- 
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/24662
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I1c7f1315b8ef44f651efd6a22fb5b854f65c0946
Gerrit-Change-Number: 24662
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210614/affe7ae5/attachment.htm>


More information about the gerrit-log mailing list