lkml.org 
[lkml]   [2015]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v5 4/6] memory: mediatek: Add SMI driver
Date
On 09/10/15 03:23, Yong Wu wrote:
[...]
> +static int mtk_smi_probe(struct platform_device *pdev)
> +{
> + struct device *dev = &pdev->dev;
> + struct mtk_smi_data *smidata;
> + int ret;
> +
> + if (!dev->pm_domain)
> + return -EPROBE_DEFER;
> +
> + smidata = devm_kzalloc(dev, sizeof(*smidata), GFP_KERNEL);
> + if (!smidata)
> + return -ENOMEM;
> +
> + smidata->clk_apb = devm_clk_get(dev, "apb");
> + if (IS_ERR(smidata->clk_apb))
> + return PTR_ERR(smidata->clk_apb);
> +
> + smidata->clk_smi = devm_clk_get(dev, "smi");
> + if (IS_ERR(smidata->clk_smi))
> + return PTR_ERR(smidata->clk_smi);
> +
> + pm_runtime_enable(dev);
> + dev_set_drvdata(dev, smidata);
> + return ret;

ret is used uninitialised here, but you might as well just "return 0;"
and get rid of the variable entirely.

> +}

Robin.



\
 
 \ /
  Last update: 2015-10-27 14:41    [W:0.101 / U:0.612 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site