This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.
Holger Hans Peter Freyther holger at freyther.deOn Thu, Dec 05, 2013 at 04:02:36PM +0100, Andreas Eversberg wrote: > +struct bts_codec_conf { > + uint8_t hr; > + uint8_t efr; > + uint8_t afs; > + uint8_t ahs; this naming appears odd. Where is it coming from? Do you know of a BTS that supports AMR on TCH/F but not on TCH/H? > +DEFUN(cfg_bts_codec1, cfg_bts_codec1_cmd, > +DEFUN(cfg_bts_codec2, cfg_bts_codec2_cmd, > +DEFUN(cfg_bts_codec3, cfg_bts_codec3_cmd, > +DEFUN(cfg_bts_codec4, cfg_bts_codec4_cmd, > +DEFUN(cfg_bts_codec5, cfg_bts_codec5_cmd, please remove the code duplication and add a VTY testcase for setting and reading this value. > +static int apply_codec_restrictions(struct gsm_bts *bts, > + struct gsm_mncc_bearer_cap *bcap) > +{ > + int i, j; > + > + /* remove unsupported speech versions from list */ > + for (i = 0, j = 0; bcap->speech_ver[i] >= 0; i++) { > + if (bcap->speech_ver[i] == 0) > + bcap->speech_ver[j++] = 0; > + if (bcap->speech_ver[i] == 2 && bts->codec.efr) > + bcap->speech_ver[j++] = 2; > + if (bcap->speech_ver[i] == 4 && bts->codec.afs) > + bcap->speech_ver[j++] = 4; > + if (bcap->speech_ver[i] == 1 && bts->codec.hr) > + bcap->speech_ver[j++] = 1; > + if (bcap->speech_ver[i] == 5 && bts->codec.ahs) > + bcap->speech_ver[j++] = 5; > + } please use the values of the gsm48_bcap_speech_ver enum instead of the magic numbers.