From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Wilcox Subject: Re: [PATCH] PCI: remove pcibios_scan_all_fns() Date: Mon, 22 Jun 2009 08:36:13 -0600 Message-ID: <20090622143613.GU19977@parisc-linux.org> References: <20090622140807.25509.54448.stgit@bob.kio> <20090622142602.GA25289@linux-mips.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from palinux.external.hp.com ([192.25.206.14]:49648 "EHLO mail.parisc-linux.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751907AbZFVOgL (ORCPT ); Mon, 22 Jun 2009 10:36:11 -0400 Content-Disposition: inline In-Reply-To: <20090622142602.GA25289@linux-mips.org> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Ralf Baechle Cc: Alex Chiang , jbarnes@virtuousgeek.org, linux-arch@vger.kernel.org, Kyle McMartin , Tony Luck , Russell King , Arnd Bergmann , Yoshinori Sato , Benjamin Herrenschmidt , Jeff Dike , linux-kernel@vger.kernel.org, David Howells , Paul Mundt , Ivan Kokshaysky , Ingo Molnar , "David S. Miller" On Mon, Jun 22, 2009 at 03:26:02PM +0100, Ralf Baechle wrote: > On Mon, Jun 22, 2009 at 08:08:07AM -0600, Alex Chiang wrote: > > This was #define'd as 0 on all platforms, so let's get rid of it. > > > > This change makes pci_scan_slot() slightly easier to read. > > I don't see why one would want to have a non-zero pcibios_scan_all_fns() > anyway. If you have real hardware, absolutely. If you have a hypervisor that gives you function 5 of a device (instead of mapping your function 0 to function 5 of the real device), you'll want something along these lines. -- Matthew Wilcox Intel Open Source Technology Centre "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step."