Attention is currently required from: matanp.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/osmo-bsc/+/35746?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified-1 by Jenkins Builder
Change subject: ctrl: Split bts_trx_ts_ctrl from bts_trx_ctrl
......................................................................
ctrl: Split bts_trx_ts_ctrl from bts_trx_ctrl
Change-Id: I9840bddd4eae409bc8373912d54b6bbfc9fc1c1a
---
M include/osmocom/bsc/ctrl.h
M src/osmo-bsc/Makefile.am
M src/osmo-bsc/bts_trx_ctrl.c
A src/osmo-bsc/bts_trx_ts_ctrl.c
4 files changed, 117 insertions(+), 98 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-bsc refs/changes/46/35746/2
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/35746?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I9840bddd4eae409bc8373912d54b6bbfc9fc1c1a
Gerrit-Change-Number: 35746
Gerrit-PatchSet: 2
Gerrit-Owner: matanp <matan1008(a)gmail.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: matanp <matan1008(a)gmail.com>
Gerrit-MessageType: newpatchset
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/35746?usp=email )
Change subject: ctrl: Split bts_trx_ts_ctrl from bts_trx_ctrl
......................................................................
Patch Set 1:
(2 comments)
File src/osmo-bsc/bts_trx_ts_ctrl.c:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-13929):
https://gerrit.osmocom.org/c/osmo-bsc/+/35746/comment/6fb0171c_f435514b
PS1, Line 34: if (gsm_arfcn2band_rc(arfcn, &unused) < 0) {
braces {} are not necessary for single statement blocks
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-13929):
https://gerrit.osmocom.org/c/osmo-bsc/+/35746/comment/d5141d3b_6709adbb
PS1, Line 67: if (gsm_arfcn2band_rc(arfcn, &unused) < 0) {
braces {} are not necessary for single statement blocks
--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/35746?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I9840bddd4eae409bc8373912d54b6bbfc9fc1c1a
Gerrit-Change-Number: 35746
Gerrit-PatchSet: 1
Gerrit-Owner: matanp <matan1008(a)gmail.com>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Tue, 30 Jan 2024 15:07:18 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: pespin.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35742?usp=email )
Change subject: NS_Emulation: expect altstep timeout entry only if timer is running
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/35742?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: If1fae965659f73fde2508b0e9158099025fa65f2
Gerrit-Change-Number: 35742
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 30 Jan 2024 14:28:36 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment