fixeria has uploaded this change for review. (
https://gerrit.osmocom.org/c/osmocom-bb/+/32917 )
Change subject: trxcon/l1sched: drop Tx prims for not supported TCH modes
......................................................................
trxcon/l1sched: drop Tx prims for not supported TCH modes
Otherwise we'll be attempting to transmit the same prim again and again.
Change-Id: I2f137a0b931e8e7d05a5d903023c50065f57bfa6
Related: OS#4396
---
M src/host/trxcon/src/sched_lchan_tchf.c
M src/host/trxcon/src/sched_lchan_tchh.c
2 files changed, 20 insertions(+), 4 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/17/32917/1
diff --git a/src/host/trxcon/src/sched_lchan_tchf.c
b/src/host/trxcon/src/sched_lchan_tchf.c
index fca2108..1a00a93 100644
--- a/src/host/trxcon/src/sched_lchan_tchf.c
+++ b/src/host/trxcon/src/sched_lchan_tchf.c
@@ -303,8 +303,10 @@
break;
}
default:
- LOGP_LCHAND(lchan, LOGL_ERROR, "Invalid TCH mode: %u\n",
lchan->tch_mode);
- return -EINVAL;
+ LOGP_LCHAND(lchan, LOGL_ERROR,
+ "TCH mode %s is unknown or not supported\n",
+ gsm48_chan_mode_name(lchan->tch_mode));
+ goto free_bad_msg;
}
if (rc) {
diff --git a/src/host/trxcon/src/sched_lchan_tchh.c
b/src/host/trxcon/src/sched_lchan_tchh.c
index e3e4cde..303d59e 100644
--- a/src/host/trxcon/src/sched_lchan_tchh.c
+++ b/src/host/trxcon/src/sched_lchan_tchh.c
@@ -523,8 +523,10 @@
break;
}
default:
- LOGP_LCHAND(lchan, LOGL_ERROR, "Invalid TCH mode: %u\n",
lchan->tch_mode);
- return -EINVAL;
+ LOGP_LCHAND(lchan, LOGL_ERROR,
+ "TCH mode %s is unknown or not supported\n",
+ gsm48_chan_mode_name(lchan->tch_mode));
+ goto free_bad_msg;
}
if (rc) {
--
To view, visit
https://gerrit.osmocom.org/c/osmocom-bb/+/32917
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I2f137a0b931e8e7d05a5d903023c50065f57bfa6
Gerrit-Change-Number: 32917
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-MessageType: newchange