All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomasz Nowicki <tn@semihalf.com>
To: Sinan Kaya <okaya@codeaurora.org>,
	bhelgaas@google.com, arnd@arndb.de, will.deacon@arm.com,
	catalin.marinas@arm.com, rjw@rjwysocki.net,
	hanjun.guo@linaro.org, Lorenzo.Pieralisi@arm.com
Cc: jiang.liu@linux.intel.com, robert.richter@caviumnetworks.com,
	Narinder.Dhillon@caviumnetworks.com, ddaney@caviumnetworks.com,
	Liviu.Dudau@arm.com, tglx@linutronix.de, wangyijing@huawei.com,
	Suravee.Suthikulpanit@amd.com, msalter@redhat.com,
	linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-acpi@vger.kernel.org, linaro-acpi@lists.linaro.org
Subject: Re: [PATCH V1 11/11] arm64, pci, acpi: Support for ACPI based PCI hostbridge init
Date: Thu, 29 Oct 2015 12:38:32 +0100	[thread overview]
Message-ID: <56320538.6000905@semihalf.com> (raw)
In-Reply-To: <563131B0.9060305@codeaurora.org>

On 28.10.2015 21:36, Sinan Kaya wrote:
> 1. ACPI code is unable to discover the interrupt numbers when objects
> are ordered as follows in the ACPI file
>
> PNP0A08 object
> PNP0C0F INTA object
> PNP0C0F INTB object
> PNP0C0F INTC object
> PNP0C0F INTD object
>
> This gives me invalid link context error.
>
> pci 0000:00:00.0: PCI INT A: no GSI
> pci 0000:01:00.0: Derived GSI for 0000:01:00.0 INT A from 0000:00:00.0
> acpi PNP0C0F:00: Invalid link context
>
> If I order it like this in the ACPI file,
>
> PNP0C0F INTA object
> PNP0C0F INTB object
> PNP0C0F INTC object
> PNP0C0F INTD object
> PNP0A08 object
>
> then, the legacy interrupt numbers can be discovered properly.

Can you show full content of your PNP0C0F and PNP0A08 objects?

Regards,
Tomasz

WARNING: multiple messages have this Message-ID (diff)
From: tn@semihalf.com (Tomasz Nowicki)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V1 11/11] arm64, pci, acpi: Support for ACPI based PCI hostbridge init
Date: Thu, 29 Oct 2015 12:38:32 +0100	[thread overview]
Message-ID: <56320538.6000905@semihalf.com> (raw)
In-Reply-To: <563131B0.9060305@codeaurora.org>

On 28.10.2015 21:36, Sinan Kaya wrote:
> 1. ACPI code is unable to discover the interrupt numbers when objects
> are ordered as follows in the ACPI file
>
> PNP0A08 object
> PNP0C0F INTA object
> PNP0C0F INTB object
> PNP0C0F INTC object
> PNP0C0F INTD object
>
> This gives me invalid link context error.
>
> pci 0000:00:00.0: PCI INT A: no GSI
> pci 0000:01:00.0: Derived GSI for 0000:01:00.0 INT A from 0000:00:00.0
> acpi PNP0C0F:00: Invalid link context
>
> If I order it like this in the ACPI file,
>
> PNP0C0F INTA object
> PNP0C0F INTB object
> PNP0C0F INTC object
> PNP0C0F INTD object
> PNP0A08 object
>
> then, the legacy interrupt numbers can be discovered properly.

Can you show full content of your PNP0C0F and PNP0A08 objects?

Regards,
Tomasz

  reply	other threads:[~2015-10-29 11:38 UTC|newest]

