Bug 1373041

Summary:	No battery detected with Dell 7537
Product:	[Fedora] Fedora	Reporter:	Dominik Gronkiewicz <gronki>
Component:	kernel	Assignee:	Kernel Maintainer List <kernel-maint>
Status:	CLOSED WONTFIX	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	24	CC:	gansalmon, ichavero, itamar, jonathan, kernel-maint, labbott, madhu.chinakonda, mchehab
Target Milestone:	---
Target Release:	---
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-09-06 16:31:07 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Dominik Gronkiewicz 2016-09-05 01:17:41 UTC

No battery is detected in Dell Inspiron 7537 laptop. However using power button or LCD backlight buttons make battery indicator re-appear. 

$ sudo dmesg | grep -i bat
[    0.767857] ACPI: Battery Slot [BAT0] (battery absent)

Adding acpi_osi=Linux acpi_rev_override to kernel boot parameters does not help.

From here https://bugzilla.kernel.org/show_bug.cgi?id=105721 it is evident that switching on the CONFIG_ACPI_REV_OVERRIDE_POSSIBLE while compiling the kernel fixes the issue.

The fix is to re-enable CONFIG_ACPI_REV_OVERRIDE_POSSIBLE.

Comment 1 Laura Abbott 2016-09-06 16:31:07 UTC

CONFIG_ACPI_REV_OVERRIDE_POSSIBLE is an ugly option. We went through this with sound on the Dell XPS and concluded it's better to keep it off. Your best option is to build a custom kernel and turn it on if you want it.

Comment 2 Dominik Gronkiewicz 2016-09-06 18:01:28 UTC

Could you explain why introducing a regression is "less ugly" than allowing users to fix their issues? Is this bug fixed in future kernels?

Comment 3 Laura Abbott 2016-09-06 21:07:52 UTC

When the option was introduced the first time, there were concerns about it possibly causing issues with secure boot. These issues never panned out but ultimately the option is still playing around with the ACPI version which may have unexpected side effects. The original use was for Dell XPS sound hardware which was lagging behind with I2S support. The issue found here is a BIOS issue which the kernel is exposing. I discussed this with other maintainers and we still want to keep this option off. Turning it on still seems riskier and more prone to unexpected issues.

Comment 4 Dominik Gronkiewicz 2016-09-07 13:49:27 UTC

Thank you for your explanation. I noticed that battery indicator stopped working between kernels 4.5 and 4.6 (I can attempt to bisect more precise version). Could it be that the reason is entirely different? I'll try building the kernel with above option turned on and report the results so anyone who encounters the problem can find it on google.

Comment 5 Dominik Gronkiewicz 2016-09-07 13:49:45 UTC

Thank you for your explanation. I noticed that battery indicator stopped working between kernels 4.5 and 4.6 (I can attempt to bisect more precise version). Could it be that the reason is entirely different? I'll try building the kernel with above option turned on and report the results so anyone who encounters the problem can find it on google.