Summary: | Keyboard backlight on Samsung Np900X5N not working due to not loading samsung-laptop.ko module | ||
---|---|---|---|
Product: | DRI | Reporter: | Jan <jv1> |
Component: | General | Assignee: | Default DRI bug account <dri-devel> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | high | CC: | corentin.chary, jv1 |
Version: | DRI git | ||
Hardware: | x86-64 (AMD64) | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: |
Description
Jan
2018-12-30 00:06:38 UTC
Never mind. Already found the maintainers on GIThub. The samsung-laptop.ko module is enabling the use of the keyboard backlight keys. I haved looked into the code and it seems that the BIOS is directly poked. The BIOS is put into CSM mode. The module is still not loaded. Therefor the backlight control is not working. Now digging into the linux source to understand what determines loading as discover does not recognize the Samsung machine anyway. As it is weekend I did some more reading and debugging of the module. To my understanding the module checks the model and type of the laptop. The known models and types are stored in the struct: static struct dmi_system_id __initdata samsung_dmi_table[] I currently finding out whether the NP900X5N notebook is included in this list. With dmidecode -t chassis it shows: Getting SMBIOS data from sysfs. SMBIOS 3.0.0 present. Handle 0x0003, DMI type 3, 22 bytes Chassis Information Manufacturer: SAMSUNG ELECTRONICS CO., LTD. Type: Notebook Lock: Not Present Version: N/A Serial Number: 0F4C91CJ900346 Asset Tag: No Asset Tag Boot-up State: Safe Power Supply State: Safe Thermal State: Other Security Status: None OEM Information: 0x00000000 Height: Unspecified Number Of Power Cords: 1 Contained Elements: 0 SKU Number: Chassis I think this should not be the problem as the struct includes: .matches = { DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."), DMI_MATCH(DMI_CHASSIS_TYPE, "10"), /* Notebook */ }, (In reply to Jan from comment #2) > The samsung-laptop.ko module is enabling the use of the keyboard backlight > keys. This bugzilla is about display and graphics, not about keyboard. Try these people and lists instead (or bugzilla.kernel.org, but there's a risk nobody notices: $ get_maintainer.pl -f ./drivers/platform/x86/samsung-laptop.c Corentin Chary <corentin.chary@gmail.com> (maintainer:SAMSUNG LAPTOP DRIVER) Darren Hart <dvhart@infradead.org> (maintainer:X86 PLATFORM DRIVERS) Andy Shevchenko <andy@infradead.org> (maintainer:X86 PLATFORM DRIVERS) platform-driver-x86@vger.kernel.org (open list:SAMSUNG LAPTOP DRIVER) linux-kernel@vger.kernel.org (open list) > The BIOS is put into CSM mode. For i915 we don't really support the CSM mode. Please use UEFI instead. NOTOURBUG Ok apologies :-) I understand now. Sorry for bothering you guys in the wrong group. I am still learning my way to the right people. I contacted them directly via get_maint.... Warm regards! Indeed nobody notices :-) Is there another way to reach the maintainers? ps: only reopened it for notification purpuses. As soon there is an answer I will close it again. Maintainers contacted me. I do not have the authority to close it. Therefor put it on RESOLVED with NOTOURBUG label. (In reply to Jan from comment #7) > Indeed nobody notices :-) Is there another way to reach the maintainers? Considering the time of the year, I think you're getting plenty of attention. ;) > ps: only reopened it for notification purpuses. As soon there is an answer I > will close it again. I think notifications and everything work just fine regardless of open/close status. (In reply to Jan from comment #8) > Maintainers contacted me. I do not have the authority to close it. Therefor > put it on RESOLVED with NOTOURBUG label. If it's on a public list or another bugzilla, please reference here for the benefit of others. Thanks. Ok I will Jani. And I express herewith my deepest respect for the whole Linux community. Also from a sociological view a true wonder how the cross-company eco-systems operates. Fascinating and very very impressive. I feel privileged to be part of it :-) Also please bear in mind that are bought NP900X5N laptops are currently not much more than an interesting hobby project, while we bought them as a replacement for the NP900X4D laptops that are getting end of life. Our only solution is currently getting support from the community and spending a tremendous amount of time in getting them properly running. Warm regards Jan Just for the sake of record I found the solution: The (UEFI based) kernel creates /sys/firmware/efi/efivars with the variable: KBDBacklitLvl-5af56f53-985c-47d5-920c-f1c531d06852 The set immutable flag can be disabled with: chattr -i /sys/firmware/efi/efivars/KBDBacklitLvl-5af56f53-985c-47d5-920c-f1c531d06852 After that the variable can be altered from 00 - 03: echo 0700000002 | xxd -p -r > /sys/firmware/efi/efivars/KBDBacklitLvl-5af56f53-985c-47d5-920c-f1c531d06852 (GUID dependent on the manufacturer) 00 - Backlight off (always) 01 - Backlight on DIM level (by low ambient light; detected by light sensor) 02 - Backlight on NORM level (by low ambient light) 03 - Backlight on FULL level (by low ambient light) Inspired by: https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/1203592 |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.