Patch Set 1: (2 comments)
https://gerrit.osmocom.org/#/c/7723/1/src/libbsc/bsc_vty.c File src/libbsc/bsc_vty.c: Line 1918: acc_ramp_init(&bts->acc_ramp, bts); > I agree. It can be left for a later patch, no problem. Line 3278: acc_ramp_enable(&bts->acc_ramp); > I don't understand what this would achieve. The 'enable' flag does not chan Indeed you are right, no issue here. I was mixing it with previous version which may have had some related issues. -- To view, visit https://gerrit.osmocom.org/7723 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia25bff85d9e5c277da76bffa11d31972e9fdc323 Gerrit-PatchSet: 1 Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Owner: Stefan Sperling <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Pau Espin Pedrol <[email protected]> Gerrit-Reviewer: Stefan Sperling <[email protected]> Gerrit-HasComments: Yes
