Merge branch 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/paulus/powerpc

* 'master' of git://git.kernel.org/pub/scm/linux/kernel/git/paulus/powerpc: (77 commits)
  [POWERPC] Abolish powerpc_flash_init()
  [POWERPC] Early serial debug support for PPC44x
  [POWERPC] Support for the Ebony 440GP reference board in arch/powerpc
  [POWERPC] Add device tree for Ebony
  [POWERPC] Add powerpc/platforms/44x, disable platforms/4xx for now
  [POWERPC] MPIC U3/U4 MSI backend
  [POWERPC] MPIC MSI allocator
  [POWERPC] Enable MSI mappings for MPIC
  [POWERPC] Tell Phyp we support MSI
  [POWERPC] RTAS MSI implementation
  [POWERPC] PowerPC MSI infrastructure
  [POWERPC] Rip out the existing powerpc msi stubs
  [POWERPC] Remove use of 4level-fixup.h for ppc32
  [POWERPC] Add powerpc PCI-E reset API implementation
  [POWERPC] Holly bootwrapper
  [POWERPC] Holly DTS
  [POWERPC] Holly defconfig
  [POWERPC] Add support for 750CL Holly board
  [POWERPC] Generalize tsi108 PCI setup
  [POWERPC] Generalize tsi108 PHY types
  ...

Fixed conflict in include/asm-powerpc/kdebug.h manually

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

12 files changed, 81 insertions, 550 deletions

diff --git a/drivers/macintosh/Kconfig b/drivers/macintosh/Kconfig
index 1a86387e23be..a32c91e27b3c 100644
--- a/drivers/macintosh/Kconfig
+++ b/drivers/macintosh/Kconfig
@@ -1,6 +1,10 @@
 
-menu "Macintosh device drivers"
+menuconfig MACINTOSH_DRIVERS
+        bool "Macintosh device drivers"
         depends on PPC || MAC || X86
+        default y
+
+if MACINTOSH_DRIVERS
 
 config ADB
         bool "Apple Desktop Bus (ADB) support"
@@ -109,7 +113,9 @@ config PMAC_SMU
 
 config PMAC_APM_EMU
         tristate "APM emulation"
-        depends on PPC_PMAC && PPC32 && PM && ADB_PMU
+        select SYS_SUPPORTS_APM_EMULATION
+        select APM_EMULATION
+        depends on ADB_PMU && PM
 
 config PMAC_MEDIABAY
         bool "Support PowerBook hotswap media bay"
@@ -234,4 +240,4 @@ config PMAC_RACKMETER
           This driver procides some support to control the front panel
           blue LEDs "vu-meter" of the XServer macs.
 
-endmenu
+endif # MACINTOSH_DRIVERS
diff --git a/drivers/macintosh/apm_emu.c b/drivers/macintosh/apm_emu.c
index cdb0bead9917..9821e6361e60 100644
--- a/drivers/macintosh/apm_emu.c
+++ b/drivers/macintosh/apm_emu.c
@@ -1,9 +1,7 @@
-/* APM emulation layer for PowerMac
- * 
- * Copyright 2001 Benjamin Herrenschmidt (benh@kernel.crashing.org)
+/*
+ * APM emulation for PMU-based machines
  *
- * Lots of code inherited from apm.c, see appropriate notice in
- *  arch/i386/kernel/apm.c
+ * Copyright 2001 Benjamin Herrenschmidt (benh@kernel.crashing.org)
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms of the GNU General Public License as published by the
@@ -18,429 +16,39 @@
  *
  */
 
-#include <linux/module.h>
-
-#include <linux/poll.h>
-#include <linux/types.h>
-#include <linux/stddef.h>
-#include <linux/timer.h>
-#include <linux/fcntl.h>
-#include <linux/slab.h>
-#include <linux/stat.h>
-#include <linux/proc_fs.h>
-#include <linux/miscdevice.h>
-#include <linux/apm_bios.h>
-#include <linux/init.h>
-#include <linux/sched.h>
-#include <linux/pm.h>
 #include <linux/kernel.h>
-#include <linux/smp_lock.h>
-
+#include <linux/module.h>
+#include <linux/apm-emulation.h>
 #include <linux/adb.h>
 #include <linux/pmu.h>
 
