fixeria has uploaded this change for review.

View Change

virt_phy: fix l1ctl_rx_rach_req(): send RACH on TS0, not TS1

Change-Id: I096e377de9e74893e9eb1927707d5e1a44b15ad9
---
M src/host/virt_phy/src/virt_prim_rach.c
1 file changed, 11 insertions(+), 3 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/72/32672/1
diff --git a/src/host/virt_phy/src/virt_prim_rach.c b/src/host/virt_phy/src/virt_prim_rach.c
index 22b8e29..5a3b61a 100644
--- a/src/host/virt_phy/src/virt_prim_rach.c
+++ b/src/host/virt_phy/src/virt_prim_rach.c
@@ -75,7 +75,6 @@
struct l1ctl_info_ul *ul = (struct l1ctl_info_ul *) l1h->data;
struct l1ctl_rach_req *rach_req = (struct l1ctl_rach_req *) ul->payload;
uint32_t fn_sched;
- uint8_t ts = 1; /* FIXME mostly, ts 1 is used for rach, where can i get that info? System info? */
uint16_t offset = ntohs(rach_req->offset);

LOGPMS(DL1C, LOGL_INFO, ms, "Rx L1CTL_RACH_REQ (ra=0x%02x, offset=%d combined=%d)\n",
@@ -90,7 +89,7 @@

/* chan_nr need to be encoded here, as it is not set by l23 for
* the rach request, but needed by virt um */
- ul->chan_nr = rsl_enc_chan_nr(RSL_CHAN_RACH, 0, ts);
+ ul->chan_nr = RSL_CHAN_RACH;
ul->link_id = LID_DEDIC;

/* sched fn calculation if we have a combined ccch channel configuration */
@@ -104,7 +103,7 @@
} else
fn_sched = l1s->current_time.fn + offset;

- virt_l1_sched_schedule(ms, msg, fn_sched, ts, &virt_l1_sched_handler_cb);
+ virt_l1_sched_schedule(ms, msg, fn_sched, 0, &virt_l1_sched_handler_cb);
}

/**

To view, visit change 32672. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I096e377de9e74893e9eb1927707d5e1a44b15ad9
Gerrit-Change-Number: 32672
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-MessageType: newchange