Refreshed the dsmg600 and nas100d power patches from nslu2-linux
authorRod Whitby <rod@whitby.id.au>
Fri, 11 Jan 2008 07:26:59 +0000 (07:26 +0000)
committerRod Whitby <rod@whitby.id.au>
Fri, 11 Jan 2008 07:26:59 +0000 (07:26 +0000)
SVN-Revision: 10168

target/linux/ixp4xx/patches-2.6.23/016-dsmg600_auto_power_on.patch
target/linux/ixp4xx/patches-2.6.23/017-nas100d_auto_power_on.patch

index 25709da..2a1f241 100644 (file)
@@ -1,13 +1,40 @@
+Upgrade the power and reset button handling for the DSMG600:
+ * Remove the superfluous declaration of ctrl_alt_del().
+ * Convert GPIO and IRQ handling to use the <asm/gpio.h> api.
+ * Perform the reset on the release of the power button, so that
+   NAS devices which have been set to auto-power-on (by bridging
+   the power button) do not continuously power cycle.
+ * Remove all superflous constants from dsmg600.h
+
+Signed-off-by: Rod Whitby <rod@whitby.id.au>
+
 ---
- arch/arm/mach-ixp4xx/dsmg600-power.c |   13 +++++++++----
- 1 file changed, 9 insertions(+), 4 deletions(-)
+ arch/arm/mach-ixp4xx/dsmg600-power.c  |   24 ++++++++++++++----------
+ include/asm-arm/arch-ixp4xx/dsmg600.h |    7 +------
+ 2 files changed, 15 insertions(+), 16 deletions(-)
 
-Index: linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/dsmg600-power.c
+Index: linux-2.6.23.12-armeb/arch/arm/mach-ixp4xx/dsmg600-power.c
 ===================================================================