-#include <asm/system.h>
-#include <asm/uaccess.h>
-#include <asm/machdep.h>
-
-#undef DEBUG
-
-#ifdef DEBUG
-#define DBG(args...) printk(KERN_DEBUG args)
-//#define DBG(args...) xmon_printf(args)
-#else
-#define DBG(args...) do { } while (0)
-#endif
-
-/*
- * The apm_bios device is one of the misc char devices.
- * This is its minor number.
- */
-#define APM_MINOR_DEV   134
-
-/*
- * Maximum number of events stored
- */
-#define APM_MAX_EVENTS          20
-
-#define FAKE_APM_BIOS_VERSION   0x0101
-
-#define APM_USER_NOTIFY_TIMEOUT (5*HZ)
-
-/*
- * The per-file APM data
- */
-struct apm_user {
-        int             magic;
-        struct apm_user *       next;
-        int             suser: 1;
-        int             suspend_waiting: 1;
-        int             suspends_pending;
-        int             suspends_read;
-        int             event_head;
-        int             event_tail;
-        apm_event_t     events[APM_MAX_EVENTS];
-};
-
-/*
- * The magic number in apm_user
- */
-#define APM_BIOS_MAGIC          0x4101
-
-/*
- * Local variables
- */
-static int                      suspends_pending;
-
-static DECLARE_WAIT_QUEUE_HEAD(apm_waitqueue);
-static DECLARE_WAIT_QUEUE_HEAD(apm_suspend_waitqueue);
-static struct apm_user *        user_list;
-
-static void apm_notify_sleep(struct pmu_sleep_notifier *self, int when);
-static struct pmu_sleep_notifier apm_sleep_notifier = {
-        apm_notify_sleep,
-        SLEEP_LEVEL_USERLAND,
-};
-
-static const char driver_version[] = "0.5";     /* no spaces */
-
-#ifdef DEBUG
-static char *   apm_event_name[] = {
-        "system standby",
-        "system suspend",
-        "normal resume",
-        "critical resume",
-        "low battery",
-        "power status change",
-        "update time",
-        "critical suspend",
-        "user standby",
-        "user suspend",
-        "system standby resume",
-        "capabilities change"
-};
-#define NR_APM_EVENT_NAME       \
-                (sizeof(apm_event_name) / sizeof(apm_event_name[0]))
-
-#endif
-
-static int queue_empty(struct apm_user *as)
-{
-        return as->event_head == as->event_tail;
-}
-
-static apm_event_t get_queued_event(struct apm_user *as)
-{
-        as->event_tail = (as->event_tail + 1) % APM_MAX_EVENTS;
-        return as->events[as->event_tail];
-}
-
-static void queue_event(apm_event_t event, struct apm_user *sender)
-{
-        struct apm_user *       as;
-
-        DBG("apm_emu: queue_event(%s)\n", apm_event_name[event-1]);
-        if (user_list == NULL)
-                return;
-        for (as = user_list; as != NULL; as = as->next) {
-                if (as == sender)
-                        continue;
-                as->event_head = (as->event_head + 1) % APM_MAX_EVENTS;
-                if (as->event_head == as->event_tail) {
-                        static int notified;
-
-                        if (notified++ == 0)
-                            printk(KERN_ERR "apm_emu: an event queue overflowed\n");
-                        as->event_tail = (as->event_tail + 1) % APM_MAX_EVENTS;
-                }
-                as->events[as->event_head] = event;
-                if (!as->suser)
-                        continue;
-                switch (event) {
-                case APM_SYS_SUSPEND:
-                case APM_USER_SUSPEND:
-                        as->suspends_pending++;
-                        suspends_pending++;
-                        break;
-                case APM_NORMAL_RESUME:
-                        as->suspend_waiting = 0;
-                        break;
-                }
-        }
-        wake_up_interruptible(&apm_waitqueue);
-}
-
-static int check_apm_user(struct apm_user *as, const char *func)
-{
-        if ((as == NULL) || (as->magic != APM_BIOS_MAGIC)) {
-                printk(KERN_ERR "apm_emu: %s passed bad filp\n", func);
-                return 1;
-        }
-        return 0;
-}
-
-static ssize_t do_read(struct file *fp, char __user *buf, size_t count, loff_t *ppos)
-{
-        struct apm_user *       as;
-        size_t                  i;
-        apm_event_t             event;
-        DECLARE_WAITQUEUE(wait, current);
-
-        as = fp->private_data;
-        if (check_apm_user(as, "read"))
-                return -EIO;
-        if (count < sizeof(apm_event_t))
-                return -EINVAL;
-        if (queue_empty(as)) {
-                if (fp->f_flags & O_NONBLOCK)
-                        return -EAGAIN;
-                add_wait_queue(&apm_waitqueue, &wait);
-repeat:
-                set_current_state(TASK_INTERRUPTIBLE);
-                if (queue_empty(as) && !signal_pending(current)) {
-                        schedule();
-                        goto repeat;
-                }
-                set_current_state(TASK_RUNNING);
-                remove_wait_queue(&apm_waitqueue, &wait);
-        }
-        i = count;
-        while ((i >= sizeof(event)) && !queue_empty(as)) {
-                event = get_queued_event(as);
-                DBG("apm_emu: do_read, returning: %s\n", apm_event_name[event-1]);
-                if (copy_to_user(buf, &event, sizeof(event))) {
-                        if (i < count)
-                                break;
-                        return -EFAULT;
-                }
-                switch (event) {
-                case APM_SYS_SUSPEND:
-                case APM_USER_SUSPEND:
-                        as->suspends_read++;
-                        break;
-                }
-                buf += sizeof(event);
-                i -= sizeof(event);
-        }
-        if (i < count)
-                return count - i;
-        if (signal_pending(current))
-                return -ERESTARTSYS;
-        return 0;
-}
-
-static unsigned int do_poll(struct file *fp, poll_table * wait)
-{
-        struct apm_user * as;
-
-        as = fp->private_data;
-        if (check_apm_user(as, "poll"))
-                return 0;
-        poll_wait(fp, &apm_waitqueue, wait);
-        if (!queue_empty(as))
-                return POLLIN | POLLRDNORM;
-        return 0;
-}
-
-static int do_ioctl(struct inode * inode, struct file *filp,
-                    u_int cmd, u_long arg)
-{
-        struct apm_user *       as;
-        DECLARE_WAITQUEUE(wait, current);
-
-        as = filp->private_data;
-        if (check_apm_user(as, "ioctl"))
-                return -EIO;
-        if (!as->suser)
-                return -EPERM;
-        switch (cmd) {
-        case APM_IOC_SUSPEND:
-                /* If a suspend message was sent to userland, we
-                 * consider this as a confirmation message
-                 */
-                if (as->suspends_read > 0) {
-                        as->suspends_read--;
-                        as->suspends_pending--;
-                        suspends_pending--;
-                } else {
-                        // Route to PMU suspend ?
-                        break;
-                }
-                as->suspend_waiting = 1;
-                add_wait_queue(&apm_waitqueue, &wait);
-                DBG("apm_emu: ioctl waking up sleep waiter !\n");
-                wake_up(&apm_suspend_waitqueue);
-                mb();
-                while(as->suspend_waiting && !signal_pending(current)) {
-                        set_current_state(TASK_INTERRUPTIBLE);
-                        schedule();
-                }
-                set_current_state(TASK_RUNNING);
-                remove_wait_queue(&apm_waitqueue, &wait);
-                break;
-        default:
-                return -EINVAL;
-        }
-        return 0;
-}
-
-static int do_release(struct inode * inode, struct file * filp)
-{
-        struct apm_user *       as;
-
-        as = filp->private_data;
-        if (check_apm_user(as, "release"))
-                return 0;
-        filp->private_data = NULL;
-        lock_kernel();
-        if (as->suspends_pending > 0) {
-                suspends_pending -= as->suspends_pending;
-                if (suspends_pending <= 0)
-                        wake_up(&apm_suspend_waitqueue);
-        }
-        if (user_list == as)
-                user_list = as->next;
-        else {
-                struct apm_user *       as1;
-
-                for (as1 = user_list;
-                     (as1 != NULL) && (as1->next != as);
-                     as1 = as1->next)
-                        ;
-                if (as1 == NULL)
-                        printk(KERN_ERR "apm: filp not in user list\n");
-                else
-                        as1->next = as->next;
-        }
-        unlock_kernel();
-        kfree(as);
-        return 0;
-}
-
-static int do_open(struct inode * inode, struct file * filp)
-{
-        struct apm_user *       as;
-
-        as = kmalloc(sizeof(*as), GFP_KERNEL);
-        if (as == NULL) {
-                printk(KERN_ERR "apm: cannot allocate struct of size %d bytes\n",
-                       sizeof(*as));
-                return -ENOMEM;
-        }
-        as->magic = APM_BIOS_MAGIC;
-        as->event_tail = as->event_head = 0;
-        as->suspends_pending = 0;
-        as->suspends_read = 0;
-        /*
-         * XXX - this is a tiny bit broken, when we consider BSD
-         * process accounting. If the device is opened by root, we
-         * instantly flag that we used superuser privs. Who knows,
-         * we might close the device immediately without doing a
-         * privileged operation -- cevans
-         */
-        as->suser = capable(CAP_SYS_ADMIN);
-        as->next = user_list;
-        user_list = as;
-        filp->private_data = as;
-
-        DBG("apm_emu: opened by %s, suser: %d\n", current->comm, (int)as->suser);
-
-        return 0;
-}
-
-/* Wait for all clients to ack the suspend request. APM API
- * doesn't provide a way to NAK, but this could be added
- * here.
- */
-static void wait_all_suspend(void)
-{
-        DECLARE_WAITQUEUE(wait, current);
-
-        add_wait_queue(&apm_suspend_waitqueue, &wait);
-        DBG("apm_emu: wait_all_suspend(), suspends_pending: %d\n", suspends_pending);
-        while(suspends_pending > 0) {
-                set_current_state(TASK_UNINTERRUPTIBLE);
-                schedule();
-        }
-        set_current_state(TASK_RUNNING);
-        remove_wait_queue(&apm_suspend_waitqueue, &wait);
-
-        DBG("apm_emu: wait_all_suspend() - complete !\n");
-}
-
-static void apm_notify_sleep(struct pmu_sleep_notifier *self, int when)
-{
-        switch(when) {
-                case PBOOK_SLEEP_REQUEST:
-                        queue_event(APM_SYS_SUSPEND, NULL);
-                        wait_all_suspend();
-                        break;
-                case PBOOK_WAKE:
-                        queue_event(APM_NORMAL_RESUME, NULL);
-                        break;
-        }
-}
-
 #define APM_CRITICAL            10
 #define APM_LOW                 30
 
