<br><br><div class="gmail_quote">On Fri, Apr 17, 2009 at 10:49 PM, Grant Likely <span dir="ltr"><<a href="mailto:grant.likely@secretlab.ca">grant.likely@secretlab.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
On Fri, Apr 17, 2009 at 11:06 AM, Stephen Neuendorffer<br>
<<a href="mailto:stephen.neuendorffer@xilinx.com">stephen.neuendorffer@xilinx.com</a>> wrote:<br>
><br>
> Can we have XILINX_DRIVERS, please? That way this can also be enabled<br>
> on any architecture that has FPGA peripherals.<br>
<br>
I've thought about this more, and I'd really rather not. The list of<br>
affected drivers is short and is not a large maintenance burden. I<br>
don't think a list of 2 or 3 architecture selects for each driver is<br>
unreasonable. A "XILINX_DRIVERS" config item doesn't really help much<br>
anyway. At any given time one of these drivers may be needed on<br>
another platform. ie. the SystemACE device is present on at least one<br>
non-virtex, non-spartan platform. </blockquote><div><br>Which is exactly why having it architecture dependent isn't right... All of these drivers<br>could be needed and used on any OF-based platform. If you have a platform (for instance, a processor connected to an FPGA which has these peripherals in the FPGA) then you should be able to enable these drivers. Just my 2 cents...<br>
<br>Steve<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
g.<br>
<br>
><br>
> Steve<br>
><br>
>> -----Original Message-----<br>
>> From: <a href="mailto:owner-microblaze-uclinux@itee.uq.edu.au">owner-microblaze-uclinux@itee.uq.edu.au</a><br>
> [mailto:<a href="mailto:owner-microblaze-uclinux@itee.uq.edu.au">owner-microblaze-uclinux@itee.uq.edu.au</a>] On<br>
>> Behalf Of monstr@monstr.eu<br>
>> Sent: Thursday, April 16, 2009 2:57 AM<br>
>> To: <a href="mailto:linux-kernel@vger.kernel.org">linux-kernel@vger.kernel.org</a><br>
>> Cc: <a href="mailto:microblaze-uclinux@itee.uq.edu.au">microblaze-uclinux@itee.uq.edu.au</a>; Michal Simek<br>
>> Subject: [microblaze-uclinux] [PATCH 11/11] microblaze: Kconfig:<br>
> Enable drivers for Microblaze<br>
>><br>
>> From: Michal Simek <monstr@monstr.eu><br>
>><br>
>> Signed-off-by: Michal Simek <monstr@monstr.eu><br>
>> ---<br>
>> drivers/block/Kconfig | 2 +-<br>
>> drivers/char/Kconfig | 2 +-<br>
>> drivers/gpio/Kconfig | 2 +-<br>
>> drivers/usb/Kconfig | 1 +<br>
>> 4 files changed, 4 insertions(+), 3 deletions(-)<br>
>><br>
>> diff --git a/drivers/block/Kconfig b/drivers/block/Kconfig<br>
>> index ddea8e4..9f1665f 100644<br>
>> --- a/drivers/block/Kconfig<br>
>> +++ b/drivers/block/Kconfig<br>
>> @@ -438,7 +438,7 @@ source "drivers/s390/block/Kconfig"<br>
>><br>
>> config XILINX_SYSACE<br>
>> tristate "Xilinx SystemACE support"<br>
>> - depends on 4xx<br>
>> + depends on 4xx || MICROBLAZE<br>
>> help<br>
>> Include support for the Xilinx SystemACE CompactFlash<br>
> interface<br>
>><br>
>> diff --git a/drivers/char/Kconfig b/drivers/char/Kconfig<br>
>> index 735bbe2..bb1a071 100644<br>
>> --- a/drivers/char/Kconfig<br>
>> +++ b/drivers/char/Kconfig<br>
>> @@ -893,7 +893,7 @@ config DTLK<br>
>><br>
>> config XILINX_HWICAP<br>
>> tristate "Xilinx HWICAP Support"<br>
>> - depends on XILINX_VIRTEX<br>
>> + depends on XILINX_VIRTEX || MICROBLAZE<br>
>> help<br>
>> This option enables support for Xilinx Internal Configuration<br>
>> Access Port (ICAP) driver. The ICAP is used on Xilinx Virtex<br>
>> diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig<br>
>> index edb0253..11f3739 100644<br>
>> --- a/drivers/gpio/Kconfig<br>
>> +++ b/drivers/gpio/Kconfig<br>
>> @@ -69,7 +69,7 @@ comment "Memory mapped GPIO expanders:"<br>
>><br>
>> config GPIO_XILINX<br>
>> bool "Xilinx GPIO support"<br>
>> - depends on PPC_OF<br>
>> + depends on PPC_OF || MICROBLAZE<br>
>> help<br>
>> Say yes here to support the Xilinx FPGA GPIO device<br>
>><br>
>> diff --git a/drivers/usb/Kconfig b/drivers/usb/Kconfig<br>
>> index c6c816b..5eee3f8 100644<br>
>> --- a/drivers/usb/Kconfig<br>
>> +++ b/drivers/usb/Kconfig<br>
>> @@ -22,6 +22,7 @@ config USB_ARCH_HAS_HCD<br>
>> default y if PCMCIA && !M32R # sl811_cs<br>
>> default y if ARM # SL-811<br>
>> default y if SUPERH # r8a66597-hcd<br>
>> + default y if MICROBLAZE<br>
>> default PCI<br>
>><br>
>> # many non-PCI SOC chips embed OHCI<br>
>> --<br>
>> 1.5.5.1<br>
>><br>
>> ___________________________<br>
>> microblaze-uclinux mailing list<br>
>> <a href="mailto:microblaze-uclinux@itee.uq.edu.au">microblaze-uclinux@itee.uq.edu.au</a><br>
>> Project Home Page :<br>
> <a href="http://www.itee.uq.edu.au/%7Ejwilliams/mblaze-uclinux" target="_blank">http://www.itee.uq.edu.au/~jwilliams/mblaze-uclinux</a><br>
>> Mailing List Archive :<br>
> <a href="http://www.itee.uq.edu.au/%7Elistarch/microblaze-uclinux/" target="_blank">http://www.itee.uq.edu.au/~listarch/microblaze-uclinux/</a><br>
>><br>
><br>
><br>
> This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.<br>
><br>
><br>
> --<br>
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in<br>
> the body of a message to <a href="mailto:majordomo@vger.kernel.org">majordomo@vger.kernel.org</a><br>
> More majordomo info at <a href="http://vger.kernel.org/majordomo-info.html" target="_blank">http://vger.kernel.org/majordomo-info.html</a><br>
> Please read the FAQ at <a href="http://www.tux.org/lkml/" target="_blank">http://www.tux.org/lkml/</a><br>
><br>
<font color="#888888"><br>
<br>
<br>
--<br>
Grant Likely, B.Sc., P.Eng.<br>
Secret Lab Technologies Ltd.<br>
_______________________________________________<br>
Linuxppc-dev mailing list<br>
<a href="mailto:Linuxppc-dev@ozlabs.org">Linuxppc-dev@ozlabs.org</a><br>
<a href="https://ozlabs.org/mailman/listinfo/linuxppc-dev" target="_blank">https://ozlabs.org/mailman/listinfo/linuxppc-dev</a><br>
</font></blockquote></div><br>