dexter has submitted this change. ( 
https://gerrit.osmocom.org/c/libosmocore/+/31456 )

 (

3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted 
one.
 )Change subject: i460_mux: add define constant for maximum number of 
subchannels
......................................................................

i460_mux: add define constant for maximum number of subchannels

Lets get rid of the magic number in struct osmo_i460_timeslot and
replace it with a define constant.

Change-Id: Id3a3782927c7dcbc873223d56129f291c04fee26
Related: OS#5198
---
M include/osmocom/isdn/i460_mux.h
1 file changed, 16 insertions(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, but someone else must approve
  daniel: Looks good to me, approved
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified




diff --git a/include/osmocom/isdn/i460_mux.h b/include/osmocom/isdn/i460_mux.h
index ad9cb24..537e325 100644
--- a/include/osmocom/isdn/i460_mux.h
+++ b/include/osmocom/isdn/i460_mux.h
@@ -22,6 +22,8 @@
 #include <osmocom/core/linuxlist.h>
 #include <osmocom/core/msgb.h>

+#define OSMO_I460_NUM_SUBCHAN 8
+
 /* I.460 sub-slot rate */
 enum osmo_i460_rate {
        OSMO_I460_RATE_NONE,            /* disabled */
@@ -69,7 +71,7 @@
 };

 struct osmo_i460_timeslot {
-       struct osmo_i460_subchan schan[8];
+       struct osmo_i460_subchan schan[OSMO_I460_NUM_SUBCHAN];
 };

 /*! description of a sub-channel; passed by caller */

--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31456
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Id3a3782927c7dcbc873223d56129f291c04fee26
Gerrit-Change-Number: 31456
Gerrit-PatchSet: 7
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <[email protected]>
Gerrit-Reviewer: dexter <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-MessageType: merged

Reply via email to