dexter submitted this change.

View Change



3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

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
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(-)

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 change 31456. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Id3a3782927c7dcbc873223d56129f291c04fee26
Gerrit-Change-Number: 31456
Gerrit-PatchSet: 7
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged