diff options
author | tim.niemeyer@mastersword.de <tim.niemeyer@mastersword.de> | 2009-03-24 11:11:20 +0100 |
---|---|---|
committer | Werner Almesberger <werner@openmoko.org> | 2009-03-26 02:55:26 -0300 |
commit | 08157c132e35e48b55f735b607ec51379e05bc53 (patch) | |
tree | 520d9a67a1f0799fb204eee7904d8212ecc43a6f /drivers | |
parent | c010bd7b90814dad534eb5544479f4316e83b712 (diff) |
Clean up pcf50606_client_dev_register
platform_device_add_data copies the data, so no need for
kmalloc.
Signed-off-by: Tim Niemeyer <reddog@mastersword.de>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/mfd/pcf50606-core.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/drivers/mfd/pcf50606-core.c b/drivers/mfd/pcf50606-core.c index 994d6f406c2..9d0bdae0cb5 100644 --- a/drivers/mfd/pcf50606-core.c +++ b/drivers/mfd/pcf50606-core.c @@ -436,7 +436,7 @@ static void pcf50606_client_dev_register(struct pcf50606 *pcf, const char *name, struct platform_device **pdev) { - struct pcf50606_subdev_pdata *subdev_pdata; + struct pcf50606_subdev_pdata subdev_pdata; int ret; *pdev = platform_device_alloc(name, -1); @@ -445,14 +445,8 @@ pcf50606_client_dev_register(struct pcf50606 *pcf, const char *name, return; } - subdev_pdata = kmalloc(sizeof(*subdev_pdata), GFP_KERNEL); - if (!subdev_pdata) { - dev_err(pcf->dev, "Error allocating subdev pdata\n"); - platform_device_put(*pdev); - } - - subdev_pdata->pcf = pcf; - platform_device_add_data(*pdev, subdev_pdata, sizeof(*subdev_pdata)); + subdev_pdata.pcf = pcf; + platform_device_add_data(*pdev, &subdev_pdata, sizeof(subdev_pdata)); (*pdev)->dev.parent = pcf->dev; |