Attention is currently required from: Paul Menzel, Reagan.
Nico Huber has posted comments on this change by Reagan. ( https://review.coreboot.org/c/coreboot/+/82458?usp=email )
Change subject: mb/razer/blade_stealth_kbl: add panel_cfg ......................................................................
Patch Set 16:
(1 comment)
File src/mainboard/razer/blade_stealth_kbl/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/82458/comment/4bf2fbea_86ca04b7?usp... : PS13, Line 163: .backlight_off_delay_ms = 5000, /* T12 */
Done. You are right, it is in 100us. At src/soc/intel/skylake/graphics. […]
Well, that's what I meant with the mapping being wrong. `down_delay_ms` *is* T10 so 50ms would be right, and that's also what the VBT says. `cycle_delay_ms` *is* T12, `backlight_on/_off` T7/T9. Overall, this is how I read the VBT: ``` Power Sequence: T3 2000 T7 10 T9 2000 T10 500 T12 5000 ```
``` .up_delay_ms = 200, .backlight_on_delay_ms = 1, .backlight_off_delay_ms = 200, .down_delay_ms = 50, .cycle_delay_ms = 500, ```