Change in osmo-bts[master]: lc15: led_sleep_cb: pass correct ptr to llist_move_tail

2018-09-21 Thread Pau Espin Pedrol
Pau Espin Pedrol has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/9534 )

Change subject: lc15: led_sleep_cb: pass correct ptr to llist_move_tail
..

lc15: led_sleep_cb: pass correct ptr to llist_move_tail

Fix compilation warning. At runtime it's not a big issue because the
"list" field is the first field of the led_list (struct
lc15bts_led_timer_list) variable. Hence, the address passed is the same.

Change-Id: Ib8bf07990800d74bfb3ad7a55eccfc65e40cd480
---
M src/osmo-bts-litecell15/misc/lc15bts_led.c
1 file changed, 1 insertion(+), 1 deletion(-)

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



diff --git a/src/osmo-bts-litecell15/misc/lc15bts_led.c 
b/src/osmo-bts-litecell15/misc/lc15bts_led.c
index 603e0fb..a93d3fb 100644
--- a/src/osmo-bts-litecell15/misc/lc15bts_led.c
+++ b/src/osmo-bts-litecell15/misc/lc15bts_led.c
@@ -151,7 +151,7 @@
/* Delete current timer */
osmo_timer_del(_list->led_timer.timer);
/* Rotate the timer list */
-llist_move_tail(led_list, >lc15bts_leds.list);
+llist_move_tail(_list->list, 
>lc15bts_leds.list);
break;
}
}

--
To view, visit https://gerrit.osmocom.org/9534
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib8bf07990800d74bfb3ad7a55eccfc65e40cd480
Gerrit-Change-Number: 9534
Gerrit-PatchSet: 2
Gerrit-Owner: Pau Espin Pedrol 
Gerrit-Reviewer: Harald Welte 
Gerrit-Reviewer: Jenkins Builder (102)
Gerrit-Reviewer: Pau Espin Pedrol 


Change in osmo-bts[master]: lc15: led_sleep_cb: pass correct ptr to llist_move_tail

2018-06-11 Thread Pau Espin Pedrol
Pau Espin Pedrol has uploaded this change for review. ( 
https://gerrit.osmocom.org/9534


Change subject: lc15: led_sleep_cb: pass correct ptr to llist_move_tail
..

lc15: led_sleep_cb: pass correct ptr to llist_move_tail

Fix compilation warning. At runtime it's not a big issue because the
"list" field is the first field of the led_list (struct
lc15bts_led_timer_list) variable. Hence, the address passed is the same.

Change-Id: Ib8bf07990800d74bfb3ad7a55eccfc65e40cd480
---
M src/osmo-bts-litecell15/misc/lc15bts_led.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-bts refs/changes/34/9534/1

diff --git a/src/osmo-bts-litecell15/misc/lc15bts_led.c 
b/src/osmo-bts-litecell15/misc/lc15bts_led.c
index 603e0fb..a93d3fb 100644
--- a/src/osmo-bts-litecell15/misc/lc15bts_led.c
+++ b/src/osmo-bts-litecell15/misc/lc15bts_led.c
@@ -151,7 +151,7 @@
/* Delete current timer */
osmo_timer_del(_list->led_timer.timer);
/* Rotate the timer list */
-llist_move_tail(led_list, >lc15bts_leds.list);
+llist_move_tail(_list->list, 
>lc15bts_leds.list);
break;
}
}

--
To view, visit https://gerrit.osmocom.org/9534
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib8bf07990800d74bfb3ad7a55eccfc65e40cd480
Gerrit-Change-Number: 9534
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol