Change in osmo-gsm-tester[master]: gnuradio_zmq_broker_remote: fix eNB port mapping for carrier aggregation

[pespin]

pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21409 )

Change subject: gnuradio_zmq_broker_remote: fix eNB port mapping for carrier aggregation
......................................................................

gnuradio_zmq_broker_remote: fix eNB port mapping for carrier aggregation

the mapping of ports didn't consider when a eNB had two ports for
different EARFCNs

Change-Id: I5548f67a3b37b03482e4b6172828fcd891e704a5
---
M src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  pespin: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py b/src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py
index fa69efd..98942f3 100755
--- a/src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py
+++ b/src/osmo_gsm_tester/external/gnuradio_zmq_broker_remote.py
@@ -22,7 +22,7 @@
 class GrBroker(gr.top_block):
 
     def __init__(self, args, cfg):
-        gr.top_block.__init__(self, "Intra Handover Flowgraph")
+        gr.top_block.__init__(self, "InterENB Handover Flowgraph")
 
         ##################################################
         # Variables
@@ -83,10 +83,10 @@
         idx = 0
         for enb in self.cfg['enb']:
             for it in enb:
-                print('Connecting ENB port %u to Adder[%u] for earfcn %u' % (it['bind_port'], blocks_add_next_avail_port[earfcn], it['earfcn']))
-                self.connect((it['gr_block_multiply'], 0), (self.blocks_add[it['earfcn']], blocks_add_next_avail_port[earfcn]))
+                print('Connecting ENB port %u to Adder[%u] for earfcn %u' % (it['bind_port'], blocks_add_next_avail_port[it['earfcn']], it['earfcn']))
+                self.connect((it['gr_block_multiply'], 0), (self.blocks_add[it['earfcn']], blocks_add_next_avail_port[it['earfcn']]))
                 # TODO: if it['use_mimo'], connect it['gr_block_multiply2'] to some adder...
-                blocks_add_next_avail_port[earfcn] += 1
+                blocks_add_next_avail_port[it['earfcn']] += 1
 
         # Connect the Adder to the UE-side (Dl):
         for earfcn, bl_add in self.blocks_add.items():

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-tester/+/21409
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Change-Id: I5548f67a3b37b03482e4b6172828fcd891e704a5
Gerrit-Change-Number: 21409
Gerrit-PatchSet: 1
Gerrit-Owner: srs_andre <andre at softwareradiosystems.com>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201130/442ceb76/attachment.htm>