---- linux-2.6.22-rc5-armeb.orig/arch/arm/mach-ixp4xx/dsmg600-power.c
-+++ linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/dsmg600-power.c
-@@ -50,6 +50,13 @@
-       if (*IXP4XX_GPIO_GPINR & DSMG600_PB_BM) {
+--- linux-2.6.23.12-armeb.orig/arch/arm/mach-ixp4xx/dsmg600-power.c    2008-01-11 16:20:26.000000000 +1030
++++ linux-2.6.23.12-armeb/arch/arm/mach-ixp4xx/dsmg600-power.c 2008-01-11 16:20:30.000000000 +1030
+@@ -26,10 +26,9 @@
+ #include <linux/jiffies.h>
+ #include <linux/timer.h>
++#include <asm/gpio.h>
+ #include <asm/mach-types.h>
+-extern void ctrl_alt_del(void);
+-
+ /* This is used to make sure the power-button pusher is serious.  The button
+  * must be held until the value of this counter reaches zero.
+  */
+@@ -47,9 +46,16 @@
+        * state of the power button.
+        */
+-      if (*IXP4XX_GPIO_GPINR & DSMG600_PB_BM) {
++      if (gpio_get_value(DSMG600_PB_GPIO)) {
  
                /* IO Pin is 1 (button pushed) */
 +              if (power_button_countdown > 0) {
@@ -20,7 +47,7 @@ Index: linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/dsmg600-power.c
                if (power_button_countdown == 0) {
                        /* Signal init to do the ctrlaltdel action, this will bypass
                         * init if it hasn't started and do a kernel_restart.
-@@ -58,11 +65,9 @@
+@@ -58,11 +64,9 @@
  
                        /* Change the state of the power LED to "blink" */
                        gpio_line_set(DSMG600_LED_PWR_GPIO, IXP4XX_GPIO_LOW);
@@ -34,3 +61,51 @@ Index: linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/dsmg600-power.c
        }
  
        mod_timer(&dsmg600_power_timer, jiffies + msecs_to_jiffies(500));
+@@ -81,12 +85,12 @@
+       if (!(machine_is_dsmg600()))
+               return 0;
+-      if (request_irq(DSMG600_RB_IRQ, &dsmg600_reset_handler,
++      if (request_irq(gpio_to_irq(DSMG600_RB_GPIO), &dsmg600_reset_handler,
+               IRQF_DISABLED | IRQF_TRIGGER_LOW, "DSM-G600 reset button",
+               NULL) < 0) {
+               printk(KERN_DEBUG "Reset Button IRQ %d not available\n",
+-                      DSMG600_RB_IRQ);
++                      gpio_to_irq(DSMG600_RB_GPIO));
+               return -EIO;
+       }
+@@ -114,7 +118,7 @@
+       del_timer_sync(&dsmg600_power_timer);
+-      free_irq(DSMG600_RB_IRQ, NULL);
++      free_irq(gpio_to_irq(DSMG600_RB_GPIO), NULL);
+ }
+ module_init(dsmg600_power_init);
+Index: linux-2.6.23.12-armeb/include/asm-arm/arch-ixp4xx/dsmg600.h
+===================================================================
+--- linux-2.6.23.12-armeb.orig/include/asm-arm/arch-ixp4xx/dsmg600.h   2008-01-11 16:20:26.000000000 +1030
++++ linux-2.6.23.12-armeb/include/asm-arm/arch-ixp4xx/dsmg600.h        2008-01-11 16:20:30.000000000 +1030
+@@ -40,18 +40,13 @@
+ /* Buttons */
+ #define DSMG600_PB_GPIO               15      /* power button */
+-#define DSMG600_PB_BM         (1L << DSMG600_PB_GPIO)
+-
+ #define DSMG600_RB_GPIO               3       /* reset button */
+-#define DSMG600_RB_IRQ                IRQ_IXP4XX_GPIO3
++/* Power control */
+ #define DSMG600_PO_GPIO               2       /* power off */
+ /* LEDs */
+ #define DSMG600_LED_PWR_GPIO  0
+-#define DSMG600_LED_PWR_BM    (1L << DSMG600_LED_PWR_GPIO)
+-
+ #define DSMG600_LED_WLAN_GPIO 14
+-#define DSMG600_LED_WLAN_BM   (1L << DSMG600_LED_WLAN_GPIO)
index 232dd9a..b77df48 100644 (file)
@@ -1,19 +1,27 @@
----
- arch/arm/mach-ixp4xx/nas100d-power.c  |   68 ++++++++++++++++++++++++++++++++--
- include/asm-arm/arch-ixp4xx/nas100d.h |   18 +++++----
- 2 files changed, 75 insertions(+), 11 deletions(-)
+Upgrade the power and reset button handling for the NAS100D:
+ * Convert GPIO and IRQ handling to use the <asm/gpio.h> api.
+ * Perform the reset only after the power button has been held down
+   for at least two seconds.  Do the reset on the release of the power
+   button, so that NAS devices which have been set to auto-power-on (by
+   bridging the power button) do not continuously power cycle.
+ * Remove all superflous constants from nas100d.h
+ * Add LED constants to nas100d.h while we're there.
+Also, update the board LED setup code to use constants.
 
-Index: linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/nas100d-power.c
+Signed-off-by: Rod Whitby <rod@whitby.id.au>
+
+Index: linux-2.6.23.12-armeb/arch/arm/mach-ixp4xx/nas100d-power.c
 ===================================================================
---- linux-2.6.22-rc5-armeb.orig/arch/arm/mach-ixp4xx/nas100d-power.c
-+++ linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/nas100d-power.c
-@@ -21,15 +21,60 @@
+--- linux-2.6.23.12-armeb.orig/arch/arm/mach-ixp4xx/nas100d-power.c    2008-01-11 16:59:20.000000000 +1030
++++ linux-2.6.23.12-armeb/arch/arm/mach-ixp4xx/nas100d-power.c 2008-01-11 17:03:23.000000000 +1030
+@@ -21,15 +21,61 @@
  #include <linux/irq.h>
  #include <linux/module.h>
  #include <linux/reboot.h>
 +#include <linux/jiffies.h>
 +#include <linux/timer.h>
  
++#include <asm/gpio.h>
  #include <asm/mach-types.h>
  
 -static irqreturn_t nas100d_reset_handler(int irq, void *dev_id)
@@ -39,7 +47,7 @@ Index: linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/nas100d-power.c
         */
 -      ctrl_alt_del();
 +
-+      if (*IXP4XX_GPIO_GPINR & NAS100D_PB_BM) {
++      if (gpio_get_value(NAS100D_PB_GPIO)) {
 +
 +              /* IO Pin is 1 (button pushed) */
 +              if (power_button_countdown > 0) {
@@ -72,7 +80,21 @@ Index: linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/nas100d-power.c
  
        return IRQ_HANDLED;
  }
-@@ -50,6 +95,19 @@
+@@ -39,17 +85,30 @@
+       if (!(machine_is_nas100d()))
+               return 0;
+-      set_irq_type(NAS100D_RB_IRQ, IRQT_LOW);
++      set_irq_type(gpio_to_irq(NAS100D_RB_GPIO), IRQT_LOW);
+-      if (request_irq(NAS100D_RB_IRQ, &nas100d_reset_handler,
++      if (request_irq(gpio_to_irq(NAS100D_RB_GPIO), &nas100d_reset_handler,
+               IRQF_DISABLED, "NAS100D reset button", NULL) < 0) {
+               printk(KERN_DEBUG "Reset Button IRQ %d not available\n",
+-                      NAS100D_RB_IRQ);
++                      gpio_to_irq(NAS100D_RB_GPIO));
                return -EIO;
        }
  
@@ -92,42 +114,75 @@ Index: linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/nas100d-power.c
        return 0;
  }
  
-@@ -58,6 +116,8 @@
+@@ -58,7 +117,9 @@
        if (!(machine_is_nas100d()))
                return;
  
+-      free_irq(NAS100D_RB_IRQ, NULL);
 +      del_timer_sync(&nas100d_power_timer);
 +
-       free_irq(NAS100D_RB_IRQ, NULL);
++      free_irq(gpio_to_irq(NAS100D_RB_GPIO), NULL);
  }
  
-Index: linux-2.6.22-rc5-armeb/include/asm-arm/arch-ixp4xx/nas100d.h
+ module_init(nas100d_power_init);
+Index: linux-2.6.23.12-armeb/include/asm-arm/arch-ixp4xx/nas100d.h
 ===================================================================
---- linux-2.6.22-rc5-armeb.orig/include/asm-arm/arch-ixp4xx/nas100d.h
-+++ linux-2.6.22-rc5-armeb/include/asm-arm/arch-ixp4xx/nas100d.h
-@@ -39,14 +39,18 @@
+--- linux-2.6.23.12-armeb.orig/include/asm-arm/arch-ixp4xx/nas100d.h   2008-01-11 16:59:20.000000000 +1030
++++ linux-2.6.23.12-armeb/include/asm-arm/arch-ixp4xx/nas100d.h        2008-01-11 17:03:23.000000000 +1030
+@@ -38,15 +38,15 @@
  /* Buttons */
  
- #define NAS100D_PB_GPIO         14
-+#define NAS100D_PB_BM           (1L << NAS100D_PB_GPIO)
+-#define NAS100D_PB_GPIO         14
+-#define NAS100D_RB_GPIO         4
++#define NAS100D_PB_GPIO         14   /* power button */
++#define NAS100D_RB_GPIO         4    /* reset button */
++
++/* Power control */
 +
- #define NAS100D_RB_GPIO         4
--#define NAS100D_PO_GPIO         12   /* power off */
+ #define NAS100D_PO_GPIO         12   /* power off */
  
 -#define NAS100D_PB_IRQ          IRQ_IXP4XX_GPIO14
- #define NAS100D_RB_IRQ          IRQ_IXP4XX_GPIO4
+-#define NAS100D_RB_IRQ          IRQ_IXP4XX_GPIO4
++/* LEDs */
  
 -/*
 -#define NAS100D_PB_BM           (1L << NAS100D_PB_GPIO)
 -#define NAS100D_PO_BM           (1L << NAS100D_PO_GPIO)
 -#define NAS100D_RB_BM           (1L << NAS100D_RB_GPIO)
 -*/
-+#define NAS100D_PO_GPIO         12   /* power off */
-+
-+/* LEDs */
-+
-+#define NAS100D_LED_PWR_GPIO  15
-+#define NAS100D_LED_PWR_BM    (1L << NAS100D_LED_PWR_GPIO)
-+
 +#define NAS100D_LED_WLAN_GPIO 0
-+#define NAS100D_LED_WLAN_BM   (1L << NAS100D_LED_WLAN_GPIO)
++#define NAS100D_LED_DISK_GPIO 3
++#define NAS100D_LED_PWR_GPIO  15
+Index: linux-2.6.23.12-armeb/arch/arm/mach-ixp4xx/nas100d-setup.c
+===================================================================
+--- linux-2.6.23.12-armeb.orig/arch/arm/mach-ixp4xx/nas100d-setup.c    2008-01-11 17:03:23.000000000 +1030
++++ linux-2.6.23.12-armeb/arch/arm/mach-ixp4xx/nas100d-setup.c 2008-01-11 17:06:15.000000000 +1030
+@@ -44,20 +44,20 @@
+ static struct resource nas100d_led_resources[] = {
+       {
+               .name           = "wlan",   /* green led */
+-              .start          = 0,
+-              .end            = 0,
++              .start          = NAS100D_LED_WLAN_GPIO,
++              .end            = NAS100D_LED_WLAN_GPIO,
+               .flags          = IXP4XX_GPIO_LOW,
+       },
+       {
+-              .name           = "ready",  /* blue power led (off is flashing!) */
+-              .start          = 15,
+-              .end            = 15,
++              .name           = "power",  /* blue power led (off is flashing!) */
++              .start          = NAS100D_LED_PWR_GPIO,
++              .end            = NAS100D_LED_PWR_GPIO,
+               .flags          = IXP4XX_GPIO_LOW,
+       },
+       {
+               .name           = "disk",   /* yellow led */
+-              .start          = 3,
+-              .end            = 3,
++              .start          = NAS100D_LED_DISK_GPIO,
++              .end            = NAS100D_LED_DISK_GPIO,
+               .flags          = IXP4XX_GPIO_LOW,
+       },
+ };