[M] Change in osmo-bts[master]: pcuif: fix TXT_IND/PCU_VERSION BTS initialization behavior

Attention is currently required from: dexter, fixeria. pespin has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/osmo-bts/+/38090?usp=email ) Change subject: pcuif: fix TXT_IND/PCU_VERSION BTS initialization behavior ...................................................................... Patch Set 1: (5 comments) Patchset: PS1: I'm not really understanding this patch tbh. if TXT_IND is sent once after socket gets connected, then it just means it's a global message and the bts_nr is simply not used, so it can be ignored... File include/osmo-bts/pcuif_proto.h: https://gerrit.osmocom.org/c/osmo-bts/+/38090/comment/c6a7a337_b3a91149?usp… : PS1, Line 262: #define PCU_IF_BTS_NR_BCAST 0xff we are jsut increasing the bts_nr in osmo-bsc to 16bit, so this doesn't look like a good idea. File src/common/pcu_sock.c: https://gerrit.osmocom.org/c/osmo-bts/+/38090/comment/2c614a92_c57b35a0?usp… : PS1, Line 868: /* we use the reception of the PCU_VERSION as a trigger to make the PCU available for This is really only osmo-bts inside a bts SM, never really going to be differnet bts (it's osmo-bts code), so if at all we should start by differentiating between cell and bts in PCUIF. I see no clear use in this right now. https://gerrit.osmocom.org/c/osmo-bts/+/38090/comment/c75935ab_1ffa86ee?usp… : PS1, Line 954: #define ENSURE_BTS_OBJECT() \ Please fix. https://gerrit.osmocom.org/c/osmo-bts/+/38090/comment/239a3946_4a1e4e74?usp… : PS1, Line 968: ENSURE_BTS_OBJECT(); This can probably be done once above the switch? -- To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/38090?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-bts Gerrit-Branch: master Gerrit-Change-Id: I5316d3b7cef416eb19bb256f4ccc1468b3efe1c6 Gerrit-Change-Number: 38090 Gerrit-PatchSet: 1 Gerrit-Owner: dexter <pmaier(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: dexter <pmaier(a)sysmocom.de> Gerrit-Comment-Date: Thu, 12 Sep 2024 12:30:41 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No