Attention is currently required from: osmith, keith.
dexter has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/32029 )
Change subject: bts: bts: Explain Ericsson's interface switch (IS)
......................................................................
Patch Set 3:
(2 comments)
File doc/manuals/chapters/bts.adoc:
https://gerrit.osmocom.org/c/osmo-bsc/+/32029/comment/a9323180_881b90cb
PS1, Line 784: to the outside must be aligned to E1 timeslot boarders.
> borders. […]
Done
File doc/manuals/chapters/bts.adoc:
https://gerrit.osmocom.org/c/osmo-bsc/+/32029/comment/58ad215d_f9239d5e
PS2, Line 700: [options="header",cols="45%,25%,25%"]
> these need to add up to 100%, with this patch the build passes: […]
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/32029
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ib6ebd7fdfe9063c0d8cacf53ffd27f6099d9038a
Gerrit-Change-Number: 32029
Gerrit-PatchSet: 3
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: keith <keith(a)rhizomatica.org>
Gerrit-Comment-Date: Fri, 24 Mar 2023 13:22:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: osmith <osmith(a)sysmocom.de>
Comment-In-Reply-To: keith <keith(a)rhizomatica.org>
Gerrit-MessageType: comment
Attention is currently required from: osmith, keith.
Hello Jenkins Builder, laforge, keith,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-bsc/+/32029
to look at the new patch set (#3).
Change subject: bts: bts: Explain Ericsson's interface switch (IS)
......................................................................
bts: bts: Explain Ericsson's interface switch (IS)
The built in interface switch in ericsson RBS base stations no where
explained. From the example configuration files alone it is not possible
to understand how the IS configuration works. Let's add a chapter that
explains how the IS configuration works.
Related: OS#5198
Change-Id: Ib6ebd7fdfe9063c0d8cacf53ffd27f6099d9038a
---
M doc/manuals/chapters/bts-examples.adoc
M doc/manuals/chapters/bts.adoc
2 files changed, 153 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/29/32029/3
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/32029
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ib6ebd7fdfe9063c0d8cacf53ffd27f6099d9038a
Gerrit-Change-Number: 32029
Gerrit-PatchSet: 3
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: keith <keith(a)rhizomatica.org>
Gerrit-MessageType: newpatchset
laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmocore/+/32048 )
Change subject: rate_ctr: Add rate_ctr_add2() similar to rate_ctr_inc2()
......................................................................
rate_ctr: Add rate_ctr_add2() similar to rate_ctr_inc2()
The convenience wrapper relieves the caller from manually resolving
the individual counter, and instead specify just the counter group
and the index.
Change-Id: If93e8b4fb0b86a87358f32d2b45438ca1887e9f3
---
M include/osmocom/core/rate_ctr.h
1 file changed, 22 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/48/32048/1
diff --git a/include/osmocom/core/rate_ctr.h b/include/osmocom/core/rate_ctr.h
index d944cc0..0a3eb2c 100644
--- a/include/osmocom/core/rate_ctr.h
+++ b/include/osmocom/core/rate_ctr.h
@@ -86,6 +86,15 @@
* \param inc quantity to increment \a ctr by */
void rate_ctr_add(struct rate_ctr *ctr, int inc);
+/*! Increment the counter by \a inc
+ * \param ctrg \ref rate_ctr_group of counter
+ * \param idx index into \a ctrg counter group
+ * \param inc quantity to increment \a ctr by */
+static inline void rate_ctr_add2(struct rate_ctr_group *ctrg, unsigned int idx, int inc)
+{
+ rate_ctr_add(rate_ctr_group_get_ctr(ctrg, idx), inc);
+}
+
/*! Increment the counter by 1
* \param ctr \ref rate_ctr to increment */
static inline void rate_ctr_inc(struct rate_ctr *ctr)
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/32048
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: If93e8b4fb0b86a87358f32d2b45438ca1887e9f3
Gerrit-Change-Number: 32048
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newchange
Attention is currently required from: dexter.
osmith has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/32029 )
Change subject: bts: bts: Explain Ericsson's interface switch (IS)
......................................................................
Patch Set 2:
(2 comments)
Patchset:
PS2:
This
File doc/manuals/chapters/bts.adoc:
https://gerrit.osmocom.org/c/osmo-bsc/+/32029/comment/8240fa09_d3eee67d
PS2, Line 700: [options="header",cols="45%,25%,25%"]
these need to add up to 100%, with this patch the build passes:
```
diff --git a/doc/manuals/chapters/bts.adoc b/doc/manuals/chapters/bts.adoc
index 111c5051f..195fc32b6 100644
--- a/doc/manuals/chapters/bts.adoc
+++ b/doc/manuals/chapters/bts.adoc
@@ -697,7 +697,7 @@ will repeat itself. This means the next usable ICP can be found at an offset
of 128.
.External connections of a BTS with two E1 ports
-[options="header",cols="45%,25%,25%"]
+[options="header",cols="50%,25%,25%"]
|===
|Function |Subslot offset (ICP) |ICP count
|E1 port A |4 |124
@@ -713,7 +713,7 @@ used to serve a single air interface timeslot. The two sets of ICPs must not be
mixed. Only one set may be used at a time.
.ICPs to use TRU with 16kbps subslots per TRAU
-[options="header",cols="45%,25%,25%"]
+[options="header",cols="50%,25%,25%"]
|===
|Function |Subslot offset (ICP) |ICP count
|TRU-0, RSL/OML |512 |4
@@ -748,7 +748,7 @@ connections on the other end may contain gaps and do not have to be
consecutive.
.ICPs to use TRU with 64kbps subslots per TRAU
-[options="header",cols="45%,25%,25%"]
+[options="header",cols="50%,25%,25%"]
|===
|Function |Subslot offset (ICP) |ICP count
|TRU-0, RSL/OML |712 |4
@@ -781,4 +781,4 @@ NOTE: In case voice TRAU frames are transferred, only the first of the four
16kbps subslots is used. When the timeslot is switched to GPRS/EGPRS, the
full 64kbps bandwidth will be used. This also means that the set of four
ICPs per TRAU must be connected consecutively. Also the connection
-to the outside must be aligned to E1 timeslot borders.
\ No newline at end of file
+to the outside must be aligned to E1 timeslot borders.
```
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/32029
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ib6ebd7fdfe9063c0d8cacf53ffd27f6099d9038a
Gerrit-Change-Number: 32029
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: keith <keith(a)rhizomatica.org>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 24 Mar 2023 13:05:07 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Hoernchen, fixeria.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmo-gprs/+/32021 )
Change subject: gmm: Initial implementation of GPRS Attach
......................................................................
Patch Set 7:
(1 comment)
File libosmo-gprs-gmm.pc.in:
https://gerrit.osmocom.org/c/libosmo-gprs/+/32021/comment/0191810f_21dd63ad
PS1, Line 9: libosmo-gprs-llc
> Ah, right. Somehow I thought I am commenting to file `contrib/libosmo-gprs.spec.in`.
So is it fine now? can this merged?
--
To view, visit https://gerrit.osmocom.org/c/libosmo-gprs/+/32021
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-gprs
Gerrit-Branch: master
Gerrit-Change-Id: I212053b3a3f27ef7d63503c3d5ef08453b2d2056
Gerrit-Change-Number: 32021
Gerrit-PatchSet: 7
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: Hoernchen <ewild(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 24 Mar 2023 12:34:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment
Attention is currently required from: dexter.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-pcu/+/32047 )
Change subject: doc: overview: put BTS and PCU on the same rank
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-pcu/+/32047
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Change-Id: Ifa170f3956b4629765a88ed62e877a204541d9ea
Gerrit-Change-Number: 32047
Gerrit-PatchSet: 1
Gerrit-Owner: dexter <pmaier(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: dexter <pmaier(a)sysmocom.de>
Gerrit-Comment-Date: Fri, 24 Mar 2023 12:32:37 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment