Dasharo / dasharo-issues

The Dasharo issue tracker
https://dasharo.com/
25 stars 0 forks source link

V540TND keyboad illumination changes on coldboot #1055

Closed philipandag closed 1 month ago

philipandag commented 2 months ago

Component

Dasharo firmware, EC firmware

Device

NovaCustom V54 14th Gen

Dasharo version

v0.9.1-rc5

Dasharo Tools Suite version

--

Test case ID

ECR25

Brief summary

The keyboard backlight setting changes after a coldboot

How reproducible

100% in 2 tries on Ubuntu and 2 tries on Windows

How to reproduce

Boot an OS, change backlight setting (alternating turned off and full brightness is the easiest to notice), disconnect all external power supplies and disconnect the battery. Connect the battery back, boot into an OS

Expected behavior

The backlight setting from before the coldboot should remain

Actual behavior

The backlight setting changes. In all tests the default was off and I have changed it to the max brightness. After coldboot it's always back to being off.

Screenshots

cbmem logs from Ubuntu, for -2 (boot before coldboot) and -1 (the coldboot) cbmem-keyboard-backlight-1.log cbmem-keyboard-backlight-2.log

Additional context

No response

Solutions you've tried

No response

mkopec commented 2 months ago

No need to disconnect the battery to perform a coldboot. Just shut down, unplug wall adapter and wait a second.

The EC saves settings upon (graceful) shutdown, so you have to go through the proper shutdown, instead of hot-removing AC and battery completely

philipandag commented 2 months ago

Does it mean that the AC needs to be plugged in for a warmboot and without it the os shutdown results in a coldboot? In that case the Dasharo test specification needs updates because it says to remove the battery. https://docs.dasharo.com/unified-test-documentation/dasharo-compatibility/31G-ec-and-superio/#ecr025001-permanent-keyboard-illumination-after-cold-boot-firmware

mkopec commented 1 month ago

Power states explained in https://docs.dasharo.com/transparent-validation/ec-information/ec-explained/ , closing