[S] Change in osmocom-bb[master]: virt_phy: fix l1ctl_rx_rach_req(): send RACH on TS0, not TS1

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/32672 ) Change subject: virt_phy: fix l1ctl_rx_rach_req(): send RACH on TS0, not TS1 ...................................................................... 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(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved 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 https://gerrit.osmocom.org/c/osmocom-bb/+/32672 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I096e377de9e74893e9eb1927707d5e1a44b15ad9 Gerrit-Change-Number: 32672 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-MessageType: merged