{"comment":{"created_at":"2011-10-26T18:46:03Z","body":"pulled","updated_at":"2011-10-26T18:46:03Z","url":"https://api.github.com/repos/evgeni/tp_smapi/issues/comments/2534394","id":2534394,"user":{"avatar_url":"https://secure.gravatar.com/avatar/5880e0806dddca1eeafbef418ea54077?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"5880e0806dddca1eeafbef418ea54077","url":"https://api.github.com/users/evgeni","id":94284,"login":"evgeni"}},"action":"created","issue":{"number":5,"created_at":"2011-10-26T04:34:26Z","pull_request":{"diff_url":null,"patch_url":null,"html_url":null},"body":"Since some time (2.6.37?) hdaps.c is in drivers/platform/x86/ and not in drivers/hwmon/\r\n\r\njust for reference I've used this patch to the Makefile to \"make patch\" it'll probably break backward compatibility\r\n\r\n```\r\nFrom 87df6f52ad3c1b71444a78078919a2c2e410de29 Mon Sep 17 00:00:00 2001\r\nFrom: Damjan Georgievski <gdamjan@gmail.com>\r\nDate: Wed, 26 Oct 2011 06:01:44 +0200\r\nSubject: [PATCH 1/2] hdaps.c is moved in recent kernels\r\n\r\nSince some time (2.6.38?) hdaps.c is in drivers/platform/x86/ and not in drivers/hwmon/\r\n---\r\n Makefile |   10 +++++-----\r\n hdaps.c  |    2 +-\r\n 2 files changed, 6 insertions(+), 6 deletions(-)\r\n\r\ndiff --git a/Makefile b/Makefile\r\nindex 3d4f876..cc59946 100644\r\n--- a/Makefile\r\n+++ b/Makefile\r\n@@ -73,7 +73,7 @@ install: modules\r\n \trm -f $(MOD_DIR)/drivers/firmware/{thinkpad_ec,tp_smapi,tp_base}.ko\r\n \trm -f $(MOD_DIR)/extra/{thinkpad_ec,tp_smapi,tp_base}.ko\r\n ifeq ($(HDAPS),1)\r\n-\trm -f $(MOD_DIR)/drivers/hwmon/hdaps.ko\r\n+\trm -f $(MOD_DIR)/drivers/platform/x86/hdaps.ko\r\n \trm -f $(MOD_DIR)/extra/hdaps.ko\r\n endif\r\n \t$(MAKE) -C $(KBUILD) M=$(PWD) O=$(KBUILD) modules_install\r\n@@ -98,9 +98,9 @@ patch: $(KSRC)\r\n \tcd $$TMPDIR &&\\\r\n \tmkdir -p $(ORG)/$(TP_DIR) &&\\\r\n \tmkdir -p $(ORG)/$(IDIR) &&\\\r\n-\tmkdir -p $(ORG)/drivers/hwmon &&\\\r\n+\tmkdir -p $(ORG)/drivers/platform/x86 &&\\\r\n \tcp $(KSRC)/$(TP_DIR)/{Kconfig,Makefile} $(ORG)/$(TP_DIR) &&\\\r\n-\tcp $(KSRC)/drivers/hwmon/{Kconfig,hdaps.c} $(ORG)/drivers/hwmon/ &&\\\r\n+\tcp $(KSRC)/drivers/platform/x86/{Kconfig,hdaps.c} $(ORG)/drivers/platform/x86/ &&\\\r\n \tcp -r $(ORG) $(NEW) &&\\\r\n \t\\\r\n \tif [ \"$(BASE_IN_PATCH)\" == 1 ]; then \\\r\n@@ -111,8 +111,8 @@ patch: $(KSRC)\r\n \t; fi &&\\\r\n \t\\\r\n \tif [ \"$(HDAPS_IN_PATCH)\" == 1 ]; then \\\r\n-\t\tcp $(PWD)/hdaps.c $(NEW)/drivers/hwmon/ &&\\\r\n-\t\tperl -i -0777 -pe 's/(config SENSORS_HDAPS\\n\\ttristate [^\\n]+\\n\\tdepends [^\\n]+\\n)/$$1\\tselect THINKPAD_EC\\n/' $(NEW)/drivers/hwmon/Kconfig  \\\r\n+\t\tcp $(PWD)/hdaps.c $(NEW)/drivers/platform/x86/ &&\\\r\n+\t\tperl -i -0777 -pe 's/(config SENSORS_HDAPS\\n\\ttristate [^\\n]+\\n\\tdepends [^\\n]+\\n)/$$1\\tselect THINKPAD_EC\\n/' $(NEW)/drivers/platform/x86/Kconfig  \\\r\n \t; fi &&\\\r\n \t\\\r\n \tif [ \"$(SMAPI_IN_PATCH)\" == 1 ]; then \\\r\ndiff --git a/hdaps.c b/hdaps.c\r\nindex 59ed481..c37d784 100644\r\n--- a/hdaps.c\r\n+++ b/hdaps.c\r\n@@ -1,5 +1,5 @@\r\n /*\r\n- * drivers/hwmon/hdaps.c - driver for IBM's Hard Drive Active Protection System\r\n+ * drivers/platform/x86/hdaps.c - driver for IBM's Hard Drive Active Protection System\r\n  *\r\n  * Copyright (C) 2005 Robert Love <rml@novell.com>\r\n  * Copyright (C) 2005 Jesper Juhl <jesper.juhl@gmail.com>\r\n-- \r\n1.7.7.1\r\n\r\n```","comments":5,"title":"[make patch] hdaps.c is moved in recent kernels","updated_at":"2011-10-26T18:46:03Z","url":"https://api.github.com/repos/evgeni/tp_smapi/issues/5","id":2053224,"assignee":null,"milestone":null,"closed_at":"2011-10-26T18:46:03Z","user":{"avatar_url":"https://secure.gravatar.com/avatar/701767a2c87efd6a3ca22c5292be8956?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"701767a2c87efd6a3ca22c5292be8956","url":"https://api.github.com/users/gdamjan","id":81654,"login":"gdamjan"},"html_url":"https://github.com/evgeni/tp_smapi/issues/5","labels":[],"state":"closed"},"legacy":{"issue_id":2053224,"comment_id":2534394}}