public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Nikita Yushchenko <nyushchenko@dev.rtsoft.ru>
To: Xianghua Xiao <x.xiao@freescale.com>,
	Jason Jin <Jason.jin@freescale.com>,
	Jon Loelier <jdl@freescale.com>,
	Kumar Gala <galak@kernel.crashing.org>
Cc: "Vadim V. Vlasov" <vvlasov@dev.rtsoft.ru>,
	"'Alexey Lugovskoy'" <lugovskoy@dev.rtsoft.ru>,
	linux-kernel@vger.kernel.org
Subject: ULI M1575 question
Date: Fri, 28 Mar 2014 19:06:08 +0400	[thread overview]
Message-ID: <53358FE0.2060105@dev.rtsoft.ru> (raw)

Hello people

I'm writing you since your addresses are in Signed-Of-By lines of commit
0e65bfe34c1 ("[POWERPC] Add initial MPC8610 HPCD Platform files") in
linux kernel git tree.

This commit has quirk_uli1575() routine, that has followinf lines:

    /* Disable INTx */
    pci_read_config_dword(dev, 0x48, &temp32);
    pci_write_config_dword(dev, 0x48, (temp32 | 1<<26));

This code is still in kernel, just moved to
  arch/powerpc/platforms/fsl_uli1575.c
at some point.

Could you please explain what "Disable INTx" comment means, and why
setting bit 2 of PCI configuration byte with offset 0x4b does that?

Per ULI1575 datasheet that we've been able to find in the net, this bit
is part of 4-bit field "PIRQG# routing table", thus setting it alone
looks very strange.


The reason for asking is the following.

We are trying to overcome a very unpleasant problem with ULI M1575 on
Freescale P2020DS and MPC8572DS boards. On both boards, when running
CONFIG_PREEMPT_RT_FULL kernel and trying to use PCI ethernet adapter in
PCI slot, we are getting irq storm on MPIC IRQ that is used by i8259,
while i8259 reports no irq.

Looks like same MPIC input is triggered by something else. Which, per
CPU reference guide, could be caused by "Assert INTx" message received
from ULI1575 over PCIe bus. However, ULI1575 document we have does not
say anything about usage of "Assert INTx" on upstream bus. Instead, it
documents how to route IRQs via built-in i8259-compatible interrupt
controller.

Mention of "Disable INTx" in your code raises some ideas that perhaps
ULI *does* use INTx, and that could be somehow disabled...


Thanks in advance.
Nikita Yushchenko
RTsoft Sortware Development Center.


                 reply	other threads:[~2014-03-28 15:06 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=53358FE0.2060105@dev.rtsoft.ru \
    --to=nyushchenko@dev.rtsoft.ru \
    --cc=Jason.jin@freescale.com \
    --cc=galak@kernel.crashing.org \
    --cc=jdl@freescale.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lugovskoy@dev.rtsoft.ru \
    --cc=vvlasov@dev.rtsoft.ru \
    --cc=x.xiao@freescale.com \
    /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