public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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 13:59:48 +0000	[thread overview]
Message-ID: <5280E2D4.9080101@imgtec.com> (raw)
In-Reply-To: <201311111433.03271.marex@denx.de>

On 11/11/13 13:33, Marek Vasut wrote:
> 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?

I'm sure the information could be passed via DT but the Malta board 
currently doesn't use DT at all either in Linux or U-boot, and I don't 
have the time right now to convert them both to DT.

>
>> 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
>

Thanks,
     Paul

  reply	other threads:[~2013-11-11 13:59 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
2013-11-11 13:59         ` Paul Burton [this message]
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=5280E2D4.9080101@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