backlight: lp855x: move backlight mode platform data
authorKim, Milo <Milo.Kim@ti.com>
Mon, 29 Apr 2013 23:18:03 +0000 (16:18 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 30 Apr 2013 01:28:18 +0000 (18:28 -0700)
The brightness of LP855x devices is controlled by I2C register or PWM
input .  This mode was selected through the platform data, but it can be
chosen by the driver internally without platform data configuration.

How to decide the control mode:
  If the PWM period has specific value, the mode is PWM input.
  On the other hand, the mode is register-based.
  This mode selection is done on the _probe().

Move 'mode' from a header file to the driver private data structure,
'lp855 x'.  And correlated code was replaced.

Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Documentation/backlight/lp855x-driver.txt
drivers/video/backlight/lp855x_bl.c
include/linux/platform_data/lp855x.h

index 18b06ca038eaa9e4845319e4a940f2fefff466b7..72e2a673f7740f170ac896480e0b8de21cabd536 100644 (file)
@@ -32,7 +32,6 @@ Platform data for lp855x
 For supporting platform specific data, the lp855x platform data can be used.
 
 * name : Backlight driver name. If it is not defined, default name is set.
-* mode : Brightness control mode. PWM or register based.
 * device_control : Value of DEVICE CONTROL register.
 * initial_brightness : Initial value of backlight brightness.
 * period_ns : Platform specific PWM period value. unit is nano.
@@ -54,7 +53,6 @@ static struct lp855x_rom_data lp8552_eeprom_arr[] = {
 
 static struct lp855x_platform_data lp8552_pdata = {
        .name = "lcd-bl",
-       .mode = REGISTER_BASED,
        .device_control = I2C_CONFIG(LP8552),
        .initial_brightness = INITIAL_BRT,
        .load_new_rom_data = 1,
@@ -65,7 +63,6 @@ static struct lp855x_platform_data lp8552_pdata = {
 example 2) lp8556 platform data : pwm input mode with default rom data
 
 static struct lp855x_platform_data lp8556_pdata = {
-       .mode = PWM_BASED,
        .device_control = PWM_CONFIG(LP8556),
        .initial_brightness = INITIAL_BRT,
        .period_ns = 1000000,
index df937ce5c2ecb39f8a89ab42e5a6e79c0bf8fcec..b94dc00cea3fd19b6f2aad5afc1c8f0345d9457d 100644 (file)
 #define DEFAULT_BL_NAME                "lcd-backlight"
 #define MAX_BRIGHTNESS         255
 
+enum lp855x_brightness_ctrl_mode {
+       PWM_BASED = 1,
+       REGISTER_BASED,
+};
+
 struct lp855x;
 
 /*
@@ -57,6 +62,7 @@ struct lp855x_device_config {
 struct lp855x {
        const char *chipname;
        enum lp855x_chip_id chip_id;
+       enum lp855x_brightness_ctrl_mode mode;
        struct lp855x_device_config *cfg;
        struct i2c_client *client;
        struct backlight_device *bl;
@@ -238,18 +244,17 @@ static void lp855x_pwm_ctrl(struct lp855x *lp, int br, int max_br)
 static int lp855x_bl_update_status(struct backlight_device *bl)
 {
        struct lp855x *lp = bl_get_data(bl);
-       enum lp855x_brightness_ctrl_mode mode = lp->pdata->mode;
 
        if (bl->props.state & BL_CORE_SUSPENDED)
                bl->props.brightness = 0;
 
-       if (mode == PWM_BASED) {
+       if (lp->mode == PWM_BASED) {
                int br = bl->props.brightness;
                int max_br = bl->props.max_brightness;
 
                lp855x_pwm_ctrl(lp, br, max_br);
 
-       } else if (mode == REGISTER_BASED) {
+       } else if (lp->mode == REGISTER_BASED) {
                u8 val = bl->props.brightness;
                lp855x_write_byte(lp, lp->cfg->reg_brightness, val);
        }
@@ -310,12 +315,11 @@ static ssize_t lp855x_get_bl_ctl_mode(struct device *dev,
                                     struct device_attribute *attr, char *buf)
 {
        struct lp855x *lp = dev_get_drvdata(dev);
-       enum lp855x_brightness_ctrl_mode mode = lp->pdata->mode;
        char *strmode = NULL;
 
-       if (mode == PWM_BASED)
+       if (lp->mode == PWM_BASED)
                strmode = "pwm based";
-       else if (mode == REGISTER_BASED)
+       else if (lp->mode == REGISTER_BASED)
                strmode = "register based";
 
        return scnprintf(buf, PAGE_SIZE, "%s\n", strmode);
@@ -352,6 +356,11 @@ static int lp855x_probe(struct i2c_client *cl, const struct i2c_device_id *id)
        if (!lp)
                return -ENOMEM;
 
+       if (pdata->period_ns > 0)
+               lp->mode = PWM_BASED;
+       else
+               lp->mode = REGISTER_BASED;
+
        lp->client = cl;
        lp->dev = &cl->dev;
        lp->pdata = pdata;
index d135939d392a9a8b1ceddc436b1d51bfcb35824c..e0954dfea7ee41b6297f8420becc13e1ba140b31 100644 (file)
@@ -69,11 +69,6 @@ enum lp855x_chip_id {
        LP8557,
 };
 
-enum lp855x_brightness_ctrl_mode {
-       PWM_BASED = 1,
-       REGISTER_BASED,
-};
-
 enum lp8550_brighntess_source {
        LP8550_PWM_ONLY,
        LP8550_I2C_ONLY = 2,
@@ -116,7 +111,6 @@ struct lp855x_rom_data {
 /**
  * struct lp855x_platform_data
  * @name : Backlight driver name. If it is not defined, default name is set.
- * @mode : brightness control by pwm or lp855x register
  * @device_control : value of DEVICE CONTROL register
  * @initial_brightness : initial value of backlight brightness
  * @period_ns : platform specific pwm period value. unit is nano.
@@ -129,7 +123,6 @@ struct lp855x_rom_data {
  */
 struct lp855x_platform_data {
        const char *name;
-       enum lp855x_brightness_ctrl_mode mode;
        u8 device_control;
        int initial_brightness;
        unsigned int period_ns;