From: Cezary Jackiewicz <cezary.jackiewicz@gmail.com>
Date: Thu, 12 Jun 2008 20:08:59 +0000 (+0200)
Subject: compal-laptop: remove unnecessary lcd_level attribute
X-Git-Tag: v2.6.27-rc1~1050^2~76
X-Git-Url: http://pilppa.com/gitweb/?a=commitdiff_plain;h=87dc5e3218ba3d7a9293f9113f58455747a233ac;p=linux-2.6-omap-h63xx.git

compal-laptop: remove unnecessary lcd_level attribute

Signed-off-by: Cezary Jackiewicz <cezary.jackiewicz@gmail.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
---

diff --git a/drivers/misc/compal-laptop.c b/drivers/misc/compal-laptop.c
index 0c1f5875fbb..344b790a625 100644
--- a/drivers/misc/compal-laptop.c
+++ b/drivers/misc/compal-laptop.c
@@ -28,9 +28,6 @@
  *
  * This driver exports a few files in /sys/devices/platform/compal-laptop/:
  *
- *   lcd_level - Screen brightness: contains a single integer in the
- *   range 0..7. (rw)
- *
  *   wlan - wlan subsystem state: contains 0 or 1 (rw)
  *
  *   bluetooth - Bluetooth subsystem state: contains 0 or 1 (rw)
@@ -44,7 +41,7 @@
  * This driver might work on other laptops produced by Compal. If you
  * want to try it you can pass force=1 as argument to the module which
  * will force it to load even when the DMI data doesn't identify the
- * laptop as IFL90.
+ * laptop as FL9x.
  */
 
 #include <linux/module.h>
@@ -56,7 +53,7 @@
 #include <linux/platform_device.h>
 #include <linux/autoconf.h>
 
-#define COMPAL_DRIVER_VERSION "0.2.5"
+#define COMPAL_DRIVER_VERSION "0.2.6"
 
 #define COMPAL_LCD_LEVEL_MAX 8
 
@@ -209,34 +206,6 @@ static ssize_t show_bluetooth(struct device *dev,
 	return sprintf(buf, "%i\n", enabled);
 }
 
-static ssize_t show_lcd_level(struct device *dev,
-	struct device_attribute *attr, char *buf)
-{
-	int ret;
-
-	ret = get_lcd_level();
-	if (ret < 0)
-		return ret;
-
-	return sprintf(buf, "%i\n", ret);
-}
-
-static ssize_t store_lcd_level(struct device *dev,
-	struct device_attribute *attr, const char *buf, size_t count)
-{
-	int level, ret;
-
-	if (sscanf(buf, "%i", &level) != 1 ||
-		(level < 0 || level >= COMPAL_LCD_LEVEL_MAX))
-		return -EINVAL;
-
-	ret = set_lcd_level(level);
-	if (ret < 0)
-		return ret;
-
-	return count;
-}
-
 static ssize_t store_wlan_state(struct device *dev,
 	struct device_attribute *attr, const char *buf, size_t count)
 {
@@ -267,13 +236,11 @@ static ssize_t store_bluetooth_state(struct device *dev,
 	return count;
 }
 
-static DEVICE_ATTR(lcd_level, 0644, show_lcd_level, store_lcd_level);
 static DEVICE_ATTR(bluetooth, 0644, show_bluetooth, store_bluetooth_state);
 static DEVICE_ATTR(wlan, 0644, show_wlan, store_wlan_state);
 static DEVICE_ATTR(raw, 0444, show_raw, NULL);
 
 static struct attribute *compal_attributes[] = {
-	&dev_attr_lcd_level.attr,
 	&dev_attr_bluetooth.attr,
 	&dev_attr_wlan.attr,
 	&dev_attr_raw.attr,