matthew/sonic-buildimage
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[brcm] Fix and simplify start_led.sh (#7548)

Browse Source 
LED_PROC_INIT_SOC variable was incorrectly referenced as LED_SOC_INIT_SOC. Introduced in #5483

Rather than fixing the typo, I decided to simplify the script, removing the need for the conditional altogether by moving the bcmcmd call inside the conditional which checks for the presence of LED_SOC_INIT_SOC.
This commit is contained in:
Joe LeVeque 2021-05-07 01:55:40 -07:00 committed by GitHub
parent 9daec6f20b
commit 3dddbf22fa
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
platform/broadcom/docker-syncd-brcm/start_led.sh
View File

@ -4,11 +4,6 @@ PLATFORM_DIR=/usr/share/sonic/platform
SYNCD_SOCKET_FILE=/var/run/sswsyncd/sswsyncd.socket SYNCD_SOCKET_FILE=/var/run/sswsyncd/sswsyncd.socket
LED_PROC_INIT_SOC=${PLATFORM_DIR}/led_proc_init.soc LED_PROC_INIT_SOC=${PLATFORM_DIR}/led_proc_init.soc
if [ ! -f "$LED_PROC_INIT_SOC" ]; then
echo "No soc led configuration found under $LED_SOC_INIT_SOC"
exit 0
fi
# Function: wait until syncd has created the socket for bcmcmd to connect to # Function: wait until syncd has created the socket for bcmcmd to connect to
wait_syncd() { wait_syncd() {
while true; do while true; do
@ -38,6 +33,6 @@ wait_syncd() {
# If this platform has an initialization file for the Broadcom LED microprocessor, load it # If this platform has an initialization file for the Broadcom LED microprocessor, load it
if [[ -r "$LED_PROC_INIT_SOC" && ! -f /var/warmboot/warm-starting ]]; then if [[ -r "$LED_PROC_INIT_SOC" && ! -f /var/warmboot/warm-starting ]]; then
wait_syncd wait_syncd
/usr/bin/bcmcmd -t 60 "rcload $LED_PROC_INIT_SOC"
fi fi
/usr/bin/bcmcmd -t 60 "rcload $LED_PROC_INIT_SOC"