[PATCH v2 2/3] dmaengine: imx-sdma: sdma_get_firmware does not need to copy fw_name

Uwe Kleine-König u.kleine-koenig at pengutronix.de
Mon Sep 5 19:20:35 EST 2011


On Fri, Jul 15, 2011 at 05:44:25PM +0800, Shawn Guo wrote:
> It does not need to allocate space and copy fw_name in function
> sdma_get_firmware().
> 
> Signed-off-by: Shawn Guo <shawn.guo at linaro.org>
> Cc: Vinod Koul <vinod.koul at intel.com>
> Cc: Sascha Hauer <s.hauer at pengutronix.de>
Acked-by: Uwe Kleine-König <u.kleine-koenig at pengutronix.de>
(appointed to act for Sascha who is on vacation)

Thanks
Uwe

> ---
>  drivers/dma/imx-sdma.c |   12 ++----------
>  1 files changed, 2 insertions(+), 10 deletions(-)
> 
> diff --git a/drivers/dma/imx-sdma.c b/drivers/dma/imx-sdma.c
> index 4a7aa72..6b839a2 100644
> --- a/drivers/dma/imx-sdma.c
> +++ b/drivers/dma/imx-sdma.c
> @@ -1125,22 +1125,14 @@ static int __init sdma_get_firmware(struct sdma_engine *sdma,
>  		const char *fw_name)
>  {
>  	const struct firmware *fw;
> -	char *fwname;
>  	const struct sdma_firmware_header *header;
>  	int ret;
>  	const struct sdma_script_start_addrs *addr;
>  	unsigned short *ram_code;
>  
> -	fwname = kasprintf(GFP_KERNEL, "%s", fw_name);
> -	if (!fwname)
> -		return -ENOMEM;
> -
> -	ret = request_firmware(&fw, fwname, sdma->dev);
> -	if (ret) {
> -		kfree(fwname);
> +	ret = request_firmware(&fw, fw_name, sdma->dev);
> +	if (ret)
>  		return ret;
> -	}
> -	kfree(fwname);
>  
>  	if (fw->size < sizeof(*header))
>  		goto err_firmware;
> -- 
> 1.7.4.1
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |


More information about the devicetree-discuss mailing list