diff mbox

[osmo-bts] src/osmo-bts-sysmo/l1_if.c: Forgot add utils header

Message ID 1399994078-17246-1-git-send-email-anayuso@sysmocom.de
State Changes Requested
Headers show

Commit Message

Alvaro Neira May 13, 2014, 3:14 p.m. UTC 
From: Álvaro Neira Ayuso <anayuso@sysmocom.de>

In the case that we use sysmobts_get_nominal_power inside the l1_if we receive
a warning for forgive include that header.

Signed-off-by: Alvaro Neira Ayuso <anayuso@sysmocom.de>
---
 src/osmo-bts-sysmo/l1_if.c |    1 +
 1 file changed, 1 insertion(+)

Comments

Holger Freyther May 14, 2014, 5:12 a.m. UTC | #1 
On Tue, May 13, 2014 at 05:14:38PM +0200, Alvaro Neira Ayuso wrote:

Good Morning,

> In the case that we use sysmobts_get_nominal_power inside the l1_if we receive
> a warning for forgive include that header.

Please try again. "forgive" is certainly not the right word here. Be
more explicit with the "in the case". It applies to sysmobts-v2 and
all versions above 2.2.0.

Include the warning generated by the compiler in the commit message
as well. E.g. look at 666fec7ff23e125730d7e13aa6d68112cde082de for
an example.

cheers
	holger
diff mbox

Patch

diff --git a/src/osmo-bts-sysmo/l1_if.c b/src/osmo-bts-sysmo/l1_if.c
index 79d506a..6834240 100644
--- a/src/osmo-bts-sysmo/l1_if.c
+++ b/src/osmo-bts-sysmo/l1_if.c
@@ -60,6 +60,7 @@ 
 #include "hw_misc.h"
 #include "misc/sysmobts_par.h"
 #include "eeprom.h"
+#include "utils.h"
 
 extern int pcu_direct;