leds-lp55xx: replace name of data structure
Milo(Woogyom) Kim [Tue, 5 Feb 2013 09:02:26 +0000 (18:02 +0900)]
 Change the name of chip data structure and platform data.
 This patch is a preceding step for cleaning up lp5521/5523 probe and remove.
 These data will be replaced with new lp55xx common data structures
 in next patch.

Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: Bryan Wu <cooloney@gmail.com>

drivers/leds/leds-lp5521.c
drivers/leds/leds-lp5523.c

index e042a09..8ef8f44 100644 (file)
@@ -872,35 +872,35 @@ static void lp5521_unregister_leds(struct lp5521_chip *chip)
 static int lp5521_probe(struct i2c_client *client,
                        const struct i2c_device_id *id)
 {
-       struct lp5521_chip              *chip;
-       struct lp5521_platform_data     *pdata;
+       struct lp5521_chip              *old_chip;
+       struct lp5521_platform_data     *old_pdata;
        int ret;
 
-       chip = devm_kzalloc(&client->dev, sizeof(*chip), GFP_KERNEL);
-       if (!chip)
+       old_chip = devm_kzalloc(&client->dev, sizeof(*old_chip), GFP_KERNEL);
+       if (!old_chip)
                return -ENOMEM;
 
-       i2c_set_clientdata(client, chip);
-       chip->client = client;
+       i2c_set_clientdata(client, old_chip);
+       old_chip->client = client;
 
-       pdata = client->dev.platform_data;
+       old_pdata = client->dev.platform_data;
 
-       if (!pdata) {
+       if (!old_pdata) {
                dev_err(&client->dev, "no platform data\n");
                return -EINVAL;
        }
 
-       mutex_init(&chip->lock);
+       mutex_init(&old_chip->lock);
 
-       chip->pdata   = pdata;
+       old_chip->pdata   = old_pdata;
 
-       ret = lp5521_init_device(chip);
+       ret = lp5521_init_device(old_chip);
        if (ret)
                goto err_init;
 
        dev_info(&client->dev, "%s programmable led chip found\n", id->name);
 
-       ret = lp5521_register_leds(chip);
+       ret = lp5521_register_leds(old_chip);
        if (ret)
                goto fail2;
 
@@ -911,22 +911,22 @@ static int lp5521_probe(struct i2c_client *client,
        }
        return ret;
 fail2:
-       lp5521_unregister_leds(chip);
-       lp5521_deinit_device(chip);
+       lp5521_unregister_leds(old_chip);
+       lp5521_deinit_device(old_chip);
 err_init:
        return ret;
 }
 
 static int lp5521_remove(struct i2c_client *client)
 {
-       struct lp5521_chip *chip = i2c_get_clientdata(client);
+       struct lp5521_chip *old_chip = i2c_get_clientdata(client);
 
-       lp5521_run_led_pattern(PATTERN_OFF, chip);
+       lp5521_run_led_pattern(PATTERN_OFF, old_chip);
        lp5521_unregister_sysfs(client);
 
-       lp5521_unregister_leds(chip);
+       lp5521_unregister_leds(old_chip);
 
-       lp5521_deinit_device(chip);
+       lp5521_deinit_device(old_chip);
        return 0;
 }
 
index fefe27c..49b9762 100644 (file)
@@ -1010,44 +1010,44 @@ static void lp5523_deinit_device(struct lp5523_chip *chip)
 static int lp5523_probe(struct i2c_client *client,
                        const struct i2c_device_id *id)
 {
-       struct lp5523_chip              *chip;
-       struct lp5523_platform_data     *pdata;
+       struct lp5523_chip              *old_chip;
+       struct lp5523_platform_data     *old_pdata;
        int ret, i;
 
-       chip = devm_kzalloc(&client->dev, sizeof(*chip), GFP_KERNEL);
-       if (!chip)
+       old_chip = devm_kzalloc(&client->dev, sizeof(*old_chip), GFP_KERNEL);
+       if (!old_chip)
                return -ENOMEM;
 
-       i2c_set_clientdata(client, chip);
-       chip->client = client;
+       i2c_set_clientdata(client, old_chip);
+       old_chip->client = client;
 
-       pdata = client->dev.platform_data;
+       old_pdata = client->dev.platform_data;
 
-       if (!pdata) {
+       if (!old_pdata) {
                dev_err(&client->dev, "no platform data\n");
                return -EINVAL;
        }
 
-       mutex_init(&chip->lock);
+       mutex_init(&old_chip->lock);
 
-       chip->pdata   = pdata;
+       old_chip->pdata   = old_pdata;
 
-       ret = lp5523_init_device(chip);
+       ret = lp5523_init_device(old_chip);
        if (ret)
                goto err_init;
 
        dev_info(&client->dev, "%s Programmable led chip found\n", id->name);
 
        /* Initialize engines */
-       for (i = 0; i < ARRAY_SIZE(chip->engines); i++) {
-               ret = lp5523_init_engine(&chip->engines[i], i + 1);
+       for (i = 0; i < ARRAY_SIZE(old_chip->engines); i++) {
+               ret = lp5523_init_engine(&old_chip->engines[i], i + 1);
                if (ret) {
                        dev_err(&client->dev, "error initializing engine\n");
                        goto fail1;
                }
        }
 
-       ret = lp5523_register_leds(chip, id->name);
+       ret = lp5523_register_leds(old_chip, id->name);
        if (ret)
                goto fail2;
 
@@ -1058,25 +1058,25 @@ static int lp5523_probe(struct i2c_client *client,
        }
        return ret;
 fail2:
-       lp5523_unregister_leds(chip);
+       lp5523_unregister_leds(old_chip);
 fail1:
-       lp5523_deinit_device(chip);
+       lp5523_deinit_device(old_chip);
 err_init:
        return ret;
 }
 
 static int lp5523_remove(struct i2c_client *client)
 {
-       struct lp5523_chip *chip = i2c_get_clientdata(client);
+       struct lp5523_chip *old_chip = i2c_get_clientdata(client);
 
        /* Disable engine mode */
        lp5523_write(client, LP5523_REG_OP_MODE, LP5523_CMD_DISABLED);
 
        lp5523_unregister_sysfs(client);
 
-       lp5523_unregister_leds(chip);
+       lp5523_unregister_leds(old_chip);
 
-       lp5523_deinit_device(chip);
+       lp5523_deinit_device(old_chip);
        return 0;
 }