sonic-buildimage

matthew/sonic-buildimage

Fork 0

Commit Graph

Author	SHA1	Message	Date
Joe LeVeque	3dddbf22fa	[brcm] Fix and simplify start_led.sh (#7548 ) 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.	2021-05-07 01:55:40 -07:00
Samuel Angebault	db6fa1d293	[led]: Skip ledinit if there is no led_proc_init.soc file for broadcom platform (#5483 ) Some platforms don't leverage the brcm led coprocessor. However ledinit will try to load a non existing file and exit with an error code. This change is a cosmetic fix mostly. - How to verify it Boot a platform without the configuration and verify in the syslog that the exit status of ledinit is 0 Boot a platform with the configuration and verify in the syslog that the exit status of ledinit is 0 and the leds are working. Verified by adding a dumb led_proc_init.soc on an Arista platform which usually doesn't use it.	2020-10-02 09:16:05 -07:00
Guohan Lu	bb40300cad	[docker-syncd-brcm]: use service dependency in supervisord to start services	2020-05-22 11:01:28 -07:00

Author

SHA1

Message

Date

Joe LeVeque

3dddbf22fa

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

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.

2021-05-07 01:55:40 -07:00

Samuel Angebault

db6fa1d293

[led]: Skip ledinit if there is no led_proc_init.soc file for broadcom platform (#5483 )

Some platforms don't leverage the brcm led coprocessor.
However ledinit will try to load a non existing file and exit with an
error code.
This change is a cosmetic fix mostly.

- How to verify it

Boot a platform without the configuration and verify in the syslog that the exit status of ledinit is 0
Boot a platform with the configuration and verify in the syslog that the exit status of ledinit is 0 and the leds are working.
Verified by adding a dumb led_proc_init.soc on an Arista platform which usually doesn't use it.

2020-10-02 09:16:05 -07:00

Guohan Lu

bb40300cad

[docker-syncd-brcm]: use service dependency in supervisord to start services

2020-05-22 11:01:28 -07:00

3 Commits