Review at  https://gerrit.osmocom.org/2639

sercomm: Move HDLC related defines into C file

Those values are not relevant to the sercomm user, only to the
implementation and thus can remain inside sercomm.c

Change-Id: I5700a45985b7c119c6338932171aae62ee4e2d22
---
M include/osmocom/core/sercomm.h
M src/sercomm.c
2 files changed, 7 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/39/2639/1

diff --git a/include/osmocom/core/sercomm.h b/include/osmocom/core/sercomm.h
index 3d986c6..0583390 100644
--- a/include/osmocom/core/sercomm.h
+++ b/include/osmocom/core/sercomm.h
@@ -3,13 +3,6 @@
 
 #include <osmocom/core/msgb.h>
 
-#define HDLC_FLAG      0x7E
-#define HDLC_ESCAPE    0x7D
-
-#define HDLC_C_UI      0x03
-#define HDLC_C_P_BIT   (1 << 4)
-#define HDLC_C_F_BIT   (1 << 4)
-
 /* a low sercomm_dlci means high priority.  A high DLCI means low priority */
 enum sercomm_dlci {
        SC_DLCI_HIGHEST = 0,
diff --git a/src/sercomm.c b/src/sercomm.c
index dbe3d05..75e7968 100644
--- a/src/sercomm.c
+++ b/src/sercomm.c
@@ -58,6 +58,13 @@
 
 #endif
 
+#define HDLC_FLAG      0x7E
+#define HDLC_ESCAPE    0x7D
+
+#define HDLC_C_UI      0x03
+#define HDLC_C_P_BIT   (1 << 4)
+#define HDLC_C_F_BIT   (1 << 4)
+
 enum rx_state {
        RX_ST_WAIT_START,
        RX_ST_ADDR,

-- 
To view, visit https://gerrit.osmocom.org/2639
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5700a45985b7c119c6338932171aae62ee4e2d22
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <[email protected]>

Reply via email to