+ * 25-April-2009: Version 0.5:
+ * - ported to 2.6.30
+ * - removed kthread and used polling of thermal api
Version information in comments isn't generally useful
There's no other thermal device_mode state than ENABLED/DISABLED.
Maybe just if/else?
+/* print the name of the trip point */
+static int get_trip_type(struct thermal_zone_device *thermal,
+ int trip, enum thermal_trip_type *type)
+{
+ if (trip == 0)
+ *type = THERMAL_TRIP_ACTIVE;
+ return 0;
+}
Comment doesn't match code. I see no printing here.
You would #define pr_fmt(fmt) "acerhd: " fmt
instead of prefixing all the printks with "acerhd:"