tnt has uploaded this change for review. ( https://gerrit.osmocom.org/c/dahdi-linux/+/36654?usp=email )
Change subject: icE1usb: Use ICE1USB_TX_MODE_TS0_CRC4_E to set E bits automatically
......................................................................
icE1usb: Use ICE1USB_TX_MODE_TS0_CRC4_E to set E bits automatically
We want the hardware to take care of the CRC4 error bit autonomously
without us having to do anything ...
Change-Id: Iea936a6c908ef5e175be3090a8fbcafaeb5a0aed
Signed-off-by: Sylvain Munaut <tnt(a)246tNt.com>
---
M drivers/dahdi/icE1usb/icE1usb.c
1 file changed, 14 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/dahdi-linux refs/changes/54/36654/1
diff --git a/drivers/dahdi/icE1usb/icE1usb.c b/drivers/dahdi/icE1usb/icE1usb.c
index 7ab751e..2262deb 100644
--- a/drivers/dahdi/icE1usb/icE1usb.c
+++ b/drivers/dahdi/icE1usb/icE1usb.c
@@ -686,7 +686,7 @@
}
if (span->lineconfig & DAHDI_CONFIG_CRC4)
- ieu->cfg.tx.mode = ICE1USB_TX_MODE_TS0_CRC4;
+ ieu->cfg.tx.mode = ICE1USB_TX_MODE_TS0_CRC4_E;
else
ieu->cfg.tx.mode = ICE1USB_TX_MODE_TS0;
--
To view, visit https://gerrit.osmocom.org/c/dahdi-linux/+/36654?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: dahdi-linux
Gerrit-Branch: master
Gerrit-Change-Id: Iea936a6c908ef5e175be3090a8fbcafaeb5a0aed
Gerrit-Change-Number: 36654
Gerrit-PatchSet: 1
Gerrit-Owner: tnt <tnt(a)246tNt.com>
Gerrit-MessageType: newchange
Attention is currently required from: laforge.
Hello Jenkins Builder, laforge,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-e1d/+/36653?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+1 by laforge, Verified+1 by Jenkins Builder
Change subject: iCE1usb: Add support to configure CRC4 mode for TX/RX
......................................................................
iCE1usb: Add support to configure CRC4 mode for TX/RX
Signed-off-by: Sylvain Munaut <tnt(a)246tNt.com>
Change-Id: I9b627a9617af4ae9ed98214d672c7d0391f801eb
---
M src/e1d.h
M src/usb.c
M src/vty.c
3 files changed, 78 insertions(+), 1 deletion(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-e1d refs/changes/53/36653/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/36653?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1d
Gerrit-Branch: master
Gerrit-Change-Id: I9b627a9617af4ae9ed98214d672c7d0391f801eb
Gerrit-Change-Number: 36653
Gerrit-PatchSet: 2
Gerrit-Owner: tnt <tnt(a)246tNt.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: fixeria, neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/36546?usp=email )
Change subject: add API logging_vty_subsys_strip_leading_char()
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Yes, the current approach of when or where to strip that 'd' is not logical. It is - like many things - just because it was done a certain way without thinking about long-term implications.
But do we really make life easier by offering (now, after probably more than a decade of the existing behavior) multiple behavioral options, increasing the complexity for application developers even further?
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/36546?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I5faedf7d6525d744a734ebe54c185fcc904f763e
Gerrit-Change-Number: 36546
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Sun, 28 Apr 2024 12:21:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: fixeria, neels.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/libosmocore/+/36545?usp=email )
Change subject: formalize log subsys stripping for vty
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
Yes, the current approach of when or where to strip that 'd' is not logical. It is - like many things - just because it was done a certain way without thinking about long-term implications.
But do we really make life easier by offering (now, after probably more than a decade of the existing behavior) multiple behavioral options, increasing the complexity for application developers even further?
--
To view, visit https://gerrit.osmocom.org/c/libosmocore/+/36545?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I5f81343e8c7b714a4630e64ba654e391435c4244
Gerrit-Change-Number: 36545
Gerrit-PatchSet: 1
Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-CC: laforge <laforge(a)osmocom.org>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de>
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Comment-Date: Sun, 28 Apr 2024 12:20:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: laforge.
tnt has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-e1d/+/36653?usp=email )
Change subject: iCE1usb: Add support to configure CRC4 mode for TX/RX
......................................................................
Patch Set 1:
(1 comment)
File src/vty.c:
https://gerrit.osmocom.org/c/osmo-e1d/+/36653/comment/012c4c7d_0303d0b1
PS1, Line 434: framing-tx (no-crc4|crc4)",
> this could be one command for both rx and tx like "framing (rx|tx) (no-crc4|crc4)". […]
Ack, will change.
I was trying to change it to "framing [(tx|rx)] (no-crc4|crc4)" so that you could just do both at once, but that doesn't see to work. Is optional middle argument not supported ?
--
To view, visit https://gerrit.osmocom.org/c/osmo-e1d/+/36653?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-e1d
Gerrit-Branch: master
Gerrit-Change-Id: I9b627a9617af4ae9ed98214d672c7d0391f801eb
Gerrit-Change-Number: 36653
Gerrit-PatchSet: 1
Gerrit-Owner: tnt <tnt(a)246tNt.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Sun, 28 Apr 2024 12:19:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment
laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-sgsn/+/36629?usp=email )
Change subject: docs: front page: use https:// instead of http://
......................................................................
docs: front page: use https:// instead of http://
Change-Id: If3c3b8e79f94da7a3bcc9262808da09f6a5a601c
---
M doc/manuals/osmosgsn-usermanual-docinfo.xml
1 file changed, 11 insertions(+), 2 deletions(-)
Approvals:
pespin: Looks good to me, but someone else must approve
Jenkins Builder: Verified
fixeria: Looks good to me, approved
diff --git a/doc/manuals/osmosgsn-usermanual-docinfo.xml b/doc/manuals/osmosgsn-usermanual-docinfo.xml
index 7c7de2c..9eea626 100644
--- a/doc/manuals/osmosgsn-usermanual-docinfo.xml
+++ b/doc/manuals/osmosgsn-usermanual-docinfo.xml
@@ -66,8 +66,8 @@
</para>
<para>
The Asciidoc source code of this manual can be found at
- <ulink url="http://git.osmocom.org/osmo-gsm-manuals/">
- http://git.osmocom.org/osmo-gsm-manuals/
+ <ulink url="https://git.osmocom.org/osmo-gsm-manuals/">
+ https://git.osmocom.org/osmo-gsm-manuals/
</ulink>
</para>
</legalnotice>
--
To view, visit https://gerrit.osmocom.org/c/osmo-sgsn/+/36629?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Change-Id: If3c3b8e79f94da7a3bcc9262808da09f6a5a601c
Gerrit-Change-Number: 36629
Gerrit-PatchSet: 3
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged