From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH 1/5] pci/of: Match PCI devices to OF nodes dynamically Date: Thu, 07 Apr 2011 15:29:37 -0700 (PDT) Message-ID: <20110407.152937.59699180.davem@davemloft.net> References: <1302146796-26825-1-git-send-email-benh@kernel.crashing.org> <1302146796-26825-2-git-send-email-benh@kernel.crashing.org> <20110407.152707.39195948.davem@davemloft.net> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Return-path: Received: from 74-93-104-97-Washington.hfc.comcastbusiness.net ([74.93.104.97]:51818 "EHLO sunset.davemloft.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755867Ab1DGWaO (ORCPT ); Thu, 7 Apr 2011 18:30:14 -0400 In-Reply-To: <20110407.152707.39195948.davem@davemloft.net> Sender: linux-arch-owner@vger.kernel.org List-ID: To: benh@kernel.crashing.org Cc: linux-pci@vger.kernel.org, linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, bheglaas@google.com, monstr@monstr.eu, tglx@linuxtronix.de, bigeasy@linuxtronix.de From: David Miller Date: Thu, 07 Apr 2011 15:27:07 -0700 (PDT) > However, of_pci_find_child_device() lives in drivers/of/of_pci.c which is not > built into the tree because you haven't provided the necessary changes to > drivers/of/Makefile in this patch. Oh I see, it exists already but it's only built into the tree for microblaze, x86, and ppc. There's a lot of stuff in there which is superfluous and will never be used on sparc. Everything in there except the new of_pci_find_child_device() function will be never ever used on sparc and therefore dead code.