Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932329AbVIJVkA (ORCPT ); Sat, 10 Sep 2005 17:40:00 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932328AbVIJVkA (ORCPT ); Sat, 10 Sep 2005 17:40:00 -0400 Received: from mail.dvmed.net ([216.237.124.58]:19861 "EHLO mail.dvmed.net") by vger.kernel.org with ESMTP id S932325AbVIJVj7 (ORCPT ); Sat, 10 Sep 2005 17:39:59 -0400 Message-ID: <432352A8.3010605@pobox.com> Date: Sat, 10 Sep 2005 17:39:52 -0400 From: Jeff Garzik User-Agent: Mozilla Thunderbird 1.0.6-1.1.fc4 (X11/20050720) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Greg KH CC: Jiri Slaby , Greg KH , Linux Kernel Mailing List , linux-pci@atrey.karlin.mff.cuni.cz, linux-ide@vger.kernel.org, B.Zolnierkiewicz@elka.pw.edu.pl Subject: Re: [PATCH] include: pci_find_device remove (include/asm-i386/ide.h) References: <200509102032.j8AKWxMC006246@localhost.localdomain> <4323482E.2090409@pobox.com> <20050910211932.GA13679@kroah.com> In-Reply-To: <20050910211932.GA13679@kroah.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Score: 0.0 (/) Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1096 Lines: 37 Greg KH wrote: > On Sat, Sep 10, 2005 at 04:55:10PM -0400, Jeff Garzik wrote: > >>Jiri Slaby wrote: >> >>>diff --git a/include/asm-i386/ide.h b/include/asm-i386/ide.h >>>--- a/include/asm-i386/ide.h >>>+++ b/include/asm-i386/ide.h >>>@@ -41,7 +41,12 @@ static __inline__ int ide_default_irq(un >>> >>>static __inline__ unsigned long ide_default_io_base(int index) >>>{ >>>- if (pci_find_device(PCI_ANY_ID, PCI_ANY_ID, NULL) == NULL) { >>>+ struct pci_dev *pdev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, NULL); >>>+ unsigned int a = !pdev; >>>+ >>>+ pci_dev_put(pdev); >> >> >>Looks like we need to resurrect pci_present() from the ancient past. > > > Heh, ick, no :) > > Jiri, any other way to do this instead? Look at what the IDE code is trying to do. All it cares about is whether -any PCI device at all- is present, a boolean value. Jeff - 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/