All of lore.kernel.org
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/7] arm/versatile PCI support, second try
Date: Sat, 16 Oct 2010 23:54:20 +0200	[thread overview]
Message-ID: <201010162354.20418.arnd@arndb.de> (raw)
In-Reply-To: <AANLkTikc5Vc0Q0z4rE4raX7LTyB=LiE+YuzycMjsXUaM@mail.gmail.com>

On Saturday 16 October 2010, Linus Walleij wrote:
> 2010/10/14 Arnd Bergmann <arnd@arndb.de>:
> 
> > With lots of help and testing from Peter Maydell, I've completed
> > a second version of the PCI support for the versatile platform,
> > which works much better now on real hardware, although it does
> > not improve existing qemu setups.
> 
> I've got nothing better to do tonite than playing with the PB1176
> board so tried this out...
> 
> Some fixup is needed in 6/7, the EB patch because I get this:
> arch/arm/mach-realview/realview_eb.c:94: error:
> 'REALVIEW_PCI_CORE_BASE' undeclared here (not in a function)
> arch/arm/mach-realview/realview_eb.c:110: error:
> 'REALVIEW_PCI_IO_BASE' undeclared here (not in a function)
> arch/arm/mach-realview/realview_eb.c:111: error:
> 'REALVIEW_PCI_IO_BASE_SIZE' undeclared here (not in a function)
> make[3]: *** [arch/arm/mach-realview/realview_eb.o] Error 1
> make[2]: *** [arch/arm/mach-realview] Error 2
> make[2]: *** Waiting for unfinished jobs....
> 
> No these are indeed named
> REALVIEW_EB_PCI_CORE_BASE
> REALVIEW_EB_PCI_IO_BASE0
> etc...

Ah right, I removed these when I prepared the patches for submission,
initially I was using the common definitions for the realview boards
but I figured it would be bad since newer realviews have a different
controller. Apparently I forgot to move over the EB code to use the
new definitions I added later.

I'll fix these up.

> If I disable the EB support it compiles and boots. Nothing much
> else happens, not even if I enable PCI debugging. But I guess
> that's normal since I don't have any PCI enclosure expansion
> board for this card?

Yes, without a PCI card to plug into it it's pretty pointless.

> The only visible change is that /sys/bus/pci now exists with no
> slots nor devices in it.
> 
> But it's not causing any regressions for me so with a fix
> for the compile problem it's
> Tested-by: Linus Walleij <linus.walleij@stericsson.com>

Thanks,

	Arnd

  reply	other threads:[~2010-10-16 21:54 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-14 16:10 [PATCH 0/7] arm/versatile PCI support, second try Arnd Bergmann
2010-10-14 16:10 ` [PATCH 1/7] arm/versatile: move pci code to plat-versatile Arnd Bergmann
2010-10-18 15:20   ` Linus Walleij
2010-10-18 15:35     ` Arnd Bergmann
2010-10-14 16:10 ` [PATCH 2/7] arm/versatile: boot-time configure xilinx-pci Arnd Bergmann
2010-10-14 16:10 ` [PATCH 3/7] arm/versatile: enable PCI I/O space Arnd Bergmann
2010-10-14 16:10 ` [PATCH 4/7] arm/versatile: use correct PCI IRQ swizzling Arnd Bergmann
2010-10-14 16:10 ` [PATCH 5/7] arm/realview: fix building PB-A8 and PBX with CONFIG_PCI Arnd Bergmann
2010-10-14 16:10 ` [PATCH 6/7] arm/realview: enable PCI for realview-eb and realview-pb1176 Arnd Bergmann
2010-10-14 16:10 ` [PATCH 7/7] arm: Enable support for virtio Arnd Bergmann
2010-10-16 20:52 ` [PATCH 0/7] arm/versatile PCI support, second try Linus Walleij
2010-10-16 21:54   ` Arnd Bergmann [this message]
2010-10-18  8:54   ` Colin Tuckley
2010-10-18 15:23     ` Arnd Bergmann
     [not found]   ` <-1220762139438340023@unknownmsgid>
2010-10-18 15:08     ` Linus Walleij
2010-10-18 18:13 ` Peter Maydell

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=201010162354.20418.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.