From 37e906af5c2e7ea6680088576d5a854c2d7a3a26 Mon Sep 17 00:00:00 2001 From: Rod Whitby Date: Fri, 11 Jan 2008 07:26:59 +0000 Subject: [PATCH] Refreshed the dsmg600 and nas100d power patches from nslu2-linux SVN-Revision: 10168 --- .../016-dsmg600_auto_power_on.patch | 93 ++++++++++++-- .../017-nas100d_auto_power_on.patch | 113 +++++++++++++----- 2 files changed, 168 insertions(+), 38 deletions(-) diff --git a/target/linux/ixp4xx/patches-2.6.23/016-dsmg600_auto_power_on.patch b/target/linux/ixp4xx/patches-2.6.23/016-dsmg600_auto_power_on.patch index 25709da4f0..2a1f2415f7 100644 --- a/target/linux/ixp4xx/patches-2.6.23/016-dsmg600_auto_power_on.patch +++ b/target/linux/ixp4xx/patches-2.6.23/016-dsmg600_auto_power_on.patch @@ -1,13 +1,40 @@ ---- - arch/arm/mach-ixp4xx/dsmg600-power.c | 13 +++++++++---- - 1 file changed, 9 insertions(+), 4 deletions(-) +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 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 -Index: linux-2.6.22-rc5-armeb/arch/arm/mach-ixp4xx/dsmg600-power.c +Signed-off-by: Rod Whitby + +--- + 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.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 + #include + ++#include + #include + +-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) diff --git a/target/linux/ixp4xx/patches-2.6.23/017-nas100d_auto_power_on.patch b/target/linux/ixp4xx/patches-2.6.23/017-nas100d_auto_power_on.patch index 232dd9a8c5..b77df4850b 100644 --- a/target/linux/ixp4xx/patches-2.6.23/017-nas100d_auto_power_on.patch +++ b/target/linux/ixp4xx/patches-2.6.23/017-nas100d_auto_power_on.patch @@ -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 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 + +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 #include #include +#include +#include ++#include #include -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 */ + - #define NAS100D_RB_GPIO 4 --#define NAS100D_PO_GPIO 12 /* power off */ ++/* Power control */ ++ + #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, + }, + };