I am sorry, but here you aren't fixing the problem but simply adopting the test suite expectations to the current behavior of osmo-bsc. As I said, the problem is in osmo-bsc and it needs to be fixed there. Hence CR-2.
Patch set 1:Code-Review -2
1 comment:
Patch Set #1, Line 2595: int2bit(enum2int(rlid.c), 2));
This is wrong. The encoding of the 'Control Channel type' is different between RSL and BSSAP:
=== RSL
'00'B - FACCH_SDCCH
'01'B - SACCH
== BSSAP
'00'B - control channel is not further specified
'10'B - represents the FACCH or the SDCCH (this is what we expect)
'11'B - represents the SACCH
See 3GPP TS 48.006, section 9.3.2. Quick link (page 34):
https://www.etsi.org/deliver/etsi_ts/148000_148099/148006/16.00.00_60/ts_148006v160000p.pdf
To view, visit change 26891. To unsubscribe, or for help writing mail filters, visit settings.