diff options
author | Cong Ding <dinggnu@gmail.com> | 2013-01-14 22:39:18 +0100 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2013-01-24 13:03:26 +0100 |
commit | b9975dabe3f0a6e4d1af52c47f66b5558df207a3 (patch) | |
tree | b52446b890c904e00faba1bc7fb8042776553211 /arch/x86/kernel/apb_timer.c | |
parent | d29a4a5fe8d687cee3ea372be00c61a09860b656 (diff) | |
download | lwn-b9975dabe3f0a6e4d1af52c47f66b5558df207a3.tar.gz lwn-b9975dabe3f0a6e4d1af52c47f66b5558df207a3.zip |
x86/apb/timer: Remove unnecessary "if"
adev has no chance to be NULL, so we don't need to check it. It
is also dereferenced just before the check .
Signed-off-by: Cong Ding <dinggnu@gmail.com>
Cc: Sasha Levin <sasha.levin@oracle.com>
Link: http://lkml.kernel.org/r/1358199561-15518-1-git-send-email-dinggnu@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'arch/x86/kernel/apb_timer.c')
-rw-r--r-- | arch/x86/kernel/apb_timer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/kernel/apb_timer.c b/arch/x86/kernel/apb_timer.c index afdc3f756dea..cf9273571917 100644 --- a/arch/x86/kernel/apb_timer.c +++ b/arch/x86/kernel/apb_timer.c @@ -240,7 +240,7 @@ static int apbt_cpuhp_notify(struct notifier_block *n, dw_apb_clockevent_pause(adev->timer); if (system_state == SYSTEM_RUNNING) { pr_debug("skipping APBT CPU %lu offline\n", cpu); - } else if (adev) { + } else { pr_debug("APBT clockevent for cpu %lu offline\n", cpu); dw_apb_clockevent_stop(adev->timer); } |