[Hackndev-commit] SF.net SVN: hackndev: [1294] linux4palm/linux/trunk/drivers/leds/ leds-palmtt3.c

[<mar...@us...>]

Revision: 1294
          http://hackndev.svn.sourceforge.net/hackndev/?rev=1294&view=rev
Author:   marex_z71
Date:     2007-09-02 03:08:10 -0700 (Sun, 02 Sep 2007)

Log Message:
-----------
PalmTT3: led driver cleanup, alignment to 80 characters

Modified Paths:
--------------
    linux4palm/linux/trunk/drivers/leds/leds-palmtt3.c

Modified: linux4palm/linux/trunk/drivers/leds/leds-palmtt3.c
===================================================================
--- linux4palm/linux/trunk/drivers/leds/leds-palmtt3.c	2007-09-02 08:47:26 UTC (rev 1293)
+++ linux4palm/linux/trunk/drivers/leds/leds-palmtt3.c	2007-09-02 10:08:10 UTC (rev 1294)
@@ -35,49 +35,40 @@
 	tps65010_set_vib(palmtt3led_work.vibra);
 }
 
-static void palmtt3led_red_set(struct led_classdev *led_cdev, enum led_brightness value)
+static void palmtt3led_red_set(struct led_classdev *led_cdev,
+			       enum led_brightness value)
 {
-	if (value)
-		palmtt3led_work.led2 = ON;
-		else
-		palmtt3led_work.led2 = OFF;
+	palmtt3led_work.led2 = value ? ON : OFF;
 	schedule_work(&(palmtt3led_work.work));
 }
 
-static void palmtt3led_green_set(struct led_classdev *led_cdev, enum led_brightness value)
+static void palmtt3led_green_set(struct led_classdev *led_cdev,
+				 enum led_brightness value)
 {
-	if (value)
-		palmtt3led_work.led1 = ON;
-		else
-/* NOTE: This is set to OFF, not to OFF... It shows charging status - plugged/unplugged... */
-		palmtt3led_work.led1 = UNDER_CHG_CTRL;
+	/* NOTE: This is set to OFF, not to OFF... 
+	   It shows charging status - plugged/unplugged... */
+	palmtt3led_work.led1 = value ? ON : UNDER_CHG_CTRL;
 	schedule_work(&(palmtt3led_work.work));
 }
 
-static void palmtt3led_red_blink_set(struct led_classdev *led_cdev, enum led_brightness value)
+static void palmtt3led_red_blink_set(struct led_classdev *led_cdev,
+				     enum led_brightness value)
 {
-	if (value)
-		palmtt3led_work.led2 = BLINK;
-		else
-		palmtt3led_work.led2 = OFF;
+	palmtt3led_work.led2 = value ? BLINK : OFF;
 	schedule_work(&(palmtt3led_work.work));
 }
 
-static void palmtt3led_green_blink_set(struct led_classdev *led_cdev, enum led_brightness value)
+static void palmtt3led_green_blink_set(struct led_classdev *led_cdev,
+				       enum led_brightness value)
 {
-	if (value)
-		palmtt3led_work.led1 = BLINK;
-		else
-		palmtt3led_work.led1 = OFF;
+	palmtt3led_work.led1 = value ? BLINK : OFF;
 	schedule_work(&(palmtt3led_work.work));
 }
 
-static void palmtt3_vibra_set(struct led_classdev *led_cdev, enum led_brightness value)
+static void palmtt3_vibra_set(struct led_classdev *led_cdev,
+			      enum led_brightness value)
 {
-	if (value)
-		palmtt3led_work.vibra = ON;
-		else
-		palmtt3led_work.vibra = OFF;
+	palmtt3led_work.vibra = value ? ON : OFF;
 	schedule_work(&(palmtt3led_work.work));
 }
 
@@ -110,7 +101,8 @@
 
 
 #ifdef CONFIG_PM
-static int palmtt3led_suspend(struct platform_device *dev, pm_message_t state)
+static int palmtt3led_suspend(struct platform_device *dev,
+			      pm_message_t state)
 {
 	led_classdev_suspend(&palmtt3_red_led);
 	led_classdev_suspend(&palmtt3_green_led);
@@ -120,7 +112,8 @@
 	return 0;
 }
 
-static int palmtt3led_suspend_late(struct platform_device *dev, pm_message_t state)
+static int palmtt3led_suspend_late(struct platform_device *dev,
+				   pm_message_t state)
 {
 	tps65010_set_led(LED1, OFF);
 	tps65010_set_led(LED2, OFF);


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.