From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/7] arm/versatile: move pci code to plat-versatile
Date: Mon, 18 Oct 2010 17:35:50 +0200 [thread overview]
Message-ID: <201010181735.51248.arnd@arndb.de> (raw)
In-Reply-To: <AANLkTik4StWAGR2eygMTT48Vbv-pjUjTcACBOQBMas6j@mail.gmail.com>
On Monday 18 October 2010, Linus Walleij wrote:
> 2010/10/14 Arnd Bergmann <arnd@arndb.de>:
>
> > The Xilinx PCI macro is used on both versatile-pb and
> > realview-eb, so we should move the implementation to place
> > where it can be shared.
>
> > arch/arm/Kconfig | 5 +
> > arch/arm/mach-versatile/Makefile | 1 -
> > arch/arm/mach-versatile/pci.c | 361 ----------------------------------
> > arch/arm/plat-versatile/Makefile | 1 +
> > arch/arm/plat-versatile/xilinx-pci.c | 361 ++++++++++++++++++++++++++++++++++
>
> What about arch/arm/common?
>
> PCI drivers seem like they could be useful for others than ARM reference
> designs too. Xilinx FPGAs are probably quite common anyway.
I don't care either way while we don't have any other users. Most likely
a new platform using the same code would need some extra tweaks so we
could just as well move it again then.
OTOH it may also be nice as a generic PCI implementation for qemu
on other platforms which don't have their own PCI. If we ever get device
tree support running, qemu could just define the PCI host on any platform
and Linux can instantiate it from the dt.
> > (...)
> > +obj-$(CONFIG_PCI_HOST_XILINX) += xilinx-pci.o
> > diff --git a/arch/arm/plat-versatile/xilinx-pci.c b/arch/arm/plat-versatile/xilinx-pci.c
> > new file mode 100644
> > index 0000000..13c7e5f
> > --- /dev/null
> > +++ b/arch/arm/plat-versatile/xilinx-pci.c
> > @@ -0,0 +1,361 @@
> > +/*
> > + * linux/arch/arm/mach-versatile/pci.c
>
> Hm this file path seems inappropriate why not just delete it.
Right, will do.
Thanks,
Arnd
next prev parent reply other threads:[~2010-10-18 15:35 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 [this message]
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
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=201010181735.51248.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.