Attention is currently required from: daniel.

pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/35084?usp=email )

Change subject: gtp: Add net GTP cause values and a function to check for success ......................................................................

Patch Set 2:

(2 comments)

File gtp/gtp.h:

https://gerrit.osmocom.org/c/osmo-ggsn/+/35084/comment/155c0048_6102e8d1 PS2, Line 106: #define GTPCAUSE_129 129 /* For future use 129-176 */ You broke the API compat here. If we deem these not necessary, then simply leave the comment and remove the defines, so that we don't keep updating (removing and adding) them.

https://gerrit.osmocom.org/c/osmo-ggsn/+/35084/comment/b4a98099_63dc652a PS2, Line 142: static inline bool gtp_cause_successful(uint8_t cause) not sure this is really useful to have here, I'd would have left it for the specific app. In any case one usually wants to check the exact cause in order to act differently here AFAIR.