-static int apm_emu_get_info(char *buf, char **start, off_t fpos, int length)
+static void pmu_apm_get_power_status(struct apm_power_info *info)
 {
-        /* Arguments, with symbols from linux/apm_bios.h.  Information is
-           from the Get Power Status (0x0a) call unless otherwise noted.
+        int percentage = -1;
+        int batteries = 0;
+        int time_units = -1;
+        int real_count = 0;
+        int i;
+        char charging = 0;
+        long charge = -1;
+        long amperage = 0;
+        unsigned long btype = 0;
+
+        info->battery_status = APM_BATTERY_STATUS_UNKNOWN;
+        info->battery_flag = APM_BATTERY_FLAG_UNKNOWN;
+        info->units = APM_UNITS_MINS;
+
+        if (pmu_power_flags & PMU_PWR_AC_PRESENT)
+                info->ac_line_status = APM_AC_ONLINE;
+        else
+                info->ac_line_status = APM_AC_OFFLINE;
 
-           0) Linux driver version (this will change if format changes)
-           1) APM BIOS Version.  Usually 1.0, 1.1 or 1.2.
-           2) APM flags from APM Installation Check (0x00):
-              bit 0: APM_16_BIT_SUPPORT
-              bit 1: APM_32_BIT_SUPPORT
-              bit 2: APM_IDLE_SLOWS_CLOCK
-              bit 3: APM_BIOS_DISABLED
-              bit 4: APM_BIOS_DISENGAGED
-           3) AC line status
-              0x00: Off-line
-              0x01: On-line
-              0x02: On backup power (BIOS >= 1.1 only)
-              0xff: Unknown
-           4) Battery status
-              0x00: High
-              0x01: Low
-              0x02: Critical
-              0x03: Charging
-              0x04: Selected battery not present (BIOS >= 1.2 only)
-              0xff: Unknown
-           5) Battery flag
-              bit 0: High
-              bit 1: Low
-              bit 2: Critical
-              bit 3: Charging
-              bit 7: No system battery
-              0xff: Unknown
-           6) Remaining battery life (percentage of charge):
-              0-100: valid
-              -1: Unknown
-           7) Remaining battery life (time units):
-              Number of remaining minutes or seconds
-              -1: Unknown
-           8) min = minutes; sec = seconds */
-
-        unsigned short  ac_line_status;
-        unsigned short  battery_status = 0;
-        unsigned short  battery_flag   = 0xff;
-        int             percentage     = -1;
-        int             time_units     = -1;
-        int             real_count     = 0;
-        int             i;
-        char *          p = buf;
-        char            charging       = 0;
-        long            charge         = -1;
-        long            amperage       = 0;
-        unsigned long   btype          = 0;
-
-        ac_line_status = ((pmu_power_flags & PMU_PWR_AC_PRESENT) != 0);
         for (i=0; i<pmu_battery_count; i++) {
                 if (pmu_batteries[i].flags & PMU_BATT_PRESENT) {
-                        battery_status++;
+                        batteries++;
                         if (percentage < 0)
                                 percentage = 0;
                         if (charge < 0)
@@ -456,9 +64,9 @@ static int apm_emu_get_info(char *buf, char **start, off_t fpos, int length)
                                 charging++;
                 }
         }
-        if (0 == battery_status)
-                ac_line_status = 1;
-        battery_status = 0xff;
+        if (batteries == 0)
+                info->ac_line_status = APM_AC_ONLINE;
+
         if (real_count) {
                 if (amperage < 0) {
                         if (btype == PMU_BATT_TYPE_SMART)
@@ -468,85 +76,44 @@ static int apm_emu_get_info(char *buf, char **start, off_t fpos, int length)
                 }
                 percentage /= real_count;
                 if (charging > 0) {
-                        battery_status = 0x03;
-                        battery_flag = 0x08;
+                        info->battery_status = APM_BATTERY_STATUS_CHARGING;
+                        info->battery_flag = APM_BATTERY_FLAG_CHARGING;
                 } else if (percentage <= APM_CRITICAL) {
-                        battery_status = 0x02;
-                        battery_flag = 0x04;
+                        info->battery_status = APM_BATTERY_STATUS_CRITICAL;
+                        info->battery_flag = APM_BATTERY_FLAG_CRITICAL;
                 } else if (percentage <= APM_LOW) {
-                        battery_status = 0x01;
-                        battery_flag = 0x02;
+                        info->battery_status = APM_BATTERY_STATUS_LOW;
+                        info->battery_flag = APM_BATTERY_FLAG_LOW;
                 } else {
-                        battery_status = 0x00;
-                        battery_flag = 0x01;
+                        info->battery_status = APM_BATTERY_STATUS_HIGH;
+                        info->battery_flag = APM_BATTERY_FLAG_HIGH;
                 }
         }
-        p += sprintf(p, "%s %d.%d 0x%02x 0x%02x 0x%02x 0x%02x %d%% %d %s\n",
-                     driver_version,
-                     (FAKE_APM_BIOS_VERSION >> 8) & 0xff,
-                     FAKE_APM_BIOS_VERSION & 0xff,
-                     0,
-                     ac_line_status,
-                     battery_status,
-                     battery_flag,
-                     percentage,
-                     time_units,
-                     "min");
 
-        return p - buf;
+        info->battery_life = percentage;
+        info->time = time_units;
 }
 
