Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S266200AbUJLQsb (ORCPT ); Tue, 12 Oct 2004 12:48:31 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S266196AbUJLQsb (ORCPT ); Tue, 12 Oct 2004 12:48:31 -0400 Received: from e2.ny.us.ibm.com ([32.97.182.102]:50847 "EHLO e2.ny.us.ibm.com") by vger.kernel.org with ESMTP id S266200AbUJLQsX (ORCPT ); Tue, 12 Oct 2004 12:48:23 -0400 Date: Tue, 12 Oct 2004 09:48:20 -0700 From: Hanna Linder To: sfeldma@pobox.com cc: Hanna Linder , lkml , kernel-janitors , benh@kernel.crashing.org, paulus@samba.org, greg@kroah.com Subject: Re: [Kernel-janitors] [PATCH 2.6][9/12] pplus.c replace pci_find_device with pci_get_device Message-ID: <138690000.1097599700@w-hlinder.beaverton.ibm.com> In-Reply-To: <1097341320.3903.3.camel@sfeldma-mobl2.dsl-verizon.net> References: <33930000.1097191565@w-hlinder.beaverton.ibm.com> <1097341320.3903.3.camel@sfeldma-mobl2.dsl-verizon.net> X-Mailer: Mulberry/2.2.1 (Linux/x86) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 780 Lines: 31 --On Saturday, October 09, 2004 10:02:01 AM -0700 Scott Feldman wrote: > On Thu, 2004-10-07 at 16:26, Hanna Linder wrote: >> - if ((dev = pci_find_device(PCI_VENDOR_ID_WINBOND, >> + if ((dev = pci_get_device(PCI_VENDOR_ID_WINBOND, >> PCI_DEVICE_ID_WINBOND_82C105, dev))) { >> /* >> * Disable LEGIRQ mode so PCI INTS are routed > > Missing cleanup at the bottom of func? > > if(dev) > pci_dev_put(dev); > > -scott > > OK. I see what you mean now. Will reroll patches today. Thanks. Hanna - 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/