Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933858Ab0FRRb6 (ORCPT ); Fri, 18 Jun 2010 13:31:58 -0400 Received: from mail-pw0-f46.google.com ([209.85.160.46]:47124 "EHLO mail-pw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933739Ab0FRRb4 (ORCPT ); Fri, 18 Jun 2010 13:31:56 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; b=Lamkqyt6P5x2z7bXM++JO1yvfbq1GapvpfG3zksz2dNOJtKkc0wviWxFYsi52aGIp4 XeX4R7TeM96ckWn6a8kv0vSCnEzzHff4LeB61JZEA6a1qzBsI4/qPQKuMTml4JS4kI+O 7JWvdmyA2X3YmB52X7sf5o8VOoaeEj9SCBTsI= Message-ID: <4C1BAD88.9050903@gmail.com> Date: Fri, 18 Jun 2010 10:31:52 -0700 From: "Justin P. Mattock" User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.4pre) Gecko/20091114 Lightning/1.0pre Thunderbird/3.0b4 MIME-Version: 1.0 To: Jesse Barnes CC: linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, linux-pci@vger.kernel.org, linux-scsi@vger.kernel.org, Yinghai Lu Subject: Re: [PATCH 4/5]pci:setup_bus.c Fix warning: variable 'retval' set but not used References: <1276666434-11227-1-git-send-email-justinmattock@gmail.com> <1276666434-11227-5-git-send-email-justinmattock@gmail.com> <20100618102330.29eccb5b@virtuousgeek.org> In-Reply-To: <20100618102330.29eccb5b@virtuousgeek.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2164 Lines: 55 On 06/18/2010 10:23 AM, Jesse Barnes wrote: > On Tue, 15 Jun 2010 22:33:53 -0700 > "Justin P. Mattock" wrote: > >> The below patch fixes a warning message when using gcc 4.6.0 >> CC drivers/pci/setup-bus.o >> drivers/pci/setup-bus.c: In function 'pci_assign_unassigned_bridge_resources': >> drivers/pci/setup-bus.c:868:6: warning: variable 'retval' set but not used >> >> Signed-off-by: Justin P. Mattock >> >> --- >> drivers/pci/setup-bus.c | 2 -- >> 1 files changed, 0 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c >> index 19b1113..215590b 100644 >> --- a/drivers/pci/setup-bus.c >> +++ b/drivers/pci/setup-bus.c >> @@ -865,7 +865,6 @@ void pci_assign_unassigned_bridge_resources(struct pci_dev *bridge) >> struct pci_bus *parent = bridge->subordinate; >> int tried_times = 0; >> struct resource_list_x head, *list; >> - int retval; >> unsigned long type_mask = IORESOURCE_IO | IORESOURCE_MEM | >> IORESOURCE_PREFETCH; >> >> @@ -874,7 +873,6 @@ void pci_assign_unassigned_bridge_resources(struct pci_dev *bridge) >> again: >> pci_bus_size_bridges(parent); >> __pci_bridge_assign_resources(bridge,&head); >> - retval = pci_reenable_device(bridge); >> pci_set_master(bridge); >> pci_enable_bridges(parent); >> > > This has the same problem as your earlier bridge_enable patch; you need > to keep the call to pci_reenable_device. I should have caught this > issue when Yinghai's patch went in: the right way to silence a warning > about not checking a return value isn't to simply assign it to > something, you really should *do* something as well, at least some > debug output would be nice, but ideally handling the error in a sane > way. > yeah I was still looking into the scsi patch with adding a printk then I can look at this as well. Justin P. Mattock -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/