[lm-sensors] [PATCH] dell-smm-hwmon: Blacklist Dell Studio XPS 8100

[lm-sensors] [PATCH] dell-smm-hwmon: Blacklist Dell Studio XPS 8100

[Pali Rohár]

Q1BVIGZhbiBzcGVlZCBnb2luZyB1cCBhbmQgZG93biBvbiBEZWxsIFN0dWRpbyBYUFMgODEwMCBm
b3IKdW5rbm93biByZWFzb24uIFdpdGhvdXQgZnV0dXJlIGRlYnVnZ25pbmcgb24gYWZmZWN0ZWQg
bWFjaGluZQppdCBpcyBub3QgcG9zc2libGUgdG8gZGV0ZWN0IHdoZXJlIGlzIHByb2JsZW0uIEZv
ciBtb3JlIHNlZToKaHR0cHM6Ly9idWd6aWxsYS5rZXJuZWwub3JnL3Nob3dfYnVnLmNnaT9pZD0x
MDAxMjEKClNpZ25lZC1vZmYtYnk6IFBhbGkgUm9ow6FyIDxwYWxpLnJvaGFyQGdtYWlsLmNvbT4K
VGVzdGVkLWJ5OiBKYW4gQyBQZXRlcnMgPGpjcGV0ZXJzODlAZ21haWwuY29tPgpDYzogc3RhYmxl
QHZnZXIua2VybmVsLm9yZyAjIGJlZm9yZSB2NC4yIGFwcGx5IHRvIGZpbGUgZHJpdmVycy9jaGFy
L2k4ay5jCi0tLQogZHJpdmVycy9od21vbi9kZWxsLXNtbS1od21vbi5jIHwgICAyMCArKysrKysr
KysrKysrKysrKysrLQogMSBmaWxlIGNoYW5nZWQsIDE5IGluc2VydGlvbnMoKyksIDEgZGVsZXRp
b24oLSkKCmRpZmYgLS1naXQgYS9kcml2ZXJzL2h3bW9uL2RlbGwtc21tLWh3bW9uLmMgYi9kcml2
ZXJzL2h3bW9uL2RlbGwtc21tLWh3bW9uLmMKaW5kZXggMzdjMTZhZi4uOTJjNTcyYyAxMDA2NDQK
LS0tIGEvZHJpdmVycy9od21vbi9kZWxsLXNtbS1od21vbi5jCisrKyBiL2RyaXZlcnMvaHdtb24v
ZGVsbC1zbW0taHdtb24uYwpAQCAtOTI5LDYgKzkyOSwyMyBAQCBzdGF0aWMgc3RydWN0IGRtaV9z
eXN0ZW1faWQgaThrX2RtaV90YWJsZVtdIF9faW5pdGRhdGEgPSB7CiAKIE1PRFVMRV9ERVZJQ0Vf
VEFCTEUoZG1pLCBpOGtfZG1pX3RhYmxlKTsKIAorc3RhdGljIHN0cnVjdCBkbWlfc3lzdGVtX2lk
IGk4a19ibGFja2xpc3RfZG1pX3RhYmxlW10gX19pbml0ZGF0YSA9IHsKKwl7CisJCS8qCisJCSAq
IENQVSBmYW4gc3BlZWQgZ29pbmcgdXAgYW5kIGRvd24gb24gRGVsbCBTdHVkaW8gWFBTIDgxMDAg
Zm9yCisJCSAqIHVua25vd24gcmVhc29uLiBXaXRob3V0IGZ1dHVyZSBkZWJ1Z2duaW5nIG9uIGFm
ZmVjdGVkIG1hY2hpbmUKKwkJICogaXQgaXMgbm90IHBvc3NpYmxlIHRvIGRldGVjdCB3aGVyZSBp
cyBwcm9ibGVtLiBGb3IgbW9yZSBzZWU6CisJCSAqIGh0dHBzOi8vYnVnemlsbGEua2VybmVsLm9y
Zy9zaG93X2J1Zy5jZ2k/aWQ9MTAwMTIxCisJCSAqLworCQkuaWRlbnQgPSAiRGVsbCBTdHVkaW8g
WFBTIDgxMDAiLAorCQkubWF0Y2hlcyA9IHsKKwkJCURNSV9FWEFDVF9NQVRDSChETUlfU1lTX1ZF
TkRPUiwgIkRlbGwgSW5jLiIpLAorCQkJRE1JX0VYQUNUX01BVENIKERNSV9QUk9EVUNUX05BTUUs
ICJTdHVkaW8gWFBTIDgxMDAiKSwKKwkJfSwKKwl9LAorCXsgfQorfTsKKwogLyoKICAqIFByb2Jl
IGZvciB0aGUgcHJlc2VuY2Ugb2YgYSBzdXBwb3J0ZWQgbGFwdG9wLgogICovCkBAIC05NDAsNyAr
OTU3LDggQEAgc3RhdGljIGludCBfX2luaXQgaThrX3Byb2JlKHZvaWQpCiAJLyoKIAkgKiBHZXQg
RE1JIGluZm9ybWF0aW9uCiAJICovCi0JaWYgKCFkbWlfY2hlY2tfc3lzdGVtKGk4a19kbWlfdGFi
bGUpKSB7CisJaWYgKCFkbWlfY2hlY2tfc3lzdGVtKGk4a19kbWlfdGFibGUpIHx8CisJICAgIGRt
aV9jaGVja19zeXN0ZW0oaThrX2JsYWNrbGlzdF9kbWlfdGFibGUpKSB7CiAJCWlmICghaWdub3Jl
X2RtaSAmJiAhZm9yY2UpCiAJCQlyZXR1cm4gLUVOT0RFVjsKIAotLSAKMS43LjkuNQoKCl9fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxtLXNlbnNvcnMgbWFp
bGluZyBsaXN0CmxtLXNlbnNvcnNAbG0tc2Vuc29ycy5vcmcKaHR0cDovL2xpc3RzLmxtLXNlbnNv
cnMub3JnL21haWxtYW4vbGlzdGluZm8vbG0tc2Vuc29ycw=

