mirror of
https://github.com/torvalds/linux.git
synced 2024-11-12 07:01:57 +00:00
mmc: meson-mx-sdio: Cleanup IS_ERR() checks
Using PTR_ERR_OR_ZERO() instead of IS_ERR() works, but it's not how you're supposed to write these conditions. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
This commit is contained in:
parent
7599b84932
commit
2f129d39ad
@ -602,7 +602,7 @@ static int meson_mx_mmc_register_clks(struct meson_mx_mmc_host *host)
|
||||
|
||||
host->fixed_factor_clk = devm_clk_register(host->controller_dev,
|
||||
&host->fixed_factor.hw);
|
||||
if (WARN_ON(PTR_ERR_OR_ZERO(host->fixed_factor_clk)))
|
||||
if (WARN_ON(IS_ERR(host->fixed_factor_clk)))
|
||||
return PTR_ERR(host->fixed_factor_clk);
|
||||
|
||||
clk_div_parent = __clk_get_name(host->fixed_factor_clk);
|
||||
@ -620,7 +620,7 @@ static int meson_mx_mmc_register_clks(struct meson_mx_mmc_host *host)
|
||||
|
||||
host->cfg_div_clk = devm_clk_register(host->controller_dev,
|
||||
&host->cfg_div.hw);
|
||||
if (WARN_ON(PTR_ERR_OR_ZERO(host->cfg_div_clk)))
|
||||
if (WARN_ON(IS_ERR(host->cfg_div_clk)))
|
||||
return PTR_ERR(host->cfg_div_clk);
|
||||
|
||||
return 0;
|
||||
|
Loading…
Reference in New Issue
Block a user