From 5e5024591f66064ad8b7116b1af471ff539fa27d Mon Sep 17 00:00:00 2001
From: David Brownell <dbrownell@users.sourceforge.net>
Date: Thu, 27 Mar 2008 00:59:02 +0000
Subject: [PATCH] --- yaml --- r: 87964 b: refs/heads/master c:
 306dd85c1d41e20a07d1e821e27f5df4927bb5e7 h: refs/heads/master v: v3

---
 [refs]                         |  2 +-
 trunk/drivers/leds/leds-gpio.c | 12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/[refs] b/[refs]
index 2b0e2d862cf8..c2ece9070060 100644
--- a/[refs]
+++ b/[refs]
@@ -1,2 +1,2 @@
 ---
-refs/heads/master: d95cbe6158a679d50e03787bb1dc21ff73dac372
+refs/heads/master: 306dd85c1d41e20a07d1e821e27f5df4927bb5e7
diff --git a/trunk/drivers/leds/leds-gpio.c b/trunk/drivers/leds/leds-gpio.c
index d13b622419df..4c0b05852cb6 100644
--- a/trunk/drivers/leds/leds-gpio.c
+++ b/trunk/drivers/leds/leds-gpio.c
@@ -49,13 +49,13 @@ static void gpio_led_set(struct led_classdev *led_cdev,
 	if (led_dat->active_low)
 		level = !level;
 
-	/* setting GPIOs with I2C/etc requires a preemptible task context */
+	/* Setting GPIOs with I2C/etc requires a task context, and we don't
+	 * seem to have a reliable way to know if we're already in one; so
+	 * let's just assume the worst.
+	 */
 	if (led_dat->can_sleep) {
-		if (preempt_count()) {
-			led_dat->new_level = level;
-			schedule_work(&led_dat->work);
-		} else
-			gpio_set_value_cansleep(led_dat->gpio, level);
+		led_dat->new_level = level;
+		schedule_work(&led_dat->work);
 	} else
 		gpio_set_value(led_dat->gpio, level);
 }