Patch Set 1: (1 comment)
https://gerrit.osmocom.org/#/c/6623/1/include/osmocom/vty/ports.h File include/osmocom/vty/ports.h: Line 34: #define OSMO_VTY_PORT_TRX 4263 I would argue for a lower port number. Trdaitionally we've had lower port numbers for lower elements in the stack (i.e. BTS < BSC < MSC < HLR). So I would go for numbers below 2439 if possible, putting it closer to PCU+BTS. -- To view, visit https://gerrit.osmocom.org/6623 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib79cdb62d45d8c78445c7b064e58eb7e9faeccf9 Gerrit-PatchSet: 1 Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Owner: Pau Espin Pedrol <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-HasComments: Yes
