leds-lp5523: turn off the LED engines on unloading the driver
authorKim, Milo <Milo.Kim@ti.com>
Wed, 12 Sep 2012 12:16:00 +0000 (20:16 +0800)
committerBryan Wu <bryan.wu@canonical.com>
Thu, 13 Sep 2012 03:48:34 +0000 (11:48 +0800)
The LP5523 has 3 engines which are used for running LED patterns.
These engines should be off while unloading the driver.
Obviously, LP5523 platform data are used for releasing the resource
such like enable()/release_resource(), but these are not mandatory.
Therefore this patch is required without the platform data dependency.

Signed-off-by: Milo(Woogyom) Kim <milo.kim@ti.com>
Signed-off-by: Bryan Wu <bryan.wu@canonical.com>
drivers/leds/leds-lp5523.c

index 3d60428..5ebdc92 100644 (file)
@@ -997,6 +997,9 @@ static int lp5523_remove(struct i2c_client *client)
        struct lp5523_chip *chip = i2c_get_clientdata(client);
        int i;
 
+       /* Disable engine mode */
+       lp5523_write(client, LP5523_REG_OP_MODE, LP5523_CMD_DISABLED);
+
        lp5523_unregister_sysfs(client);
 
        for (i = 0; i < chip->num_leds; i++) {