Thread overview: 131+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-27 16:38 [PATCH V1 00/11] MMCONFIG refactoring and ARM64 PCI hostbridge init based on ACPI Tomasz Nowicki
2015-10-27 16:38 ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 01/11] x86, pci: Reorder logic of pci_mmconfig_insert() function Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 02/11] x86, pci, acpi: Move arch-agnostic MMCONFIG (aka ECAM) and ACPI code out of arch/x86/ directory Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 03/11] pci, acpi, mcfg: Provide generic implementation of MCFG code initialization Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 04/11] x86, pci: mmconfig_{32,64}.c code refactoring - remove code duplication Tomasz Nowicki
2015-10-27 16:38   ` [PATCH V1 04/11] x86, pci: mmconfig_{32, 64}.c " Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 05/11] x86, pci, ecam: mmconfig_64.c becomes default implementation for ECAM driver Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 06/11] pci, acpi, mcfg: Provide default RAW ACPI PCI config space accessors Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 07/11] XEN / PCI: Remove the dependence on arch x86 when PCI_MMCONFIG=y Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:47   ` [Linaro-acpi] " Tomasz Nowicki
2015-10-27 16:47     ` Tomasz Nowicki
2015-10-27 17:25     ` Boris Ostrovsky
2015-10-27 17:25       ` Boris Ostrovsky
2015-10-28 10:49       ` Stefano Stabellini
2015-10-28 10:49         ` Stefano Stabellini
2015-10-28 10:49         ` Stefano Stabellini
2015-10-28 10:56         ` Tomasz Nowicki
2015-10-28 10:56           ` Tomasz Nowicki
2015-10-28 13:45           ` Hanjun Guo
2015-10-28 13:45             ` Hanjun Guo
2015-10-28 14:07             ` Boris Ostrovsky
2015-10-28 14:07               ` Boris Ostrovsky
2015-10-27 16:38 ` [PATCH V1 08/11] pci, acpi, ecam: Add flag to indicate whether ECAM region was hot added or not Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 09/11] x86, pci: Use previously added ECAM hot_added flag to remove ECAM regions Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 10/11] pci, acpi: Provide generic way to assign bus domain number Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-28 11:38   ` Liviu.Dudau
2015-10-28 11:38     ` Liviu.Dudau at arm.com
2015-10-28 12:47     ` [Linaro-acpi] " Tomasz Nowicki
2015-10-28 12:47       ` Tomasz Nowicki
2015-11-03 16:10   ` Lorenzo Pieralisi
2015-11-03 16:10     ` Lorenzo Pieralisi
2015-11-04 10:04     ` [Linaro-acpi] " Tomasz Nowicki
2015-11-04 10:04       ` Tomasz Nowicki
2015-10-27 16:38 ` [PATCH V1 11/11] arm64, pci, acpi: Support for ACPI based PCI hostbridge init Tomasz Nowicki
2015-10-27 16:38   ` Tomasz Nowicki
2015-10-28 11:49   ` Liviu.Dudau
2015-10-28 11:49     ` Liviu.Dudau at arm.com
2015-10-28 11:49     ` Liviu.Dudau
2015-10-28 13:42     ` Tomasz Nowicki
2015-10-28 13:42       ` Tomasz Nowicki
2015-10-28 13:51       ` Liviu.Dudau
2015-10-28 13:51         ` Liviu.Dudau at arm.com
2015-11-03 14:32     ` Lorenzo Pieralisi
2015-11-03 14:32       ` Lorenzo Pieralisi
2015-11-03 16:28       ` Liviu.Dudau
2015-11-03 16:28         ` Liviu.Dudau at arm.com
2015-10-28 18:46   ` Sinan Kaya
2015-10-28 18:46     ` Sinan Kaya
2015-10-28 20:36     ` Sinan Kaya
2015-10-28 20:36       ` Sinan Kaya
2015-10-29 11:38       ` Tomasz Nowicki [this message]
2015-10-29 11:38         ` Tomasz Nowicki
2015-10-29 15:01         ` Sinan Kaya
2015-10-29 15:01           ` Sinan Kaya
2015-10-29 15:53           ` Tomasz Nowicki
2015-10-29 15:53             ` Tomasz Nowicki
2015-10-29 16:20             ` Sinan Kaya
2015-10-29 16:20               ` Sinan Kaya
2015-10-29 14:57       ` Sinan Kaya
2015-10-29 14:57         ` Sinan Kaya
2015-10-29 16:27         ` Tomasz Nowicki
2015-10-29 16:27           ` Tomasz Nowicki
2015-11-03 14:15     ` Lorenzo Pieralisi
2015-11-03 14:15       ` Lorenzo Pieralisi
2015-11-03 14:39       ` Tomasz Nowicki
2015-11-03 14:39         ` Tomasz Nowicki
2015-11-03 15:10         ` Sinan Kaya
2015-11-03 15:10           ` Sinan Kaya
2015-11-03 15:59           ` Arnd Bergmann
2015-11-03 15:59             ` Arnd Bergmann
2015-11-03 16:33             ` Sinan Kaya
2015-11-03 16:33               ` Sinan Kaya
2015-11-03 16:55               ` Arnd Bergmann
2015-11-03 16:55                 ` Arnd Bergmann
2015-11-03 17:43                 ` Sinan Kaya
2015-11-03 17:43                   ` Sinan Kaya
2015-11-05 14:48                   ` [Linaro-acpi] " Sinan Kaya
2015-11-05 14:48                     ` Sinan Kaya
2015-11-03 15:19       ` Hanjun Guo
2015-11-03 15:19         ` Hanjun Guo
2015-11-03 15:19         ` Hanjun Guo
2015-11-03 17:39         ` David Daney
2015-11-03 17:39           ` David Daney
2015-11-03 17:39           ` David Daney
2015-11-03 18:00           ` Gabriele Paoloni
2015-11-03 18:00             ` Gabriele Paoloni
2015-11-03 18:00             ` Gabriele Paoloni
2015-11-03 16:55   ` Lorenzo Pieralisi
2015-11-03 16:55     ` Lorenzo Pieralisi
2015-11-04  9:59     ` Tomasz Nowicki
2015-11-04  9:59       ` Tomasz Nowicki
2015-11-04 10:11     ` Tomasz Nowicki
2015-11-04 10:11       ` Tomasz Nowicki
2015-10-30  4:07 ` [PATCH V1 00/11] MMCONFIG refactoring and ARM64 PCI hostbridge init based on ACPI Jon Masters
2015-10-30  4:07   ` Jon Masters
2015-10-30  4:50   ` Hanjun Guo
2015-10-30  4:50     ` Hanjun Guo
2015-10-30  4:50     ` Hanjun Guo
2015-10-30  8:26   ` Tomasz Nowicki
2015-10-30  8:26     ` Tomasz Nowicki
2015-10-30 16:38 ` Suravee Suthikulpanit
2015-10-30 16:38   ` Suravee Suthikulpanit
2015-10-30 16:38   ` Suravee Suthikulpanit
2015-12-07 20:31 ` Bjorn Helgaas
2015-12-07 20:31   ` Bjorn Helgaas
2015-12-09 10:01   ` Jayachandran C.
2015-12-09 10:01     ` Jayachandran C.
2015-12-09 15:55     ` Lorenzo Pieralisi
2015-12-09 15:55       ` Lorenzo Pieralisi
2015-12-16 12:51       ` Jayachandran C.
2015-12-16 12:51         ` Jayachandran C.
2015-12-16 14:05         ` Lorenzo Pieralisi
2015-12-16 14:05           ` Lorenzo Pieralisi
2015-12-08 17:43 ` Jeremy Linton
2015-12-08 17:43   ` Jeremy Linton

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=56320538.6000905@semihalf.com \
    --to=tn@semihalf.com \
    --cc=Liviu.Dudau@arm.com \
    --cc=Lorenzo.Pieralisi@arm.com \
    --cc=Narinder.Dhillon@caviumnetworks.com \
    --cc=Suravee.Suthikulpanit@amd.com \
    --cc=arnd@arndb.de \
    --cc=bhelgaas@google.com \
    --cc=catalin.marinas@arm.com \
    --cc=ddaney@caviumnetworks.com \
    --cc=hanjun.guo@linaro.org \
    --cc=jiang.liu@linux.intel.com \
    --cc=linaro-acpi@lists.linaro.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=msalter@redhat.com \
    --cc=okaya@codeaurora.org \
    --cc=rjw@rjwysocki.net \
    --cc=robert.richter@caviumnetworks.com \
    --cc=tglx@linutronix.de \
    --cc=wangyijing@huawei.com \
    --cc=will.deacon@arm.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.