From: Paul Burton <paul.burton@imgtec.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 16/18] malta: setup PIIX4 interrupt route
Date: Mon, 11 Nov 2013 10:57:25 +0000 [thread overview]
Message-ID: <5280B815.4090603@imgtec.com> (raw)
In-Reply-To: <201311102109.30137.marex@denx.de>
On 10/11/13 20:09, Marek Vasut wrote:
> Dear Paul Burton,
>
>> Without setting up the PIRQ[A:D] interrupt routes, PCI interrupts will
>> be left disabled. Linux does not set up this routing but relies upon it
>> having been set up by the bootloader, reading back the IRQ lines which
>> the PIRQ[A:D] signals have been routed to.
>
> Did you also submit a fix to Linux guys?
>
> Best regards,
> Marek Vasut
>
No, I haven't. Although Linux is reliant upon these values having been
programmed it's quite clear from reading its code that that depence is
intentional. It explicitely reads the route setup by the bootloader
rather than clobbering it, and to me that does make sense in this case.
However the pcnet32 ethernet driver seems to be reliant upon the
interrupt and doesn't handle the case of a PCI device with no assigned
interrupt, so I'll send a patch to fail its probe in that case shortly.
Thanks,
Paul
next prev parent reply other threads:[~2013-11-11 10:57 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-08 11:18 [U-Boot] [PATCH v1+ 00/18] MIPS Malta board support Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v2 01/18] mips32: detect L1 cache sizes if they're not defined Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v2 02/18] pcnet: code style cleanup Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v2 03/18] pcnet: s/le16_to_cpu/cpu_to_le16/ in pcnet_send Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v2 04/18] pcnet: add cache flushing & invalidation Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v2 05/18] pcnet: enable the NOUFLO feature Paul Burton
2013-11-10 20:02 ` Marek Vasut
2013-11-11 10:22 ` Paul Burton
2013-11-11 13:30 ` Marek Vasut
2013-11-08 11:18 ` [U-Boot] [PATCH v2 06/18] pci.h: allow inclusion in assembly source Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v4 07/18] qemu-malta: rename to just "malta" Paul Burton
2013-11-09 0:53 ` Daniel Schwierzeck
2013-11-09 10:22 ` [U-Boot] [PATCH v5 " Paul Burton
2013-11-10 20:05 ` Marek Vasut
2013-11-11 10:24 ` Paul Burton
2013-11-11 13:30 ` Marek Vasut
2013-11-08 11:18 ` [U-Boot] [PATCH v3 08/18] malta: setup super I/O UARTs Paul Burton
2013-11-10 20:06 ` Marek Vasut
2013-11-11 10:29 ` Paul Burton
2013-11-11 13:31 ` Marek Vasut
2013-11-08 11:18 ` [U-Boot] [PATCH v3 09/18] malta: support for coreFPGA6 boards Paul Burton
2013-11-10 20:07 ` Marek Vasut
2013-11-11 10:32 ` Paul Burton
2013-11-11 10:52 ` Daniel Schwierzeck
2013-11-11 11:03 ` [U-Boot] [PATCH] malta: arch/mips/include/asm/malta.h SPDX license tag Paul Burton
2013-11-11 11:34 ` Daniel Schwierzeck
2013-11-08 11:18 ` [U-Boot] [PATCH v3 10/18] malta: display "U-boot" on the LCD screen Paul Burton
2013-11-10 20:08 ` Marek Vasut
2013-11-11 10:36 ` Paul Burton
2013-11-11 13:32 ` Marek Vasut
2013-11-08 11:18 ` [U-Boot] [PATCH v2 11/18] malta: enable CONFIG_PCNET_79C973, PCNET_HAS_PROM, CONFIG_CMD_DHCP Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v2 12/18] malta: remove cache size definitions Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v3 13/18] malta: disable L2 caches Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v2 14/18] malta: enable RTC support Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH 15/18] malta: store environment in flash Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH 16/18] malta: setup PIIX4 interrupt route Paul Burton
2013-11-10 20:09 ` Marek Vasut
2013-11-11 10:57 ` Paul Burton [this message]
2013-11-11 13:33 ` Marek Vasut
2013-11-11 13:59 ` Paul Burton
2013-11-11 14:01 ` Marek Vasut
2013-11-08 11:18 ` [U-Boot] [PATCH v3 17/18] malta: add script & instructions to flash U-boot Paul Burton
2013-11-08 11:18 ` [U-Boot] [PATCH v3 18/18] malta: add myself to maintainers Paul Burton
2013-11-09 20:44 ` [U-Boot] [PATCH v1+ 00/18] MIPS Malta board support Daniel Schwierzeck
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=5280B815.4090603@imgtec.com \
--to=paul.burton@imgtec.com \
--cc=u-boot@lists.denx.de \
/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