From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760561AbXJXQgu (ORCPT ); Wed, 24 Oct 2007 12:36:50 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1758994AbXJXQYc (ORCPT ); Wed, 24 Oct 2007 12:24:32 -0400 Received: from emailhub.stusta.mhn.de ([141.84.69.5]:60931 "EHLO mailhub.stusta.mhn.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1759585AbXJXQYb (ORCPT ); Wed, 24 Oct 2007 12:24:31 -0400 Date: Wed, 24 Oct 2007 18:25:00 +0200 From: Adrian Bunk To: gregkh@suse.de Cc: linux-kernel@vger.kernel.org, linux-pci@atrey.karlin.mff.cuni.cz Subject: [2.6 patch] cpqhp_pushbutton_thread(): remove a pointless if() check Message-ID: <20071024162500.GS30533@stusta.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline User-Agent: Mutt/1.5.16 (2007-06-11) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org The Coverity checker spotted that we'd have already oops'ed if "ctrl" was NULL. Additionally, "func" had just been checked for not being NULL. Signed-off-by: Adrian Bunk --- This patch has been sent on: - 14 Aug 2007 --- linux-2.6.23-rc1-mm2/drivers/pci/hotplug/cpqphp_ctrl.c.old 2007-08-08 14:00:24.000000000 +0200 +++ linux-2.6.23-rc1-mm2/drivers/pci/hotplug/cpqphp_ctrl.c 2007-08-08 14:01:16.000000000 +0200 @@ -1951,30 +1951,28 @@ void cpqhp_pushbutton_thread(unsigned lo if (is_slot_enabled(ctrl, hp_slot)) { p_slot->state = POWEROFF_STATE; /* power Down board */ func = cpqhp_slot_find(p_slot->bus, p_slot->device, 0); dbg("In power_down_board, func = %p, ctrl = %p\n", func, ctrl); if (!func) { dbg("Error! func NULL in %s\n", __FUNCTION__); return ; } - if (func != NULL && ctrl != NULL) { - if (cpqhp_process_SS(ctrl, func) != 0) { - amber_LED_on (ctrl, hp_slot); - green_LED_on (ctrl, hp_slot); - - set_SOGO(ctrl); + if (cpqhp_process_SS(ctrl, func) != 0) { + amber_LED_on (ctrl, hp_slot); + green_LED_on (ctrl, hp_slot); + + set_SOGO(ctrl); - /* Wait for SOBS to be unset */ - wait_for_ctrl_irq (ctrl); - } + /* Wait for SOBS to be unset */ + wait_for_ctrl_irq (ctrl); } p_slot->state = STATIC_STATE; } else { p_slot->state = POWERON_STATE; /* slot is off */ func = cpqhp_slot_find(p_slot->bus, p_slot->device, 0); dbg("In add_board, func = %p, ctrl = %p\n", func, ctrl); if (!func) {