From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 16/18] malta: setup PIIX4 interrupt route
Date: Mon, 11 Nov 2013 14:33:03 +0100 [thread overview]
Message-ID: <201311111433.03271.marex@denx.de> (raw)
In-Reply-To: <5280B815.4090603@imgtec.com>
Dear Paul Burton,
> 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.
Can this setup not be passed via a well-designed DT binding then?
> 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!
Best regards,
Marek Vasut
next prev parent reply other threads:[~2013-11-11 13:33 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
2013-11-11 13:33 ` Marek Vasut [this message]
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=201311111433.03271.marex@denx.de \
--to=marex@denx.de \
--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