Attention is currently required from: msuraev. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/31664 ) Change subject: layer23: get rid of lapdm_channel_init() warning ...................................................................... Patch Set 2: (2 comments) File src/host/layer23/include/osmocom/bb/common/ms.h: https://gerrit.osmocom.org/c/osmocom-bb/+/31664/comment/db03dd66_1f4adc17 PS2, Line 106: enum gsm_chan_t chan_t, const char *name); using chan_t for the name of the variable is confusing, it looks like you are passing a type. even gerrit is confused with colors. File src/host/layer23/src/common/ms.c: https://gerrit.osmocom.org/c/osmocom-bb/+/31664/comment/41799a23_eb37ee01 PS2, Line 50: lapdm_channel_init3(&ms->lapdm_channel, LAPDM_MODE_MS, t200_ms_dcch, t200_ms_acch, chan_t, name); if these are to be set to some other value then they will most probably be set by the VTY after the object has been created, so I don't really see the point in having those params passed in the constructor, but rather having an API to set/update them. So I'd keep 1 ms_alloc() containing defaults and then add an extra API to set it after the MS is allocated. -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/31664 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: If3be53a9f20e0f9e7cfb9ccd04ed54a3dcde44b8 Gerrit-Change-Number: 31664 Gerrit-PatchSet: 2 Gerrit-Owner: msuraev <msuraev@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> Gerrit-Reviewer: laforge <laforge@osmocom.org> Gerrit-Reviewer: pespin <pespin@sysmocom.de> Gerrit-Attention: msuraev <msuraev@sysmocom.de> Gerrit-Comment-Date: Tue, 07 Mar 2023 09:36:28 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment