hwmon: (asus_atk0110) fix uninitialized data access
[pandora-kernel.git] / drivers / hwmon / asus_atk0110.c
1 /*
2  * Copyright (C) 2007-2009 Luca Tettamanti <kronos.it@gmail.com>
3  *
4  * This file is released under the GPLv2
5  * See COPYING in the top level directory of the kernel tree.
6  */
7
8 #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
9
10 #include <linux/debugfs.h>
11 #include <linux/kernel.h>
12 #include <linux/hwmon.h>
13 #include <linux/list.h>
14 #include <linux/module.h>
15 #include <linux/slab.h>
16 #include <linux/dmi.h>
17
18 #include <acpi/acpi.h>
19 #include <acpi/acpixf.h>
20 #include <acpi/acpi_drivers.h>
21 #include <acpi/acpi_bus.h>
22
23
24 #define ATK_HID "ATK0110"
25
26 static bool new_if;
27 module_param(new_if, bool, 0);
28 MODULE_PARM_DESC(new_if, "Override detection heuristic and force the use of the new ATK0110 interface");
29
30 static const struct dmi_system_id __initconst atk_force_new_if[] = {
31         {
32                 /* Old interface has broken MCH temp monitoring */
33                 .ident = "Asus Sabertooth X58",
34                 .matches = {
35                         DMI_MATCH(DMI_BOARD_NAME, "SABERTOOTH X58")
36                 }
37         }, {
38                 /* Old interface reads the same sensor for fan0 and fan1 */
39                 .ident = "Asus M5A78L",
40                 .matches = {
41                         DMI_MATCH(DMI_BOARD_NAME, "M5A78L")
42                 }
43         },
44         { }
45 };
46
47 /* Minimum time between readings, enforced in order to avoid
48  * hogging the CPU.
49  */
50 #define CACHE_TIME              HZ
51
52 #define BOARD_ID                "MBIF"
53 #define METHOD_ENUMERATE        "GGRP"
54 #define METHOD_READ             "GITM"
55 #define METHOD_WRITE            "SITM"
56 #define METHOD_OLD_READ_TMP     "RTMP"
57 #define METHOD_OLD_READ_VLT     "RVLT"
58 #define METHOD_OLD_READ_FAN     "RFAN"
59 #define METHOD_OLD_ENUM_TMP     "TSIF"
60 #define METHOD_OLD_ENUM_VLT     "VSIF"
61 #define METHOD_OLD_ENUM_FAN     "FSIF"
62
63 #define ATK_MUX_HWMON           0x00000006ULL
64 #define ATK_MUX_MGMT            0x00000011ULL
65
66 #define ATK_CLASS_MASK          0xff000000ULL
67 #define ATK_CLASS_FREQ_CTL      0x03000000ULL
68 #define ATK_CLASS_FAN_CTL       0x04000000ULL
69 #define ATK_CLASS_HWMON         0x06000000ULL
70 #define ATK_CLASS_MGMT          0x11000000ULL
71
72 #define ATK_TYPE_MASK           0x00ff0000ULL
73 #define HWMON_TYPE_VOLT         0x00020000ULL
74 #define HWMON_TYPE_TEMP         0x00030000ULL
75 #define HWMON_TYPE_FAN          0x00040000ULL
76
77 #define ATK_ELEMENT_ID_MASK     0x0000ffffULL
78
79 #define ATK_EC_ID               0x11060004ULL
80
81 enum atk_pack_member {
82         HWMON_PACK_FLAGS,
83         HWMON_PACK_NAME,
84         HWMON_PACK_LIMIT1,
85         HWMON_PACK_LIMIT2,
86         HWMON_PACK_ENABLE
87 };
88
89 /* New package format */
90 #define _HWMON_NEW_PACK_SIZE    7
91 #define _HWMON_NEW_PACK_FLAGS   0
92 #define _HWMON_NEW_PACK_NAME    1
93 #define _HWMON_NEW_PACK_UNK1    2
94 #define _HWMON_NEW_PACK_UNK2    3
95 #define _HWMON_NEW_PACK_LIMIT1  4
96 #define _HWMON_NEW_PACK_LIMIT2  5
97 #define _HWMON_NEW_PACK_ENABLE  6
98
99 /* Old package format */
100 #define _HWMON_OLD_PACK_SIZE    5
101 #define _HWMON_OLD_PACK_FLAGS   0
102 #define _HWMON_OLD_PACK_NAME    1
103 #define _HWMON_OLD_PACK_LIMIT1  2
104 #define _HWMON_OLD_PACK_LIMIT2  3
105 #define _HWMON_OLD_PACK_ENABLE  4
106
107
108 struct atk_data {
109         struct device *hwmon_dev;
110         acpi_handle atk_handle;
111         struct acpi_device *acpi_dev;
112
113         bool old_interface;
114
115         /* old interface */
116         acpi_handle rtmp_handle;
117         acpi_handle rvlt_handle;
118         acpi_handle rfan_handle;
119         /* new inteface */
120         acpi_handle enumerate_handle;
121         acpi_handle read_handle;
122         acpi_handle write_handle;
123
124         bool disable_ec;
125
126         int voltage_count;
127         int temperature_count;
128         int fan_count;
129         struct list_head sensor_list;
130
131         struct {
132                 struct dentry *root;
133                 u32 id;
134         } debugfs;
135 };
136
137
138 typedef ssize_t (*sysfs_show_func)(struct device *dev,
139                         struct device_attribute *attr, char *buf);
140
141 static const struct acpi_device_id atk_ids[] = {
142         {ATK_HID, 0},
143         {"", 0},
144 };
145 MODULE_DEVICE_TABLE(acpi, atk_ids);
146
147 #define ATTR_NAME_SIZE 16 /* Worst case is "tempN_input" */
148
149 struct atk_sensor_data {
150         struct list_head list;
151         struct atk_data *data;
152         struct device_attribute label_attr;
153         struct device_attribute input_attr;
154         struct device_attribute limit1_attr;
155         struct device_attribute limit2_attr;
156         char label_attr_name[ATTR_NAME_SIZE];
157         char input_attr_name[ATTR_NAME_SIZE];
158         char limit1_attr_name[ATTR_NAME_SIZE];
159         char limit2_attr_name[ATTR_NAME_SIZE];
160         u64 id;
161         u64 type;
162         u64 limit1;
163         u64 limit2;
164         u64 cached_value;
165         unsigned long last_updated; /* in jiffies */
166         bool is_valid;
167         char const *acpi_name;
168 };
169
170 /* Return buffer format:
171  * [0-3] "value" is valid flag
172  * [4-7] value
173  * [8- ] unknown stuff on newer mobos
174  */
175 struct atk_acpi_ret_buffer {
176         u32 flags;
177         u32 value;
178         u8 data[];
179 };
180
181 /* Input buffer used for GITM and SITM methods */
182 struct atk_acpi_input_buf {
183         u32 id;
184         u32 param1;
185         u32 param2;
186 };
187
188 static int atk_add(struct acpi_device *device);
189 static int atk_remove(struct acpi_device *device, int type);
190 static void atk_print_sensor(struct atk_data *data, union acpi_object *obj);
191 static int atk_read_value(struct atk_sensor_data *sensor, u64 *value);
192 static void atk_free_sensors(struct atk_data *data);
193
194 static struct acpi_driver atk_driver = {
195         .name   = ATK_HID,
196         .class  = "hwmon",
197         .ids    = atk_ids,
198         .ops    = {
199                 .add    = atk_add,
200                 .remove = atk_remove,
201         },
202 };
203
204 #define input_to_atk_sensor(attr) \
205         container_of(attr, struct atk_sensor_data, input_attr)
206
207 #define label_to_atk_sensor(attr) \
208         container_of(attr, struct atk_sensor_data, label_attr)
209
210 #define limit1_to_atk_sensor(attr) \
211         container_of(attr, struct atk_sensor_data, limit1_attr)
212
213 #define limit2_to_atk_sensor(attr) \
214         container_of(attr, struct atk_sensor_data, limit2_attr)
215
216 static ssize_t atk_input_show(struct device *dev,
217                 struct device_attribute *attr, char *buf)
218 {
219         struct atk_sensor_data *s = input_to_atk_sensor(attr);
220         u64 value;
221         int err;
222
223         err = atk_read_value(s, &value);
224         if (err)
225                 return err;
226
227         if (s->type == HWMON_TYPE_TEMP)
228                 /* ACPI returns decidegree */
229                 value *= 100;
230
231         return sprintf(buf, "%llu\n", value);
232 }
233
234 static ssize_t atk_label_show(struct device *dev,
235                 struct device_attribute *attr, char *buf)
236 {
237         struct atk_sensor_data *s = label_to_atk_sensor(attr);
238
239         return sprintf(buf, "%s\n", s->acpi_name);
240 }
241
242 static ssize_t atk_limit1_show(struct device *dev,
243                 struct device_attribute *attr, char *buf)
244 {
245         struct atk_sensor_data *s = limit1_to_atk_sensor(attr);
246         u64 value = s->limit1;
247
248         if (s->type == HWMON_TYPE_TEMP)
249                 value *= 100;
250
251         return sprintf(buf, "%lld\n", value);
252 }
253
254 static ssize_t atk_limit2_show(struct device *dev,
255                 struct device_attribute *attr, char *buf)
256 {
257         struct atk_sensor_data *s = limit2_to_atk_sensor(attr);
258         u64 value = s->limit2;
259
260         if (s->type == HWMON_TYPE_TEMP)
261                 value *= 100;
262
263         return sprintf(buf, "%lld\n", value);
264 }
265
266 static ssize_t atk_name_show(struct device *dev,
267                                 struct device_attribute *attr, char *buf)
268 {
269         return sprintf(buf, "atk0110\n");
270 }
271 static struct device_attribute atk_name_attr =
272                 __ATTR(name, 0444, atk_name_show, NULL);
273
274 static void atk_init_attribute(struct device_attribute *attr, char *name,
275                 sysfs_show_func show)
276 {
277         sysfs_attr_init(&attr->attr);
278         attr->attr.name = name;
279         attr->attr.mode = 0444;
280         attr->show = show;
281         attr->store = NULL;
282 }
283
284
285 static union acpi_object *atk_get_pack_member(struct atk_data *data,
286                                                 union acpi_object *pack,
287                                                 enum atk_pack_member m)
288 {
289         bool old_if = data->old_interface;
290         int offset;
291
292         switch (m) {
293         case HWMON_PACK_FLAGS:
294                 offset = old_if ? _HWMON_OLD_PACK_FLAGS : _HWMON_NEW_PACK_FLAGS;
295                 break;
296         case HWMON_PACK_NAME:
297                 offset = old_if ? _HWMON_OLD_PACK_NAME : _HWMON_NEW_PACK_NAME;
298                 break;
299         case HWMON_PACK_LIMIT1:
300                 offset = old_if ? _HWMON_OLD_PACK_LIMIT1 :
301                                   _HWMON_NEW_PACK_LIMIT1;
302                 break;
303         case HWMON_PACK_LIMIT2:
304                 offset = old_if ? _HWMON_OLD_PACK_LIMIT2 :
305                                   _HWMON_NEW_PACK_LIMIT2;
306                 break;
307         case HWMON_PACK_ENABLE:
308                 offset = old_if ? _HWMON_OLD_PACK_ENABLE :
309                                   _HWMON_NEW_PACK_ENABLE;
310                 break;
311         default:
312                 return NULL;
313         }
314
315         return &pack->package.elements[offset];
316 }
317
318
319 /* New package format is:
320  * - flag (int)
321  *      class - used for de-muxing the request to the correct GITn
322  *      type (volt, temp, fan)
323  *      sensor id |
324  *      sensor id - used for de-muxing the request _inside_ the GITn
325  * - name (str)
326  * - unknown (int)
327  * - unknown (int)
328  * - limit1 (int)
329  * - limit2 (int)
330  * - enable (int)
331  *
332  * The old package has the same format but it's missing the two unknown fields.
333  */
334 static int validate_hwmon_pack(struct atk_data *data, union acpi_object *obj)
335 {
336         struct device *dev = &data->acpi_dev->dev;
337         union acpi_object *tmp;
338         bool old_if = data->old_interface;
339         int const expected_size = old_if ? _HWMON_OLD_PACK_SIZE :
340                                            _HWMON_NEW_PACK_SIZE;
341
342         if (obj->type != ACPI_TYPE_PACKAGE) {
343                 dev_warn(dev, "Invalid type: %d\n", obj->type);
344                 return -EINVAL;
345         }
346
347         if (obj->package.count != expected_size) {
348                 dev_warn(dev, "Invalid package size: %d, expected: %d\n",
349                                 obj->package.count, expected_size);
350                 return -EINVAL;
351         }
352
353         tmp = atk_get_pack_member(data, obj, HWMON_PACK_FLAGS);
354         if (tmp->type != ACPI_TYPE_INTEGER) {
355                 dev_warn(dev, "Invalid type (flag): %d\n", tmp->type);
356                 return -EINVAL;
357         }
358
359         tmp = atk_get_pack_member(data, obj, HWMON_PACK_NAME);
360         if (tmp->type != ACPI_TYPE_STRING) {
361                 dev_warn(dev, "Invalid type (name): %d\n", tmp->type);
362                 return -EINVAL;
363         }
364
365         /* Don't check... we don't know what they're useful for anyway */
366 #if 0
367         tmp = &obj->package.elements[HWMON_PACK_UNK1];
368         if (tmp->type != ACPI_TYPE_INTEGER) {
369                 dev_warn(dev, "Invalid type (unk1): %d\n", tmp->type);
370                 return -EINVAL;
371         }
372
373         tmp = &obj->package.elements[HWMON_PACK_UNK2];
374         if (tmp->type != ACPI_TYPE_INTEGER) {
375                 dev_warn(dev, "Invalid type (unk2): %d\n", tmp->type);
376                 return -EINVAL;
377         }
378 #endif
379
380         tmp = atk_get_pack_member(data, obj, HWMON_PACK_LIMIT1);
381         if (tmp->type != ACPI_TYPE_INTEGER) {
382                 dev_warn(dev, "Invalid type (limit1): %d\n", tmp->type);
383                 return -EINVAL;
384         }
385
386         tmp = atk_get_pack_member(data, obj, HWMON_PACK_LIMIT2);
387         if (tmp->type != ACPI_TYPE_INTEGER) {
388                 dev_warn(dev, "Invalid type (limit2): %d\n", tmp->type);
389                 return -EINVAL;
390         }
391
392         tmp = atk_get_pack_member(data, obj, HWMON_PACK_ENABLE);
393         if (tmp->type != ACPI_TYPE_INTEGER) {
394                 dev_warn(dev, "Invalid type (enable): %d\n", tmp->type);
395                 return -EINVAL;
396         }
397
398         atk_print_sensor(data, obj);
399
400         return 0;
401 }
402
403 #ifdef DEBUG
404 static char const *atk_sensor_type(union acpi_object *flags)
405 {
406         u64 type = flags->integer.value & ATK_TYPE_MASK;
407         char const *what;
408
409         switch (type) {
410         case HWMON_TYPE_VOLT:
411                 what = "voltage";
412                 break;
413         case HWMON_TYPE_TEMP:
414                 what = "temperature";
415                 break;
416         case HWMON_TYPE_FAN:
417                 what = "fan";
418                 break;
419         default:
420                 what = "unknown";
421                 break;
422         }
423
424         return what;
425 }
426 #endif
427
428 static void atk_print_sensor(struct atk_data *data, union acpi_object *obj)
429 {
430 #ifdef DEBUG
431         struct device *dev = &data->acpi_dev->dev;
432         union acpi_object *flags;
433         union acpi_object *name;
434         union acpi_object *limit1;
435         union acpi_object *limit2;
436         union acpi_object *enable;
437         char const *what;
438
439         flags = atk_get_pack_member(data, obj, HWMON_PACK_FLAGS);
440         name = atk_get_pack_member(data, obj, HWMON_PACK_NAME);
441         limit1 = atk_get_pack_member(data, obj, HWMON_PACK_LIMIT1);
442         limit2 = atk_get_pack_member(data, obj, HWMON_PACK_LIMIT2);
443         enable = atk_get_pack_member(data, obj, HWMON_PACK_ENABLE);
444
445         what = atk_sensor_type(flags);
446
447         dev_dbg(dev, "%s: %#llx %s [%llu-%llu] %s\n", what,
448                         flags->integer.value,
449                         name->string.pointer,
450                         limit1->integer.value, limit2->integer.value,
451                         enable->integer.value ? "enabled" : "disabled");
452 #endif
453 }
454
455 static int atk_read_value_old(struct atk_sensor_data *sensor, u64 *value)
456 {
457         struct atk_data *data = sensor->data;
458         struct device *dev = &data->acpi_dev->dev;
459         struct acpi_object_list params;
460         union acpi_object id;
461         acpi_status status;
462         acpi_handle method;
463
464         switch (sensor->type) {
465         case HWMON_TYPE_VOLT:
466                 method = data->rvlt_handle;
467                 break;
468         case HWMON_TYPE_TEMP:
469                 method = data->rtmp_handle;
470                 break;
471         case HWMON_TYPE_FAN:
472                 method = data->rfan_handle;
473                 break;
474         default:
475                 return -EINVAL;
476         }
477
478         id.type = ACPI_TYPE_INTEGER;
479         id.integer.value = sensor->id;
480
481         params.count = 1;
482         params.pointer = &id;
483
484         status = acpi_evaluate_integer(method, NULL, &params, value);
485         if (status != AE_OK) {
486                 dev_warn(dev, "%s: ACPI exception: %s\n", __func__,
487                                 acpi_format_exception(status));
488                 return -EIO;
489         }
490
491         return 0;
492 }
493
494 static union acpi_object *atk_ggrp(struct atk_data *data, u16 mux)
495 {
496         struct device *dev = &data->acpi_dev->dev;
497         struct acpi_buffer buf;
498         acpi_status ret;
499         struct acpi_object_list params;
500         union acpi_object id;
501         union acpi_object *pack;
502
503         id.type = ACPI_TYPE_INTEGER;
504         id.integer.value = mux;
505         params.count = 1;
506         params.pointer = &id;
507
508         buf.length = ACPI_ALLOCATE_BUFFER;
509         ret = acpi_evaluate_object(data->enumerate_handle, NULL, &params, &buf);
510         if (ret != AE_OK) {
511                 dev_err(dev, "GGRP[%#x] ACPI exception: %s\n", mux,
512                                 acpi_format_exception(ret));
513                 return ERR_PTR(-EIO);
514         }
515         pack = buf.pointer;
516         if (pack->type != ACPI_TYPE_PACKAGE) {
517                 /* Execution was successful, but the id was not found */
518                 ACPI_FREE(pack);
519                 return ERR_PTR(-ENOENT);
520         }
521
522         if (pack->package.count < 1) {
523                 dev_err(dev, "GGRP[%#x] package is too small\n", mux);
524                 ACPI_FREE(pack);
525                 return ERR_PTR(-EIO);
526         }
527         return pack;
528 }
529
530 static union acpi_object *atk_gitm(struct atk_data *data, u64 id)
531 {
532         struct device *dev = &data->acpi_dev->dev;
533         struct atk_acpi_input_buf buf;
534         union acpi_object tmp;
535         struct acpi_object_list params;
536         struct acpi_buffer ret;
537         union acpi_object *obj;
538         acpi_status status;
539
540         buf.id = id;
541         buf.param1 = 0;
542         buf.param2 = 0;
543
544         tmp.type = ACPI_TYPE_BUFFER;
545         tmp.buffer.pointer = (u8 *)&buf;
546         tmp.buffer.length = sizeof(buf);
547
548         params.count = 1;
549         params.pointer = (void *)&tmp;
550
551         ret.length = ACPI_ALLOCATE_BUFFER;
552         status = acpi_evaluate_object_typed(data->read_handle, NULL, &params,
553                         &ret, ACPI_TYPE_BUFFER);
554         if (status != AE_OK) {
555                 dev_warn(dev, "GITM[%#llx] ACPI exception: %s\n", id,
556                                 acpi_format_exception(status));
557                 return ERR_PTR(-EIO);
558         }
559         obj = ret.pointer;
560
561         /* Sanity check */
562         if (obj->buffer.length < 8) {
563                 dev_warn(dev, "Unexpected ASBF length: %u\n",
564                                 obj->buffer.length);
565                 ACPI_FREE(obj);
566                 return ERR_PTR(-EIO);
567         }
568         return obj;
569 }
570
571 static union acpi_object *atk_sitm(struct atk_data *data,
572                 struct atk_acpi_input_buf *buf)
573 {
574         struct device *dev = &data->acpi_dev->dev;
575         struct acpi_object_list params;
576         union acpi_object tmp;
577         struct acpi_buffer ret;
578         union acpi_object *obj;
579         acpi_status status;
580
581         tmp.type = ACPI_TYPE_BUFFER;
582         tmp.buffer.pointer = (u8 *)buf;
583         tmp.buffer.length = sizeof(*buf);
584
585         params.count = 1;
586         params.pointer = &tmp;
587
588         ret.length = ACPI_ALLOCATE_BUFFER;
589         status = acpi_evaluate_object_typed(data->write_handle, NULL, &params,
590                         &ret, ACPI_TYPE_BUFFER);
591         if (status != AE_OK) {
592                 dev_warn(dev, "SITM[%#x] ACPI exception: %s\n", buf->id,
593                                 acpi_format_exception(status));
594                 return ERR_PTR(-EIO);
595         }
596         obj = ret.pointer;
597
598         /* Sanity check */
599         if (obj->buffer.length < 8) {
600                 dev_warn(dev, "Unexpected ASBF length: %u\n",
601                                 obj->buffer.length);
602                 ACPI_FREE(obj);
603                 return ERR_PTR(-EIO);
604         }
605         return obj;
606 }
607
608 static int atk_read_value_new(struct atk_sensor_data *sensor, u64 *value)
609 {
610         struct atk_data *data = sensor->data;
611         struct device *dev = &data->acpi_dev->dev;
612         union acpi_object *obj;
613         struct atk_acpi_ret_buffer *buf;
614         int err = 0;
615
616         obj = atk_gitm(data, sensor->id);
617         if (IS_ERR(obj))
618                 return PTR_ERR(obj);
619
620         buf = (struct atk_acpi_ret_buffer *)obj->buffer.pointer;
621         if (buf->flags == 0) {
622                 /* The reading is not valid, possible causes:
623                  * - sensor failure
624                  * - enumeration was FUBAR (and we didn't notice)
625                  */
626                 dev_warn(dev, "Read failed, sensor = %#llx\n", sensor->id);
627                 err = -EIO;
628                 goto out;
629         }
630
631         *value = buf->value;
632 out:
633         ACPI_FREE(obj);
634         return err;
635 }
636
637 static int atk_read_value(struct atk_sensor_data *sensor, u64 *value)
638 {
639         int err;
640
641         if (!sensor->is_valid ||
642             time_after(jiffies, sensor->last_updated + CACHE_TIME)) {
643                 if (sensor->data->old_interface)
644                         err = atk_read_value_old(sensor, value);
645                 else
646                         err = atk_read_value_new(sensor, value);
647
648                 if (err)
649                         return err;
650
651                 sensor->is_valid = true;
652                 sensor->last_updated = jiffies;
653                 sensor->cached_value = *value;
654         } else {
655                 *value = sensor->cached_value;
656                 err = 0;
657         }
658
659         return err;
660 }
661
662 #ifdef CONFIG_DEBUG_FS
663 static int atk_debugfs_gitm_get(void *p, u64 *val)
664 {
665         struct atk_data *data = p;
666         union acpi_object *ret;
667         struct atk_acpi_ret_buffer *buf;
668         int err = 0;
669
670         if (!data->read_handle)
671                 return -ENODEV;
672
673         if (!data->debugfs.id)
674                 return -EINVAL;
675
676         ret = atk_gitm(data, data->debugfs.id);
677         if (IS_ERR(ret))
678                 return PTR_ERR(ret);
679
680         buf = (struct atk_acpi_ret_buffer *)ret->buffer.pointer;
681         if (buf->flags)
682                 *val = buf->value;
683         else
684                 err = -EIO;
685
686         ACPI_FREE(ret);
687         return err;
688 }
689
690 DEFINE_SIMPLE_ATTRIBUTE(atk_debugfs_gitm,
691                         atk_debugfs_gitm_get,
692                         NULL,
693                         "0x%08llx\n")
694
695 static int atk_acpi_print(char *buf, size_t sz, union acpi_object *obj)
696 {
697         int ret = 0;
698
699         switch (obj->type) {
700         case ACPI_TYPE_INTEGER:
701                 ret = snprintf(buf, sz, "0x%08llx\n", obj->integer.value);
702                 break;
703         case ACPI_TYPE_STRING:
704                 ret = snprintf(buf, sz, "%s\n", obj->string.pointer);
705                 break;
706         }
707
708         return ret;
709 }
710
711 static void atk_pack_print(char *buf, size_t sz, union acpi_object *pack)
712 {
713         int ret;
714         int i;
715
716         for (i = 0; i < pack->package.count; i++) {
717                 union acpi_object *obj = &pack->package.elements[i];
718
719                 ret = atk_acpi_print(buf, sz, obj);
720                 if (ret >= sz)
721                         break;
722                 buf += ret;
723                 sz -= ret;
724         }
725 }
726
727 static int atk_debugfs_ggrp_open(struct inode *inode, struct file *file)
728 {
729         struct atk_data *data = inode->i_private;
730         char *buf = NULL;
731         union acpi_object *ret;
732         u8 cls;
733         int i;
734
735         if (!data->enumerate_handle)
736                 return -ENODEV;
737         if (!data->debugfs.id)
738                 return -EINVAL;
739
740         cls = (data->debugfs.id & 0xff000000) >> 24;
741         ret = atk_ggrp(data, cls);
742         if (IS_ERR(ret))
743                 return PTR_ERR(ret);
744
745         for (i = 0; i < ret->package.count; i++) {
746                 union acpi_object *pack = &ret->package.elements[i];
747                 union acpi_object *id;
748
749                 if (pack->type != ACPI_TYPE_PACKAGE)
750                         continue;
751                 if (!pack->package.count)
752                         continue;
753                 id = &pack->package.elements[0];
754                 if (id->integer.value == data->debugfs.id) {
755                         /* Print the package */
756                         buf = kzalloc(512, GFP_KERNEL);
757                         if (!buf) {
758                                 ACPI_FREE(ret);
759                                 return -ENOMEM;
760                         }
761                         atk_pack_print(buf, 512, pack);
762                         break;
763                 }
764         }
765         ACPI_FREE(ret);
766
767         if (!buf)
768                 return -EINVAL;
769
770         file->private_data = buf;
771
772         return nonseekable_open(inode, file);
773 }
774
775 static ssize_t atk_debugfs_ggrp_read(struct file *file, char __user *buf,
776                 size_t count, loff_t *pos)
777 {
778         char *str = file->private_data;
779         size_t len = strlen(str);
780
781         return simple_read_from_buffer(buf, count, pos, str, len);
782 }
783
784 static int atk_debugfs_ggrp_release(struct inode *inode, struct file *file)
785 {
786         kfree(file->private_data);
787         return 0;
788 }
789
790 static const struct file_operations atk_debugfs_ggrp_fops = {
791         .read           = atk_debugfs_ggrp_read,
792         .open           = atk_debugfs_ggrp_open,
793         .release        = atk_debugfs_ggrp_release,
794         .llseek         = no_llseek,
795 };
796
797 static void atk_debugfs_init(struct atk_data *data)
798 {
799         struct dentry *d;
800         struct dentry *f;
801
802         data->debugfs.id = 0;
803
804         d = debugfs_create_dir("asus_atk0110", NULL);
805         if (!d || IS_ERR(d))
806                 return;
807
808         f = debugfs_create_x32("id", S_IRUSR | S_IWUSR, d, &data->debugfs.id);
809         if (!f || IS_ERR(f))
810                 goto cleanup;
811
812         f = debugfs_create_file("gitm", S_IRUSR, d, data,
813                         &atk_debugfs_gitm);
814         if (!f || IS_ERR(f))
815                 goto cleanup;
816
817         f = debugfs_create_file("ggrp", S_IRUSR, d, data,
818                         &atk_debugfs_ggrp_fops);
819         if (!f || IS_ERR(f))
820                 goto cleanup;
821
822         data->debugfs.root = d;
823
824         return;
825 cleanup:
826         debugfs_remove_recursive(d);
827 }
828
829 static void atk_debugfs_cleanup(struct atk_data *data)
830 {
831         debugfs_remove_recursive(data->debugfs.root);
832 }
833
834 #else /* CONFIG_DEBUG_FS */
835
836 static void atk_debugfs_init(struct atk_data *data)
837 {
838 }
839
840 static void atk_debugfs_cleanup(struct atk_data *data)
841 {
842 }
843 #endif
844
845 static int atk_add_sensor(struct atk_data *data, union acpi_object *obj)
846 {
847         struct device *dev = &data->acpi_dev->dev;
848         union acpi_object *flags;
849         union acpi_object *name;
850         union acpi_object *limit1;
851         union acpi_object *limit2;
852         union acpi_object *enable;
853         struct atk_sensor_data *sensor;
854         char const *base_name;
855         char const *limit1_name;
856         char const *limit2_name;
857         u64 type;
858         int err;
859         int *num;
860         int start;
861
862         if (obj->type != ACPI_TYPE_PACKAGE) {
863                 /* wft is this? */
864                 dev_warn(dev, "Unknown type for ACPI object: (%d)\n",
865                                 obj->type);
866                 return -EINVAL;
867         }
868
869         err = validate_hwmon_pack(data, obj);
870         if (err)
871                 return err;
872
873         /* Ok, we have a valid hwmon package */
874         type = atk_get_pack_member(data, obj, HWMON_PACK_FLAGS)->integer.value
875                & ATK_TYPE_MASK;
876
877         switch (type) {
878         case HWMON_TYPE_VOLT:
879                 base_name = "in";
880                 limit1_name = "min";
881                 limit2_name = "max";
882                 num = &data->voltage_count;
883                 start = 0;
884                 break;
885         case HWMON_TYPE_TEMP:
886                 base_name = "temp";
887                 limit1_name = "max";
888                 limit2_name = "crit";
889                 num = &data->temperature_count;
890                 start = 1;
891                 break;
892         case HWMON_TYPE_FAN:
893                 base_name = "fan";
894                 limit1_name = "min";
895                 limit2_name = "max";
896                 num = &data->fan_count;
897                 start = 1;
898                 break;
899         default:
900                 dev_warn(dev, "Unknown sensor type: %#llx\n", type);
901                 return -EINVAL;
902         }
903
904         enable = atk_get_pack_member(data, obj, HWMON_PACK_ENABLE);
905         if (!enable->integer.value)
906                 /* sensor is disabled */
907                 return 0;
908
909         flags = atk_get_pack_member(data, obj, HWMON_PACK_FLAGS);
910         name = atk_get_pack_member(data, obj, HWMON_PACK_NAME);
911         limit1 = atk_get_pack_member(data, obj, HWMON_PACK_LIMIT1);
912         limit2 = atk_get_pack_member(data, obj, HWMON_PACK_LIMIT2);
913
914         sensor = kzalloc(sizeof(*sensor), GFP_KERNEL);
915         if (!sensor)
916                 return -ENOMEM;
917
918         sensor->acpi_name = kstrdup(name->string.pointer, GFP_KERNEL);
919         if (!sensor->acpi_name) {
920                 err = -ENOMEM;
921                 goto out;
922         }
923
924         INIT_LIST_HEAD(&sensor->list);
925         sensor->type = type;
926         sensor->data = data;
927         sensor->id = flags->integer.value;
928         sensor->limit1 = limit1->integer.value;
929         if (data->old_interface)
930                 sensor->limit2 = limit2->integer.value;
931         else
932                 /* The upper limit is expressed as delta from lower limit */
933                 sensor->limit2 = sensor->limit1 + limit2->integer.value;
934
935         snprintf(sensor->input_attr_name, ATTR_NAME_SIZE,
936                         "%s%d_input", base_name, start + *num);
937         atk_init_attribute(&sensor->input_attr,
938                         sensor->input_attr_name,
939                         atk_input_show);
940
941         snprintf(sensor->label_attr_name, ATTR_NAME_SIZE,
942                         "%s%d_label", base_name, start + *num);
943         atk_init_attribute(&sensor->label_attr,
944                         sensor->label_attr_name,
945                         atk_label_show);
946
947         snprintf(sensor->limit1_attr_name, ATTR_NAME_SIZE,
948                         "%s%d_%s", base_name, start + *num, limit1_name);
949         atk_init_attribute(&sensor->limit1_attr,
950                         sensor->limit1_attr_name,
951                         atk_limit1_show);
952
953         snprintf(sensor->limit2_attr_name, ATTR_NAME_SIZE,
954                         "%s%d_%s", base_name, start + *num, limit2_name);
955         atk_init_attribute(&sensor->limit2_attr,
956                         sensor->limit2_attr_name,
957                         atk_limit2_show);
958
959         list_add(&sensor->list, &data->sensor_list);
960         (*num)++;
961
962         return 1;
963 out:
964         kfree(sensor->acpi_name);
965         kfree(sensor);
966         return err;
967 }
968
969 static int atk_enumerate_old_hwmon(struct atk_data *data)
970 {
971         struct device *dev = &data->acpi_dev->dev;
972         struct acpi_buffer buf;
973         union acpi_object *pack;
974         acpi_status status;
975         int i, ret;
976         int count = 0;
977
978         /* Voltages */
979         buf.length = ACPI_ALLOCATE_BUFFER;
980         status = acpi_evaluate_object_typed(data->atk_handle,
981                         METHOD_OLD_ENUM_VLT, NULL, &buf, ACPI_TYPE_PACKAGE);
982         if (status != AE_OK) {
983                 dev_warn(dev, METHOD_OLD_ENUM_VLT ": ACPI exception: %s\n",
984                                 acpi_format_exception(status));
985
986                 return -ENODEV;
987         }
988
989         pack = buf.pointer;
990         for (i = 1; i < pack->package.count; i++) {
991                 union acpi_object *obj = &pack->package.elements[i];
992
993                 ret = atk_add_sensor(data, obj);
994                 if (ret > 0)
995                         count++;
996         }
997         ACPI_FREE(buf.pointer);
998
999         /* Temperatures */
1000         buf.length = ACPI_ALLOCATE_BUFFER;
1001         status = acpi_evaluate_object_typed(data->atk_handle,
1002                         METHOD_OLD_ENUM_TMP, NULL, &buf, ACPI_TYPE_PACKAGE);
1003         if (status != AE_OK) {
1004                 dev_warn(dev, METHOD_OLD_ENUM_TMP ": ACPI exception: %s\n",
1005                                 acpi_format_exception(status));
1006
1007                 ret = -ENODEV;
1008                 goto cleanup;
1009         }
1010
1011         pack = buf.pointer;
1012         for (i = 1; i < pack->package.count; i++) {
1013                 union acpi_object *obj = &pack->package.elements[i];
1014
1015                 ret = atk_add_sensor(data, obj);
1016                 if (ret > 0)
1017                         count++;
1018         }
1019         ACPI_FREE(buf.pointer);
1020
1021         /* Fans */
1022         buf.length = ACPI_ALLOCATE_BUFFER;
1023         status = acpi_evaluate_object_typed(data->atk_handle,
1024                         METHOD_OLD_ENUM_FAN, NULL, &buf, ACPI_TYPE_PACKAGE);
1025         if (status != AE_OK) {
1026                 dev_warn(dev, METHOD_OLD_ENUM_FAN ": ACPI exception: %s\n",
1027                                 acpi_format_exception(status));
1028
1029                 ret = -ENODEV;
1030                 goto cleanup;
1031         }
1032
1033         pack = buf.pointer;
1034         for (i = 1; i < pack->package.count; i++) {
1035                 union acpi_object *obj = &pack->package.elements[i];
1036
1037                 ret = atk_add_sensor(data, obj);
1038                 if (ret > 0)
1039                         count++;
1040         }
1041         ACPI_FREE(buf.pointer);
1042
1043         return count;
1044 cleanup:
1045         atk_free_sensors(data);
1046         return ret;
1047 }
1048
1049 static int atk_ec_present(struct atk_data *data)
1050 {
1051         struct device *dev = &data->acpi_dev->dev;
1052         union acpi_object *pack;
1053         union acpi_object *ec;
1054         int ret;
1055         int i;
1056
1057         pack = atk_ggrp(data, ATK_MUX_MGMT);
1058         if (IS_ERR(pack)) {
1059                 if (PTR_ERR(pack) == -ENOENT) {
1060                         /* The MGMT class does not exists - that's ok */
1061                         dev_dbg(dev, "Class %#llx not found\n", ATK_MUX_MGMT);
1062                         return 0;
1063                 }
1064                 return PTR_ERR(pack);
1065         }
1066
1067         /* Search the EC */
1068         ec = NULL;
1069         for (i = 0; i < pack->package.count; i++) {
1070                 union acpi_object *obj = &pack->package.elements[i];
1071                 union acpi_object *id;
1072
1073                 if (obj->type != ACPI_TYPE_PACKAGE)
1074                         continue;
1075
1076                 id = &obj->package.elements[0];
1077                 if (id->type != ACPI_TYPE_INTEGER)
1078                         continue;
1079
1080                 if (id->integer.value == ATK_EC_ID) {
1081                         ec = obj;
1082                         break;
1083                 }
1084         }
1085
1086         ret = (ec != NULL);
1087         if (!ret)
1088                 /* The system has no EC */
1089                 dev_dbg(dev, "EC not found\n");
1090
1091         ACPI_FREE(pack);
1092         return ret;
1093 }
1094
1095 static int atk_ec_enabled(struct atk_data *data)
1096 {
1097         struct device *dev = &data->acpi_dev->dev;
1098         union acpi_object *obj;
1099         struct atk_acpi_ret_buffer *buf;
1100         int err;
1101
1102         obj = atk_gitm(data, ATK_EC_ID);
1103         if (IS_ERR(obj)) {
1104                 dev_err(dev, "Unable to query EC status\n");
1105                 return PTR_ERR(obj);
1106         }
1107         buf = (struct atk_acpi_ret_buffer *)obj->buffer.pointer;
1108
1109         if (buf->flags == 0) {
1110                 dev_err(dev, "Unable to query EC status\n");
1111                 err = -EIO;
1112         } else {
1113                 err = (buf->value != 0);
1114                 dev_dbg(dev, "EC is %sabled\n",
1115                                 err ? "en" : "dis");
1116         }
1117
1118         ACPI_FREE(obj);
1119         return err;
1120 }
1121
1122 static int atk_ec_ctl(struct atk_data *data, int enable)
1123 {
1124         struct device *dev = &data->acpi_dev->dev;
1125         union acpi_object *obj;
1126         struct atk_acpi_input_buf sitm;
1127         struct atk_acpi_ret_buffer *ec_ret;
1128         int err = 0;
1129
1130         sitm.id = ATK_EC_ID;
1131         sitm.param1 = enable;
1132         sitm.param2 = 0;
1133
1134         obj = atk_sitm(data, &sitm);
1135         if (IS_ERR(obj)) {
1136                 dev_err(dev, "Failed to %sable the EC\n",
1137                                 enable ? "en" : "dis");
1138                 return PTR_ERR(obj);
1139         }
1140         ec_ret = (struct atk_acpi_ret_buffer *)obj->buffer.pointer;
1141         if (ec_ret->flags == 0) {
1142                 dev_err(dev, "Failed to %sable the EC\n",
1143                                 enable ? "en" : "dis");
1144                 err = -EIO;
1145         } else {
1146                 dev_info(dev, "EC %sabled\n",
1147                                 enable ? "en" : "dis");
1148         }
1149
1150         ACPI_FREE(obj);
1151         return err;
1152 }
1153
1154 static int atk_enumerate_new_hwmon(struct atk_data *data)
1155 {
1156         struct device *dev = &data->acpi_dev->dev;
1157         union acpi_object *pack;
1158         int err;
1159         int i;
1160
1161         err = atk_ec_present(data);
1162         if (err < 0)
1163                 return err;
1164         if (err) {
1165                 err = atk_ec_enabled(data);
1166                 if (err < 0)
1167                         return err;
1168                 /* If the EC was disabled we will disable it again on unload */
1169                 data->disable_ec = err;
1170
1171                 err = atk_ec_ctl(data, 1);
1172                 if (err) {
1173                         data->disable_ec = false;
1174                         return err;
1175                 }
1176         }
1177
1178         dev_dbg(dev, "Enumerating hwmon sensors\n");
1179
1180         pack = atk_ggrp(data, ATK_MUX_HWMON);
1181         if (IS_ERR(pack))
1182                 return PTR_ERR(pack);
1183
1184         for (i = 0; i < pack->package.count; i++) {
1185                 union acpi_object *obj = &pack->package.elements[i];
1186
1187                 atk_add_sensor(data, obj);
1188         }
1189
1190         err = data->voltage_count + data->temperature_count + data->fan_count;
1191
1192         ACPI_FREE(pack);
1193         return err;
1194 }
1195
1196 static int atk_create_files(struct atk_data *data)
1197 {
1198         struct atk_sensor_data *s;
1199         int err;
1200
1201         list_for_each_entry(s, &data->sensor_list, list) {
1202                 err = device_create_file(data->hwmon_dev, &s->input_attr);
1203                 if (err)
1204                         return err;
1205                 err = device_create_file(data->hwmon_dev, &s->label_attr);
1206                 if (err)
1207                         return err;
1208                 err = device_create_file(data->hwmon_dev, &s->limit1_attr);
1209                 if (err)
1210                         return err;
1211                 err = device_create_file(data->hwmon_dev, &s->limit2_attr);
1212                 if (err)
1213                         return err;
1214         }
1215
1216         err = device_create_file(data->hwmon_dev, &atk_name_attr);
1217
1218         return err;
1219 }
1220
1221 static void atk_remove_files(struct atk_data *data)
1222 {
1223         struct atk_sensor_data *s;
1224
1225         list_for_each_entry(s, &data->sensor_list, list) {
1226                 device_remove_file(data->hwmon_dev, &s->input_attr);
1227                 device_remove_file(data->hwmon_dev, &s->label_attr);
1228                 device_remove_file(data->hwmon_dev, &s->limit1_attr);
1229                 device_remove_file(data->hwmon_dev, &s->limit2_attr);
1230         }
1231         device_remove_file(data->hwmon_dev, &atk_name_attr);
1232 }
1233
1234 static void atk_free_sensors(struct atk_data *data)
1235 {
1236         struct list_head *head = &data->sensor_list;
1237         struct atk_sensor_data *s, *tmp;
1238
1239         list_for_each_entry_safe(s, tmp, head, list) {
1240                 kfree(s->acpi_name);
1241                 kfree(s);
1242         }
1243 }
1244
1245 static int atk_register_hwmon(struct atk_data *data)
1246 {
1247         struct device *dev = &data->acpi_dev->dev;
1248         int err;
1249
1250         dev_dbg(dev, "registering hwmon device\n");
1251         data->hwmon_dev = hwmon_device_register(dev);
1252         if (IS_ERR(data->hwmon_dev))
1253                 return PTR_ERR(data->hwmon_dev);
1254
1255         dev_dbg(dev, "populating sysfs directory\n");
1256         err = atk_create_files(data);
1257         if (err)
1258                 goto remove;
1259
1260         return 0;
1261 remove:
1262         /* Cleanup the registered files */
1263         atk_remove_files(data);
1264         hwmon_device_unregister(data->hwmon_dev);
1265         return err;
1266 }
1267
1268 static int atk_probe_if(struct atk_data *data)
1269 {
1270         struct device *dev = &data->acpi_dev->dev;
1271         acpi_handle ret;
1272         acpi_status status;
1273         int err = 0;
1274
1275         /* RTMP: read temperature */
1276         status = acpi_get_handle(data->atk_handle, METHOD_OLD_READ_TMP, &ret);
1277         if (ACPI_SUCCESS(status))
1278                 data->rtmp_handle = ret;
1279         else
1280                 dev_dbg(dev, "method " METHOD_OLD_READ_TMP " not found: %s\n",
1281                                 acpi_format_exception(status));
1282
1283         /* RVLT: read voltage */
1284         status = acpi_get_handle(data->atk_handle, METHOD_OLD_READ_VLT, &ret);
1285         if (ACPI_SUCCESS(status))
1286                 data->rvlt_handle = ret;
1287         else
1288                 dev_dbg(dev, "method " METHOD_OLD_READ_VLT " not found: %s\n",
1289                                 acpi_format_exception(status));
1290
1291         /* RFAN: read fan status */
1292         status = acpi_get_handle(data->atk_handle, METHOD_OLD_READ_FAN, &ret);
1293         if (ACPI_SUCCESS(status))
1294                 data->rfan_handle = ret;
1295         else
1296                 dev_dbg(dev, "method " METHOD_OLD_READ_FAN " not found: %s\n",
1297                                 acpi_format_exception(status));
1298
1299         /* Enumeration */
1300         status = acpi_get_handle(data->atk_handle, METHOD_ENUMERATE, &ret);
1301         if (ACPI_SUCCESS(status))
1302                 data->enumerate_handle = ret;
1303         else
1304                 dev_dbg(dev, "method " METHOD_ENUMERATE " not found: %s\n",
1305                                 acpi_format_exception(status));
1306
1307         /* De-multiplexer (read) */
1308         status = acpi_get_handle(data->atk_handle, METHOD_READ, &ret);
1309         if (ACPI_SUCCESS(status))
1310                 data->read_handle = ret;
1311         else
1312                 dev_dbg(dev, "method " METHOD_READ " not found: %s\n",
1313                                 acpi_format_exception(status));
1314
1315         /* De-multiplexer (write) */
1316         status = acpi_get_handle(data->atk_handle, METHOD_WRITE, &ret);
1317         if (ACPI_SUCCESS(status))
1318                 data->write_handle = ret;
1319         else
1320                 dev_dbg(dev, "method " METHOD_WRITE " not found: %s\n",
1321                                  acpi_format_exception(status));
1322
1323         /* Check for hwmon methods: first check "old" style methods; note that
1324          * both may be present: in this case we stick to the old interface;
1325          * analysis of multiple DSDTs indicates that when both interfaces
1326          * are present the new one (GGRP/GITM) is not functional.
1327          */
1328         if (new_if)
1329                 dev_info(dev, "Overriding interface detection\n");
1330         if (data->rtmp_handle && data->rvlt_handle && data->rfan_handle && !new_if)
1331                 data->old_interface = true;
1332         else if (data->enumerate_handle && data->read_handle &&
1333                         data->write_handle)
1334                 data->old_interface = false;
1335         else
1336                 err = -ENODEV;
1337
1338         return err;
1339 }
1340
1341 static int atk_add(struct acpi_device *device)
1342 {
1343         acpi_status ret;
1344         int err;
1345         struct acpi_buffer buf;
1346         union acpi_object *obj;
1347         struct atk_data *data;
1348
1349         dev_dbg(&device->dev, "adding...\n");
1350
1351         data = kzalloc(sizeof(*data), GFP_KERNEL);
1352         if (!data)
1353                 return -ENOMEM;
1354
1355         data->acpi_dev = device;
1356         data->atk_handle = device->handle;
1357         INIT_LIST_HEAD(&data->sensor_list);
1358         data->disable_ec = false;
1359
1360         buf.length = ACPI_ALLOCATE_BUFFER;
1361         ret = acpi_evaluate_object_typed(data->atk_handle, BOARD_ID, NULL,
1362                         &buf, ACPI_TYPE_PACKAGE);
1363         if (ret != AE_OK) {
1364                 dev_dbg(&device->dev, "atk: method MBIF not found\n");
1365         } else {
1366                 obj = buf.pointer;
1367                 if (obj->package.count >= 2) {
1368                         union acpi_object *id = &obj->package.elements[1];
1369                         if (id->type == ACPI_TYPE_STRING)
1370                                 dev_dbg(&device->dev, "board ID = %s\n",
1371                                         id->string.pointer);
1372                 }
1373                 ACPI_FREE(buf.pointer);
1374         }
1375
1376         err = atk_probe_if(data);
1377         if (err) {
1378                 dev_err(&device->dev, "No usable hwmon interface detected\n");
1379                 goto out;
1380         }
1381
1382         if (data->old_interface) {
1383                 dev_dbg(&device->dev, "Using old hwmon interface\n");
1384                 err = atk_enumerate_old_hwmon(data);
1385         } else {
1386                 dev_dbg(&device->dev, "Using new hwmon interface\n");
1387                 err = atk_enumerate_new_hwmon(data);
1388         }
1389         if (err < 0)
1390                 goto out;
1391         if (err == 0) {
1392                 dev_info(&device->dev,
1393                          "No usable sensor detected, bailing out\n");
1394                 err = -ENODEV;
1395                 goto out;
1396         }
1397
1398         err = atk_register_hwmon(data);
1399         if (err)
1400                 goto cleanup;
1401
1402         atk_debugfs_init(data);
1403
1404         device->driver_data = data;
1405         return 0;
1406 cleanup:
1407         atk_free_sensors(data);
1408 out:
1409         if (data->disable_ec)
1410                 atk_ec_ctl(data, 0);
1411         kfree(data);
1412         return err;
1413 }
1414
1415 static int atk_remove(struct acpi_device *device, int type)
1416 {
1417         struct atk_data *data = device->driver_data;
1418         dev_dbg(&device->dev, "removing...\n");
1419
1420         device->driver_data = NULL;
1421
1422         atk_debugfs_cleanup(data);
1423
1424         atk_remove_files(data);
1425         atk_free_sensors(data);
1426         hwmon_device_unregister(data->hwmon_dev);
1427
1428         if (data->disable_ec) {
1429                 if (atk_ec_ctl(data, 0))
1430                         dev_err(&device->dev, "Failed to disable EC\n");
1431         }
1432
1433         kfree(data);
1434
1435         return 0;
1436 }
1437
1438 static int __init atk0110_init(void)
1439 {
1440         int ret;
1441
1442         /* Make sure it's safe to access the device through ACPI */
1443         if (!acpi_resources_are_enforced()) {
1444                 pr_err("Resources not safely usable due to acpi_enforce_resources kernel parameter\n");
1445                 return -EBUSY;
1446         }
1447
1448         if (dmi_check_system(atk_force_new_if))
1449                 new_if = true;
1450
1451         ret = acpi_bus_register_driver(&atk_driver);
1452         if (ret)
1453                 pr_info("acpi_bus_register_driver failed: %d\n", ret);
1454
1455         return ret;
1456 }
1457
1458 static void __exit atk0110_exit(void)
1459 {
1460         acpi_bus_unregister_driver(&atk_driver);
1461 }
1462
1463 module_init(atk0110_init);
1464 module_exit(atk0110_exit);
1465
1466 MODULE_LICENSE("GPL");