-static const struct file_operations apm_bios_fops = {
-        .owner          = THIS_MODULE,
-        .read           = do_read,
-        .poll           = do_poll,
-        .ioctl          = do_ioctl,
-        .open           = do_open,
-        .release        = do_release,
-};
-
-static struct miscdevice apm_device = {
-        APM_MINOR_DEV,
-        "apm_bios",
-        &apm_bios_fops
-};
-
 static int __init apm_emu_init(void)
 {
-        struct proc_dir_entry *apm_proc;
-
-        if (sys_ctrler != SYS_CTRLER_PMU) {
-                printk(KERN_INFO "apm_emu: Requires a machine with a PMU.\n");
-                return -ENODEV;
-        }
-                
-        apm_proc = create_proc_info_entry("apm", 0, NULL, apm_emu_get_info);
-        if (apm_proc)
-                apm_proc->owner = THIS_MODULE;
+        apm_get_power_status = pmu_apm_get_power_status;
 
-        if (misc_register(&apm_device) != 0)
-                printk(KERN_INFO "Could not create misc. device for apm\n");
-
-        pmu_register_sleep_notifier(&apm_sleep_notifier);
-
-        printk(KERN_INFO "apm_emu: APM Emulation %s initialized.\n", driver_version);
+        printk(KERN_INFO "apm_emu: PMU APM Emulation initialized.\n");
 
         return 0;
 }
 
 static void __exit apm_emu_exit(void)
 {
-        pmu_unregister_sleep_notifier(&apm_sleep_notifier);
-        misc_deregister(&apm_device);
-        remove_proc_entry("apm", NULL);
+        if (apm_get_power_status == pmu_apm_get_power_status)
+                apm_get_power_status = NULL;
 
-        printk(KERN_INFO "apm_emu: APM Emulation removed.\n");
+        printk(KERN_INFO "apm_emu: PMU APM Emulation removed.\n");
 }
 
 module_init(apm_emu_init);
 module_exit(apm_emu_exit);
 
 MODULE_AUTHOR("Benjamin Herrenschmidt");
