From: Olof Johansson <olof@lixom.net>
To: Stephen Rothwell <sfr@canb.auug.org.au>
Cc: ppc-dev <linuxppc-dev@ozlabs.org>
Subject: Re: [PATCH 07/19] [POWERPC] iSeries: unindent and clean iSeries_pci_final_fixup
Date: Thu, 6 Dec 2007 12:33:19 -0600 [thread overview]
Message-ID: <20071206183319.GA9949@lixom.net> (raw)
In-Reply-To: <20071207015148.917436cc.sfr@canb.auug.org.au>
On Fri, Dec 07, 2007 at 01:51:48AM +1100, Stephen Rothwell wrote:
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> ---
> arch/powerpc/platforms/iseries/pci.c | 68 +++++++++++++++++----------------
> 1 files changed, 35 insertions(+), 33 deletions(-)
>
> diff --git a/arch/powerpc/platforms/iseries/pci.c b/arch/powerpc/platforms/iseries/pci.c
> index cff832a..8e2ac3d 100644
> --- a/arch/powerpc/platforms/iseries/pci.c
> +++ b/arch/powerpc/platforms/iseries/pci.c
> @@ -177,47 +177,49 @@ void __init iSeries_pci_final_fixup(void)
>
> printk("pcibios_final_fixup\n");
> for_each_pci_dev(pdev) {
> + struct pci_dn *pdn;
> + const u32 *agent;
> +
> node = find_device_node(pdev->bus->number, pdev->devfn);
> printk("pci dev %p (%x.%x), node %p\n", pdev,
> pdev->bus->number, pdev->devfn, node);
> + if (!node) {
> + printk("PCI: Device Tree not found for 0x%016lX\n",
> + (unsigned long)pdev);
Hi,
Care to add KERN_ levels, while you're touching it? (both printks above
lack them).
-Olof
next prev parent reply other threads:[~2007-12-06 18:29 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-12-06 7:00 Powerpc PCI cleanups (mainly iSeries) Stephen Rothwell
2007-12-06 7:02 ` [PATCH 01/19] [POWERPC] clean up pci-bridge.h Stephen Rothwell
2007-12-06 7:06 ` [PATCH 02/19] [POWERPC] consolidate pci_controller Stephen Rothwell
2007-12-11 0:00 ` Stephen Rothwell
2007-12-06 14:46 ` [PATCH 03/19] [POWERPC] iSeries: move find_Device_Node to avoid a forward declaration Stephen Rothwell
2007-12-06 14:48 ` [PATCH 04/19] [POWERPC] iSeries: cleanup PCI retry code a little Stephen Rothwell
2007-12-06 14:49 ` [PATCH 05/19] [POWERPC] iSeries: deCamelCase pci.c Stephen Rothwell
2007-12-06 14:50 ` [PATCH 06/19] [POWERPC] iSeries: remove some dead code from pci.c Stephen Rothwell
2007-12-06 14:51 ` [PATCH 07/19] [POWERPC] iSeries: unindent and clean iSeries_pci_final_fixup Stephen Rothwell
2007-12-06 18:33 ` Olof Johansson [this message]
2007-12-06 14:52 ` [PATCH 08/19] [POWERPC] iSeries: consoldiate PCI IO error check Stephen Rothwell
2007-12-06 14:53 ` [PATCH 09/19] [POWERPC] iSeries: remove one layer of IO routines Stephen Rothwell
2007-12-06 14:54 ` [PATCH 10/19] [POWERPC] iSeries: remove pci_dn dependency from iSeries_Device_Information Stephen Rothwell
2007-12-06 14:55 ` [PATCH 11/19] [POWERPC] iSeries: iseries_ds_addr is only used in pci.c Stephen Rothwell
2007-12-06 14:56 ` [PATCH 12/19] [POWERPC] iSeries: reduce dependence on pci_dn bussubno Stephen Rothwell
2007-12-06 14:58 ` [PATCH 13/19] [POWERPC] iSeries: make pcibios_final_fixup not depend on pci_dn Stephen Rothwell
2007-12-06 14:59 ` [PATCH 14/19] [POWERPC] Inline pci_setup_pci_controller as it has become trivial Stephen Rothwell
2007-12-11 0:02 ` Stephen Rothwell
2007-12-06 15:00 ` [PATCH 15/19] [POWERPC] iSeries: call iSeries_pcibios_init from setup_arch Stephen Rothwell
2007-12-06 15:01 ` [PATCH 16/19] [POWERPC] iSeries: hose->buid is always zero for iSeries Stephen Rothwell
2007-12-11 0:03 ` Stephen Rothwell
2007-12-06 15:03 ` [PATCH 17/19] [POWERPC] remove some iSeries platform checks from the PCI code Stephen Rothwell
2007-12-06 15:04 ` [PATCH 18/19] [POWERPC] Don't special case pci_domain_nr() for iSeries Stephen Rothwell
2007-12-06 15:05 ` [PATCH 19/19] [POWERPC] pci_controller->arch_data really is a struct device_node * Stephen Rothwell
2007-12-06 16:56 ` Jon Loeliger
2007-12-09 7:27 ` Benjamin Herrenschmidt
2007-12-10 3:33 ` Stephen Rothwell
2007-12-06 15:07 ` Powerpc PCI cleanups (mainly iSeries) Stephen Rothwell
2007-12-06 15:44 ` Josh Boyer
2007-12-09 7:29 ` Benjamin Herrenschmidt
2007-12-09 14:40 ` Josh Boyer
2007-12-07 5:08 ` [PATCH 1/3] [POWERPC] iSeries: DeCamelCase vpdinfo.c Stephen Rothwell
2007-12-07 5:09 ` [PATCH 2/3] [POWERPC] iSeries: clean up and simplify vdpinfo.c Stephen Rothwell
2007-12-07 5:10 ` [PATCH 3/3] [POWERPC] iSeries: merge vpdinfo.c intp pci.c Stephen Rothwell
2007-12-11 0:06 ` Powerpc PCI cleanups (mainly iSeries) Stephen Rothwell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20071206183319.GA9949@lixom.net \
--to=olof@lixom.net \
--cc=linuxppc-dev@ozlabs.org \
--cc=sfr@canb.auug.org.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.