[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Z3dAvgO3XEUaJfq_@google.com>
Date: Thu, 2 Jan 2025 17:43:26 -0800
From: Brian Norris <briannorris@...omium.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>
Cc: Jingoo Han <jingoohan1@...il.com>,
Lorenzo Pieralisi <lpieralisi@...nel.org>,
Bjorn Helgaas <bhelgaas@...gle.com>, linux-kernel@...r.kernel.org,
linux-pci@...r.kernel.org, Rob Herring <robh@...nel.org>,
Marc Zyngier <marc.zyngier@....com>,
Krzysztof WilczyĆski <kw@...ux.com>
Subject: Re: [PATCH] PCI: dwc: Use level-triggered handler for MSI IRQs
On Mon, Dec 30, 2024 at 10:41:45PM +0530, Manivannan Sadhasivam wrote:
> On Tue, Oct 15, 2024 at 02:12:16PM -0700, Brian Norris wrote:
> > From: Brian Norris <briannorris@...gle.com>
> >
> > Per Synopsis's documentation, the msi_ctrl_int signal is
> > level-triggered, not edge-triggered.
> >
>
> Could you please quote the spec reference?
>From the reference manual for msi_ctrl_int:
"Asserted when an MSI interrupt is pending. De-asserted when there is
no MSI interrupt pending.
...
Active State: High (level)"
The reference manual also points at the databook for more info. One
relevant excerpt from the databook:
"When any status bit remains set, then msi_ctrl_int remains asserted.
The interrupt status register provides a status bit for up to 32
interrupt vectors per Endpoint. When the decoded interrupt vector is
enabled but is masked, then the controller sets the corresponding bit
in interrupt status register but the it does not assert the top-level
controller output msi_ctrl_int.
That's essentially a prose description of level-triggering, plus
32-vector multiplexing and masking.
Did you want a v2 with this included, or did you just want it noted
here?
(Side note: I think it doesn't really matter that much whether we use
the 'level' or 'edge' variant handlers here, at least if the parent
interrupt is configured correctly as level-triggered. We're not actually
in danger of a level-triggered interrupt flood or similar issue.)
Brian
Powered by blists - more mailing lists