summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--applesmc_update.patch2293
-rw-r--r--drm-intel-big-hammer.patch12
-rw-r--r--drm_i915-check-eDP-encoder-correctly-when-setting-modes.patch33
-rw-r--r--kernel.spec6
-rw-r--r--linux-next-macbook-air-input.patch185
5 files changed, 6 insertions, 2523 deletions
diff --git a/applesmc_update.patch b/applesmc_update.patch
deleted file mode 100644
index d42146871..000000000
--- a/applesmc_update.patch
+++ /dev/null
@@ -1,2293 +0,0 @@
-Update the applesmc driver. Should work on all hardware. Upstream in .38?
-
-diff --git a/drivers/hwmon/applesmc.c b/drivers/hwmon/applesmc.c
-index b6598aa..0d7e9ae 100644
---- a/drivers/hwmon/applesmc.c
-+++ b/drivers/hwmon/applesmc.c
-@@ -4,6 +4,7 @@
- * computers.
- *
- * Copyright (C) 2007 Nicolas Boichat <nicolas@boichat.ch>
-+ * Copyright (C) 2010 Henrik Rydberg <rydberg@euromail.se>
- *
- * Based on hdaps.c driver:
- * Copyright (C) 2005 Robert Love <rml@novell.com>
-@@ -26,10 +27,12 @@
- * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
- */
-
-+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
-+
- #include <linux/delay.h>
--#include <linux/platform_device.h>
- #include <linux/input-polldev.h>
- #include <linux/kernel.h>
-+#include <linux/slab.h>
- #include <linux/module.h>
- #include <linux/timer.h>
- #include <linux/dmi.h>
-@@ -39,16 +42,23 @@
- #include <linux/leds.h>
- #include <linux/hwmon.h>
- #include <linux/workqueue.h>
-+#include <linux/slab.h>
-+#include <linux/pnp.h>
-+#include <linux/irq.h>
-+#include <linux/interrupt.h>
-
- /* data port used by Apple SMC */
--#define APPLESMC_DATA_PORT 0x300
-+#define APPLESMC_DATA_PORT 0x0
- /* command/status port used by Apple SMC */
--#define APPLESMC_CMD_PORT 0x304
-+#define APPLESMC_CMD_PORT 0x4
-+/* interrupt status port */
-+#define APPLESMC_ISR_PORT 0x1f
-
- #define APPLESMC_NR_PORTS 32 /* 0x300-0x31f */
-
- #define APPLESMC_MAX_DATA_LENGTH 32
-
-+/* wait up to 32 ms for a status change. */
- #define APPLESMC_MIN_WAIT 0x0040
- #define APPLESMC_MAX_WAIT 0x8000
-
-@@ -71,106 +81,19 @@
- #define MOTION_SENSOR_Z_KEY "MO_Z" /* r-o sp78 (2 bytes) */
- #define MOTION_SENSOR_KEY "MOCN" /* r/w ui16 */
-
-+#define NOTIFICATION_KEY "NTOK"
-+
- #define FANS_COUNT "FNum" /* r-o ui8 */
- #define FANS_MANUAL "FS! " /* r-w ui16 */
--#define FAN_ACTUAL_SPEED "F0Ac" /* r-o fpe2 (2 bytes) */
--#define FAN_MIN_SPEED "F0Mn" /* r-o fpe2 (2 bytes) */
--#define FAN_MAX_SPEED "F0Mx" /* r-o fpe2 (2 bytes) */
--#define FAN_SAFE_SPEED "F0Sf" /* r-o fpe2 (2 bytes) */
--#define FAN_TARGET_SPEED "F0Tg" /* r-w fpe2 (2 bytes) */
--#define FAN_POSITION "F0ID" /* r-o char[16] */
--
--/*
-- * Temperature sensors keys (sp78 - 2 bytes).
-- */
--static const char *temperature_sensors_sets[][41] = {
--/* Set 0: Macbook Pro */
-- { "TA0P", "TB0T", "TC0D", "TC0P", "TG0H", "TG0P", "TG0T", "Th0H",
-- "Th1H", "Tm0P", "Ts0P", "Ts1P", NULL },
--/* Set 1: Macbook2 set */
-- { "TB0T", "TC0D", "TC0P", "TM0P", "TN0P", "TN1P", "TTF0", "Th0H",
-- "Th0S", "Th1H", NULL },
--/* Set 2: Macbook set */
-- { "TB0T", "TC0D", "TC0P", "TM0P", "TN0P", "TN1P", "Th0H", "Th0S",
-- "Th1H", "Ts0P", NULL },
--/* Set 3: Macmini set */
-- { "TC0D", "TC0P", NULL },
--/* Set 4: Mac Pro (2 x Quad-Core) */
-- { "TA0P", "TCAG", "TCAH", "TCBG", "TCBH", "TC0C", "TC0D", "TC0P",
-- "TC1C", "TC1D", "TC2C", "TC2D", "TC3C", "TC3D", "THTG", "TH0P",
-- "TH1P", "TH2P", "TH3P", "TMAP", "TMAS", "TMBS", "TM0P", "TM0S",
-- "TM1P", "TM1S", "TM2P", "TM2S", "TM3S", "TM8P", "TM8S", "TM9P",
-- "TM9S", "TN0H", "TS0C", NULL },
--/* Set 5: iMac */
-- { "TC0D", "TA0P", "TG0P", "TG0D", "TG0H", "TH0P", "Tm0P", "TO0P",
-- "Tp0C", NULL },
--/* Set 6: Macbook3 set */
-- { "TB0T", "TC0D", "TC0P", "TM0P", "TN0P", "TTF0", "TW0P", "Th0H",
-- "Th0S", "Th1H", NULL },
--/* Set 7: Macbook Air */
-- { "TB0T", "TB1S", "TB1T", "TB2S", "TB2T", "TC0D", "TC0P", "TCFP",
-- "TTF0", "TW0P", "Th0H", "Tp0P", "TpFP", "Ts0P", "Ts0S", NULL },
--/* Set 8: Macbook Pro 4,1 (Penryn) */
-- { "TB0T", "TC0D", "TC0P", "TG0D", "TG0H", "TTF0", "TW0P", "Th0H",
-- "Th1H", "Th2H", "Tm0P", "Ts0P", NULL },
--/* Set 9: Macbook Pro 3,1 (Santa Rosa) */
-- { "TALP", "TB0T", "TC0D", "TC0P", "TG0D", "TG0H", "TTF0", "TW0P",
-- "Th0H", "Th1H", "Th2H", "Tm0P", "Ts0P", NULL },
--/* Set 10: iMac 5,1 */
-- { "TA0P", "TC0D", "TC0P", "TG0D", "TH0P", "TO0P", "Tm0P", NULL },
--/* Set 11: Macbook 5,1 */
-- { "TB0T", "TB1T", "TB2T", "TB3T", "TC0D", "TC0P", "TN0D", "TN0P",
-- "TTF0", "Th0H", "Th1H", "ThFH", "Ts0P", "Ts0S", NULL },
--/* Set 12: Macbook Pro 5,1 */
-- { "TB0T", "TB1T", "TB2T", "TB3T", "TC0D", "TC0F", "TC0P", "TG0D",
-- "TG0F", "TG0H", "TG0P", "TG0T", "TG1H", "TN0D", "TN0P", "TTF0",
-- "Th2H", "Tm0P", "Ts0P", "Ts0S", NULL },
--/* Set 13: iMac 8,1 */
-- { "TA0P", "TC0D", "TC0H", "TC0P", "TG0D", "TG0H", "TG0P", "TH0P",
-- "TL0P", "TO0P", "TW0P", "Tm0P", "Tp0P", NULL },
--/* Set 14: iMac 6,1 */
-- { "TA0P", "TC0D", "TC0H", "TC0P", "TG0D", "TG0H", "TG0P", "TH0P",
-- "TO0P", "Tp0P", NULL },
--/* Set 15: MacBook Air 2,1 */
-- { "TB0T", "TB1S", "TB1T", "TB2S", "TB2T", "TC0D", "TN0D", "TTF0",
-- "TV0P", "TVFP", "TW0P", "Th0P", "Tp0P", "Tp1P", "TpFP", "Ts0P",
-- "Ts0S", NULL },
--/* Set 16: Mac Pro 3,1 (2 x Quad-Core) */
-- { "TA0P", "TCAG", "TCAH", "TCBG", "TCBH", "TC0C", "TC0D", "TC0P",
-- "TC1C", "TC1D", "TC2C", "TC2D", "TC3C", "TC3D", "TH0P", "TH1P",
-- "TH2P", "TH3P", "TMAP", "TMAS", "TMBS", "TM0P", "TM0S", "TM1P",
-- "TM1S", "TM2P", "TM2S", "TM3S", "TM8P", "TM8S", "TM9P", "TM9S",
-- "TN0C", "TN0D", "TN0H", "TS0C", "Tp0C", "Tp1C", "Tv0S", "Tv1S",
-- NULL },
--/* Set 17: iMac 9,1 */
-- { "TA0P", "TC0D", "TC0H", "TC0P", "TG0D", "TG0H", "TH0P", "TL0P",
-- "TN0D", "TN0H", "TN0P", "TO0P", "Tm0P", "Tp0P", NULL },
--/* Set 18: MacBook Pro 2,2 */
-- { "TB0T", "TC0D", "TC0P", "TG0H", "TG0P", "TG0T", "TM0P", "TTF0",
-- "Th0H", "Th1H", "Tm0P", "Ts0P", NULL },
--/* Set 19: Macbook Pro 5,3 */
-- { "TB0T", "TB1T", "TB2T", "TB3T", "TC0D", "TC0F", "TC0P", "TG0D",
-- "TG0F", "TG0H", "TG0P", "TG0T", "TN0D", "TN0P", "TTF0", "Th2H",
-- "Tm0P", "Ts0P", "Ts0S", NULL },
--/* Set 20: MacBook Pro 5,4 */
-- { "TB0T", "TB1T", "TB2T", "TB3T", "TC0D", "TC0F", "TC0P", "TN0D",
-- "TN0P", "TTF0", "Th2H", "Ts0P", "Ts0S", NULL },
--/* Set 21: MacBook Pro 6,2 */
-- { "TB0T", "TB1T", "TB2T", "TC0C", "TC0D", "TC0P", "TC1C", "TG0D",
-- "TG0P", "TG0T", "TMCD", "TP0P", "TPCD", "Th1H", "Th2H", "Tm0P",
-- "Ts0P", "Ts0S", NULL },
--/* Set 22: MacBook Pro 7,1 */
-- { "TB0T", "TB1T", "TB2T", "TC0D", "TC0P", "TN0D", "TN0P", "TN0S",
-- "TN1D", "TN1F", "TN1G", "TN1S", "Th1H", "Ts0P", "Ts0S", NULL },
--};
-+#define FAN_ID_FMT "F%dID" /* r-o char[16] */
-
- /* List of keys used to read/write fan speeds */
--static const char* fan_speed_keys[] = {
-- FAN_ACTUAL_SPEED,
-- FAN_MIN_SPEED,
-- FAN_MAX_SPEED,
-- FAN_SAFE_SPEED,
-- FAN_TARGET_SPEED
-+static const char *const fan_speed_fmt[] = {
-+ "F%dAc", /* actual speed */
-+ "F%dMn", /* minimum speed (rw) */
-+ "F%dMx", /* maximum speed */
-+ "F%dSf", /* safe speed - not all models */
-+ "F%dTg", /* target speed (manual: rw) */
- };
-
- #define INIT_TIMEOUT_MSECS 5000 /* wait up to 5s for device init ... */
-@@ -184,18 +107,58 @@ static const char* fan_speed_keys[] = {
- #define SENSOR_Y 1
- #define SENSOR_Z 2
-
--/* Structure to be passed to DMI_MATCH function */
--struct dmi_match_data {
--/* Indicates whether this computer has an accelerometer. */
-- int accelerometer;
--/* Indicates whether this computer has light sensors and keyboard backlight. */
-- int light;
--/* Indicates which temperature sensors set to use. */
-- int temperature_set;
-+#define to_index(attr) (to_sensor_dev_attr(attr)->index & 0xffff)
-+#define to_option(attr) (to_sensor_dev_attr(attr)->index >> 16)
-+
-+struct applesmc_pnp_device {
-+ int iobase;
-+ int iolen;
-+ int irq;
-+};
-+
-+struct pnp_dev *pdev;
-+struct applesmc_pnp_device *pnp_device;
-+
-+/* Dynamic device node attributes */
-+struct applesmc_dev_attr {
-+ struct sensor_device_attribute sda; /* hwmon attributes */
-+ char name[32]; /* room for node file name */
-+};
-+
-+/* Dynamic device node group */
-+struct applesmc_node_group {
-+ char *format; /* format string */
-+ void *show; /* show function */
-+ void *store; /* store function */
-+ int option; /* function argument */
-+ struct applesmc_dev_attr *nodes; /* dynamic node array */
- };
-
-+/* AppleSMC entry - cached register information */
-+struct applesmc_entry {
-+ char key[5]; /* four-letter key code */
-+ u8 valid; /* set when entry is successfully read once */
-+ u8 len; /* bounded by APPLESMC_MAX_DATA_LENGTH */
-+ char type[5]; /* four-letter type code */
-+ u8 flags; /* 0x10: func; 0x40: write; 0x80: read */
-+};
-+
-+/* Register lookup and registers common to all SMCs */
-+static struct applesmc_registers {
-+ struct mutex mutex; /* register read/write mutex */
-+ unsigned int key_count; /* number of SMC registers */
-+ unsigned int fan_count; /* number of fans */
-+ unsigned int temp_count; /* number of temperature registers */
-+ unsigned int temp_begin; /* temperature lower index bound */
-+ unsigned int temp_end; /* temperature upper index bound */
-+ int num_light_sensors; /* number of light sensors */
-+ bool has_accelerometer; /* has motion sensor */
-+ bool has_key_backlight; /* has keyboard backlight */
-+ bool init_complete; /* true when fully initialized */
-+ struct applesmc_entry *cache; /* cached key entries */
-+} smcreg;
-+
- static const int debug;
--static struct platform_device *pdev;
- static s16 rest_x;
- static s16 rest_y;
- static u8 backlight_state[2];
-@@ -203,20 +166,6 @@ static u8 backlight_state[2];
- static struct device *hwmon_dev;
- static struct input_polled_dev *applesmc_idev;
-
--/* Indicates whether this computer has an accelerometer. */
--static unsigned int applesmc_accelerometer;
--
--/* Indicates whether this computer has light sensors and keyboard backlight. */
--static unsigned int applesmc_light;
--
--/* The number of fans handled by the driver */
--static unsigned int fans_handled;
--
--/* Indicates which temperature sensors set to use. */
--static unsigned int applesmc_temperature_set;
--
--static DEFINE_MUTEX(applesmc_lock);
--
- /*
- * Last index written to key_at_index sysfs file, and value to use for all other
- * key_at_index_* sysfs files.
-@@ -225,6 +174,16 @@ static unsigned int key_at_index;
-
- static struct workqueue_struct *applesmc_led_wq;
-
-+static u8 applesmc_read_reg(u8 reg)
-+{
-+ return inb(pnp_device->iobase + reg);
-+}
-+
-+static void applesmc_write_reg(u8 val, u8 reg)
-+{
-+ outb(val, pnp_device->iobase + reg);
-+}
-+
- /*
- * __wait_status - Wait up to 32ms for the status port to get a certain value
- * (masked with 0x0f), returning zero if the value is obtained. Callers must
-@@ -238,18 +197,12 @@ static int __wait_status(u8 val)
-
- for (us = APPLESMC_MIN_WAIT; us < APPLESMC_MAX_WAIT; us <<= 1) {
- udelay(us);
-- if ((inb(APPLESMC_CMD_PORT) & APPLESMC_STATUS_MASK) == val) {
-- if (debug)
-- printk(KERN_DEBUG
-- "Waited %d us for status %x\n",
-- 2 * us - APPLESMC_MIN_WAIT, val);
-+ if ((applesmc_read_reg(APPLESMC_CMD_PORT)
-+ & APPLESMC_STATUS_MASK) == val) {
- return 0;
- }
- }
-
-- printk(KERN_WARNING "applesmc: wait status failed: %x != %x\n",
-- val, inb(APPLESMC_CMD_PORT));
--
- return -EIO;
- }
-
-@@ -262,162 +215,246 @@ static int send_command(u8 cmd)
- {
- int us;
- for (us = APPLESMC_MIN_WAIT; us < APPLESMC_MAX_WAIT; us <<= 1) {
-- outb(cmd, APPLESMC_CMD_PORT);
-+ applesmc_write_reg(cmd, APPLESMC_CMD_PORT);
- udelay(us);
-- if ((inb(APPLESMC_CMD_PORT) & APPLESMC_STATUS_MASK) == 0x0c)
-+ if ((applesmc_read_reg(APPLESMC_CMD_PORT)
-+ & APPLESMC_STATUS_MASK) == 0x0c)
- return 0;
- }
-- printk(KERN_WARNING "applesmc: command failed: %x -> %x\n",
-- cmd, inb(APPLESMC_CMD_PORT));
- return -EIO;
- }
-
--/*
-- * applesmc_read_key - reads len bytes from a given key, and put them in buffer.
-- * Returns zero on success or a negative error on failure. Callers must
-- * hold applesmc_lock.
-- */
--static int applesmc_read_key(const char* key, u8* buffer, u8 len)
-+static int send_argument(const char *key)
- {
- int i;
-
-- if (len > APPLESMC_MAX_DATA_LENGTH) {
-- printk(KERN_ERR "applesmc_read_key: cannot read more than "
-- "%d bytes\n", APPLESMC_MAX_DATA_LENGTH);
-- return -EINVAL;
-- }
--
-- if (send_command(APPLESMC_READ_CMD))
-- return -EIO;
--
- for (i = 0; i < 4; i++) {
-- outb(key[i], APPLESMC_DATA_PORT);
-+ applesmc_write_reg(key[i], APPLESMC_DATA_PORT);
- if (__wait_status(0x04))
- return -EIO;
- }
-- if (debug)
-- printk(KERN_DEBUG "<%s", key);
-+ return 0;
-+}
-
-- outb(len, APPLESMC_DATA_PORT);
-- if (debug)
-- printk(KERN_DEBUG ">%x", len);
-+static int read_smc(u8 cmd, const char *key, u8 *buffer, u8 len)
-+{
-+ int i;
-+
-+ if (send_command(cmd) || send_argument(key)) {
-+ pr_warn("%s: read arg fail\n", key);
-+ return -EIO;
-+ }
-+
-+ applesmc_write_reg(len, APPLESMC_DATA_PORT);
-
- for (i = 0; i < len; i++) {
-- if (__wait_status(0x05))
-+ if (__wait_status(0x05)) {
-+ pr_warn("%s: read data fail\n", key);
- return -EIO;
-- buffer[i] = inb(APPLESMC_DATA_PORT);
-- if (debug)
-- printk(KERN_DEBUG "<%x", buffer[i]);
-+ }
-+ buffer[i] = applesmc_read_reg(APPLESMC_DATA_PORT);
- }
-- if (debug)
-- printk(KERN_DEBUG "\n");
-
- return 0;
- }
-
--/*
-- * applesmc_write_key - writes len bytes from buffer to a given key.
-- * Returns zero on success or a negative error on failure. Callers must
-- * hold applesmc_lock.
-- */
--static int applesmc_write_key(const char* key, u8* buffer, u8 len)
-+static int write_smc(u8 cmd, const char *key, const u8 *buffer, u8 len)
- {
- int i;
-
-- if (len > APPLESMC_MAX_DATA_LENGTH) {
-- printk(KERN_ERR "applesmc_write_key: cannot write more than "
-- "%d bytes\n", APPLESMC_MAX_DATA_LENGTH);
-- return -EINVAL;
-- }
--
-- if (send_command(APPLESMC_WRITE_CMD))
-+ if (send_command(cmd) || send_argument(key)) {
-+ pr_warn("%s: write arg fail\n", key);
- return -EIO;
--
-- for (i = 0; i < 4; i++) {
-- outb(key[i], APPLESMC_DATA_PORT);
-- if (__wait_status(0x04))
-- return -EIO;
- }
-
-- outb(len, APPLESMC_DATA_PORT);
-+ applesmc_write_reg(len, APPLESMC_DATA_PORT);
-
- for (i = 0; i < len; i++) {
-- if (__wait_status(0x04))
-+ if (__wait_status(0x04)) {
-+ pr_warn("%s: write data fail\n", key);
- return -EIO;
-- outb(buffer[i], APPLESMC_DATA_PORT);
-+ }
-+ applesmc_write_reg(buffer[i], APPLESMC_DATA_PORT);
- }
-
- return 0;
- }
-
-+static int read_register_count(unsigned int *count)
-+{
-+ __be32 be;
-+ int ret;
-+
-+ ret = read_smc(APPLESMC_READ_CMD, KEY_COUNT_KEY, (u8 *)&be, 4);
-+ if (ret)
-+ return ret;
-+
-+ *count = be32_to_cpu(be);
-+ return 0;
-+}
-+
- /*
-- * applesmc_get_key_at_index - get key at index, and put the result in key
-- * (char[6]). Returns zero on success or a negative error on failure. Callers
-- * must hold applesmc_lock.
-+ * Serialized I/O
-+ *
-+ * Returns zero on success or a negative error on failure.
-+ * All functions below are concurrency safe - callers should NOT hold lock.
- */
--static int applesmc_get_key_at_index(int index, char* key)
-+
-+static int applesmc_read_entry(const struct applesmc_entry *entry,
-+ u8 *buf, u8 len)
- {
-- int i;
-- u8 readkey[4];
-- readkey[0] = index >> 24;
-- readkey[1] = index >> 16;
-- readkey[2] = index >> 8;
-- readkey[3] = index;
-+ int ret;
-
-- if (send_command(APPLESMC_GET_KEY_BY_INDEX_CMD))
-- return -EIO;
-+ if (entry->len != len)
-+ return -EINVAL;
-+ mutex_lock(&smcreg.mutex);
-+ ret = read_smc(APPLESMC_READ_CMD, entry->key, buf, len);
-+ mutex_unlock(&smcreg.mutex);
-
-- for (i = 0; i < 4; i++) {
-- outb(readkey[i], APPLESMC_DATA_PORT);
-- if (__wait_status(0x04))
-- return -EIO;
-+ return ret;
-+}
-+
-+static int applesmc_write_entry(const struct applesmc_entry *entry,
-+ const u8 *buf, u8 len)
-+{
-+ int ret;
-+
-+ if (entry->len != len)
-+ return -EINVAL;
-+ mutex_lock(&smcreg.mutex);
-+ ret = write_smc(APPLESMC_WRITE_CMD, entry->key, buf, len);
-+ mutex_unlock(&smcreg.mutex);
-+ return ret;
-+}
-+
-+static const struct applesmc_entry *applesmc_get_entry_by_index(int index)
-+{
-+ struct applesmc_entry *cache = &smcreg.cache[index];
-+ u8 key[4], info[6];
-+ __be32 be;
-+ int ret = 0;
-+
-+ if (cache->valid)
-+ return cache;
-+
-+ mutex_lock(&smcreg.mutex);
-+
-+ if (cache->valid)
-+ goto out;
-+ be = cpu_to_be32(index);
-+ ret = read_smc(APPLESMC_GET_KEY_BY_INDEX_CMD, (u8 *)&be, key, 4);
-+ if (ret)
-+ goto out;
-+ ret = read_smc(APPLESMC_GET_KEY_TYPE_CMD, key, info, 6);
-+ if (ret)
-+ goto out;
-+
-+ memcpy(cache->key, key, 4);
-+ cache->len = info[0];
-+ memcpy(cache->type, &info[1], 4);
-+ cache->flags = info[5];
-+ cache->valid = 1;
-+
-+out:
-+ mutex_unlock(&smcreg.mutex);
-+ if (ret)
-+ return ERR_PTR(ret);
-+ return cache;
-+}
-+
-+static int applesmc_get_lower_bound(unsigned int *lo, const char *key)
-+{
-+ int begin = 0, end = smcreg.key_count;
-+ const struct applesmc_entry *entry;
-+
-+ while (begin != end) {
-+ int middle = begin + (end - begin) / 2;
-+ entry = applesmc_get_entry_by_index(middle);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-+ if (strcmp(entry->key, key) < 0)
-+ begin = middle + 1;
-+ else
-+ end = middle;
- }
-
-- outb(4, APPLESMC_DATA_PORT);
-+ *lo = begin;
-+ return 0;
-+}
-
-- for (i = 0; i < 4; i++) {
-- if (__wait_status(0x05))
-- return -EIO;
-- key[i] = inb(APPLESMC_DATA_PORT);
-+static int applesmc_get_upper_bound(unsigned int *hi, const char *key)
-+{
-+ int begin = 0, end = smcreg.key_count;
-+ const struct applesmc_entry *entry;
-+
-+ while (begin != end) {
-+ int middle = begin + (end - begin) / 2;
-+ entry = applesmc_get_entry_by_index(middle);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-+ if (strcmp(key, entry->key) < 0)
-+ end = middle;
-+ else
-+ begin = middle + 1;
- }
-- key[4] = 0;
-
-+ *hi = begin;
- return 0;
- }
-
--/*
-- * applesmc_get_key_type - get key type, and put the result in type (char[6]).
-- * Returns zero on success or a negative error on failure. Callers must
-- * hold applesmc_lock.
-- */
--static int applesmc_get_key_type(char* key, char* type)
-+static const struct applesmc_entry *applesmc_get_entry_by_key(const char *key)
- {
-- int i;
-+ int begin, end;
-+ int ret;
-
-- if (send_command(APPLESMC_GET_KEY_TYPE_CMD))
-- return -EIO;
-+ ret = applesmc_get_lower_bound(&begin, key);
-+ if (ret)
-+ return ERR_PTR(ret);
-+ ret = applesmc_get_upper_bound(&end, key);
-+ if (ret)
-+ return ERR_PTR(ret);
-+ if (end - begin != 1)
-+ return ERR_PTR(-EINVAL);
-
-- for (i = 0; i < 4; i++) {
-- outb(key[i], APPLESMC_DATA_PORT);
-- if (__wait_status(0x04))
-- return -EIO;
-- }
-+ return applesmc_get_entry_by_index(begin);
-+}
-
-- outb(6, APPLESMC_DATA_PORT);
-+static int applesmc_read_key(const char *key, u8 *buffer, u8 len)
-+{
-+ const struct applesmc_entry *entry;
-
-- for (i = 0; i < 6; i++) {
-- if (__wait_status(0x05))
-- return -EIO;
-- type[i] = inb(APPLESMC_DATA_PORT);
-- }
-- type[5] = 0;
-+ entry = applesmc_get_entry_by_key(key);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-+
-+ return applesmc_read_entry(entry, buffer, len);
-+}
-+
-+static int applesmc_write_key(const char *key, const u8 *buffer, u8 len)
-+{
-+ const struct applesmc_entry *entry;
-+
-+ entry = applesmc_get_entry_by_key(key);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-+
-+ return applesmc_write_entry(entry, buffer, len);
-+}
-+
-+static int applesmc_has_key(const char *key, bool *value)
-+{
-+ const struct applesmc_entry *entry;
-
-+ entry = applesmc_get_entry_by_key(key);
-+ if (IS_ERR(entry) && PTR_ERR(entry) != -EINVAL)
-+ return PTR_ERR(entry);
-+
-+ *value = !IS_ERR(entry);
- return 0;
- }
-
- /*
-- * applesmc_read_motion_sensor - Read motion sensor (X, Y or Z). Callers must
-- * hold applesmc_lock.
-+ * applesmc_read_motion_sensor - Read motion sensor (X, Y or Z).
- */
- static int applesmc_read_motion_sensor(int index, s16* value)
- {
-@@ -444,100 +481,130 @@ static int applesmc_read_motion_sensor(int index, s16* value)
- }
-
- /*
-- * applesmc_device_init - initialize the accelerometer. Returns zero on success
-- * and negative error code on failure. Can sleep.
-+ * applesmc_device_init - initialize the accelerometer. Can sleep.
- */
--static int applesmc_device_init(void)
-+static void applesmc_device_init(void)
- {
-- int total, ret = -ENXIO;
-+ int total;
- u8 buffer[2];
-
-- if (!applesmc_accelerometer)
-- return 0;
--
-- mutex_lock(&applesmc_lock);
-+ if (!smcreg.has_accelerometer)
-+ return;
-
- for (total = INIT_TIMEOUT_MSECS; total > 0; total -= INIT_WAIT_MSECS) {
-- if (debug)
-- printk(KERN_DEBUG "applesmc try %d\n", total);
- if (!applesmc_read_key(MOTION_SENSOR_KEY, buffer, 2) &&
-- (buffer[0] != 0x00 || buffer[1] != 0x00)) {
-- if (total == INIT_TIMEOUT_MSECS) {
-- printk(KERN_DEBUG "applesmc: device has"
-- " already been initialized"
-- " (0x%02x, 0x%02x).\n",
-- buffer[0], buffer[1]);
-- } else {
-- printk(KERN_DEBUG "applesmc: device"
-- " successfully initialized"
-- " (0x%02x, 0x%02x).\n",
-- buffer[0], buffer[1]);
-- }
-- ret = 0;
-- goto out;
-- }
-+ (buffer[0] != 0x00 || buffer[1] != 0x00))
-+ return;
- buffer[0] = 0xe0;
- buffer[1] = 0x00;
- applesmc_write_key(MOTION_SENSOR_KEY, buffer, 2);
- msleep(INIT_WAIT_MSECS);
- }
-
-- printk(KERN_WARNING "applesmc: failed to init the device\n");
--
--out:
-- mutex_unlock(&applesmc_lock);
-- return ret;
-+ pr_warn("failed to init the device\n");
- }
-
- /*
-- * applesmc_get_fan_count - get the number of fans. Callers must NOT hold
-- * applesmc_lock.
-+ * applesmc_init_smcreg_try - Try to initialize register cache. Idempotent.
- */
--static int applesmc_get_fan_count(void)
-+static int applesmc_init_smcreg_try(void)
- {
-+ struct applesmc_registers *s = &smcreg;
-+ bool left_light_sensor, right_light_sensor;
-+ u8 tmp[1];
- int ret;
-- u8 buffer[1];
-
-- mutex_lock(&applesmc_lock);
-+ if (s->init_complete)
-+ return 0;
-
-- ret = applesmc_read_key(FANS_COUNT, buffer, 1);
-+ mutex_init(&s->mutex);
-
-- mutex_unlock(&applesmc_lock);
-+ ret = read_register_count(&s->key_count);
- if (ret)
- return ret;
-- else
-- return buffer[0];
--}
-
--/* Device model stuff */
--static int applesmc_probe(struct platform_device *dev)
--{
-- int ret;
-+ if (!s->cache)
-+ s->cache = kcalloc(s->key_count, sizeof(*s->cache), GFP_KERNEL);
-+ if (!s->cache)
-+ return -ENOMEM;
-
-- ret = applesmc_device_init();
-+ ret = applesmc_read_key(FANS_COUNT, tmp, 1);
- if (ret)
- return ret;
-+ s->fan_count = tmp[0];
-+
-+ ret = applesmc_get_lower_bound(&s->temp_begin, "T");
-+ if (ret)
-+ return ret;
-+ ret = applesmc_get_lower_bound(&s->temp_end, "U");
-+ if (ret)
-+ return ret;
-+ s->temp_count = s->temp_end - s->temp_begin;
-+
-+ ret = applesmc_has_key(LIGHT_SENSOR_LEFT_KEY, &left_light_sensor);
-+ if (ret)
-+ return ret;
-+ ret = applesmc_has_key(LIGHT_SENSOR_RIGHT_KEY, &right_light_sensor);
-+ if (ret)
-+ return ret;
-+ ret = applesmc_has_key(MOTION_SENSOR_KEY, &s->has_accelerometer);
-+ if (ret)
-+ return ret;
-+ ret = applesmc_has_key(BACKLIGHT_KEY, &s->has_key_backlight);
-+ if (ret)
-+ return ret;
-+
-+ s->num_light_sensors = left_light_sensor + right_light_sensor;
-+ s->init_complete = true;
-+
-+ pr_info("key=%d fan=%d temp=%d acc=%d lux=%d kbd=%d\n",
-+ s->key_count, s->fan_count, s->temp_count,
-+ s->has_accelerometer,
-+ s->num_light_sensors,
-+ s->has_key_backlight);
-
-- printk(KERN_INFO "applesmc: device successfully initialized.\n");
- return 0;
- }
-
-+/*
-+ * applesmc_init_smcreg - Initialize register cache.
-+ *
-+ * Retries until initialization is successful, or the operation times out.
-+ *
-+ */
-+static int applesmc_init_smcreg(void)
-+{
-+ int ms, ret;
-+
-+ for (ms = 0; ms < INIT_TIMEOUT_MSECS; ms += INIT_WAIT_MSECS) {
-+ ret = applesmc_init_smcreg_try();
-+ if (!ret)
-+ return 0;
-+ pr_warn("slow init, retrying\n");
-+ msleep(INIT_WAIT_MSECS);
-+ }
-+
-+ return ret;
-+}
-+
-+static void applesmc_destroy_smcreg(void)
-+{
-+ kfree(smcreg.cache);
-+ memset(&smcreg, 0, sizeof(smcreg));
-+}
-+
- /* Synchronize device with memorized backlight state */
- static int applesmc_pm_resume(struct device *dev)
- {
-- mutex_lock(&applesmc_lock);
-- if (applesmc_light)
-+ if (smcreg.has_key_backlight)
- applesmc_write_key(BACKLIGHT_KEY, backlight_state, 2);
-- mutex_unlock(&applesmc_lock);
- return 0;
- }
-
- /* Reinitialize device on resume from hibernation */
- static int applesmc_pm_restore(struct device *dev)
- {
-- int ret = applesmc_device_init();
-- if (ret)
-- return ret;
-+ applesmc_device_init();
- return applesmc_pm_resume(dev);
- }
-
-@@ -546,15 +613,6 @@ static const struct dev_pm_ops applesmc_pm_ops = {
- .restore = applesmc_pm_restore,
- };
-
--static struct platform_driver applesmc_driver = {
-- .probe = applesmc_probe,
-- .driver = {
-- .name = "applesmc",
-- .owner = THIS_MODULE,
-- .pm = &applesmc_pm_ops,
-- },
--};
--
- /*
- * applesmc_calibrate - Set our "resting" values. Callers must
- * hold applesmc_lock.
-@@ -571,20 +629,15 @@ static void applesmc_idev_poll(struct input_polled_dev *dev)
- struct input_dev *idev = dev->input;
- s16 x, y;
-
-- mutex_lock(&applesmc_lock);
--
- if (applesmc_read_motion_sensor(SENSOR_X, &x))
-- goto out;
-+ return;
- if (applesmc_read_motion_sensor(SENSOR_Y, &y))
-- goto out;
-+ return;
-
- x = -x;
- input_report_abs(idev, ABS_X, x - rest_x);
- input_report_abs(idev, ABS_Y, y - rest_y);
- input_sync(idev);
--
--out:
-- mutex_unlock(&applesmc_lock);
- }
-
- /* Sysfs Files */
-@@ -601,8 +654,6 @@ static ssize_t applesmc_position_show(struct device *dev,
- int ret;
- s16 x, y, z;
-
-- mutex_lock(&applesmc_lock);
--
- ret = applesmc_read_motion_sensor(SENSOR_X, &x);
- if (ret)
- goto out;
-@@ -614,7 +665,6 @@ static ssize_t applesmc_position_show(struct device *dev,
- goto out;
-
- out:
-- mutex_unlock(&applesmc_lock);
- if (ret)
- return ret;
- else
-@@ -624,20 +674,20 @@ out:
- static ssize_t applesmc_light_show(struct device *dev,
- struct device_attribute *attr, char *sysfsbuf)
- {
-+ const struct applesmc_entry *entry;
- static int data_length;
- int ret;
- u8 left = 0, right = 0;
-- u8 buffer[10], query[6];
--
-- mutex_lock(&applesmc_lock);
-+ u8 buffer[10];
-
- if (!data_length) {
-- ret = applesmc_get_key_type(LIGHT_SENSOR_LEFT_KEY, query);
-- if (ret)
-- goto out;
-- data_length = clamp_val(query[0], 0, 10);
-- printk(KERN_INFO "applesmc: light sensor data length set to "
-- "%d\n", data_length);
-+ entry = applesmc_get_entry_by_key(LIGHT_SENSOR_LEFT_KEY);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-+ if (entry->len > 10)
-+ return -ENXIO;
-+ data_length = entry->len;
-+ pr_info("light sensor data length set to %d\n", data_length);
- }
-
- ret = applesmc_read_key(LIGHT_SENSOR_LEFT_KEY, buffer, data_length);
-@@ -653,7 +703,6 @@ static ssize_t applesmc_light_show(struct device *dev,
- right = buffer[2];
-
- out:
-- mutex_unlock(&applesmc_lock);
- if (ret)
- return ret;
- else
-@@ -664,36 +713,48 @@ out:
- static ssize_t applesmc_show_sensor_label(struct device *dev,
- struct device_attribute *devattr, char *sysfsbuf)
- {
-- struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
-- const char *key =
-- temperature_sensors_sets[applesmc_temperature_set][attr->index];
-+ int index = smcreg.temp_begin + to_index(devattr);
-+ const struct applesmc_entry *entry;
-+
-+ entry = applesmc_get_entry_by_index(index);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-
-- return snprintf(sysfsbuf, PAGE_SIZE, "%s\n", key);
-+ return snprintf(sysfsbuf, PAGE_SIZE, "%s\n", entry->key);
- }
-
- /* Displays degree Celsius * 1000 */
- static ssize_t applesmc_show_temperature(struct device *dev,
- struct device_attribute *devattr, char *sysfsbuf)
- {
-+ int index = smcreg.temp_begin + to_index(devattr);
-+ const struct applesmc_entry *entry;
- int ret;
- u8 buffer[2];
- unsigned int temp;
-- struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
-- const char* key =
-- temperature_sensors_sets[applesmc_temperature_set][attr->index];
-
-- mutex_lock(&applesmc_lock);
--
-- ret = applesmc_read_key(key, buffer, 2);
-- temp = buffer[0]*1000;
-- temp += (buffer[1] >> 6) * 250;
--
-- mutex_unlock(&applesmc_lock);
-+ entry = applesmc_get_entry_by_index(index);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-+ if (entry->len > 2)
-+ return -EINVAL;
-
-+ ret = applesmc_read_entry(entry, buffer, entry->len);
- if (ret)
- return ret;
-- else
-- return snprintf(sysfsbuf, PAGE_SIZE, "%u\n", temp);
-+
-+ if (entry->len == 2) {
-+ if (buffer[0] >= 0x80) {
-+ /* The two byte format is signed - ignore negative */
-+ return -EINVAL;
-+ }
-+ temp = buffer[0] * 1000;
-+ temp += (buffer[1] >> 6) * 250;
-+ } else {
-+ temp = buffer[0] * 4000;
-+ }
-+
-+ return snprintf(sysfsbuf, PAGE_SIZE, "%u\n", temp);
- }
-
- static ssize_t applesmc_show_fan_speed(struct device *dev,
-@@ -703,21 +764,12 @@ static ssize_t applesmc_show_fan_speed(struct device *dev,
- unsigned int speed = 0;
- char newkey[5];
- u8 buffer[2];
-- struct sensor_device_attribute_2 *sensor_attr =
-- to_sensor_dev_attr_2(attr);
-
-- newkey[0] = fan_speed_keys[sensor_attr->nr][0];
-- newkey[1] = '0' + sensor_attr->index;
-- newkey[2] = fan_speed_keys[sensor_attr->nr][2];
-- newkey[3] = fan_speed_keys[sensor_attr->nr][3];
-- newkey[4] = 0;
--
-- mutex_lock(&applesmc_lock);
-+ sprintf(newkey, fan_speed_fmt[to_option(attr)], to_index(attr));
-
- ret = applesmc_read_key(newkey, buffer, 2);
- speed = ((buffer[0] << 8 | buffer[1]) >> 2);
-
-- mutex_unlock(&applesmc_lock);
- if (ret)
- return ret;
- else
-@@ -732,27 +784,18 @@ static ssize_t applesmc_store_fan_speed(struct device *dev,
- u32 speed;
- char newkey[5];
- u8 buffer[2];
-- struct sensor_device_attribute_2 *sensor_attr =
-- to_sensor_dev_attr_2(attr);
-
- speed = simple_strtoul(sysfsbuf, NULL, 10);
-
- if (speed > 0x4000) /* Bigger than a 14-bit value */
- return -EINVAL;
-
-- newkey[0] = fan_speed_keys[sensor_attr->nr][0];
-- newkey[1] = '0' + sensor_attr->index;
-- newkey[2] = fan_speed_keys[sensor_attr->nr][2];
-- newkey[3] = fan_speed_keys[sensor_attr->nr][3];
-- newkey[4] = 0;
--
-- mutex_lock(&applesmc_lock);
-+ sprintf(newkey, fan_speed_fmt[to_option(attr)], to_index(attr));
-
- buffer[0] = (speed >> 6) & 0xff;
- buffer[1] = (speed << 2) & 0xff;
- ret = applesmc_write_key(newkey, buffer, 2);
-
-- mutex_unlock(&applesmc_lock);
- if (ret)
- return ret;
- else
-@@ -760,19 +803,15 @@ static ssize_t applesmc_store_fan_speed(struct device *dev,
- }
-
- static ssize_t applesmc_show_fan_manual(struct device *dev,
-- struct device_attribute *devattr, char *sysfsbuf)
-+ struct device_attribute *attr, char *sysfsbuf)
- {
- int ret;
- u16 manual = 0;
- u8 buffer[2];
-- struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
--
-- mutex_lock(&applesmc_lock);
-
- ret = applesmc_read_key(FANS_MANUAL, buffer, 2);
-- manual = ((buffer[0] << 8 | buffer[1]) >> attr->index) & 0x01;
-+ manual = ((buffer[0] << 8 | buffer[1]) >> to_index(attr)) & 0x01;
-
-- mutex_unlock(&applesmc_lock);
- if (ret)
- return ret;
- else
-@@ -780,28 +819,25 @@ static ssize_t applesmc_show_fan_manual(struct device *dev,
- }
-
- static ssize_t applesmc_store_fan_manual(struct device *dev,
-- struct device_attribute *devattr,
-+ struct device_attribute *attr,
- const char *sysfsbuf, size_t count)
- {
- int ret;
- u8 buffer[2];
- u32 input;
- u16 val;
-- struct sensor_device_attribute *attr = to_sensor_dev_attr(devattr);
-
- input = simple_strtoul(sysfsbuf, NULL, 10);
-
-- mutex_lock(&applesmc_lock);
--
- ret = applesmc_read_key(FANS_MANUAL, buffer, 2);
- val = (buffer[0] << 8 | buffer[1]);
- if (ret)
- goto out;
-
- if (input)
-- val = val | (0x01 << attr->index);
-+ val = val | (0x01 << to_index(attr));
- else
-- val = val & ~(0x01 << attr->index);
-+ val = val & ~(0x01 << to_index(attr));
-
- buffer[0] = (val >> 8) & 0xFF;
- buffer[1] = val & 0xFF;
-@@ -809,7 +845,6 @@ static ssize_t applesmc_store_fan_manual(struct device *dev,
- ret = applesmc_write_key(FANS_MANUAL, buffer, 2);
-
- out:
-- mutex_unlock(&applesmc_lock);
- if (ret)
- return ret;
- else
-@@ -822,21 +857,12 @@ static ssize_t applesmc_show_fan_position(struct device *dev,
- int ret;
- char newkey[5];
- u8 buffer[17];
-- struct sensor_device_attribute_2 *sensor_attr =
-- to_sensor_dev_attr_2(attr);
-
-- newkey[0] = FAN_POSITION[0];
-- newkey[1] = '0' + sensor_attr->index;
-- newkey[2] = FAN_POSITION[2];
-- newkey[3] = FAN_POSITION[3];
-- newkey[4] = 0;
--
-- mutex_lock(&applesmc_lock);
-+ sprintf(newkey, FAN_ID_FMT, to_index(attr));
-
- ret = applesmc_read_key(newkey, buffer, 16);
- buffer[16] = 0;
-
-- mutex_unlock(&applesmc_lock);
- if (ret)
- return ret;
- else
-@@ -852,18 +878,14 @@ static ssize_t applesmc_calibrate_show(struct device *dev,
- static ssize_t applesmc_calibrate_store(struct device *dev,
- struct device_attribute *attr, const char *sysfsbuf, size_t count)
- {
-- mutex_lock(&applesmc_lock);
- applesmc_calibrate();
-- mutex_unlock(&applesmc_lock);
-
- return count;
- }
-
- static void applesmc_backlight_set(struct work_struct *work)
- {
-- mutex_lock(&applesmc_lock);
- applesmc_write_key(BACKLIGHT_KEY, backlight_state, 2);
-- mutex_unlock(&applesmc_lock);
- }
- static DECLARE_WORK(backlight_work, &applesmc_backlight_set);
-
-@@ -886,13 +908,10 @@ static ssize_t applesmc_key_count_show(struct device *dev,
- u8 buffer[4];
- u32 count;
-
-- mutex_lock(&applesmc_lock);
--
- ret = applesmc_read_key(KEY_COUNT_KEY, buffer, 4);
- count = ((u32)buffer[0]<<24) + ((u32)buffer[1]<<16) +
- ((u32)buffer[2]<<8) + buffer[3];
-
-- mutex_unlock(&applesmc_lock);
- if (ret)
- return ret;
- else
-@@ -902,113 +921,53 @@ static ssize_t applesmc_key_count_show(struct device *dev,
- static ssize_t applesmc_key_at_index_read_show(struct device *dev,
- struct device_attribute *attr, char *sysfsbuf)
- {
-- char key[5];
-- char info[6];
-+ const struct applesmc_entry *entry;
- int ret;
-
-- mutex_lock(&applesmc_lock);
--
-- ret = applesmc_get_key_at_index(key_at_index, key);
--
-- if (ret || !key[0]) {
-- mutex_unlock(&applesmc_lock);
--
-- return -EINVAL;
-- }
--
-- ret = applesmc_get_key_type(key, info);
--
-- if (ret) {
-- mutex_unlock(&applesmc_lock);
--
-+ entry = applesmc_get_entry_by_index(key_at_index);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-+ ret = applesmc_read_entry(entry, sysfsbuf, entry->len);
-+ if (ret)
- return ret;
-- }
--
-- /*
-- * info[0] maximum value (APPLESMC_MAX_DATA_LENGTH) is much lower than
-- * PAGE_SIZE, so we don't need any checks before writing to sysfsbuf.
-- */
-- ret = applesmc_read_key(key, sysfsbuf, info[0]);
--
-- mutex_unlock(&applesmc_lock);
-
-- if (!ret) {
-- return info[0];
-- } else {
-- return ret;
-- }
-+ return entry->len;
- }
-
- static ssize_t applesmc_key_at_index_data_length_show(struct device *dev,
- struct device_attribute *attr, char *sysfsbuf)
- {
-- char key[5];
-- char info[6];
-- int ret;
--
-- mutex_lock(&applesmc_lock);
-+ const struct applesmc_entry *entry;
-
-- ret = applesmc_get_key_at_index(key_at_index, key);
-+ entry = applesmc_get_entry_by_index(key_at_index);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-
-- if (ret || !key[0]) {
-- mutex_unlock(&applesmc_lock);
--
-- return -EINVAL;
-- }
--
-- ret = applesmc_get_key_type(key, info);
--
-- mutex_unlock(&applesmc_lock);
--
-- if (!ret)
-- return snprintf(sysfsbuf, PAGE_SIZE, "%d\n", info[0]);
-- else
-- return ret;
-+ return snprintf(sysfsbuf, PAGE_SIZE, "%d\n", entry->len);
- }
-
- static ssize_t applesmc_key_at_index_type_show(struct device *dev,
- struct device_attribute *attr, char *sysfsbuf)
- {
-- char key[5];
-- char info[6];
-- int ret;
-+ const struct applesmc_entry *entry;
-
-- mutex_lock(&applesmc_lock);
-+ entry = applesmc_get_entry_by_index(key_at_index);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-
-- ret = applesmc_get_key_at_index(key_at_index, key);
--
-- if (ret || !key[0]) {
-- mutex_unlock(&applesmc_lock);
--
-- return -EINVAL;
-- }
--
-- ret = applesmc_get_key_type(key, info);
--
-- mutex_unlock(&applesmc_lock);
--
-- if (!ret)
-- return snprintf(sysfsbuf, PAGE_SIZE, "%s\n", info+1);
-- else
-- return ret;
-+ return snprintf(sysfsbuf, PAGE_SIZE, "%s\n", entry->type);
- }
-
- static ssize_t applesmc_key_at_index_name_show(struct device *dev,
- struct device_attribute *attr, char *sysfsbuf)
- {
-- char key[5];
-- int ret;
--
-- mutex_lock(&applesmc_lock);
--
-- ret = applesmc_get_key_at_index(key_at_index, key);
-+ const struct applesmc_entry *entry;
-
-- mutex_unlock(&applesmc_lock);
-+ entry = applesmc_get_entry_by_index(key_at_index);
-+ if (IS_ERR(entry))
-+ return PTR_ERR(entry);
-
-- if (!ret && key[0])
-- return snprintf(sysfsbuf, PAGE_SIZE, "%s\n", key);
-- else
-- return -EINVAL;
-+ return snprintf(sysfsbuf, PAGE_SIZE, "%s\n", entry->key);
- }
-
- static ssize_t applesmc_key_at_index_show(struct device *dev,
-@@ -1020,12 +979,8 @@ static ssize_t applesmc_key_at_index_show(struct device *dev,
- static ssize_t applesmc_key_at_index_store(struct device *dev,
- struct device_attribute *attr, const char *sysfsbuf, size_t count)
- {
-- mutex_lock(&applesmc_lock);
--
- key_at_index = simple_strtoul(sysfsbuf, NULL, 10);
-
-- mutex_unlock(&applesmc_lock);
--
- return count;
- }
-
-@@ -1035,387 +990,101 @@ static struct led_classdev applesmc_backlight = {
- .brightness_set = applesmc_brightness_set,
- };
-
--static DEVICE_ATTR(name, 0444, applesmc_name_show, NULL);
--
--static DEVICE_ATTR(position, 0444, applesmc_position_show, NULL);
--static DEVICE_ATTR(calibrate, 0644,
-- applesmc_calibrate_show, applesmc_calibrate_store);
--
--static struct attribute *accelerometer_attributes[] = {
-- &dev_attr_position.attr,
-- &dev_attr_calibrate.attr,
-- NULL
--};
--
--static const struct attribute_group accelerometer_attributes_group =
-- { .attrs = accelerometer_attributes };
--
--static DEVICE_ATTR(light, 0444, applesmc_light_show, NULL);
--
--static DEVICE_ATTR(key_count, 0444, applesmc_key_count_show, NULL);
--static DEVICE_ATTR(key_at_index, 0644,
-- applesmc_key_at_index_show, applesmc_key_at_index_store);
--static DEVICE_ATTR(key_at_index_name, 0444,
-- applesmc_key_at_index_name_show, NULL);
--static DEVICE_ATTR(key_at_index_type, 0444,
-- applesmc_key_at_index_type_show, NULL);
--static DEVICE_ATTR(key_at_index_data_length, 0444,
-- applesmc_key_at_index_data_length_show, NULL);
--static DEVICE_ATTR(key_at_index_data, 0444,
-- applesmc_key_at_index_read_show, NULL);
--
--static struct attribute *key_enumeration_attributes[] = {
-- &dev_attr_key_count.attr,
-- &dev_attr_key_at_index.attr,
-- &dev_attr_key_at_index_name.attr,
-- &dev_attr_key_at_index_type.attr,
-- &dev_attr_key_at_index_data_length.attr,
-- &dev_attr_key_at_index_data.attr,
-- NULL
--};
--
--static const struct attribute_group key_enumeration_group =
-- { .attrs = key_enumeration_attributes };
--
--/*
-- * Macro defining SENSOR_DEVICE_ATTR for a fan sysfs entries.
-- * - show actual speed
-- * - show/store minimum speed
-- * - show maximum speed
-- * - show safe speed
-- * - show/store target speed
-- * - show/store manual mode
-- */
--#define sysfs_fan_speeds_offset(offset) \
--static SENSOR_DEVICE_ATTR_2(fan##offset##_input, S_IRUGO, \
-- applesmc_show_fan_speed, NULL, 0, offset-1); \
--\
--static SENSOR_DEVICE_ATTR_2(fan##offset##_min, S_IRUGO | S_IWUSR, \
-- applesmc_show_fan_speed, applesmc_store_fan_speed, 1, offset-1); \
--\
--static SENSOR_DEVICE_ATTR_2(fan##offset##_max, S_IRUGO, \
-- applesmc_show_fan_speed, NULL, 2, offset-1); \
--\
--static SENSOR_DEVICE_ATTR_2(fan##offset##_safe, S_IRUGO, \
-- applesmc_show_fan_speed, NULL, 3, offset-1); \
--\
--static SENSOR_DEVICE_ATTR_2(fan##offset##_output, S_IRUGO | S_IWUSR, \
-- applesmc_show_fan_speed, applesmc_store_fan_speed, 4, offset-1); \
--\
--static SENSOR_DEVICE_ATTR(fan##offset##_manual, S_IRUGO | S_IWUSR, \
-- applesmc_show_fan_manual, applesmc_store_fan_manual, offset-1); \
--\
--static SENSOR_DEVICE_ATTR(fan##offset##_label, S_IRUGO, \
-- applesmc_show_fan_position, NULL, offset-1); \
--\
--static struct attribute *fan##offset##_attributes[] = { \
-- &sensor_dev_attr_fan##offset##_input.dev_attr.attr, \
-- &sensor_dev_attr_fan##offset##_min.dev_attr.attr, \
-- &sensor_dev_attr_fan##offset##_max.dev_attr.attr, \
-- &sensor_dev_attr_fan##offset##_safe.dev_attr.attr, \
-- &sensor_dev_attr_fan##offset##_output.dev_attr.attr, \
-- &sensor_dev_attr_fan##offset##_manual.dev_attr.attr, \
-- &sensor_dev_attr_fan##offset##_label.dev_attr.attr, \
-- NULL \
-+static struct applesmc_node_group info_group[] = {
-+ { "name", applesmc_name_show },
-+ { "key_count", applesmc_key_count_show },
-+ { "key_at_index", applesmc_key_at_index_show, applesmc_key_at_index_store },
-+ { "key_at_index_name", applesmc_key_at_index_name_show },
-+ { "key_at_index_type", applesmc_key_at_index_type_show },
-+ { "key_at_index_data_length", applesmc_key_at_index_data_length_show },
-+ { "key_at_index_data", applesmc_key_at_index_read_show },
-+ { }
- };
-
--/*
-- * Create the needed functions for each fan using the macro defined above
-- * (4 fans are supported)
-- */
--sysfs_fan_speeds_offset(1);
--sysfs_fan_speeds_offset(2);
--sysfs_fan_speeds_offset(3);
--sysfs_fan_speeds_offset(4);
--
--static const struct attribute_group fan_attribute_groups[] = {
-- { .attrs = fan1_attributes },
-- { .attrs = fan2_attributes },
-- { .attrs = fan3_attributes },
-- { .attrs = fan4_attributes },
-+static struct applesmc_node_group accelerometer_group[] = {
-+ { "position", applesmc_position_show },
-+ { "calibrate", applesmc_calibrate_show, applesmc_calibrate_store },
-+ { }
- };
-
--/*
-- * Temperature sensors sysfs entries.
-- */
--static SENSOR_DEVICE_ATTR(temp1_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 0);
--static SENSOR_DEVICE_ATTR(temp2_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 1);
--static SENSOR_DEVICE_ATTR(temp3_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 2);
--static SENSOR_DEVICE_ATTR(temp4_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 3);
--static SENSOR_DEVICE_ATTR(temp5_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 4);
--static SENSOR_DEVICE_ATTR(temp6_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 5);
--static SENSOR_DEVICE_ATTR(temp7_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 6);
--static SENSOR_DEVICE_ATTR(temp8_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 7);
--static SENSOR_DEVICE_ATTR(temp9_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 8);
--static SENSOR_DEVICE_ATTR(temp10_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 9);
--static SENSOR_DEVICE_ATTR(temp11_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 10);
--static SENSOR_DEVICE_ATTR(temp12_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 11);
--static SENSOR_DEVICE_ATTR(temp13_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 12);
--static SENSOR_DEVICE_ATTR(temp14_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 13);
--static SENSOR_DEVICE_ATTR(temp15_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 14);
--static SENSOR_DEVICE_ATTR(temp16_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 15);
--static SENSOR_DEVICE_ATTR(temp17_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 16);
--static SENSOR_DEVICE_ATTR(temp18_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 17);
--static SENSOR_DEVICE_ATTR(temp19_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 18);
--static SENSOR_DEVICE_ATTR(temp20_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 19);
--static SENSOR_DEVICE_ATTR(temp21_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 20);
--static SENSOR_DEVICE_ATTR(temp22_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 21);
--static SENSOR_DEVICE_ATTR(temp23_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 22);
--static SENSOR_DEVICE_ATTR(temp24_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 23);
--static SENSOR_DEVICE_ATTR(temp25_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 24);
--static SENSOR_DEVICE_ATTR(temp26_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 25);
--static SENSOR_DEVICE_ATTR(temp27_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 26);
--static SENSOR_DEVICE_ATTR(temp28_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 27);
--static SENSOR_DEVICE_ATTR(temp29_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 28);
--static SENSOR_DEVICE_ATTR(temp30_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 29);
--static SENSOR_DEVICE_ATTR(temp31_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 30);
--static SENSOR_DEVICE_ATTR(temp32_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 31);
--static SENSOR_DEVICE_ATTR(temp33_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 32);
--static SENSOR_DEVICE_ATTR(temp34_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 33);
--static SENSOR_DEVICE_ATTR(temp35_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 34);
--static SENSOR_DEVICE_ATTR(temp36_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 35);
--static SENSOR_DEVICE_ATTR(temp37_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 36);
--static SENSOR_DEVICE_ATTR(temp38_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 37);
--static SENSOR_DEVICE_ATTR(temp39_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 38);
--static SENSOR_DEVICE_ATTR(temp40_label, S_IRUGO,
-- applesmc_show_sensor_label, NULL, 39);
--static SENSOR_DEVICE_ATTR(temp1_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 0);
--static SENSOR_DEVICE_ATTR(temp2_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 1);
--static SENSOR_DEVICE_ATTR(temp3_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 2);
--static SENSOR_DEVICE_ATTR(temp4_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 3);
--static SENSOR_DEVICE_ATTR(temp5_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 4);
--static SENSOR_DEVICE_ATTR(temp6_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 5);
--static SENSOR_DEVICE_ATTR(temp7_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 6);
--static SENSOR_DEVICE_ATTR(temp8_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 7);
--static SENSOR_DEVICE_ATTR(temp9_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 8);
--static SENSOR_DEVICE_ATTR(temp10_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 9);
--static SENSOR_DEVICE_ATTR(temp11_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 10);
--static SENSOR_DEVICE_ATTR(temp12_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 11);
--static SENSOR_DEVICE_ATTR(temp13_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 12);
--static SENSOR_DEVICE_ATTR(temp14_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 13);
--static SENSOR_DEVICE_ATTR(temp15_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 14);
--static SENSOR_DEVICE_ATTR(temp16_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 15);
--static SENSOR_DEVICE_ATTR(temp17_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 16);
--static SENSOR_DEVICE_ATTR(temp18_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 17);
--static SENSOR_DEVICE_ATTR(temp19_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 18);
--static SENSOR_DEVICE_ATTR(temp20_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 19);
--static SENSOR_DEVICE_ATTR(temp21_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 20);
--static SENSOR_DEVICE_ATTR(temp22_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 21);
--static SENSOR_DEVICE_ATTR(temp23_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 22);
--static SENSOR_DEVICE_ATTR(temp24_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 23);
--static SENSOR_DEVICE_ATTR(temp25_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 24);
--static SENSOR_DEVICE_ATTR(temp26_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 25);
--static SENSOR_DEVICE_ATTR(temp27_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 26);
--static SENSOR_DEVICE_ATTR(temp28_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 27);
--static SENSOR_DEVICE_ATTR(temp29_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 28);
--static SENSOR_DEVICE_ATTR(temp30_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 29);
--static SENSOR_DEVICE_ATTR(temp31_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 30);
--static SENSOR_DEVICE_ATTR(temp32_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 31);
--static SENSOR_DEVICE_ATTR(temp33_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 32);
--static SENSOR_DEVICE_ATTR(temp34_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 33);
--static SENSOR_DEVICE_ATTR(temp35_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 34);
--static SENSOR_DEVICE_ATTR(temp36_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 35);
--static SENSOR_DEVICE_ATTR(temp37_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 36);
--static SENSOR_DEVICE_ATTR(temp38_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 37);
--static SENSOR_DEVICE_ATTR(temp39_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 38);
--static SENSOR_DEVICE_ATTR(temp40_input, S_IRUGO,
-- applesmc_show_temperature, NULL, 39);
--
--static struct attribute *label_attributes[] = {
-- &sensor_dev_attr_temp1_label.dev_attr.attr,
-- &sensor_dev_attr_temp2_label.dev_attr.attr,
-- &sensor_dev_attr_temp3_label.dev_attr.attr,
-- &sensor_dev_attr_temp4_label.dev_attr.attr,
-- &sensor_dev_attr_temp5_label.dev_attr.attr,
-- &sensor_dev_attr_temp6_label.dev_attr.attr,
-- &sensor_dev_attr_temp7_label.dev_attr.attr,
-- &sensor_dev_attr_temp8_label.dev_attr.attr,
-- &sensor_dev_attr_temp9_label.dev_attr.attr,
-- &sensor_dev_attr_temp10_label.dev_attr.attr,
-- &sensor_dev_attr_temp11_label.dev_attr.attr,
-- &sensor_dev_attr_temp12_label.dev_attr.attr,
-- &sensor_dev_attr_temp13_label.dev_attr.attr,
-- &sensor_dev_attr_temp14_label.dev_attr.attr,
-- &sensor_dev_attr_temp15_label.dev_attr.attr,
-- &sensor_dev_attr_temp16_label.dev_attr.attr,
-- &sensor_dev_attr_temp17_label.dev_attr.attr,
-- &sensor_dev_attr_temp18_label.dev_attr.attr,
-- &sensor_dev_attr_temp19_label.dev_attr.attr,
-- &sensor_dev_attr_temp20_label.dev_attr.attr,
-- &sensor_dev_attr_temp21_label.dev_attr.attr,
-- &sensor_dev_attr_temp22_label.dev_attr.attr,
-- &sensor_dev_attr_temp23_label.dev_attr.attr,
-- &sensor_dev_attr_temp24_label.dev_attr.attr,
-- &sensor_dev_attr_temp25_label.dev_attr.attr,
-- &sensor_dev_attr_temp26_label.dev_attr.attr,
-- &sensor_dev_attr_temp27_label.dev_attr.attr,
-- &sensor_dev_attr_temp28_label.dev_attr.attr,
-- &sensor_dev_attr_temp29_label.dev_attr.attr,
-- &sensor_dev_attr_temp30_label.dev_attr.attr,
-- &sensor_dev_attr_temp31_label.dev_attr.attr,
-- &sensor_dev_attr_temp32_label.dev_attr.attr,
-- &sensor_dev_attr_temp33_label.dev_attr.attr,
-- &sensor_dev_attr_temp34_label.dev_attr.attr,
-- &sensor_dev_attr_temp35_label.dev_attr.attr,
-- &sensor_dev_attr_temp36_label.dev_attr.attr,
-- &sensor_dev_attr_temp37_label.dev_attr.attr,
-- &sensor_dev_attr_temp38_label.dev_attr.attr,
-- &sensor_dev_attr_temp39_label.dev_attr.attr,
-- &sensor_dev_attr_temp40_label.dev_attr.attr,
-- NULL
-+static struct applesmc_node_group light_sensor_group[] = {
-+ { "light", applesmc_light_show },
-+ { }
- };
-
--static struct attribute *temperature_attributes[] = {
-- &sensor_dev_attr_temp1_input.dev_attr.attr,
-- &sensor_dev_attr_temp2_input.dev_attr.attr,
-- &sensor_dev_attr_temp3_input.dev_attr.attr,
-- &sensor_dev_attr_temp4_input.dev_attr.attr,
-- &sensor_dev_attr_temp5_input.dev_attr.attr,
-- &sensor_dev_attr_temp6_input.dev_attr.attr,
-- &sensor_dev_attr_temp7_input.dev_attr.attr,
-- &sensor_dev_attr_temp8_input.dev_attr.attr,
-- &sensor_dev_attr_temp9_input.dev_attr.attr,
-- &sensor_dev_attr_temp10_input.dev_attr.attr,
-- &sensor_dev_attr_temp11_input.dev_attr.attr,
-- &sensor_dev_attr_temp12_input.dev_attr.attr,
-- &sensor_dev_attr_temp13_input.dev_attr.attr,
-- &sensor_dev_attr_temp14_input.dev_attr.attr,
-- &sensor_dev_attr_temp15_input.dev_attr.attr,
-- &sensor_dev_attr_temp16_input.dev_attr.attr,
-- &sensor_dev_attr_temp17_input.dev_attr.attr,
-- &sensor_dev_attr_temp18_input.dev_attr.attr,
-- &sensor_dev_attr_temp19_input.dev_attr.attr,
-- &sensor_dev_attr_temp20_input.dev_attr.attr,
-- &sensor_dev_attr_temp21_input.dev_attr.attr,
-- &sensor_dev_attr_temp22_input.dev_attr.attr,
-- &sensor_dev_attr_temp23_input.dev_attr.attr,
-- &sensor_dev_attr_temp24_input.dev_attr.attr,
-- &sensor_dev_attr_temp25_input.dev_attr.attr,
-- &sensor_dev_attr_temp26_input.dev_attr.attr,
-- &sensor_dev_attr_temp27_input.dev_attr.attr,
-- &sensor_dev_attr_temp28_input.dev_attr.attr,
-- &sensor_dev_attr_temp29_input.dev_attr.attr,
-- &sensor_dev_attr_temp30_input.dev_attr.attr,
-- &sensor_dev_attr_temp31_input.dev_attr.attr,
-- &sensor_dev_attr_temp32_input.dev_attr.attr,
-- &sensor_dev_attr_temp33_input.dev_attr.attr,
-- &sensor_dev_attr_temp34_input.dev_attr.attr,
-- &sensor_dev_attr_temp35_input.dev_attr.attr,
-- &sensor_dev_attr_temp36_input.dev_attr.attr,
-- &sensor_dev_attr_temp37_input.dev_attr.attr,
-- &sensor_dev_attr_temp38_input.dev_attr.attr,
-- &sensor_dev_attr_temp39_input.dev_attr.attr,
-- &sensor_dev_attr_temp40_input.dev_attr.attr,
-- NULL
-+static struct applesmc_node_group fan_group[] = {
-+ { "fan%d_label", applesmc_show_fan_position },
-+ { "fan%d_input", applesmc_show_fan_speed, NULL, 0 },
-+ { "fan%d_min", applesmc_show_fan_speed, applesmc_store_fan_speed, 1 },
-+ { "fan%d_max", applesmc_show_fan_speed, NULL, 2 },
-+ { "fan%d_safe", applesmc_show_fan_speed, NULL, 3 },
-+ { "fan%d_output", applesmc_show_fan_speed, applesmc_store_fan_speed, 4 },
-+ { "fan%d_manual", applesmc_show_fan_manual, applesmc_store_fan_manual },
-+ { }
- };
-
--static const struct attribute_group temperature_attributes_group =
-- { .attrs = temperature_attributes };
--
--static const struct attribute_group label_attributes_group = {
-- .attrs = label_attributes
-+static struct applesmc_node_group temp_group[] = {
-+ { "temp%d_label", applesmc_show_sensor_label },
-+ { "temp%d_input", applesmc_show_temperature },
-+ { }
- };
-
- /* Module stuff */
-
- /*
-- * applesmc_dmi_match - found a match. return one, short-circuiting the hunt.
-+ * applesmc_destroy_nodes - remove files and free associated memory
- */
--static int applesmc_dmi_match(const struct dmi_system_id *id)
-+static void applesmc_destroy_nodes(struct applesmc_node_group *groups)
- {
-- int i = 0;
-- struct dmi_match_data* dmi_data = id->driver_data;
-- printk(KERN_INFO "applesmc: %s detected:\n", id->ident);
-- applesmc_accelerometer = dmi_data->accelerometer;
-- printk(KERN_INFO "applesmc: - Model %s accelerometer\n",
-- applesmc_accelerometer ? "with" : "without");
-- applesmc_light = dmi_data->light;
-- printk(KERN_INFO "applesmc: - Model %s light sensors and backlight\n",
-- applesmc_light ? "with" : "without");
--
-- applesmc_temperature_set = dmi_data->temperature_set;
-- while (temperature_sensors_sets[applesmc_temperature_set][i] != NULL)
-- i++;
-- printk(KERN_INFO "applesmc: - Model with %d temperature sensors\n", i);
-- return 1;
-+ struct applesmc_node_group *grp;
-+ struct applesmc_dev_attr *node;
-+
-+ for (grp = groups; grp->nodes; grp++) {
-+ for (node = grp->nodes; node->sda.dev_attr.attr.name; node++)
-+ sysfs_remove_file(&pdev->dev.kobj,
-+ &node->sda.dev_attr.attr);
-+ kfree(grp->nodes);
-+ grp->nodes = NULL;
-+ }
-+}
-+
-+/*
-+ * applesmc_create_nodes - create a two-dimensional group of sysfs files
-+ */
-+static int applesmc_create_nodes(struct applesmc_node_group *groups, int num)
-+{
-+ struct applesmc_node_group *grp;
-+ struct applesmc_dev_attr *node;
-+ struct attribute *attr;
-+ int ret, i;
-+
-+ for (grp = groups; grp->format; grp++) {
-+ grp->nodes = kcalloc(num + 1, sizeof(*node), GFP_KERNEL);
-+ if (!grp->nodes) {
-+ ret = -ENOMEM;
-+ goto out;
-+ }
-+ for (i = 0; i < num; i++) {
-+ node = &grp->nodes[i];
-+ sprintf(node->name, grp->format, i + 1);
-+ node->sda.index = (grp->option << 16) | (i & 0xffff);
-+ node->sda.dev_attr.show = grp->show;
-+ node->sda.dev_attr.store = grp->store;
-+ attr = &node->sda.dev_attr.attr;
-+ attr->name = node->name;
-+ attr->mode = S_IRUGO | (grp->store ? S_IWUSR : 0);
-+ ret = sysfs_create_file(&pdev->dev.kobj, attr);
-+ if (ret) {
-+ attr->name = NULL;
-+ goto out;
-+ }
-+ }
-+ }
-+
-+ return 0;
-+out:
-+ applesmc_destroy_nodes(groups);
-+ return ret;
- }
-
- /* Create accelerometer ressources */
-@@ -1424,8 +1093,10 @@ static int applesmc_create_accelerometer(void)
- struct input_dev *idev;
- int ret;
-
-- ret = sysfs_create_group(&pdev->dev.kobj,
-- &accelerometer_attributes_group);
-+ if (!smcreg.has_accelerometer)
-+ return 0;
-+
-+ ret = applesmc_create_nodes(accelerometer_group, 1);
- if (ret)
- goto out;
-
-@@ -1462,282 +1133,132 @@ out_idev:
- input_free_polled_device(applesmc_idev);
-
- out_sysfs:
-- sysfs_remove_group(&pdev->dev.kobj, &accelerometer_attributes_group);
-+ applesmc_destroy_nodes(accelerometer_group);
-
- out:
-- printk(KERN_WARNING "applesmc: driver init failed (ret=%d)!\n", ret);
-+ pr_warn("driver init failed (ret=%d)!\n", ret);
- return ret;
- }
-
- /* Release all ressources used by the accelerometer */
- static void applesmc_release_accelerometer(void)
- {
-+ if (!smcreg.has_accelerometer)
-+ return;
- input_unregister_polled_device(applesmc_idev);
- input_free_polled_device(applesmc_idev);
-- sysfs_remove_group(&pdev->dev.kobj, &accelerometer_attributes_group);
-+ applesmc_destroy_nodes(accelerometer_group);
- }
-
--static __initdata struct dmi_match_data applesmc_dmi_data[] = {
--/* MacBook Pro: accelerometer, backlight and temperature set 0 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 0 },
--/* MacBook2: accelerometer and temperature set 1 */
-- { .accelerometer = 1, .light = 0, .temperature_set = 1 },
--/* MacBook: accelerometer and temperature set 2 */
-- { .accelerometer = 1, .light = 0, .temperature_set = 2 },
--/* MacMini: temperature set 3 */
-- { .accelerometer = 0, .light = 0, .temperature_set = 3 },
--/* MacPro: temperature set 4 */
-- { .accelerometer = 0, .light = 0, .temperature_set = 4 },
--/* iMac: temperature set 5 */
-- { .accelerometer = 0, .light = 0, .temperature_set = 5 },
--/* MacBook3, MacBook4: accelerometer and temperature set 6 */
-- { .accelerometer = 1, .light = 0, .temperature_set = 6 },
--/* MacBook Air: accelerometer, backlight and temperature set 7 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 7 },
--/* MacBook Pro 4: accelerometer, backlight and temperature set 8 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 8 },
--/* MacBook Pro 3: accelerometer, backlight and temperature set 9 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 9 },
--/* iMac 5: light sensor only, temperature set 10 */
-- { .accelerometer = 0, .light = 0, .temperature_set = 10 },
--/* MacBook 5: accelerometer, backlight and temperature set 11 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 11 },
--/* MacBook Pro 5: accelerometer, backlight and temperature set 12 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 12 },
--/* iMac 8: light sensor only, temperature set 13 */
-- { .accelerometer = 0, .light = 0, .temperature_set = 13 },
--/* iMac 6: light sensor only, temperature set 14 */
-- { .accelerometer = 0, .light = 0, .temperature_set = 14 },
--/* MacBook Air 2,1: accelerometer, backlight and temperature set 15 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 15 },
--/* MacPro3,1: temperature set 16 */
-- { .accelerometer = 0, .light = 0, .temperature_set = 16 },
--/* iMac 9,1: light sensor only, temperature set 17 */
-- { .accelerometer = 0, .light = 0, .temperature_set = 17 },
--/* MacBook Pro 2,2: accelerometer, backlight and temperature set 18 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 18 },
--/* MacBook Pro 5,3: accelerometer, backlight and temperature set 19 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 19 },
--/* MacBook Pro 5,4: accelerometer, backlight and temperature set 20 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 20 },
--/* MacBook Pro 6,2: accelerometer, backlight and temperature set 21 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 21 },
--/* MacBook Pro 7,1: accelerometer, backlight and temperature set 22 */
-- { .accelerometer = 1, .light = 1, .temperature_set = 22 },
--};
-+static int applesmc_create_light_sensor(void)
-+{
-+ if (!smcreg.num_light_sensors)
-+ return 0;
-+ return applesmc_create_nodes(light_sensor_group, 1);
-+}
-
--/* Note that DMI_MATCH(...,"MacBook") will match "MacBookPro1,1".
-- * So we need to put "Apple MacBook Pro" before "Apple MacBook". */
--static __initdata struct dmi_system_id applesmc_whitelist[] = {
-- { applesmc_dmi_match, "Apple MacBook Air 2", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookAir2") },
-- &applesmc_dmi_data[15]},
-- { applesmc_dmi_match, "Apple MacBook Air", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookAir") },
-- &applesmc_dmi_data[7]},
-- { applesmc_dmi_match, "Apple MacBook Pro 7", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro7") },
-- &applesmc_dmi_data[22]},
-- { applesmc_dmi_match, "Apple MacBook Pro 5,4", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro5,4") },
-- &applesmc_dmi_data[20]},
-- { applesmc_dmi_match, "Apple MacBook Pro 5,3", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro5,3") },
-- &applesmc_dmi_data[19]},
-- { applesmc_dmi_match, "Apple MacBook Pro 6", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro6") },
-- &applesmc_dmi_data[21]},
-- { applesmc_dmi_match, "Apple MacBook Pro 5", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro5") },
-- &applesmc_dmi_data[12]},
-- { applesmc_dmi_match, "Apple MacBook Pro 4", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro4") },
-- &applesmc_dmi_data[8]},
-- { applesmc_dmi_match, "Apple MacBook Pro 3", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro3") },
-- &applesmc_dmi_data[9]},
-- { applesmc_dmi_match, "Apple MacBook Pro 2,2", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple Computer, Inc."),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro2,2") },
-- &applesmc_dmi_data[18]},
-- { applesmc_dmi_match, "Apple MacBook Pro", {
-- DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME,"MacBookPro") },
-- &applesmc_dmi_data[0]},
-- { applesmc_dmi_match, "Apple MacBook (v2)", {
-- DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME,"MacBook2") },
-- &applesmc_dmi_data[1]},
-- { applesmc_dmi_match, "Apple MacBook (v3)", {
-- DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME,"MacBook3") },
-- &applesmc_dmi_data[6]},
-- { applesmc_dmi_match, "Apple MacBook 4", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBook4") },
-- &applesmc_dmi_data[6]},
-- { applesmc_dmi_match, "Apple MacBook 5", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacBook5") },
-- &applesmc_dmi_data[11]},
-- { applesmc_dmi_match, "Apple MacBook", {
-- DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME,"MacBook") },
-- &applesmc_dmi_data[2]},
-- { applesmc_dmi_match, "Apple Macmini", {
-- DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME,"Macmini") },
-- &applesmc_dmi_data[3]},
-- { applesmc_dmi_match, "Apple MacPro2", {
-- DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME,"MacPro2") },
-- &applesmc_dmi_data[4]},
-- { applesmc_dmi_match, "Apple MacPro3", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacPro3") },
-- &applesmc_dmi_data[16]},
-- { applesmc_dmi_match, "Apple MacPro", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "MacPro") },
-- &applesmc_dmi_data[4]},
-- { applesmc_dmi_match, "Apple iMac 9,1", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple Inc."),
-- DMI_MATCH(DMI_PRODUCT_NAME, "iMac9,1") },
-- &applesmc_dmi_data[17]},
-- { applesmc_dmi_match, "Apple iMac 8", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "iMac8") },
-- &applesmc_dmi_data[13]},
-- { applesmc_dmi_match, "Apple iMac 6", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "iMac6") },
-- &applesmc_dmi_data[14]},
-- { applesmc_dmi_match, "Apple iMac 5", {
-- DMI_MATCH(DMI_BOARD_VENDOR, "Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME, "iMac5") },
-- &applesmc_dmi_data[10]},
-- { applesmc_dmi_match, "Apple iMac", {
-- DMI_MATCH(DMI_BOARD_VENDOR,"Apple"),
-- DMI_MATCH(DMI_PRODUCT_NAME,"iMac") },
-- &applesmc_dmi_data[5]},
-- { .ident = NULL }
--};
-+static void applesmc_release_light_sensor(void)
-+{
-+ if (!smcreg.num_light_sensors)
-+ return;
-+ applesmc_destroy_nodes(light_sensor_group);
-+}
-
--static int __init applesmc_init(void)
-+static int applesmc_create_key_backlight(void)
-+{
-+ if (!smcreg.has_key_backlight)
-+ return 0;
-+ applesmc_led_wq = create_singlethread_workqueue("applesmc-led");
-+ if (!applesmc_led_wq)
-+ return -ENOMEM;
-+ return led_classdev_register(&pdev->dev, &applesmc_backlight);
-+}
-+
-+static void applesmc_release_key_backlight(void)
-+{
-+ if (!smcreg.has_key_backlight)
-+ return;
-+ led_classdev_unregister(&applesmc_backlight);
-+ destroy_workqueue(applesmc_led_wq);
-+}
-+
-+static void applesmc_set_interrupt(u8 enable)
-+{
-+ applesmc_write_key(NOTIFICATION_KEY, &enable, 1);
-+}
-+
-+static irqreturn_t applesmc_interrupt(int irq, void *dev)
-+{
-+ printk("SMC Interrupt\n");
-+ return IRQ_HANDLED;
-+}
-+
-+static int __devinit applesmc_pnp_probe(struct pnp_dev *dev,
-+ const struct pnp_device_id *dev_id)
- {
- int ret;
-- int count;
-- int i;
-+ struct resource *res;
-+ struct applesmc_pnp_device *applesmc_pnp_device;
-
-- if (!dmi_check_system(applesmc_whitelist)) {
-- printk(KERN_WARNING "applesmc: supported laptop not found!\n");
-- ret = -ENODEV;
-+ pdev = dev;
-+
-+ applesmc_pnp_device = kzalloc(sizeof(struct applesmc_pnp_device),
-+ GFP_KERNEL);
-+
-+ if (!applesmc_pnp_device) {
-+ ret = -ENOMEM;
- goto out;
- }
-
-- if (!request_region(APPLESMC_DATA_PORT, APPLESMC_NR_PORTS,
-- "applesmc")) {
-+ pnp_device = applesmc_pnp_device;
-+
-+ pnp_set_drvdata(dev, applesmc_pnp_device);
-+
-+ res = pnp_get_resource(dev, IORESOURCE_IO, 0);
-+
-+ if (!res) {
- ret = -ENXIO;
- goto out;
- }
-
-- ret = platform_driver_register(&applesmc_driver);
-- if (ret)
-- goto out_region;
-+ applesmc_pnp_device->iobase = res->start;
-+ applesmc_pnp_device->iolen = res->end - res->start + 1;
-
-- pdev = platform_device_register_simple("applesmc", APPLESMC_DATA_PORT,
-- NULL, 0);
-- if (IS_ERR(pdev)) {
-- ret = PTR_ERR(pdev);
-- goto out_driver;
-+ if (!request_region(pnp_device->iobase, pnp_device->iolen, "applesmc")) {
-+ ret = -ENXIO;
-+ goto out;
- }
-
-- ret = sysfs_create_file(&pdev->dev.kobj, &dev_attr_name.attr);
-+ /* create register cache */
-+ ret = applesmc_init_smcreg();
- if (ret)
-- goto out_device;
--
-- /* Create key enumeration sysfs files */
-- ret = sysfs_create_group(&pdev->dev.kobj, &key_enumeration_group);
-- if (ret)
-- goto out_name;
--
-- /* create fan files */
-- count = applesmc_get_fan_count();
-- if (count < 0)
-- printk(KERN_ERR "applesmc: Cannot get the number of fans.\n");
-- else
-- printk(KERN_INFO "applesmc: %d fans found.\n", count);
-+ goto out_region;
-
-- if (count > 4) {
-- count = 4;
-- printk(KERN_WARNING "applesmc: More than 4 fans found,"
-- " but at most 4 fans are supported"
-- " by the driver.\n");
-- }
-+ applesmc_device_init();
-
-- while (fans_handled < count) {
-- ret = sysfs_create_group(&pdev->dev.kobj,
-- &fan_attribute_groups[fans_handled]);
-- if (ret)
-- goto out_fans;
-- fans_handled++;
-- }
-+ ret = applesmc_create_nodes(info_group, 1);
-+ if (ret)
-+ goto out_smcreg;
-
-- for (i = 0;
-- temperature_sensors_sets[applesmc_temperature_set][i] != NULL;
-- i++) {
-- if (temperature_attributes[i] == NULL ||
-- label_attributes[i] == NULL) {
-- printk(KERN_ERR "applesmc: More temperature sensors "
-- "in temperature_sensors_sets (at least %i)"
-- "than available sysfs files in "
-- "temperature_attributes (%i), please report "
-- "this bug.\n", i, i-1);
-- goto out_temperature;
-- }
-- ret = sysfs_create_file(&pdev->dev.kobj,
-- temperature_attributes[i]);
-- if (ret)
-- goto out_temperature;
-- ret = sysfs_create_file(&pdev->dev.kobj,
-- label_attributes[i]);
-- if (ret)
-- goto out_temperature;
-- }
-+ ret = applesmc_create_nodes(fan_group, smcreg.fan_count);
-+ if (ret)
-+ goto out_info;
-
-- if (applesmc_accelerometer) {
-- ret = applesmc_create_accelerometer();
-- if (ret)
-- goto out_temperature;
-- }
-+ ret = applesmc_create_nodes(temp_group, smcreg.temp_count);
-+ if (ret)
-+ goto out_fans;
-
-- if (applesmc_light) {
-- /* Add light sensor file */
-- ret = sysfs_create_file(&pdev->dev.kobj, &dev_attr_light.attr);
-- if (ret)
-- goto out_accelerometer;
-+ ret = applesmc_create_accelerometer();
-+ if (ret)
-+ goto out_temperature;
-
-- /* Create the workqueue */
-- applesmc_led_wq = create_singlethread_workqueue("applesmc-led");
-- if (!applesmc_led_wq) {
-- ret = -ENOMEM;
-- goto out_light_sysfs;
-- }
-+ ret = applesmc_create_light_sensor();
-+ if (ret)
-+ goto out_accelerometer;
-
-- /* register as a led device */
-- ret = led_classdev_register(&pdev->dev, &applesmc_backlight);
-- if (ret < 0)
-- goto out_light_wq;
-- }
-+ ret = applesmc_create_key_backlight();
-+ if (ret)
-+ goto out_light_sysfs;
-
- hwmon_dev = hwmon_device_register(&pdev->dev);
- if (IS_ERR(hwmon_dev)) {
-@@ -1745,65 +1266,85 @@ static int __init applesmc_init(void)
- goto out_light_ledclass;
- }
-
-- printk(KERN_INFO "applesmc: driver successfully loaded.\n");
-+ res = pnp_get_resource(dev, IORESOURCE_IRQ, 0);
-+
-+ if (res) {
-+ applesmc_pnp_device->irq = res->start;
-+ ret = request_irq(res->start, applesmc_interrupt, IRQF_SHARED,
-+ "applesmc", dev);
-+ if (ret)
-+ applesmc_pnp_device->irq = 0;
-+ else
-+ applesmc_set_interrupt(1);
-+ }
-
- return 0;
-
- out_light_ledclass:
-- if (applesmc_light)
-- led_classdev_unregister(&applesmc_backlight);
--out_light_wq:
-- if (applesmc_light)
-- destroy_workqueue(applesmc_led_wq);
-+ applesmc_release_key_backlight();
- out_light_sysfs:
-- if (applesmc_light)
-- sysfs_remove_file(&pdev->dev.kobj, &dev_attr_light.attr);
-+ applesmc_release_light_sensor();
- out_accelerometer:
-- if (applesmc_accelerometer)
-- applesmc_release_accelerometer();
-+ applesmc_release_accelerometer();
- out_temperature:
-- sysfs_remove_group(&pdev->dev.kobj, &label_attributes_group);
-- sysfs_remove_group(&pdev->dev.kobj, &temperature_attributes_group);
-+ applesmc_destroy_nodes(temp_group);
- out_fans:
-- while (fans_handled)
-- sysfs_remove_group(&pdev->dev.kobj,
-- &fan_attribute_groups[--fans_handled]);
-- sysfs_remove_group(&pdev->dev.kobj, &key_enumeration_group);
--out_name:
-- sysfs_remove_file(&pdev->dev.kobj, &dev_attr_name.attr);
--out_device:
-- platform_device_unregister(pdev);
--out_driver:
-- platform_driver_unregister(&applesmc_driver);
-+ applesmc_destroy_nodes(fan_group);
-+out_info:
-+ applesmc_destroy_nodes(info_group);
-+out_smcreg:
-+ applesmc_destroy_smcreg();
- out_region:
-- release_region(APPLESMC_DATA_PORT, APPLESMC_NR_PORTS);
-+ release_region(pnp_device->iobase, pnp_device->iolen);
- out:
-- printk(KERN_WARNING "applesmc: driver init failed (ret=%d)!\n", ret);
-+ pr_warn("driver init failed (ret=%d)!\n", ret);
- return ret;
- }
-
--static void __exit applesmc_exit(void)
-+static void __devexit applesmc_pnp_remove(struct pnp_dev *dev)
- {
-- hwmon_device_unregister(hwmon_dev);
-- if (applesmc_light) {
-- led_classdev_unregister(&applesmc_backlight);
-- destroy_workqueue(applesmc_led_wq);
-- sysfs_remove_file(&pdev->dev.kobj, &dev_attr_light.attr);
-+ struct applesmc_pnp_device *applesmc_pnp_device = pnp_get_drvdata(dev);
-+
-+ if (applesmc_pnp_device->irq) {
-+ applesmc_set_interrupt(0);
-+ free_irq(applesmc_pnp_device->irq, dev);
- }
-- if (applesmc_accelerometer)
-- applesmc_release_accelerometer();
-- sysfs_remove_group(&pdev->dev.kobj, &label_attributes_group);
-- sysfs_remove_group(&pdev->dev.kobj, &temperature_attributes_group);
-- while (fans_handled)
-- sysfs_remove_group(&pdev->dev.kobj,
-- &fan_attribute_groups[--fans_handled]);
-- sysfs_remove_group(&pdev->dev.kobj, &key_enumeration_group);
-- sysfs_remove_file(&pdev->dev.kobj, &dev_attr_name.attr);
-- platform_device_unregister(pdev);
-- platform_driver_unregister(&applesmc_driver);
-- release_region(APPLESMC_DATA_PORT, APPLESMC_NR_PORTS);
--
-- printk(KERN_INFO "applesmc: driver unloaded.\n");
-+
-+ hwmon_device_unregister(hwmon_dev);
-+ applesmc_release_key_backlight();
-+ applesmc_release_light_sensor();
-+ applesmc_release_accelerometer();
-+ applesmc_destroy_nodes(temp_group);
-+ applesmc_destroy_nodes(fan_group);
-+ applesmc_destroy_nodes(info_group);
-+ applesmc_destroy_smcreg();
-+ release_region(pnp_device->iobase, pnp_device->iolen);
-+ kfree(applesmc_pnp_device);
-+}
-+
-+static const struct pnp_device_id applesmc_dev_table[] = {
-+ {"APP0001", 0},
-+ {"", 0},
-+};
-+
-+static struct pnp_driver applesmc_pnp_driver = {
-+ .name = "Apple SMC",
-+ .probe = applesmc_pnp_probe,
-+ .remove = applesmc_pnp_remove,
-+ .id_table = applesmc_dev_table,
-+ .driver = {
-+ .pm = &applesmc_pm_ops,
-+ },
-+};
-+
-+static int __init applesmc_init(void)
-+{
-+ return pnp_register_driver(&applesmc_pnp_driver);
-+}
-+
-+static void __exit applesmc_exit(void)
-+{
-+ pnp_unregister_driver(&applesmc_pnp_driver);
- }
-
- module_init(applesmc_init);
-@@ -1812,4 +1353,4 @@ module_exit(applesmc_exit);
- MODULE_AUTHOR("Nicolas Boichat");
- MODULE_DESCRIPTION("Apple SMC");
- MODULE_LICENSE("GPL v2");
--MODULE_DEVICE_TABLE(dmi, applesmc_whitelist);
-+MODULE_DEVICE_TABLE(pnp, applesmc_dev_table);
diff --git a/drm-intel-big-hammer.patch b/drm-intel-big-hammer.patch
index 97bb2e8cd..bf152af3b 100644
--- a/drm-intel-big-hammer.patch
+++ b/drm-intel-big-hammer.patch
@@ -1,13 +1,13 @@
omgwtfbbqchainsaw?
---
- drivers/gpu/drm/i915/i915_gem.c | 5 +++++
+ drivers/gpu/drm/i915/i915_gem_execbuffer.c | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)
-diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
-index 8eb8453..36fa9d7 100644
---- a/drivers/gpu/drm/i915/i915_gem.c
-+++ b/drivers/gpu/drm/i915/i915_gem.c
-@@ -3692,6 +3692,11 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data,
+diff --git a/drivers/gpu/drm/i915/i915_gem_execbuffer.c b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
+index e698343..21e601d1 100644
+--- a/drivers/gpu/drm/i915/i915_gem_execbuffer.c
++++ b/drivers/gpu/drm/i915/i915_gem_execbuffer.c
+@@ -1090,6 +1090,11 @@ i915_gem_do_execbuffer(struct drm_device *dev, void *data,
if (ret)
goto pre_mutex_err;
diff --git a/drm_i915-check-eDP-encoder-correctly-when-setting-modes.patch b/drm_i915-check-eDP-encoder-correctly-when-setting-modes.patch
deleted file mode 100644
index 9f6800a4b..000000000
--- a/drm_i915-check-eDP-encoder-correctly-when-setting-modes.patch
+++ /dev/null
@@ -1,33 +0,0 @@
-From patchwork Tue Jan 4 18:46:49 2011
-Content-Type: text/plain; charset="utf-8"
-MIME-Version: 1.0
-Content-Transfer-Encoding: 7bit
-Subject: drm/i915: check eDP encoder correctly when setting modes
-Date: Tue, 04 Jan 2011 18:46:49 -0000
-From: Jesse Barnes <jbarnes@virtuousgeek.org>
-X-Patchwork-Id: 451441
-Message-Id: <1294166809-3316-1-git-send-email-jbarnes@virtuousgeek.org>
-To: intel-gfx@lists.freedesktop.org
-
-We were using a stale pointer in the check which caused us to use CPU
-attached DP params when we should have been using PCH attached params.
-
-Signed-off-by: Jesse Barnes <jbarnes@virtuousgeek.org>
-
----
-drivers/gpu/drm/i915/intel_display.c | 2 +-
- 1 files changed, 1 insertions(+), 1 deletions(-)
-
-diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
-index fe56cb3..28d58ef 100644
---- a/drivers/gpu/drm/i915/intel_display.c
-+++ b/drivers/gpu/drm/i915/intel_display.c
-@@ -3949,7 +3949,7 @@ static int intel_crtc_mode_set(struct drm_crtc *crtc,
- int lane = 0, link_bw, bpp;
- /* CPU eDP doesn't require FDI link, so just set DP M/N
- according to current link config */
-- if (has_edp_encoder && !intel_encoder_is_pch_edp(&encoder->base)) {
-+ if (has_edp_encoder && !intel_encoder_is_pch_edp(&has_edp_encoder->base)) {
- target_clock = mode->clock;
- intel_edp_link_config(has_edp_encoder,
- &lane, &link_bw);
diff --git a/kernel.spec b/kernel.spec
index 8dca74336..e1323ccc2 100644
--- a/kernel.spec
+++ b/kernel.spec
@@ -678,7 +678,6 @@ Patch1824: drm-intel-next.patch
# make sure the lvds comes back on lid open
Patch1825: drm-intel-make-lvds-work.patch
Patch1826: drm-intel-edp-fixes.patch
-Patch1827: drm_i915-check-eDP-encoder-correctly-when-setting-modes.patch
Patch1900: linux-2.6-intel-iommu-igfx.patch
@@ -720,10 +719,8 @@ Patch12018: neuter_intel_microcode_load.patch
Patch12030: tpm-fix-stall-on-boot.patch
-Patch12100: applesmc_update.patch
Patch12101: apple_backlight.patch
Patch12102: efifb_update.patch
-Patch12103: linux-next-macbook-air-input.patch
Patch12200: acpi_reboot.patch
Patch12210: efi_default_physical.patch
@@ -1300,7 +1297,6 @@ ApplyPatch drm-intel-big-hammer.patch
ApplyPatch drm-intel-make-lvds-work.patch
ApplyPatch linux-2.6-intel-iommu-igfx.patch
ApplyPatch drm-intel-edp-fixes.patch
-ApplyPatch drm_i915-check-eDP-encoder-correctly-when-setting-modes.patch
# linux1394 git patches
#ApplyPatch linux-2.6-firewire-git-update.patch
@@ -1338,10 +1334,8 @@ ApplyPatch neuter_intel_microcode_load.patch
ApplyPatch tpm-fix-stall-on-boot.patch
# various fixes for Apple and EFI
-ApplyPatch applesmc_update.patch
ApplyPatch apple_backlight.patch
ApplyPatch efifb_update.patch
-ApplyPatch linux-next-macbook-air-input.patch
ApplyPatch acpi_reboot.patch
ApplyPatch efi_default_physical.patch
diff --git a/linux-next-macbook-air-input.patch b/linux-next-macbook-air-input.patch
deleted file mode 100644
index 16d4d6d39..000000000
--- a/linux-next-macbook-air-input.patch
+++ /dev/null
@@ -1,185 +0,0 @@
-Support for USB HID devices on the new Macbook Airs. From -next.
-
-diff --git a/drivers/hid/hid-apple.c b/drivers/hid/hid-apple.c
-index eaeca56..6c52203 100644
---- a/drivers/hid/hid-apple.c
-+++ b/drivers/hid/hid-apple.c
-@@ -59,6 +59,27 @@ struct apple_key_translation {
- u8 flags;
- };
-
-+static const struct apple_key_translation macbookair_fn_keys[] = {
-+ { KEY_BACKSPACE, KEY_DELETE },
-+ { KEY_ENTER, KEY_INSERT },
-+ { KEY_F1, KEY_BRIGHTNESSDOWN, APPLE_FLAG_FKEY },
-+ { KEY_F2, KEY_BRIGHTNESSUP, APPLE_FLAG_FKEY },
-+ { KEY_F3, KEY_SCALE, APPLE_FLAG_FKEY },
-+ { KEY_F4, KEY_DASHBOARD, APPLE_FLAG_FKEY },
-+ { KEY_F6, KEY_PREVIOUSSONG, APPLE_FLAG_FKEY },
-+ { KEY_F7, KEY_PLAYPAUSE, APPLE_FLAG_FKEY },
-+ { KEY_F8, KEY_NEXTSONG, APPLE_FLAG_FKEY },
-+ { KEY_F9, KEY_MUTE, APPLE_FLAG_FKEY },
-+ { KEY_F10, KEY_VOLUMEDOWN, APPLE_FLAG_FKEY },
-+ { KEY_F11, KEY_VOLUMEUP, APPLE_FLAG_FKEY },
-+ { KEY_F12, KEY_EJECTCD, APPLE_FLAG_FKEY },
-+ { KEY_UP, KEY_PAGEUP },
-+ { KEY_DOWN, KEY_PAGEDOWN },
-+ { KEY_LEFT, KEY_HOME },
-+ { KEY_RIGHT, KEY_END },
-+ { }
-+};
-+
- static const struct apple_key_translation apple_fn_keys[] = {
- { KEY_BACKSPACE, KEY_DELETE },
- { KEY_ENTER, KEY_INSERT },
-@@ -157,10 +178,15 @@ static int hidinput_apple_event(struct hid_device *hid, struct input_dev *input,
- if (fnmode) {
- int do_translate;
-
-- trans = apple_find_translation((hid->product < 0x21d ||
-- hid->product >= 0x300) ?
-- powerbook_fn_keys : apple_fn_keys,
-- usage->code);
-+ if(hid->product >= USB_DEVICE_ID_APPLE_WELLSPRING4_ANSI &&
-+ hid->product <= USB_DEVICE_ID_APPLE_WELLSPRING4A_JIS) {
-+ trans = apple_find_translation(macbookair_fn_keys, usage->code);
-+ } else if (hid->product < 0x21d || hid->product >= 0x300) {
-+ trans = apple_find_translation(powerbook_fn_keys, usage->code);
-+ } else {
-+ trans = apple_find_translation(apple_fn_keys, usage->code);
-+ }
-+
- if (trans) {
- if (test_bit(usage->code, asc->pressed_fn))
- do_translate = 1;
-@@ -436,6 +462,18 @@ static const struct hid_device_id apple_devices[] = {
- .driver_data = APPLE_HAS_FN | APPLE_ISO_KEYBOARD },
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING3_JIS),
- .driver_data = APPLE_HAS_FN | APPLE_RDESC_JIS },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_ANSI),
-+ .driver_data = APPLE_HAS_FN },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_ISO),
-+ .driver_data = APPLE_HAS_FN | APPLE_ISO_KEYBOARD },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_JIS),
-+ .driver_data = APPLE_HAS_FN | APPLE_RDESC_JIS },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_ANSI),
-+ .driver_data = APPLE_HAS_FN },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_ISO),
-+ .driver_data = APPLE_HAS_FN | APPLE_ISO_KEYBOARD },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_JIS),
-+ .driver_data = APPLE_HAS_FN | APPLE_RDESC_JIS },
- { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ANSI),
- .driver_data = APPLE_NUMLOCK_EMULATION | APPLE_HAS_FN },
- { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ISO),
-diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c
-index b3393e1..53ac909 100644
---- a/drivers/hid/hid-core.c
-+++ b/drivers/hid/hid-core.c
-@@ -1274,6 +1274,12 @@ static const struct hid_device_id hid_blacklist[] = {
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING3_ANSI) },
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING3_ISO) },
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING3_JIS) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_ANSI) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_ISO) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_JIS) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_ANSI) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_ISO) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_JIS) },
- { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ANSI) },
- { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ISO) },
- { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_JIS) },
-@@ -1760,6 +1766,12 @@ static const struct hid_device_id hid_mouse_ignore_list[] = {
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING3_ANSI) },
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING3_ISO) },
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING3_JIS) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_ANSI) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_ISO) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4_JIS) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_ANSI) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_ISO) },
-+ { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_WELLSPRING4A_JIS) },
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_FOUNTAIN_TP_ONLY) },
- { HID_USB_DEVICE(USB_VENDOR_ID_APPLE, USB_DEVICE_ID_APPLE_GEYSER1_TP_ONLY) },
- { }
-diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h
-index 68114db..104b9f9 100644
---- a/drivers/hid/hid-ids.h
-+++ b/drivers/hid/hid-ids.h
-@@ -97,6 +97,12 @@
- #define USB_DEVICE_ID_APPLE_WELLSPRING3_ANSI 0x0236
- #define USB_DEVICE_ID_APPLE_WELLSPRING3_ISO 0x0237
- #define USB_DEVICE_ID_APPLE_WELLSPRING3_JIS 0x0238
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4_ANSI 0x023f
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4_ISO 0x0240
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4_JIS 0x0241
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4A_ANSI 0x0242
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4A_ISO 0x0243
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4A_JIS 0x0244
- #define USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ANSI 0x0239
- #define USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_ISO 0x023a
- #define USB_DEVICE_ID_APPLE_ALU_WIRELESS_2009_JIS 0x023b
-
-diff --git a/drivers/input/mouse/bcm5974.c b/drivers/input/mouse/bcm5974.c
-index b952317..ee82851 100644
---- a/drivers/input/mouse/bcm5974.c
-+++ b/drivers/input/mouse/bcm5974.c
-@@ -55,6 +55,14 @@
- #define USB_DEVICE_ID_APPLE_WELLSPRING3_ANSI 0x0236
- #define USB_DEVICE_ID_APPLE_WELLSPRING3_ISO 0x0237
- #define USB_DEVICE_ID_APPLE_WELLSPRING3_JIS 0x0238
-+/* MacbookAir3,2 (unibody), aka wellspring5 */
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4_ANSI 0x023f
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4_ISO 0x0240
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4_JIS 0x0241
-+/* MacbookAir3,1 (unibody), aka wellspring4 */
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4A_ANSI 0x0242
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4A_ISO 0x0243
-+#define USB_DEVICE_ID_APPLE_WELLSPRING4A_JIS 0x0244
-
- #define BCM5974_DEVICE(prod) { \
- .match_flags = (USB_DEVICE_ID_MATCH_DEVICE | \
-@@ -80,6 +88,14 @@ static const struct usb_device_id bcm5974_table[] = {
- BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING3_ANSI),
- BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING3_ISO),
- BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING3_JIS),
-+ /* MacbookAir3,2 */
-+ BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING4_ANSI),
-+ BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING4_ISO),
-+ BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING4_JIS),
-+ /* MacbookAir3,1 */
-+ BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING4A_ANSI),
-+ BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING4A_ISO),
-+ BCM5974_DEVICE(USB_DEVICE_ID_APPLE_WELLSPRING4A_JIS),
- /* Terminating entry */
- {}
- };
-@@ -234,6 +250,30 @@ static const struct bcm5974_config bcm5974_config_table[] = {
- { DIM_X, DIM_X / SN_COORD, -4460, 5166 },
- { DIM_Y, DIM_Y / SN_COORD, -75, 6700 }
- },
-+ {
-+ USB_DEVICE_ID_APPLE_WELLSPRING4_ANSI,
-+ USB_DEVICE_ID_APPLE_WELLSPRING4_ISO,
-+ USB_DEVICE_ID_APPLE_WELLSPRING4_JIS,
-+ HAS_INTEGRATED_BUTTON,
-+ 0x84, sizeof(struct bt_data),
-+ 0x81, TYPE2, FINGER_TYPE2, FINGER_TYPE2 + SIZEOF_ALL_FINGERS,
-+ { DIM_PRESSURE, DIM_PRESSURE / SN_PRESSURE, 0, 300 },
-+ { DIM_WIDTH, DIM_WIDTH / SN_WIDTH, 0, 2048 },
-+ { DIM_X, DIM_X / SN_COORD, -4620, 5140 },
-+ { DIM_Y, DIM_Y / SN_COORD, -150, 6600 }
-+ },
-+ {
-+ USB_DEVICE_ID_APPLE_WELLSPRING4A_ANSI,
-+ USB_DEVICE_ID_APPLE_WELLSPRING4A_ISO,
-+ USB_DEVICE_ID_APPLE_WELLSPRING4A_JIS,
-+ HAS_INTEGRATED_BUTTON,
-+ 0x84, sizeof(struct bt_data),
-+ 0x81, TYPE2, FINGER_TYPE2, FINGER_TYPE2 + SIZEOF_ALL_FINGERS,
-+ { DIM_PRESSURE, DIM_PRESSURE / SN_PRESSURE, 0, 300 },
-+ { DIM_WIDTH, DIM_WIDTH / SN_WIDTH, 0, 2048 },
-+ { DIM_X, DIM_X / SN_COORD, -4616, 5112 },
-+ { DIM_Y, DIM_Y / SN_COORD, -142, 5234 }
-+ },
- {}
- };