-MODULE_DESCRIPTION("APM emulation layer for PowerMac");
+MODULE_DESCRIPTION("APM emulation for PowerMac");
 MODULE_LICENSE("GPL");
-
diff --git a/drivers/macintosh/mac_hid.c b/drivers/macintosh/mac_hid.c
index 1599dc34f15f..76c1e8e4a487 100644
--- a/drivers/macintosh/mac_hid.c
+++ b/drivers/macintosh/mac_hid.c
@@ -24,7 +24,7 @@ static int mouse_last_keycode;
 
 #if defined(CONFIG_SYSCTL)
 /* file(s) in /proc/sys/dev/mac_hid */
-ctl_table mac_hid_files[] = {
+static ctl_table mac_hid_files[] = {
         {
                 .ctl_name       = DEV_MAC_HID_MOUSE_BUTTON_EMULATION,
                 .procname       = "mouse_button_emulation",
@@ -53,7 +53,7 @@ ctl_table mac_hid_files[] = {
 };
 
 /* dir in /proc/sys/dev */
-ctl_table mac_hid_dir[] = {
+static ctl_table mac_hid_dir[] = {
         {
                 .ctl_name       = DEV_MAC_HID,
                 .procname       = "mac_hid",
@@ -65,7 +65,7 @@ ctl_table mac_hid_dir[] = {
 };
 
 /* /proc/sys/dev itself, in case that is not there yet */
-ctl_table mac_hid_root_dir[] = {
+static ctl_table mac_hid_root_dir[] = {
         {
                 .ctl_name       = CTL_DEV,
                 .procname       = "dev",
@@ -127,7 +127,7 @@ static int emumousebtn_input_register(void)
         return ret;
 }
 
-int __init mac_hid_init(void)
+static int __init mac_hid_init(void)
 {
         int err;
 
diff --git a/drivers/macintosh/macio_sysfs.c b/drivers/macintosh/macio_sysfs.c
index cc8267912656..112e5ef728f1 100644
--- a/drivers/macintosh/macio_sysfs.c
+++ b/drivers/macintosh/macio_sysfs.c
@@ -41,28 +41,15 @@ compatible_show (struct device *dev, struct device_attribute *attr, char *buf)
 static ssize_t modalias_show (struct device *dev, struct device_attribute *attr,
                               char *buf)
 {
-        struct of_device *of;
-        const char *compat;
-        int cplen;
-        int length;
+        struct of_device *ofdev = to_of_device(dev);
+        int len;
 
-        of = &to_macio_device (dev)->ofdev;
-        compat = of_get_property(of->node, "compatible", &cplen);
-        if (!compat) compat = "", cplen = 1;
-        length = sprintf (buf, "of:N%sT%s", of->node->name, of->node->type);
-        buf += length;
-        while (cplen > 0) {
-                int l;
-                l = sprintf (buf, "C%s", compat);
-                length += l;
-                buf += l;
-                l = strlen (compat) + 1;
-                compat += l;
-                cplen -= l;
-        }
-        length += sprintf(buf, "\n");
+        len = of_device_get_modalias(ofdev, buf, PAGE_SIZE);
 
-        return length;
+        buf[len] = '\n';
+        buf[len+1] = 0;
+
+        return len+1;
 }
 
 macio_config_of_attr (name, "%s\n");
diff --git a/drivers/macintosh/smu.c b/drivers/macintosh/smu.c
index a98a328b1cfc..f8e1a135bf9d 100644
--- a/drivers/macintosh/smu.c
+++ b/drivers/macintosh/smu.c
@@ -606,7 +606,7 @@ static void smu_expose_childs(struct work_struct *unused)
         struct device_node *np;
 
         for (np = NULL; (np = of_get_next_child(smu->of_node, np)) != NULL;)
-                if (device_is_compatible(np, "smu-sensors"))
+                if (of_device_is_compatible(np, "smu-sensors"))
                         of_platform_device_create(np, "smu-sensors",
                                                   &smu->of_dev->dev);
 }
diff --git a/drivers/macintosh/therm_adt746x.c b/drivers/macintosh/therm_adt746x.c
index 6c3734325f8e..bd55e6ab99fc 100644
--- a/drivers/macintosh/therm_adt746x.c
+++ b/drivers/macintosh/therm_adt746x.c
@@ -559,9 +559,9 @@ thermostat_init(void)
         np = of_find_node_by_name(NULL, "fan");
         if (!np)
                 return -ENODEV;
-        if (device_is_compatible(np, "adt7460"))
+        if (of_device_is_compatible(np, "adt7460"))
                 therm_type = ADT7460;
-        else if (device_is_compatible(np, "adt7467"))
+        else if (of_device_is_compatible(np, "adt7467"))
                 therm_type = ADT7467;
         else
                 return -ENODEV;
diff --git a/drivers/macintosh/via-pmu-led.c b/drivers/macintosh/via-pmu-led.c
index fc89a7047cd0..55ad95671387 100644
--- a/drivers/macintosh/via-pmu-led.c
+++ b/drivers/macintosh/via-pmu-led.c
@@ -31,7 +31,6 @@ static spinlock_t pmu_blink_lock;
 static struct adb_request pmu_blink_req;
 /* -1: no change, 0: request off, 1: request on */
 static int requested_change;
-static int sleeping;
 
 static void pmu_req_done(struct adb_request * req)
 {
@@ -41,7 +40,7 @@ static void pmu_req_done(struct adb_request * req)
         /* if someone requested a change in the meantime
          * (we only see the last one which is fine)
          * then apply it now */
-        if (requested_change != -1 && !sleeping)
+        if (requested_change != -1 && !pmu_sys_suspended)
                 pmu_request(&pmu_blink_req, NULL, 4, 0xee, 4, 0, requested_change);
         /* reset requested change */
         requested_change = -1;
@@ -66,7 +65,7 @@ static void pmu_led_set(struct led_classdev *led_cdev,
                 break;
         }
         /* if request isn't done, then don't do anything */
-        if (pmu_blink_req.complete && !sleeping)
+        if (pmu_blink_req.complete && !pmu_sys_suspended)
                 pmu_request(&pmu_blink_req, NULL, 4, 0xee, 4, 0, requested_change);
  out:
         spin_unlock_irqrestore(&pmu_blink_lock, flags);
@@ -80,32 +79,6 @@ static struct led_classdev pmu_led = {
         .brightness_set = pmu_led_set,
 };
 
-#ifdef CONFIG_PM
-static void pmu_led_sleep_call(struct pmu_sleep_notifier *self, int when)
-{
-        unsigned long flags;
-
-        spin_lock_irqsave(&pmu_blink_lock, flags);
-
-        switch (when) {
-        case PBOOK_SLEEP_REQUEST:
-                sleeping = 1;
-                break;
-        case PBOOK_WAKE:
-                sleeping = 0;
-                break;
-        default:
-                /* do nothing */
-                break;
-        }
-        spin_unlock_irqrestore(&pmu_blink_lock, flags);
-}
-
-static struct pmu_sleep_notifier via_pmu_led_sleep_notif = {
-        .notifier_call = pmu_led_sleep_call,
-};
-#endif
-
 static int __init via_pmu_led_init(void)
 {
         struct device_node *dt;
@@ -135,9 +108,7 @@ static int __init via_pmu_led_init(void)
         /* no outstanding req */
         pmu_blink_req.complete = 1;
         pmu_blink_req.done = pmu_req_done;
-#ifdef CONFIG_PM
-        pmu_register_sleep_notifier(&via_pmu_led_sleep_notif);
-#endif
+
         return led_classdev_register(NULL, &pmu_led);
 }
 
diff --git a/drivers/macintosh/via-pmu.c b/drivers/macintosh/via-pmu.c
index 1729d3fd7a11..157080b3b468 100644
--- a/drivers/macintosh/via-pmu.c
+++ b/drivers/macintosh/via-pmu.c
@@ -310,14 +310,14 @@ int __init find_via_pmu(void)
                         PMU_INT_TICK;
         
         if (vias->parent->name && ((strcmp(vias->parent->name, "ohare") == 0)
-            || device_is_compatible(vias->parent, "ohare")))
+            || of_device_is_compatible(vias->parent, "ohare")))
                 pmu_kind = PMU_OHARE_BASED;
-        else if (device_is_compatible(vias->parent, "paddington"))
+        else if (of_device_is_compatible(vias->parent, "paddington"))
                 pmu_kind = PMU_PADDINGTON_BASED;
-        else if (device_is_compatible(vias->parent, "heathrow"))
+        else if (of_device_is_compatible(vias->parent, "heathrow"))
                 pmu_kind = PMU_HEATHROW_BASED;
-        else if (device_is_compatible(vias->parent, "Keylargo")
-                 || device_is_compatible(vias->parent, "K2-Keylargo")) {
+        else if (of_device_is_compatible(vias->parent, "Keylargo")
+                 || of_device_is_compatible(vias->parent, "K2-Keylargo")) {
                 struct device_node *gpiop;
                 struct device_node *adbp;
                 u64 gaddr = OF_BAD_ADDR;
@@ -2759,7 +2759,7 @@ pmu_polled_request(struct adb_request *req)
 
 #if defined(CONFIG_PM) && defined(CONFIG_PPC32)
 
-static int pmu_sys_suspended;
+int pmu_sys_suspended;
 
 static int pmu_sys_suspend(struct sys_device *sysdev, pm_message_t state)
 {
diff --git a/drivers/macintosh/windfarm_lm75_sensor.c b/drivers/macintosh/windfarm_lm75_sensor.c
index ab4d1b63f63e..a0fabf3c2008 100644
--- a/drivers/macintosh/windfarm_lm75_sensor.c
+++ b/drivers/macintosh/windfarm_lm75_sensor.c
@@ -188,10 +188,10 @@ static int wf_lm75_attach(struct i2c_adapter *adapter)
                 if (loc == NULL || addr == 0)
                         continue;
                 /* real lm75 */
-                if (device_is_compatible(dev, "lm75"))
+                if (of_device_is_compatible(dev, "lm75"))
                         wf_lm75_create(adapter, addr, 0, loc);
                 /* ds1775 (compatible, better resolution */
-                else if (device_is_compatible(dev, "ds1775"))
+                else if (of_device_is_compatible(dev, "ds1775"))
                         wf_lm75_create(adapter, addr, 1, loc);
         }
         return 0;
diff --git a/drivers/macintosh/windfarm_max6690_sensor.c b/drivers/macintosh/windfarm_max6690_sensor.c
index eaa74afa175b..5f03aab9fb5d 100644
--- a/drivers/macintosh/windfarm_max6690_sensor.c
+++ b/drivers/macintosh/windfarm_max6690_sensor.c
@@ -131,7 +131,7 @@ static int wf_max6690_attach(struct i2c_adapter *adapter)
                  */
                 if (!pmac_i2c_match_adapter(dev, adapter))
                         continue;
-                if (!device_is_compatible(dev, "max6690"))
+                if (!of_device_is_compatible(dev, "max6690"))
                         continue;
                 addr = pmac_i2c_get_dev_addr(dev);
                 loc = of_get_property(dev, "hwsensor-location", NULL);
diff --git a/drivers/macintosh/windfarm_smu_controls.c b/drivers/macintosh/windfarm_smu_controls.c
index ff398adc0283..58c2590f05ec 100644
--- a/drivers/macintosh/windfarm_smu_controls.c
+++ b/drivers/macintosh/windfarm_smu_controls.c
@@ -263,7 +263,7 @@ static int __init smu_controls_init(void)
         /* Look for RPM fans */
         for (fans = NULL; (fans = of_get_next_child(smu, fans)) != NULL;)
                 if (!strcmp(fans->name, "rpm-fans") ||
-                    device_is_compatible(fans, "smu-rpm-fans"))
+                    of_device_is_compatible(fans, "smu-rpm-fans"))
                         break;
         for (fan = NULL;
              fans && (fan = of_get_next_child(fans, fan)) != NULL;) {
diff --git a/drivers/macintosh/windfarm_smu_sat.c b/drivers/macintosh/windfarm_smu_sat.c
index 9a6c2cf8fd0e..1043b39aa123 100644
--- a/drivers/macintosh/windfarm_smu_sat.c
+++ b/drivers/macintosh/windfarm_smu_sat.c
@@ -380,7 +380,7 @@ static int wf_sat_attach(struct i2c_adapter *adapter)
         busnode = pmac_i2c_get_bus_node(bus);
 
         while ((dev = of_get_next_child(busnode, dev)) != NULL)
-                if (device_is_compatible(dev, "smu-sat"))
+                if (of_device_is_compatible(dev, "smu-sat"))
                         wf_sat_create(adapter, dev);
         return 0;
 }