[M] Change in osmo-sgsn[master]: move gsm48_gmm_att_tlvdef into gprs_gmm_util

Attention is currently required from: daniel, fixeria, laforge, pespin. lynxis lazus has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/37868?usp=email ) Change subject: move gsm48_gmm_att_tlvdef into gprs_gmm_util ...................................................................... Patch Set 18: (6 comments) File include/osmocom/sgsn/gprs_gmm_util.h: https://gerrit.osmocom.org/c/osmo-sgsn/+/37868/comment/a1839f7a_bcc5ee8a?us… : PS15, Line 1: #ifndef _GPRS_GMM_UTIL_H If this is a rule, we should add this to the checkpatch. I'm changing it to #pragma. https://gerrit.osmocom.org/c/osmo-sgsn/+/37868/comment/34c42696_cfc012e0?us… : PS15, Line 6: struct msgb; it is a refactoring left over. the next commit is requiring it. I'll move this into the next commit. https://gerrit.osmocom.org/c/osmo-sgsn/+/37868/comment/62a7eda5_b6a2c099?us… : PS15, Line 6: struct msgb; most likely because a commit later in this chain will File src/sgsn/gprs_gmm_util.c: https://gerrit.osmocom.org/c/osmo-sgsn/+/37868/comment/ffee5ea3_4f2869b7?us… : PS15, Line 9: Author: Alexander Couzens &lt;lynxis(a)fe80.eu&gt; Not sure if this is even be good enough for any copyright until I add more code to it. This is only simple tlv. @laforge@gnumonks.org what is the right solution here? What should I add here? How should the copyright look like? https://gerrit.osmocom.org/c/osmo-sgsn/+/37868/comment/2d0004ec_a740b6ec?us… : PS15, Line 9: Author: Alexander Couzens &lt;lynxis(a)fe80.eu&gt; Yes you're right, but on the other side, I don't care. I would argue here, this code isn't high enough to cover any copyright until I add more code in here from another commit. https://gerrit.osmocom.org/c/osmo-sgsn/+/37868/comment/e64c3ce7_14b0b4fd?us… : PS15, Line 25: #include <osmocom/gsm/gsm48.h> Done -- To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/37868?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-sgsn Gerrit-Branch: master Gerrit-Change-Id: I67dcdb986fd01dc093501d324b5c376246a5d30d Gerrit-Change-Number: 37868 Gerrit-PatchSet: 18 Gerrit-Owner: lynxis lazus &lt;lynxis(a)fe80.eu&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: daniel &lt;dwillmann(a)sysmocom.de&gt; Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-CC: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-CC: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Attention: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Attention: fixeria &lt;vyanitskiy(a)sysmocom.de&gt; Gerrit-Attention: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-Attention: daniel &lt;dwillmann(a)sysmocom.de&gt; Gerrit-Comment-Date: Fri, 11 Oct 2024 20:11:29 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: fixeria &lt;vyanitskiy(a)sysmocom.de&gt;