[PATCH 2/3] nitb/ctrl: Add ctrl command to get/set band

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.de
Wed May 7 07:19:37 UTC 2014


On Tue, May 06, 2014 at 07:35:58PM +0400, Ivan Kluchnikov wrote:

> +	if ((int)gsm_band_parse(value) < 0) {
> +		return -1;
> +	}


Same as with the previous patch. Either add a bottom element to
to the gsm_band or return int from the method. But the above is
the wrong thing to do.

> +    def testBtsBand(self):
> +        bands = ['GSM450', 'GSM480', 'GSM750', 'GSM810',
> +                 'GSM850', 'GSM900', 'DCS1800', 'PCS1900']

We handle PCS1800 or DCS900 as well. Maybe just make it 'public'
and add these to the test. E.g. a matrix out of {'GSM','DCS',
'PCS'}x{450, 480, ...} for the test




More information about the OpenBSC mailing list