From: Michal Simek <michal.simek@petalogix.com>
To: Grant Likely <grant.likely@secretlab.ca>,
linuxppc-dev@ozlabs.org, benh@kernel.crashing.org,
thunderbird2k@gmx.net,
John Williams <john.williams@petalogix.com>,
Arnd Bergmann <arnd@arndb.de>,
LKML <linux-kernel@vger.kernel.org>
Subject: xilinx-pci driver and pci in general
Date: Wed, 13 Jan 2010 16:23:57 +0100 [thread overview]
Message-ID: <4B4DE58D.10008@petalogix.com> (raw)
Hi guys,
We (John and partially I) did initial support for pci on Microblaze.
It is based on powerpc files and almost everything is the same.
There are some small differences which could be easily removed that's
why I think that will be good to move that file to any generic location.
That's why I would like to know your opinion about this step.
If you don't like or prefer we will add that modified versions to
arch/microblaze and in future we can merge them and move to any other
location.
Affected files: xilinx-pci.c, pci_32.c pci-common.c, indirect_pci.c and
some headers.
The main problems are:
ppc use ppc_md struct which we don't have it on Microblaze.
xilinx-pci driver uses exclude_device function. This function is used in
indirect_pci.c too. There could be a way to move that function directly
to pci_controller structure which could be useful for other controllers
too. What do you think?
Then there are some other ppc_md. calling like pcibios_after_init which
if I see correctly not used for ppc too.
The next thing is that some files contains asm/machdep.h which could be
added to asm/pci-bridge.h and the same is for asm/ppc-pci.h
Files contains CONFIG_PPC_OF and we would like to use only CONFIG_OF.
I remember any discuss around but not sure what was the conclusion on
powerpc.
Part of headers are the same that's why there will be a space to move
them to asm-generic.
Anyway: I look at your dma-mapping.h and you can use
asm-generic/dma-mapping-common.h as I am using.
Then I have some question about EARLY_PCI_OP in ppc_32.c. Is there any
reason to use early_##rw##_config_##size fucntions instead of proper
pci_bus_##rw##_config_##size functions?
There is one comment that these functions are used before PCI scanning
is done but there are used the same function as are in driver/pci/access.c.
Is there any "secret" reason to do it in this way?
Thanks for this early discuss. I would like to hear your opinion and
then I will choose solution how to add our pci support to mainline.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng)
PetaLogix - Linux Solutions for a Reconfigurable World
w: www.petalogix.com p: +61-7-30090663,+42-0-721842854 f: +61-7-30090663
next reply other threads:[~2010-01-13 15:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-13 15:23 Michal Simek [this message]
2010-01-14 2:07 ` xilinx-pci driver and pci in general Benjamin Herrenschmidt
2010-01-15 9:23 ` Michal Simek
2010-01-18 17:39 ` Grant Likely
2010-01-18 17:38 ` Grant Likely
2010-01-18 17:36 ` Grant Likely
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=4B4DE58D.10008@petalogix.com \
--to=michal.simek@petalogix.com \
--cc=arnd@arndb.de \
--cc=benh@kernel.crashing.org \
--cc=grant.likely@secretlab.ca \
--cc=john.williams@petalogix.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=thunderbird2k@gmx.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox