[MERGED] osmocom-bb[master]: VIRT_PHY: add missing L1CTL_BURST_IND to l1ctlPrimNames

2018-04-16 Thread Vadim Yanitskiy 
Vadim Yanitskiy has submitted this change and it was merged.

Change subject: VIRT_PHY: add missing L1CTL_BURST_IND to l1ctlPrimNames
..


VIRT_PHY: add missing L1CTL_BURST_IND to l1ctlPrimNames

Change-Id: I442305c034bbba5eaed080fb262a61895623eb4f
---
M src/host/virt_phy/src/logging.c
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/host/virt_phy/src/logging.c b/src/host/virt_phy/src/logging.c
index 631a6e6..a3e63bc 100644
--- a/src/host/virt_phy/src/logging.c
+++ b/src/host/virt_phy/src/logging.c
@@ -57,6 +57,7 @@
 "L1CTL_TRAFFIC_REQ",
 "L1CTL_TRAFFIC_CONF",
 "L1CTL_TRAFFIC_IND",
+"L1CTL_BURST_IND",
 "L1CTL_TBF_CFG_REQ",
 "L1CTL_TBF_CFG_CONF",
 "L1CTL_DATA_TBF_REQ",

-- 
To view, visit https://gerrit.osmocom.org/7808
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I442305c034bbba5eaed080fb262a61895623eb4f
Gerrit-PatchSet: 1
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Owner: Vadim Yanitskiy 
Gerrit-Reviewer: Harald Welte 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Vadim Yanitskiy

osmocom-bb[master]: VIRT_PHY: add missing L1CTL_BURST_IND to l1ctlPrimNames

2018-04-15 Thread Harald Welte 

Patch Set 1: Code-Review+2

(1 comment)

https://gerrit.osmocom.org/#/c/7808/1/src/host/virt_phy/src/logging.c
File src/host/virt_phy/src/logging.c:

Line 64: "L1CTL_DATA_TBF_CONF"
it's probably a good time to convert this to value_string.  Having an array 
with *implicit* index is really error-prone and should be avoided.


-- 
To view, visit https://gerrit.osmocom.org/7808
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I442305c034bbba5eaed080fb262a61895623eb4f
Gerrit-PatchSet: 1
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Owner: Vadim Yanitskiy 
Gerrit-Reviewer: Harald Welte 
Gerrit-Reviewer: Jenkins Builder
Gerrit-HasComments: Yes

[PATCH] osmocom-bb[master]: VIRT_PHY: add missing L1CTL_BURST_IND to l1ctlPrimNames

2018-04-15 Thread Vadim Yanitskiy 

Review at  https://gerrit.osmocom.org/7808

VIRT_PHY: add missing L1CTL_BURST_IND to l1ctlPrimNames

Change-Id: I442305c034bbba5eaed080fb262a61895623eb4f
---
M src/host/virt_phy/src/logging.c
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/08/7808/1

diff --git a/src/host/virt_phy/src/logging.c b/src/host/virt_phy/src/logging.c
index 631a6e6..a3e63bc 100644
--- a/src/host/virt_phy/src/logging.c
+++ b/src/host/virt_phy/src/logging.c
@@ -57,6 +57,7 @@
 "L1CTL_TRAFFIC_REQ",
 "L1CTL_TRAFFIC_CONF",
 "L1CTL_TRAFFIC_IND",
+"L1CTL_BURST_IND",
 "L1CTL_TBF_CFG_REQ",
 "L1CTL_TBF_CFG_CONF",
 "L1CTL_DATA_TBF_REQ",

-- 
To view, visit https://gerrit.osmocom.org/7808
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I442305c034bbba5eaed080fb262a61895623eb4f
Gerrit-PatchSet: 1
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Owner: Vadim Yanitskiy