laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/21568 )
Change subject: power_control: make raise/lower step limitation configurable ...................................................................... Patch Set 1: (2 comments) https://gerrit.osmocom.org/c/osmo-bts/+/21568/1/include/osmo-bts/bts.h File include/osmo-bts/bts.h: https://gerrit.osmocom.org/c/osmo-bts/+/21568/1/include/osmo-bts/bts.h@110 PS1, Line 110: as maximum at maximum (also below) https://gerrit.osmocom.org/c/osmo-bts/+/21568/1/include/osmo-bts/bts.h@111 PS1, Line 111: in the variable name nor the comment really indicate it is about the _step_ size. So maybe make that more clear. -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/21568 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: Ic37742f46f533865043b3dbcf16ea702e1746f98 Gerrit-Change-Number: 21568 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin <[email protected]> Gerrit-CC: laforge <[email protected]> Gerrit-Comment-Date: Tue, 08 Dec 2020 09:44:56 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment
