osmo-bsc[master]: abis_nm / ipa_nanobts_omlattr: use gsm48_ra_id instead of buf

2018-02-28 Thread Neels Hofmeyr
Patch Set 4: Code-Review-2 > do we have unit test coverage for this? sorry, was updating the patches without noticing this comment. will have a look. -- To view, visit https://gerrit.osmocom.org/6667 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-

osmo-bsc[master]: abis_nm / ipa_nanobts_omlattr: use gsm48_ra_id instead of buf

2018-02-27 Thread Harald Welte
Patch Set 3: do we have unit test coverage for this? -- To view, visit https://gerrit.osmocom.org/6667 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0d3908fb8ca1e2e669d257b5d59b40675fa85d06 Gerrit-PatchSet: 3 Gerrit-Project: osmo-bsc G

osmo-bsc[master]: abis_nm / ipa_nanobts_omlattr: use gsm48_ra_id instead of buf

2018-02-22 Thread Vadim Yanitskiy
Patch Set 2: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/6667 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I0d3908fb8ca1e2e669d257b5d59b40675fa85d06 Gerrit-PatchSet: 2 Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit

[PATCH] osmo-bsc[master]: abis_nm / ipa_nanobts_omlattr: use gsm48_ra_id instead of buf

2018-02-22 Thread Neels Hofmeyr
Hello Vadim Yanitskiy, Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/6667 to look at the new patch set (#2). abis_nm / ipa_nanobts_omlattr: use gsm48_ra_id instead of buf Move from using deprecated gsm48_construct_ra(), which uses a buf, to g