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

fake_trx: Send positive response to FAKE_TOA commands

Now that ctrl_if.py is capable of sending back the response to where
the command originated from, we can just as well send a positive
response back after executing the related commands.

Change-Id: Icba138835149a7264f4db3a6b05f54ca501c4d54
---
M src/target/fake_trx/ctrl_if_bb.py
M src/target/fake_trx/ctrl_if_bts.py
2 files changed, 4 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/12/7012/1

diff --git a/src/target/fake_trx/ctrl_if_bb.py 
b/src/target/fake_trx/ctrl_if_bb.py
index 11533d8..15a7eb0 100644
--- a/src/target/fake_trx/ctrl_if_bb.py
+++ b/src/target/fake_trx/ctrl_if_bb.py
@@ -138,8 +138,7 @@
                        self.burst_fwd.toa256_ul_base = int(request[1])
                        self.burst_fwd.toa256_ul_threshold = int(request[2])
 
-                       # TODO: avoid sending response
-                       return -1
+                       return 0
 
                # Timing of Arrival simulation for Uplink
                # Relative form: CMD FAKE_TOA <+-BASE_DELTA>
@@ -149,8 +148,7 @@
                        # Parse and apply delta
                        self.burst_fwd.toa256_ul_base += int(request[1])
 
-                       # TODO: avoid sending response
-                       return -1
+                       return 0
 
                # Wrong / unknown command
                else:
diff --git a/src/target/fake_trx/ctrl_if_bts.py 
b/src/target/fake_trx/ctrl_if_bts.py
index 8c1d8e5..53b9acb 100644
--- a/src/target/fake_trx/ctrl_if_bts.py
+++ b/src/target/fake_trx/ctrl_if_bts.py
@@ -106,8 +106,7 @@
                        self.burst_fwd.toa256_dl_base = int(request[1])
                        self.burst_fwd.toa256_dl_threshold = int(request[2])
 
-                       # TODO: avoid sending response
-                       return -1
+                       return 0
 
                # Timing of Arrival simulation for Downlink
                # Relative form: CMD FAKE_TOA <+-BASE_DELTA>
@@ -117,8 +116,7 @@
                        # Parse and apply delta
                        self.burst_fwd.toa256_dl_base += int(request[1])
 
-                       # TODO: avoid sending response
-                       return -1
+                       return 0
 
                # Wrong / unknown command
                else:

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icba138835149a7264f4db3a6b05f54ca501c4d54
Gerrit-PatchSet: 1
Gerrit-Project: osmocom-bb
Gerrit-Branch: fixeria/trx
Gerrit-Owner: Harald Welte <lafo...@gnumonks.org>

Reply via email to