Attention is currently required from: osmith, arehbein, laforge.

pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31877 )

Change subject: gm_12_21.h: Add cross-component parameter defaults (NSE timing) ......................................................................

Patch Set 2: Code-Review-1

(1 comment)

File include/osmocom/gsm/protocol/gsm_12_21.h:

https://gerrit.osmocom.org/c/libosmocore/+/31877/comment/c9681c15_3731b5c3 PS2, Line 294: enum abis_nm_par_defaults { it makes no sense to have this as enums, because they are not different types of the same thing nor need to have different values (as you can see below, values are repeated). Please use defines here.

BTW, I guess all these default values come from specs (TS 12.21?) because if they don't come from some spec reference, they have no place here, since it would actually be some implementation specific value of each app that we chose.