fixeria has submitted this change. (
https://gerrit.osmocom.org/c/osmo-bts/+/33076 )
(
6 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted
one.
)Change subject: osmo-bts-trx: visualize rx_tch[fh]_fn() functions
......................................................................
osmo-bts-trx: visualize rx_tch[fh]_fn() functions
Change-Id: I373dbbc3d427858f76d07ff85633e07fe2600770
Related: OS#1572
---
A doc/trx_sched_tch.txt
M src/osmo-bts-trx/sched_lchan_tchf.c
M src/osmo-bts-trx/sched_lchan_tchh.c
3 files changed, 112 insertions(+), 2 deletions(-)
Approvals:
pespin: Looks good to me, but someone else must approve
osmith: Looks good to me, but someone else must approve
laforge: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/doc/trx_sched_tch.txt b/doc/trx_sched_tch.txt
new file mode 100644
index 0000000..f8c79ff
--- /dev/null
+++ b/doc/trx_sched_tch.txt
@@ -0,0 +1,98 @@
+== rx_tchf_fn(): TCH/FS, TCH/EFS, TCH/AFS, TCH/F2.4, and FACCH/F
+
+ 00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18
19 20 21 22 23
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 4
+| | | | | | | | | | | | | | | | | | | |
| a | b | c | d | Rx bid={0,1,2,3}, decode
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 4
+| | | | | | | | | | | | | | | | | a | b | c |
d | e | f | g | h | Rx bid={0,1,2,3}, decode
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 4
+ |
+
|<~~~~~~~~~~~~~~~~~~~~~~~~~~~~~>| frame A
+ |
|<~~~~~~~~~~~~~~~~~~~~~~~~~~~~~>| frame B
+ @ decoding
from here
+
+
+== rx_tchf_fn(): TCH/F14.4, TCH/F9.6, TCH/F4.8
+
+ 00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18
19 20 21 22 23
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 4
+| | | | | | | | | | | | | | | | | | | |
| a | b | c | d | Rx bid={0,1,2,3}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 4
+| | | | | | | | | | | | | | | | | a | b | c |
d | e | f | g | h | Rx bid={0,1,2,3}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 4
+| | | | | | | | | | | | | a | b | c | d | e | f | g |
h | i | j | k | l | Rx bid={0,1,2,3}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 4
+| | | | | | | | | a | b | c | d | e | f | g | h | i | j | k |
l | m | n | o | p | Rx bid={0,1,2,3}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 4
+| | | | | a | b | c | d | e | f | g | h | i | j | k | l | m | n | o |
p | q | r | s | t | Rx bid={0,1,2,3}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 4
+| a | b | c | d | e | f | g | h | i | j | k | l | m | n | o | p | q | r | s |
t | u | v | w | x | Rx bid={0,1,2,3}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 4
+|
+|<~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~>|
frame A
+|
|<~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~>|
frame B
+@ decoding from here
+
+
+== rx_tchh_fn(): TCH/HS, TCH/AHS
+
+ 00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18
19 20 21 22 23
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | | | |
| a | b | | | Rx bid={0,1}, decode
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | | | a |
b | c | d | | | Rx bid={0,1}, decode
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | a | b | c |
d | e | f | | | Rx bid={0,1}, decode
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 2
+ |
+
|<~~~~~~~~~~~~~>| frame A
+ |
|<~~~~~~~~~~~~~>| frame B
+ @ decoding
from here
+
+
+== rx_tchh_fn(): FACCH/H
+
+ 00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18
19 20 21 22 23
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | | | |
| a | b | | | Rx bid={0,1}, decode
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | | | a |
b | c | d | | | Rx bid={0,1}
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | a | b | c |
d | e | f | | | Rx bid={0,1}, decode
++---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---|---+---+---+---+---+---+---+---+
<< 2
+ |
+
|<~~~~~~~~~~~~~~~~~~~~~>| frame A
+ |
|<~~~~~~~~~~~~~~~~~~~~~>| frame B
+ @ decoding
from here
+
+
+== rx_tchh_fn(): TCH/H4.8, TCH/H2.4
+
+ 00 01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18
19 20 21 22 23
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | | | |
| a | b | | | Rx bid={0,1}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | | | a |
b | c | d | | | Rx bid={0,1}
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | | | a | b | c |
d | e | f | | | Rx bid={0,1}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | | | a | b | c | d | e |
f | g | h | | | Rx bid={0,1}
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | | | a | b | c | d | e | f | g |
h | i | j | | | Rx bid={0,1}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | | | a | b | c | d | e | f | g | h | i |
j | k | l | | | Rx bid={0,1}
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | | | a | b | c | d | e | f | g | h | i | j | k |
l | m | n | | | Rx bid={0,1}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | | | a | b | c | d | e | f | g | h | i | j | k | l | m |
n | o | p | | | Rx bid={0,1}
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | | | a | b | c | d | e | f | g | h | i | j | k | l | m | n | o |
p | q | r | | | Rx bid={0,1}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| | | a | b | c | d | e | f | g | h | i | j | k | l | m | n | o | p | q |
r | s | t | | | Rx bid={0,1}
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+| a | b | c | d | e | f | g | h | i | j | k | l | m | n | o | p | q | r | s |
t | u | v | | | Rx bid={0,1}, decode
+|---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+---+
<< 2
+|
+|<~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~>|
frame A
+|
|<~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~>|
frame B
+@ decoding from here
diff --git a/src/osmo-bts-trx/sched_lchan_tchf.c
b/src/osmo-bts-trx/sched_lchan_tchf.c
index c3e9239..2b2d1f4 100644
--- a/src/osmo-bts-trx/sched_lchan_tchf.c
+++ b/src/osmo-bts-trx/sched_lchan_tchf.c
@@ -97,7 +97,8 @@
return GSM_MACBLOCK_LEN;
}
-/*! \brief a single TCH/F burst was received by the PHY, process it */
+/* Process a single Uplink TCH/F burst received by the PHY.
+ * This function is visualized in file 'doc/trx_sched_tch.txt'. */
int rx_tchf_fn(struct l1sched_ts *l1ts, const struct trx_ul_burst_ind *bi)
{
struct l1sched_chan_state *chan_state = &l1ts->chan_state[bi->chan];
diff --git a/src/osmo-bts-trx/sched_lchan_tchh.c
b/src/osmo-bts-trx/sched_lchan_tchh.c
index 1f65d4f..d7f2227 100644
--- a/src/osmo-bts-trx/sched_lchan_tchh.c
+++ b/src/osmo-bts-trx/sched_lchan_tchh.c
@@ -154,7 +154,8 @@
return GSM_MACBLOCK_LEN;
}
-/*! \brief a single TCH/H burst was received by the PHY, process it */
+/* Process a single Uplink TCH/H burst received by the PHY.
+ * This function is visualized in file 'doc/trx_sched_tch.txt'. */
int rx_tchh_fn(struct l1sched_ts *l1ts, const struct trx_ul_burst_ind *bi)
{
struct l1sched_chan_state *chan_state = &l1ts->chan_state[bi->chan];
--
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/33076
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I373dbbc3d427858f76d07ff85633e07fe2600770
Gerrit-Change-Number: 33076
Gerrit-PatchSet: 9
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged
