mirror of
https://github.com/torvalds/linux.git
synced 2024-12-30 06:41:43 +00:00
c762b293d4
This fixes a small memory leak that was only hit in very unlikely error paths Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
92 lines
2.3 KiB
C
92 lines
2.3 KiB
C
/*
|
|
* Copyright 2008 Sascha Hauer, kernel@pengutronix.de
|
|
*
|
|
* This program is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU General Public License
|
|
* as published by the Free Software Foundation; either version 2
|
|
* of the License, or (at your option) any later version.
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program; if not, write to the Free Software
|
|
* Foundation, Inc., 51 Franklin Street, Fifth Floor,
|
|
* Boston, MA 02110-1301, USA.
|
|
*/
|
|
|
|
#include <linux/kernel.h>
|
|
#include <linux/slab.h>
|
|
#include <linux/init.h>
|
|
#include <linux/err.h>
|
|
#include <linux/platform_device.h>
|
|
#include <mach/common.h>
|
|
|
|
int __init mxc_register_device(struct platform_device *pdev, void *data)
|
|
{
|
|
int ret;
|
|
|
|
pdev->dev.platform_data = data;
|
|
|
|
ret = platform_device_register(pdev);
|
|
if (ret)
|
|
pr_debug("Unable to register platform device '%s': %d\n",
|
|
pdev->name, ret);
|
|
|
|
return ret;
|
|
}
|
|
|
|
struct platform_device *__init imx_add_platform_device_dmamask(
|
|
const char *name, int id,
|
|
const struct resource *res, unsigned int num_resources,
|
|
const void *data, size_t size_data, u64 dmamask)
|
|
{
|
|
int ret = -ENOMEM;
|
|
struct platform_device *pdev;
|
|
|
|
pdev = platform_device_alloc(name, id);
|
|
if (!pdev)
|
|
goto err;
|
|
|
|
if (dmamask) {
|
|
/*
|
|
* This memory isn't freed when the device is put,
|
|
* I don't have a nice idea for that though. Conceptually
|
|
* dma_mask in struct device should not be a pointer.
|
|
* See http://thread.gmane.org/gmane.linux.kernel.pci/9081
|
|
*/
|
|
pdev->dev.dma_mask =
|
|
kmalloc(sizeof(*pdev->dev.dma_mask), GFP_KERNEL);
|
|
if (!pdev->dev.dma_mask)
|
|
/* ret is still -ENOMEM; */
|
|
goto err;
|
|
|
|
*pdev->dev.dma_mask = dmamask;
|
|
pdev->dev.coherent_dma_mask = dmamask;
|
|
}
|
|
|
|
if (res) {
|
|
ret = platform_device_add_resources(pdev, res, num_resources);
|
|
if (ret)
|
|
goto err;
|
|
}
|
|
|
|
if (data) {
|
|
ret = platform_device_add_data(pdev, data, size_data);
|
|
if (ret)
|
|
goto err;
|
|
}
|
|
|
|
ret = platform_device_add(pdev);
|
|
if (ret) {
|
|
err:
|
|
if (dmamask)
|
|
kfree(pdev->dev.dma_mask);
|
|
platform_device_put(pdev);
|
|
return ERR_PTR(ret);
|
|
}
|
|
|
|
return pdev;
|
|
}
|