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

Change subject: trxcon: allow advancing FBSB timeout (quirks for slow PHYs) ......................................................................

trxcon: allow advancing FBSB timeout (quirks for slow PHYs)

This is needed for SDR based PHYs, because for them it takes longer to tune, flush the buffers and so on. Add a field to the trxcon_inst structure and a command line option (-F) for the trxcon app.

Change-Id: Ia68954c5bdacda45fc871ffea0ccdf2460936408 Related: OS#5599 --- M src/host/trxcon/include/osmocom/bb/trxcon/trxcon.h M src/host/trxcon/src/trxcon_fsm.c M src/host/trxcon/src/trxcon_inst.c M src/host/trxcon/src/trxcon_main.c 4 files changed, 27 insertions(+), 3 deletions(-)

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

diff --git a/src/host/trxcon/include/osmocom/bb/trxcon/trxcon.h b/src/host/trxcon/include/osmocom/bb/trxcon/trxcon.h index 9148771..10527e3 100644 --- a/src/host/trxcon/include/osmocom/bb/trxcon/trxcon.h +++ b/src/host/trxcon/include/osmocom/bb/trxcon/trxcon.h @@ -32,6 +32,12 @@ uint8_t tx_power; int8_t ta; } l1p; + + /* PHY specific quirks */ + struct { + /* FBSB timeout advance (in TDMA FNs) */ + unsigned int fbsb_advance_fns; + } phy_quirks; };

enum trxcon_log_cat { diff --git a/src/host/trxcon/src/trxcon_fsm.c b/src/host/trxcon/src/trxcon_fsm.c index 7c0057a..ea7471b 100644 --- a/src/host/trxcon/src/trxcon_fsm.c +++ b/src/host/trxcon/src/trxcon_fsm.c @@ -159,9 +159,11 @@ case TRXCON_EV_FBSB_SEARCH_REQ: { const struct trxcon_param_fbsb_search_req *req = data; - unsigned long timeout_ms; + unsigned long timeout_fns, timeout_ms;

- timeout_ms = req->timeout_fns * GSM_TDMA_FN_DURATION_uS / 1000; + /* Some PHYs need additional time to tune (in TDMA FNs) */ + timeout_fns = req->timeout_fns + trxcon->phy_quirks.fbsb_advance_fns; + timeout_ms = timeout_fns * GSM_TDMA_FN_DURATION_uS / 1000; osmo_fsm_inst_state_chg_ms(fi, TRXCON_ST_FBSB_SEARCH, timeout_ms, 0);

l1sched_configure_ts(trxcon->sched, 0, req->pchan_config); diff --git a/src/host/trxcon/src/trxcon_inst.c b/src/host/trxcon/src/trxcon_inst.c index 7d3813e..e967cd8 100644 --- a/src/host/trxcon/src/trxcon_inst.c +++ b/src/host/trxcon/src/trxcon_inst.c @@ -91,6 +91,8 @@ return NULL; }

+ trxcon->phy_quirks.fbsb_advance_fns = 0; + return trxcon; }

diff --git a/src/host/trxcon/src/trxcon_main.c b/src/host/trxcon/src/trxcon_main.c index 95e5549..0f9e616 100644 --- a/src/host/trxcon/src/trxcon_main.c +++ b/src/host/trxcon/src/trxcon_main.c @@ -67,6 +67,9 @@ uint16_t trx_base_port; uint32_t trx_fn_advance;

+ /* PHY quirk: FBSB timeout advance (in TDMA FNs) */ + int phyq_fbsb_advance_fns; + /* GSMTAP specific */ struct gsmtap_inst *gsmtap; const char *gsmtap_ip; @@ -77,6 +80,7 @@ .trx_bind_ip = "0.0.0.0", .trx_base_port = 6700, .trx_fn_advance = 3, + .phyq_fbsb_advance_fns = 0, };

static void *tall_trxcon_ctx = NULL; @@ -153,6 +157,7 @@ }

trxcon->gsmtap = app_data.gsmtap; + trxcon->phy_quirks.fbsb_advance_fns = app_data.phyq_fbsb_advance_fns; }

static void l1ctl_conn_close_cb(struct l1ctl_client *l1c) @@ -179,6 +184,7 @@ printf(" -i --trx-remote TRX remote IP address (default 127.0.0.1)\n"); printf(" -p --trx-port Base port of TRX instance (default 6700)\n"); printf(" -f --trx-advance Uplink burst scheduling advance (default 3)\n"); + printf(" -F --fbsb-advance FBSB timeout advance (in TDMA FNs, default 0)\n"); printf(" -s --socket Listening socket for layer23 (default /tmp/osmocom_l2)\n"); printf(" -g --gsmtap-ip The destination IP used for GSMTAP (disabled by default)\n"); printf(" -C --max-clients Maximum number of L1CTL connections (default 1)\n"); @@ -200,13 +206,14 @@ {"trx-remote", 1, 0, 'i'}, {"trx-port", 1, 0, 'p'}, {"trx-advance", 1, 0, 'f'}, + {"fbsb-advance", 1, 0, 'F'}, {"gsmtap-ip", 1, 0, 'g'}, {"max-clients", 1, 0, 'C'}, {"daemonize", 0, 0, 'D'}, {0, 0, 0, 0} };

- c = getopt_long(argc, argv, "d:b:i:p:f:s:g:C:Dh", + c = getopt_long(argc, argv, "d:b:i:p:f:F:s:g:C:Dh", long_options, &option_index); if (c == -1) break; @@ -232,6 +239,13 @@ case 'f': app_data.trx_fn_advance = atoi(optarg); break; + case 'F': + app_data.phyq_fbsb_advance_fns = atoi(optarg); + if (app_data.phyq_fbsb_advance_fns < 0) { + fprintf(stderr, "Error: FBSB advance shall be a positive value\n"); + exit(1); + } + break; case 's': app_data.bind_socket = optarg; break;