Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmocom-bb/+/29868
to look at the new patch set (#2).
Change subject: trxcon: implement an abstract PHYIF API
......................................................................
trxcon: implement an abstract PHYIF API
This change is aimed to simplify the integration of trxcon with
osmo-trx-ms by allowing to replace the TRXC/TRXD interface with
direct API calls.
Change-Id: I3d7c717cfc3616809d22efb1903abbf843594258
Related: OS#5599
---
M src/host/trxcon/include/osmocom/bb/trxcon/Makefile.am
A src/host/trxcon/include/osmocom/bb/trxcon/phyif.h
M src/host/trxcon/include/osmocom/bb/trxcon/trx_if.h
M src/host/trxcon/src/trx_if.c
M src/host/trxcon/src/trxcon.c
M src/host/trxcon/src/trxcon_fsm.c
6 files changed, 300 insertions(+), 116 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/68/29868/2
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/29868
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: I3d7c717cfc3616809d22efb1903abbf843594258
Gerrit-Change-Number: 29868
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
Attention is currently required from: laforge.
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918 )
Change subject: import latest e1-tracer host software
......................................................................
Patch Set 2:
(51 comments)
File software/e1-tracer/crc4itu.h:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/daa377cb_4047…
PS2, Line 18: * The crc4itu_init() function returns the inital \c crc value and must be called
'inital' may be misspelled - perhaps 'initial'?
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/398ef445_b494…
PS2, Line 65: typedef uint_fast8_t crc_t;
do not add new typedefs
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/07c4220b_1435…
PS2, Line 75: return 0x0;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/f1ecdc82_71a7…
PS2, Line 98: return crc;
please, no spaces at the start of a line
File software/e1-tracer/crc4itu.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/d5cce08e_c2d4…
PS2, Line 26: 0x00, 0x03, 0x06, 0x05, 0x0c, 0x0f, 0x0a, 0x09, 0x0b, 0x08, 0x0d, 0x0e, 0x07, 0x04, 0x01, 0x02,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/cdc17eab_478c…
PS2, Line 27: 0x05, 0x06, 0x03, 0x00, 0x09, 0x0a, 0x0f, 0x0c, 0x0e, 0x0d, 0x08, 0x0b, 0x02, 0x01, 0x04, 0x07,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/c1cab83f_2e1f…
PS2, Line 28: 0x0a, 0x09, 0x0c, 0x0f, 0x06, 0x05, 0x00, 0x03, 0x01, 0x02, 0x07, 0x04, 0x0d, 0x0e, 0x0b, 0x08,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/12e8294f_faac…
PS2, Line 29: 0x0f, 0x0c, 0x09, 0x0a, 0x03, 0x00, 0x05, 0x06, 0x04, 0x07, 0x02, 0x01, 0x08, 0x0b, 0x0e, 0x0d,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/1dd200e3_3d3e…
PS2, Line 30: 0x07, 0x04, 0x01, 0x02, 0x0b, 0x08, 0x0d, 0x0e, 0x0c, 0x0f, 0x0a, 0x09, 0x00, 0x03, 0x06, 0x05,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/e0fd9a52_d2e8…
PS2, Line 31: 0x02, 0x01, 0x04, 0x07, 0x0e, 0x0d, 0x08, 0x0b, 0x09, 0x0a, 0x0f, 0x0c, 0x05, 0x06, 0x03, 0x00,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/27a072f3_6041…
PS2, Line 32: 0x0d, 0x0e, 0x0b, 0x08, 0x01, 0x02, 0x07, 0x04, 0x06, 0x05, 0x00, 0x03, 0x0a, 0x09, 0x0c, 0x0f,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/5f56ea96_fbd8…
PS2, Line 33: 0x08, 0x0b, 0x0e, 0x0d, 0x04, 0x07, 0x02, 0x01, 0x03, 0x00, 0x05, 0x06, 0x0f, 0x0c, 0x09, 0x0a,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/b22ee3fb_2ef9…
PS2, Line 34: 0x0e, 0x0d, 0x08, 0x0b, 0x02, 0x01, 0x04, 0x07, 0x05, 0x06, 0x03, 0x00, 0x09, 0x0a, 0x0f, 0x0c,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/fdf71f35_6c34…
PS2, Line 35: 0x0b, 0x08, 0x0d, 0x0e, 0x07, 0x04, 0x01, 0x02, 0x00, 0x03, 0x06, 0x05, 0x0c, 0x0f, 0x0a, 0x09,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/270a71ee_8cfb…
PS2, Line 36: 0x04, 0x07, 0x02, 0x01, 0x08, 0x0b, 0x0e, 0x0d, 0x0f, 0x0c, 0x09, 0x0a, 0x03, 0x00, 0x05, 0x06,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/3d953c67_12db…
PS2, Line 37: 0x01, 0x02, 0x07, 0x04, 0x0d, 0x0e, 0x0b, 0x08, 0x0a, 0x09, 0x0c, 0x0f, 0x06, 0x05, 0x00, 0x03,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/7fa94014_da3e…
PS2, Line 38: 0x09, 0x0a, 0x0f, 0x0c, 0x05, 0x06, 0x03, 0x00, 0x02, 0x01, 0x04, 0x07, 0x0e, 0x0d, 0x08, 0x0b,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/45ffd902_62d6…
PS2, Line 39: 0x0c, 0x0f, 0x0a, 0x09, 0x00, 0x03, 0x06, 0x05, 0x07, 0x04, 0x01, 0x02, 0x0b, 0x08, 0x0d, 0x0e,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/dc0353cf_8fc1…
PS2, Line 40: 0x03, 0x00, 0x05, 0x06, 0x0f, 0x0c, 0x09, 0x0a, 0x08, 0x0b, 0x0e, 0x0d, 0x04, 0x07, 0x02, 0x01,
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/ffaa5b0a_35af…
PS2, Line 41: 0x06, 0x05, 0x00, 0x03, 0x0a, 0x09, 0x0c, 0x0f, 0x0d, 0x0e, 0x0b, 0x08, 0x01, 0x02, 0x07, 0x04
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/98c5b8d7_6c56…
PS2, Line 47: const unsigned char *d = (const unsigned char *)data;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/6259b4a2_f023…
PS2, Line 48: unsigned int tbl_idx;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/4f753585_c4a0…
PS2, Line 50: while (data_len--) {
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/cbcf93b0_3d1c…
PS2, Line 51: tbl_idx = (crc << 4) ^ *d;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/64564a0c_9ab8…
PS2, Line 51: tbl_idx = (crc << 4) ^ *d;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/d3bc14da_cc16…
PS2, Line 52: crc = crc_table[tbl_idx] & 0xf;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/53b7e38c_4645…
PS2, Line 52: crc = crc_table[tbl_idx] & 0xf;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/323b1f03_e3b3…
PS2, Line 53: d++;
code indent should use tabs where possible
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/08f1ecd5_7190…
PS2, Line 53: d++;
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/f392879b_7554…
PS2, Line 54: }
please, no spaces at the start of a line
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/26c7535a_e9f4…
PS2, Line 55: return crc & 0xf;
please, no spaces at the start of a line
File software/e1-tracer/dump.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/2415ae3f_522d…
PS2, Line 17: * with this program; if not, write to the Free Software Foundation, Inc.,
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/3b11581c_ecba…
PS2, Line 18: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
File software/e1-tracer/hdlc-decode-pipe.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/642763b0_2ef4…
PS2, Line 17: * with this program; if not, write to the Free Software Foundation, Inc.,
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/8af61bbf_1147…
PS2, Line 18: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
File software/e1-tracer/idt82v2081.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/8a11e675_c7ed…
PS2, Line 17: * with this program; if not, write to the Free Software Foundation, Inc.,
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/e8185bed_4ca7…
PS2, Line 18: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
File software/e1-tracer/idt82v2081_regs.h:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/02f943ed_ca39…
PS2, Line 37: #define IDT_GCF_T1E1_E1 (0 << 2)
please, no space before tabs
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/ea829004_d1f0…
PS2, Line 38: #define IDT_GCF_T1E1_T1 (1 << 2)
please, no space before tabs
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/2d638c56_af3c…
PS2, Line 39: #define IDT_GCF_T1E1_MASK (1 << 2)
please, no space before tabs
File software/e1-tracer/idt82v2081_usb.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/cd5727dc_e8bc…
PS2, Line 17: * with this program; if not, write to the Free Software Foundation, Inc.,
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/dc53bb79_1773…
PS2, Line 18: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
File software/e1-tracer/main.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/ddce8ec0_3c28…
PS2, Line 18: * with this program; if not, write to the Free Software Foundation, Inc.,
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/aac7f872_c666…
PS2, Line 19: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
File software/e1-tracer/osmo_e1f.h:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/afc01f6d_51c0…
PS2, Line 22: * with this program; if not, write to the Free Software Foundation, Inc.,
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/e97e54cc_077c…
PS2, Line 23: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
File software/e1-tracer/osmo_e1f.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/c3e8231e_7ce4…
PS2, Line 20: * with this program; if not, write to the Free Software Foundation, Inc.,
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/808f3fcf_e133…
PS2, Line 21: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/a4749033_2591…
PS2, Line 205: } else {
else is not generally useful after a break or return
File software/e1-tracer/replay.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/4d9fd898_3a16…
PS2, Line 17: * with this program; if not, write to the Free Software Foundation, Inc.,
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-571):
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918/comment/d469d3d5_fe65…
PS2, Line 18: * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
Do not include the paragraph about writing to the Free Software Foundation's mailing address from the sample GPL notice. The FSF has changed addresses in the past, and may do so again. This Osmocom project should already include a copy of the (A)GPL.
--
To view, visit https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1-hardware
Gerrit-Branch: master
Gerrit-Change-Id: I9319c1eb8f822830307c1a181d357c58ce43efba
Gerrit-Change-Number: 29918
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Sun, 30 Oct 2022 21:37:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29919 )
Change subject: sw/e1-tracer: split recorder + analysis tools in sub-directories
......................................................................
sw/e1-tracer: split recorder + analysis tools in sub-directories
this way it's easier to see which part of the code is used where
Change-Id: I7d727263383ec1d6d61dd31fcfebf4a4b1a04765
---
R software/e1-tracer/analyze/Makefile
R software/e1-tracer/analyze/crc4itu.c
R software/e1-tracer/analyze/crc4itu.h
R software/e1-tracer/analyze/dump.c
R software/e1-tracer/analyze/dump_ts.py
R software/e1-tracer/analyze/hdlc-decode-pipe.c
R software/e1-tracer/analyze/osmo_e1f.c
R software/e1-tracer/analyze/osmo_e1f.h
R software/e1-tracer/analyze/replay.c
R software/e1-tracer/record/idt82v2081.c
R software/e1-tracer/record/idt82v2081.h
R software/e1-tracer/record/idt82v2081_regs.h
R software/e1-tracer/record/idt82v2081_usb.c
R software/e1-tracer/record/idt82v2081_usb.h
R software/e1-tracer/record/main.c
15 files changed, 1 insertion(+), 3 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-e1-hardware refs/changes/19/29919/1
diff --git a/software/e1-tracer/Makefile b/software/e1-tracer/analyze/Makefile
similarity index 71%
rename from software/e1-tracer/Makefile
rename to software/e1-tracer/analyze/Makefile
index cc3c2cf..7bd89f4 100644
--- a/software/e1-tracer/Makefile
+++ b/software/e1-tracer/analyze/Makefile
@@ -2,12 +2,10 @@
CFLAGS=`pkg-config libusb-1.0 libosmocore libosmoabis --cflags` -O2 -Wall
LDLIBS=`pkg-config libusb-1.0 libosmocore --libs`
-OBJS=main dump replay hdlc-decode-pipe
+OBJS=dump replay hdlc-decode-pipe
all: $(OBJS)
-main: idt82v2081.o idt82v2081_usb.o main.o
-
dump: dump.o crc4itu.o osmo_e1f.o
clean:
diff --git a/software/e1-tracer/crc4itu.c b/software/e1-tracer/analyze/crc4itu.c
similarity index 100%
rename from software/e1-tracer/crc4itu.c
rename to software/e1-tracer/analyze/crc4itu.c
diff --git a/software/e1-tracer/crc4itu.h b/software/e1-tracer/analyze/crc4itu.h
similarity index 100%
rename from software/e1-tracer/crc4itu.h
rename to software/e1-tracer/analyze/crc4itu.h
diff --git a/software/e1-tracer/dump.c b/software/e1-tracer/analyze/dump.c
similarity index 100%
rename from software/e1-tracer/dump.c
rename to software/e1-tracer/analyze/dump.c
diff --git a/software/e1-tracer/dump_ts.py b/software/e1-tracer/analyze/dump_ts.py
similarity index 100%
rename from software/e1-tracer/dump_ts.py
rename to software/e1-tracer/analyze/dump_ts.py
diff --git a/software/e1-tracer/hdlc-decode-pipe.c b/software/e1-tracer/analyze/hdlc-decode-pipe.c
similarity index 100%
rename from software/e1-tracer/hdlc-decode-pipe.c
rename to software/e1-tracer/analyze/hdlc-decode-pipe.c
diff --git a/software/e1-tracer/osmo_e1f.c b/software/e1-tracer/analyze/osmo_e1f.c
similarity index 100%
rename from software/e1-tracer/osmo_e1f.c
rename to software/e1-tracer/analyze/osmo_e1f.c
diff --git a/software/e1-tracer/osmo_e1f.h b/software/e1-tracer/analyze/osmo_e1f.h
similarity index 100%
rename from software/e1-tracer/osmo_e1f.h
rename to software/e1-tracer/analyze/osmo_e1f.h
diff --git a/software/e1-tracer/replay.c b/software/e1-tracer/analyze/replay.c
similarity index 100%
rename from software/e1-tracer/replay.c
rename to software/e1-tracer/analyze/replay.c
diff --git a/software/e1-tracer/idt82v2081.c b/software/e1-tracer/record/idt82v2081.c
similarity index 100%
rename from software/e1-tracer/idt82v2081.c
rename to software/e1-tracer/record/idt82v2081.c
diff --git a/software/e1-tracer/idt82v2081.h b/software/e1-tracer/record/idt82v2081.h
similarity index 100%
rename from software/e1-tracer/idt82v2081.h
rename to software/e1-tracer/record/idt82v2081.h
diff --git a/software/e1-tracer/idt82v2081_regs.h b/software/e1-tracer/record/idt82v2081_regs.h
similarity index 100%
rename from software/e1-tracer/idt82v2081_regs.h
rename to software/e1-tracer/record/idt82v2081_regs.h
diff --git a/software/e1-tracer/idt82v2081_usb.c b/software/e1-tracer/record/idt82v2081_usb.c
similarity index 100%
rename from software/e1-tracer/idt82v2081_usb.c
rename to software/e1-tracer/record/idt82v2081_usb.c
diff --git a/software/e1-tracer/idt82v2081_usb.h b/software/e1-tracer/record/idt82v2081_usb.h
similarity index 100%
rename from software/e1-tracer/idt82v2081_usb.h
rename to software/e1-tracer/record/idt82v2081_usb.h
diff --git a/software/e1-tracer/main.c b/software/e1-tracer/record/main.c
similarity index 100%
rename from software/e1-tracer/main.c
rename to software/e1-tracer/record/main.c
--
To view, visit https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29919
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1-hardware
Gerrit-Branch: master
Gerrit-Change-Id: I7d727263383ec1d6d61dd31fcfebf4a4b1a04765
Gerrit-Change-Number: 29919
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newchange
Attention is currently required from: laforge.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918
to look at the new patch set (#2).
Change subject: import latest e1-tracer host software
......................................................................
import latest e1-tracer host software
This was originally in https://github.com/smunaut/ice40-playground
but has meanwhile been removed, only a fork at
https://github.com/laf0rge/ice40-playground remained.
Let's import the code here. The history is not all that exciting
so I'm saving myself the effort of git filter-branch or the like
to import the actual old commits.
The commit log is here for reference:
commit bd36d40cc30acbae68978df79eff03f0539a9a21
Author: Harald Welte <laforge(a)osmocom.org>
Date: Fri Sep 4 10:21:51 2020 +0200
e1-recorder main.c: Don't exist just because one ISO transfer failed
I get LIBUSB_TRANSFER_ERROR every so often (once every few hours) on my
laptop. Sure, it likely means there was some data lost, but the trace
can continue juts fine after this error. So make it non-fatal and
continue running.
commit 9b02b3e87400ba3d27bff05e2632fbd5a0698e8c
Author: Harald Welte <laforge(a)osmocom.org>
Date: Mon Aug 31 10:01:20 2020 +0200
Add 'replay' to re-play 2Mbps stream with realistic speed (to stdout)
commit 23103d3a8f9cc8b3f0ba602dbb4240040b2d008a
Author: Harald Welte <laforge(a)osmocom.org>
Date: Sun Aug 30 15:41:12 2020 +0200
dump: Use osmo_e1f to decode; send HLDC over GSMTAP
commit 7acd063a1245a34835d4488be3caad592f3c19d5
Author: Harald Welte <laforge(a)osmocom.org>
Date: Thu Aug 20 16:48:15 2020 +0200
dump.c: Add error plotting support
commit 90b90d04fff494b70ad220bd82ac48d43f3096c9
Author: Harald Welte <laforge(a)osmocom.org>
Date: Mon Jan 13 18:20:39 2020 +0100
dump: Add support for writing LAPD PCAP of E1 superchannel
commit f3a0cc34f0901c2cf257367351dddfddc6f7301e
Author: Harald Welte <laforge(a)osmocom.org>
Date: Mon Jan 13 16:37:23 2020 +0100
dump.c: Move handling of frames to separate function
commit 8deffb5907979e43e3a4236ea7bb8d54513513ac
Author: Harald Welte <laforge(a)osmocom.org>
Date: Mon Jan 13 16:34:39 2020 +0100
dump.c: remove local hexdump.[ch] and use libosmocore proper
commit 80d81e78737237ba3b6bae1deb7d5107d7f613b6
Author: Harald Welte <laforge(a)osmocom.org>
Date: Tue Sep 1 13:52:52 2020 +0200
tracer: use 0x6151 as PID
See https://github.com/openmoko/openmoko-usb-oui/commit/5e68831b26dbf912d831a0c…
commit b219af27b34bbd6deb2419aa38852d8d227b9fc0
Author: Harald Welte <laforge(a)osmocom.org>
Date: Mon Jan 6 13:47:28 2020 +0100
e1-recorder: Don't silently ignore write failures
The disk may be full, for example. We should report that to the user
and terminate the program.
commit ead99fdfae6f1db54c79a91ef407a86c67352d87
Author: Harald Welte <laforge(a)osmocom.org>
Date: Mon Jan 6 12:41:23 2020 +0100
e1-recorder: Exit process on transfer + resubmit failure
This is particularly importnat in case the USB device for some reason
gets disconnected and re-connects. We want the process to fail fast
and have systemd respawn us.
commit a521e026c76056a60531c490fa2ef84e8e633c9a
Author: Harald Welte <laforge(a)osmocom.org>
Date: Mon Jan 6 12:35:20 2020 +0100
e1-recorder: use libusb_strerror() to decode error messages
commit 43a8fbdfdc35f5ba7564cfe5c92f43b124abf710
Author: Harald Welte <laforge(a)osmocom.org>
Date: Sun Jan 5 19:49:58 2020 +0100
e1-recorder: Acytually make '-r' (SCHED_RR) work
commit 84e28225d9ba1de9fb09660a96bd3b0b798c88d9
Author: Harald Welte <laforge(a)osmocom.org>
Date: Sun Jan 5 17:53:11 2020 +0100
e1-recorder: Add small 'dump' program
This program will print one line for each E1 frame, where each line
consists of 32 hex bytes: one for each timeslot
commit 0c697b40620b5d18480e626991dc639daadf58a6
Author: Sylvain Munaut <tnt(a)246tNt.com>
Date: Sun Dec 29 13:13:17 2019 +0100
projects/riscv_usb: Host software for the dual channel sniffer
Signed-off-by: Sylvain Munaut <tnt(a)246tNt.com>
Closes: OS#5673
Change-Id: I9319c1eb8f822830307c1a181d357c58ce43efba
---
A software/e1-tracer/Makefile
A software/e1-tracer/crc4itu.c
A software/e1-tracer/crc4itu.h
A software/e1-tracer/dump.c
A software/e1-tracer/dump_ts.py
A software/e1-tracer/hdlc-decode-pipe.c
A software/e1-tracer/idt82v2081.c
A software/e1-tracer/idt82v2081.h
A software/e1-tracer/idt82v2081_regs.h
A software/e1-tracer/idt82v2081_usb.c
A software/e1-tracer/idt82v2081_usb.h
A software/e1-tracer/main.c
A software/e1-tracer/osmo_e1f.c
A software/e1-tracer/osmo_e1f.h
A software/e1-tracer/replay.c
15 files changed, 2,437 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-e1-hardware refs/changes/18/29918/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-e1-hardware/+/29918
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1-hardware
Gerrit-Branch: master
Gerrit-Change-Id: I9319c1eb8f822830307c1a181d357c58ce43efba
Gerrit-Change-Number: 29918
Gerrit-PatchSet: 2
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: neels, fixeria, pespin.
lynxis lazus has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/29788 )
Change subject: Add BTS setup ramping to prevent BSC overloading
......................................................................
Patch Set 10:
(11 comments)
Patchset:
PS4:
> I hope I've addressed all your comments. […]
Done
File src/osmo-bsc/bsc_vty.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/74679f07_5ba40524
PS3, Line 3089: "Enable BTS setup ramping to limit the number of BTS which configure within a time window.\n"
> ... the amount of BTS to configure ...
Done
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/f750c9cb_f9b4f79a
PS3, Line 3090: "Limit number of BTS.\n"
> amount. […]
Done
File src/osmo-bsc/bts_setup_ramp.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/a8319faa_b7c2f525
PS4, Line 86: { 0, NULL },
> unaligned
Done
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/d7ac14f7_5ef653e8
PS4, Line 101: net->bts_setup_ramp.active
> Use bts_setup_ramp_active()?
because it's a single line. If ramp_active() is checking more, i'll use it here too.
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/f6a1bd64_7a9ed357
PS4, Line 107: net->bts_setup_ramp.active
> Use !bts_setup_ramp_active()?
same
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/1cf8509e_d30a6021
PS4, Line 164: net->bts_setup_ramp.active
> Why not using the API bts_setup_ramp_active() for that?
same
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/845e595d_c66f67ff
PS4, Line 201: INIT_LLIST_HEAD(&bts->bts_setup_ramp.list);
> exactly why we want to have a well defined value for empty llist_entries.
Done
File src/osmo-bsc/nm_bb_transc_fsm.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/3e4a01a0_724a3ebe
PS1, Line 345: case NM_EV_RAMP_GO:
> I don't see why you'd need to add it in all FSM states. […]
If I do it as you ask, I would have to check in which state the fsm is to prevent the LOGL_ERROR line about an invalid event. Further I don't like to call config_loop() when the MO is already in enabled mode. In theory it could go there if there is a bug in the BTS (e.g. ipaccess).
File src/osmo-bsc/nm_bts_ramp.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/6c4910ba_1997bfbf
PS1, Line 63: if (llist_entry_empty(&bts->bts_ramp.list))
> I don't like to have this llist set to itself. (INIT_LLIST_HEAD) […]
Done
https://gerrit.osmocom.org/c/osmo-bsc/+/29788/comment/9bd50c06_6f7e844e
PS1, Line 86: net->bts_ramp.count >= net->bts_ramp.max_bts)
> Fixed. […]
Done
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/29788
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Id56dde6d58f3d0d20352f6c306598d2cccc6345d
Gerrit-Change-Number: 29788
Gerrit-PatchSet: 10
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Sun, 30 Oct 2022 12:53:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Comment-In-Reply-To: fixeria <vyanitskiy(a)sysmocom.de>
Comment-In-Reply-To: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-MessageType: comment
Attention is currently required from: neels, fixeria, pespin.
Hello Jenkins Builder, fixeria, pespin,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-bsc/+/29788
to look at the new patch set (#10).
Change subject: Add BTS setup ramping to prevent BSC overloading
......................................................................
Add BTS setup ramping to prevent BSC overloading
Prevent BSC overloading in the event of too many BTS try to connect.
E.g. a network outage between the BSC and BTS.
The BSC will accept incoming OML connection, but will delay sending
any BSC originated messages.
Depends: Id010ade76de83ccf428f2d18e9f85bcce1d1ea2c (libosmocore)
Change-Id: Id56dde6d58f3d0d20352f6c306598d2cccc6345d
---
M include/osmocom/bsc/bts.h
A include/osmocom/bsc/bts_setup_ramp.h
M include/osmocom/bsc/gsm_data.h
M include/osmocom/bsc/nm_common_fsm.h
M src/osmo-bsc/Makefile.am
M src/osmo-bsc/bsc_vty.c
M src/osmo-bsc/bts.c
M src/osmo-bsc/bts_ipaccess_nanobts.c
A src/osmo-bsc/bts_setup_ramp.c
M src/osmo-bsc/bts_vty.c
M src/osmo-bsc/net_init.c
M src/osmo-bsc/nm_bb_transc_fsm.c
M src/osmo-bsc/nm_bts_fsm.c
M src/osmo-bsc/nm_bts_sm_fsm.c
M src/osmo-bsc/nm_channel_fsm.c
M src/osmo-bsc/nm_common_fsm.c
M src/osmo-bsc/nm_gprs_cell_fsm.c
M src/osmo-bsc/nm_gprs_nse_fsm.c
M src/osmo-bsc/nm_gprs_nsvc_fsm.c
M src/osmo-bsc/nm_rcarrier_fsm.c
M src/osmo-bsc/osmo_bsc_main.c
21 files changed, 590 insertions(+), 18 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/88/29788/10
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/29788
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Id56dde6d58f3d0d20352f6c306598d2cccc6345d
Gerrit-Change-Number: 29788
Gerrit-PatchSet: 10
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset