Commit 422650e6 authored by Stephen Warren's avatar Stephen Warren Committed by Mark Brown

ASoC: tegra: s/IS_ERR_OR_NULL/IS_ERR/ for clk_get_sys

A recent discussion on linux-arm-kernel noted that the value returned by
clk_get_sys is an opaque token, and not strictly a pointer; it is
meaningful only to the clock API, clients should not dereference the value,
and the clock API must accept any non-IS_ERR value it returned.

Hence, only IS_ERR is appropriate to interpret the result, not
IS_ERR_OR_NULL.

I checked that clk_get_sys in both ASoC's for-next and Tegra's for-next
do behave as described; NULL is not returned in the case of error.
Signed-off-by: default avatarStephen Warren <swarren@nvidia.com>
Acked-by: default avatarLiam Girdwood <lrg@slimlogic.co.uk>
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 1500b7b5
......@@ -113,35 +113,33 @@ int tegra_asoc_utils_init(void)
int ret;
clk_pll_a = clk_get_sys(NULL, "pll_a");
if (IS_ERR_OR_NULL(clk_pll_a)) {
if (IS_ERR(clk_pll_a)) {
pr_err(PREFIX "Can't retrieve clk pll_a\n");
ret = PTR_ERR(clk_pll_a);
goto err;
}
clk_pll_a_out0 = clk_get_sys(NULL, "pll_a_out0");
if (IS_ERR_OR_NULL(clk_pll_a_out0)) {
if (IS_ERR(clk_pll_a_out0)) {
pr_err(PREFIX "Can't retrieve clk pll_a_out0\n");
ret = PTR_ERR(clk_pll_a_out0);
goto err;
goto err_put_pll_a;
}
clk_cdev1 = clk_get_sys(NULL, "cdev1");
if (IS_ERR_OR_NULL(clk_cdev1)) {
if (IS_ERR(clk_cdev1)) {
pr_err(PREFIX "Can't retrieve clk cdev1\n");
ret = PTR_ERR(clk_cdev1);
goto err;
goto err_put_pll_a_out0;
}
return 0;
err_put_pll_a_out0:
clk_put(clk_pll_a_out0);
err_put_pll_a:
clk_put(clk_pll_a);
err:
if (!IS_ERR_OR_NULL(clk_cdev1))
clk_put(clk_cdev1);
if (!IS_ERR_OR_NULL(clk_pll_a_out0))
clk_put(clk_pll_a_out0);
if (!IS_ERR_OR_NULL(clk_pll_a))
clk_put(clk_pll_a);
return ret;
}
......
......@@ -385,7 +385,7 @@ static __devinit int tegra_i2s_platform_probe(struct platform_device *pdev)
snprintf(clk_name, sizeof(clk_name), DRV_NAME ".%d", pdev->id);
i2s->clk_i2s = clk_get_sys(clk_name, NULL);
if (IS_ERR_OR_NULL(i2s->clk_i2s)) {
if (IS_ERR(i2s->clk_i2s)) {
pr_err("Can't retrieve i2s clock\n");
ret = PTR_ERR(i2s->clk_i2s);
goto err_free;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment