aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/i2c/chips
diff options
context:
space:
mode:
authorJean Delvare <khali@linux-fr.org>2008-04-29 17:11:38 -0400
committerJean Delvare <khali@hyperion.delvare>2008-04-29 17:11:38 -0400
commit6d072d78f87e8fe0fe30d096991b83af07f8bdfe (patch)
tree5af6f58271aa711a2f8480f1e7449657ebc55204 /drivers/i2c/chips
parent1842cc2eeb345c4eef069ffd46e95359fb37b4b5 (diff)
i2c/tps65010: Add missing intialization of client data
tps65010_remove() calls i2c_get_clientdata(client) but the client data is never set during initialization, so it gets a NULL pointer at best. I guess it was never spotted because the tps65010 driver is typically not built modular so this function is discarded. Signed-off-by: Jean Delvare <khali@linux-fr.org> Cc: David Brownell <dbrownell@users.sourceforge.net>
Diffstat (limited to 'drivers/i2c/chips')
-rw-r--r--drivers/i2c/chips/tps65010.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/i2c/chips/tps65010.c b/drivers/i2c/chips/tps65010.c
index b67f69c2e7f3..feabd12c081c 100644
--- a/drivers/i2c/chips/tps65010.c
+++ b/drivers/i2c/chips/tps65010.c
@@ -527,6 +527,7 @@ static int __exit tps65010_remove(struct i2c_client *client)
527 flush_scheduled_work(); 527 flush_scheduled_work();
528 debugfs_remove(tps->file); 528 debugfs_remove(tps->file);
529 kfree(tps); 529 kfree(tps);
530 i2c_set_clientdata(client, NULL);
530 the_tps = NULL; 531 the_tps = NULL;
531 return 0; 532 return 0;
532} 533}
@@ -615,6 +616,7 @@ static int tps65010_probe(struct i2c_client *client)
615 i2c_smbus_read_byte_data(client, TPS_DEFGPIO), 616 i2c_smbus_read_byte_data(client, TPS_DEFGPIO),
616 i2c_smbus_read_byte_data(client, TPS_MASK3)); 617 i2c_smbus_read_byte_data(client, TPS_MASK3));
617 618
619 i2c_set_clientdata(client, tps);
618 the_tps = tps; 620 the_tps = tps;
619 621
620#if defined(CONFIG_USB_GADGET) && !defined(CONFIG_USB_OTG) 622#if defined(CONFIG_USB_GADGET) && !defined(CONFIG_USB_OTG)