Mirrors/u-boot
2
0
Fork 0
mirror of https://github.com/AsahiLinux/u-boot synced 2024-11-10 15:14:43 +00:00
Code Issues Projects Releases Packages Wiki Activity

clk: sandbox: don't check clk ID against 0

Browse source 
clk->id is unsigned, so it can't be < 0. Remove the check for that.

FWIW, this issue was introduced when the clock API converted e.g.
clk_get_rate()'s clock ID parameter from an int to an unsigned long
(with a struct clk), without removing this check.

Fixes: 135aa95002 ("clk: convert API to match reset/mailbox style")
Reported-by: Coverity Scan
Signed-off-by: Stephen Warren <swarren@nvidia.com>
Acked-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Stephen Warren 2016-06-21 13:32:07 -06:00 committed by Tom Rini
parent bd62e2419b
commit df8b0a0373
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
drivers/clk/clk_sandbox.c
View file

@ -19,7 +19,7 @@ static ulong sandbox_clk_get_rate(struct clk *clk)
{ {
struct sandbox_clk_priv *priv = dev_get_priv(clk->dev); struct sandbox_clk_priv *priv = dev_get_priv(clk->dev);
if (clk->id < 0 || clk->id >= SANDBOX_CLK_ID_COUNT) if (clk->id >= SANDBOX_CLK_ID_COUNT)
return -EINVAL; return -EINVAL;
return priv->rate[clk->id]; return priv->rate[clk->id];
@ -30,7 +30,7 @@ static ulong sandbox_clk_set_rate(struct clk *clk, ulong rate)
struct sandbox_clk_priv *priv = dev_get_priv(clk->dev); struct sandbox_clk_priv *priv = dev_get_priv(clk->dev);
ulong old_rate; ulong old_rate;
if (clk->id < 0 || clk->id >= SANDBOX_CLK_ID_COUNT) if (clk->id >= SANDBOX_CLK_ID_COUNT)
return -EINVAL; return -EINVAL;
if (!rate) if (!rate)
@ -46,7 +46,7 @@ static int sandbox_clk_enable(struct clk *clk)
{ {
struct sandbox_clk_priv *priv = dev_get_priv(clk->dev); struct sandbox_clk_priv *priv = dev_get_priv(clk->dev);
if (clk->id < 0 || clk->id >= SANDBOX_CLK_ID_COUNT) if (clk->id >= SANDBOX_CLK_ID_COUNT)
return -EINVAL; return -EINVAL;
priv->enabled[clk->id] = true; priv->enabled[clk->id] = true;
@ -58,7 +58,7 @@ static int sandbox_clk_disable(struct clk *clk)
{ {
struct sandbox_clk_priv *priv = dev_get_priv(clk->dev); struct sandbox_clk_priv *priv = dev_get_priv(clk->dev);
if (clk->id < 0 || clk->id >= SANDBOX_CLK_ID_COUNT) if (clk->id >= SANDBOX_CLK_ID_COUNT)
return -EINVAL; return -EINVAL;
priv->enabled[clk->id] = false; priv->enabled[clk->id] = false;