On Wed, 26 Feb 2020 15:00:58 +0100 Pavel Machek <pavel@xxxxxx> wrote: > > + switch (num) { > > + case IP30_LED_SYSTEM: > > + data->cdev.name = "white:system"; > > + break; > > + case IP30_LED_FAULT: > > + data->cdev.name = "red:fault"; > > + break; > > If the white LED normally serves as power indicator for the system, we > might want to specify that. Does it? it could be seen as such, but it will be turned on only after POST has finished successfull. So it's off if POST failed, which doesn't make it a real power led IMHO. Thomas. -- SUSE Software Solutions Germany GmbH HRB 36809 (AG Nürnberg) Geschäftsführer: Felix Imendörffer