[PATCH] dell-smm-hwmon: Blacklist Dell Studio XPS 8100

[Pali Rohár]

CPU fan speed going up and down on Dell Studio XPS 8100 for
unknown reason. Without future debuggning on affected machine
it is not possible to detect where is problem. For more see:
https://bugzilla.kernel.org/show_bug.cgi?id=100121

Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
Tested-by: Jan C Peters <jcpeters89@gmail.com>
Cc: stable@vger.kernel.org # before v4.2 apply to file drivers/char/i8k.c
---
 drivers/hwmon/dell-smm-hwmon.c |   20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-smm-hwmon.c
index 37c16af..92c572c 100644
--- a/drivers/hwmon/dell-smm-hwmon.c
+++ b/drivers/hwmon/dell-smm-hwmon.c
@@ -929,6 +929,23 @@ static struct dmi_system_id i8k_dmi_table[] __initdata = {
 
 MODULE_DEVICE_TABLE(dmi, i8k_dmi_table);
 
+static struct dmi_system_id i8k_blacklist_dmi_table[] __initdata = {
+	{
+		/*
+		 * CPU fan speed going up and down on Dell Studio XPS 8100 for
+		 * unknown reason. Without future debuggning on affected machine
+		 * it is not possible to detect where is problem. For more see:
+		 * https://bugzilla.kernel.org/show_bug.cgi?id=100121
+		 */
+		.ident = "Dell Studio XPS 8100",
+		.matches = {
+			DMI_EXACT_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+			DMI_EXACT_MATCH(DMI_PRODUCT_NAME, "Studio XPS 8100"),
+		},
+	},
+	{ }
+};
+
 /*
  * Probe for the presence of a supported laptop.
  */
@@ -940,7 +957,8 @@ static int __init i8k_probe(void)
 	/*
 	 * Get DMI information
 	 */
-	if (!dmi_check_system(i8k_dmi_table)) {
+	if (!dmi_check_system(i8k_dmi_table) ||
+	    dmi_check_system(i8k_blacklist_dmi_table)) {
 		if (!ignore_dmi && !force)
 			return -ENODEV;
 
-- 
1.7.9.5

Re: [lm-sensors] [PATCH] dell-smm-hwmon: Blacklist Dell Studio XPS 8100

[Guenter Roeck]

T24gMDcvMzAvMjAxNSAxMTo0MSBBTSwgUGFsaSBSb2jDoXIgd3JvdGU6Cj4gQ1BVIGZhbiBzcGVl
ZCBnb2luZyB1cCBhbmQgZG93biBvbiBEZWxsIFN0dWRpbyBYUFMgODEwMCBmb3IKPiB1bmtub3du
IHJlYXNvbi4gV2l0aG91dCBmdXR1cmUgZGVidWdnbmluZyBvbiBhZmZlY3RlZCBtYWNoaW5lCj4g
aXQgaXMgbm90IHBvc3NpYmxlIHRvIGRldGVjdCB3aGVyZSBpcyBwcm9ibGVtLiBGb3IgbW9yZSBz
ZWU6Cj4gaHR0cHM6Ly9idWd6aWxsYS5rZXJuZWwub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDAxMjEK
Pgo+IFNpZ25lZC1vZmYtYnk6IFBhbGkgUm9ow6FyIDxwYWxpLnJvaGFyQGdtYWlsLmNvbT4KPiBU
ZXN0ZWQtYnk6IEphbiBDIFBldGVycyA8amNwZXRlcnM4OUBnbWFpbC5jb20+Cj4gQ2M6IHN0YWJs
ZUB2Z2VyLmtlcm5lbC5vcmcgIyBiZWZvcmUgdjQuMiBhcHBseSB0byBmaWxlIGRyaXZlcnMvY2hh
ci9pOGsuYwoKQXBwbGllZCwgYWZ0ZXIgZml4aW5nIHVwIGRlc2NyaXB0aW9uIGFuZCBjb21tZW50
cy4KCkFmdGVyIGl0IGlzIGFwcGxpZWQgdG8gbWFpbmxpbmUsIHlvdSdsbCBwcm9iYWJseSBnZXQg
YSBub3RlCmZyb20gR3JlZywgYXNraW5nIHlvdSB0byBiYWNrLXBvcnQgdGhlIHBhdGNoIHRvIDQu
MS4KClRoYW5rcywKR3VlbnRlcgoKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fCmxtLXNlbnNvcnMgbWFpbGluZyBsaXN0CmxtLXNlbnNvcnNAbG0tc2Vuc29y
cy5vcmcKaHR0cDovL2xpc3RzLmxtLXNlbnNvcnMub3JnL21haWxtYW4vbGlzdGluZm8vbG0tc2Vu
c29ycw=

Re: [PATCH] dell-smm-hwmon: Blacklist Dell Studio XPS 8100

[Guenter Roeck]

On 07/30/2015 11:41 AM, Pali Rohár wrote:
> CPU fan speed going up and down on Dell Studio XPS 8100 for
> unknown reason. Without future debuggning on affected machine
> it is not possible to detect where is problem. For more see:
> https://bugzilla.kernel.org/show_bug.cgi?id=100121
>
> Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
> Tested-by: Jan C Peters <jcpeters89@gmail.com>
> Cc: stable@vger.kernel.org # before v4.2 apply to file drivers/char/i8k.c

Applied, after fixing up description and comments.

After it is applied to mainline, you'll probably get a note
from Greg, asking you to back-port the patch to 4.1.

Thanks,
Guenter