summaryrefslogtreecommitdiff
path: root/arch
diff options
context:
space:
mode:
authorxypron.glpk@gmx.de <xypron.glpk@gmx.de>2017-07-30 20:15:51 +0200
committerTom Rini <trini@konsulko.com>2017-08-13 15:17:26 -0400
commitb69a0849e75e5adff081d1f0f5557da03d703482 (patch)
tree5dbba25d678264a26f744a52af4d777f7a4ee805 /arch
parent9730bcdc2f6c237fde66b7d81681e380ef0bdedf (diff)
downloadu-boot-b69a0849e75e5adff081d1f0f5557da03d703482.tar.gz
u-boot-b69a0849e75e5adff081d1f0f5557da03d703482.tar.bz2
u-boot-b69a0849e75e5adff081d1f0f5557da03d703482.zip
arm: bcm235xx: avoid possible NULL dereference
It does not make sense to first dereference c and then check if it is NULL. Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
Diffstat (limited to 'arch')
-rw-r--r--arch/arm/cpu/armv7/bcm235xx/clk-core.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/cpu/armv7/bcm235xx/clk-core.c b/arch/arm/cpu/armv7/bcm235xx/clk-core.c
index 79fafa08ed..ee4b34574a 100644
--- a/arch/arm/cpu/armv7/bcm235xx/clk-core.c
+++ b/arch/arm/cpu/armv7/bcm235xx/clk-core.c
@@ -479,9 +479,9 @@ unsigned long clk_get_rate(struct clk *c)
{
unsigned long rate;
- debug("%s: %s\n", __func__, c->name);
if (!c || !c->ops || !c->ops->get_rate)
return 0;
+ debug("%s: %s\n", __func__, c->name);
rate = c->ops->get_rate(c);
debug("%s: rate = %ld\n", __func__, rate);