public inbox for linux-pci@vger.kernel.org
 help / color / mirror / Atom feed
From: Niklas Cassel <cassel@kernel.org>
To: Manivannan Sadhasivam <manivannan.sadhasivam@oss.qualcomm.com>
Cc: "Jingoo Han" <jingoohan1@gmail.com>,
	"Lorenzo Pieralisi" <lpieralisi@kernel.org>,
	"Krzysztof Wilczyński" <kwilczynski@kernel.org>,
	"Rob Herring" <robh@kernel.org>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Frank Li" <Frank.Li@nxp.com>,
	"Krishna Chaitanya Chundru" <krishna.chundru@oss.qualcomm.com>,
	"Randolph Lin" <randolph@andestech.com>,
	"Samuel Holland" <samuel.holland@sifive.com>,
	"Charles Mirabile" <cmirabil@redhat.com>,
	tim609@andestech.com, dlemoal@kernel.org,
	"Maciej W. Rozycki" <macro@orcam.me.uk>,
	linux-pci@vger.kernel.org
Subject: Re: [PATCH v5 0/3] PCI: dwc: Clean up iATU index mess
Date: Thu, 5 Feb 2026 14:23:16 +0100	[thread overview]
Message-ID: <aYSZxCtACmQyC0XT@ryzen> (raw)
In-Reply-To: <177029621115.8023.3797281385109418780.b4-ty@kernel.org>

On Thu, Feb 05, 2026 at 06:26:51PM +0530, Manivannan Sadhasivam wrote:
> 
> On Tue, 27 Jan 2026 16:10:38 +0100, Niklas Cassel wrote:
> > This series tries to clean up the iATU index mess, by treating
> > iATU indexing for outbound and inbound address translation in
> > a consistent way (increment the index after assignment, and only
> > if the function call succeeded).
> > 
> > 
> > Changes since v4:
> > -Squashed patch 1 and 2, as suggested by Damien.
> > -Added Cc: stable+noautosel for the ECAM patch, such that stable
> >  maintainers will see the dependency needed by this patch.
> > 
> > [...]
> 
> Applied, thanks!
> 
> [1/3] PCI: dwc: Fix msg_atu_index assignment
>       commit: 58fbf08935d9c4396417e5887df89a4e681fa7e3
> [2/3] PCI: dwc: Clean up iATU index usage in dw_pcie_iatu_setup()
>       commit: b5dab9b38da0a05949458276dde9227c38aa1b39
> [3/3] PCI: dwc: Fix missing iATU setup when ECAM is enabled
>       commit: 43d324eeb08c3dd9fff7eb9a2c617afd3b96e65c
> 
> Best regards,
> -- 
> Manivannan Sadhasivam <mani@kernel.org>
> 

Thank you Mani!

If possible, I think it would be nice if we could get:
https://lore.kernel.org/linux-pci/20260202145407.503348-1-den@valinux.co.jp/T/#t

Since it is fixing some minor things with the new feature inbound subrange
mappings.

You can probably even squash it if that is preferred.
Although, if you squash merge it,

this commit


commit 72cb5ed2a5c6d87f71a409347f7d3b228fee6bee
Author: Aksh Garg <a-garg7@ti.com>
Date:   Fri Jan 30 17:25:15 2026 +0530

    PCI: dwc: ep: Add per-PF BAR and inbound ATU mapping support

    (...)
    
    Fixes: 24ede430fa49 ("PCI: designware-ep: Add multiple PFs support for DWC")
    Fixes: cc839bef7727 ("PCI: dwc: ep: Support BAR subrange inbound mapping via Address Match Mode iATU")


has a Fixes that that references the commit that adds subrange inbound
mapping. (So if you squash merge it, you probably want to update the Fixes tag too.)


Kind regards,
Niklas

      reply	other threads:[~2026-02-05 13:23 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-27 15:10 [PATCH v5 0/3] PCI: dwc: Clean up iATU index mess Niklas Cassel
2026-01-27 15:10 ` [PATCH v5 1/3] PCI: dwc: Fix msg_atu_index assignment Niklas Cassel
2026-01-28  6:07   ` Hans Zhang
2026-01-28  6:36   ` Damien Le Moal
2026-02-04 16:04   ` Maciej W. Rozycki
2026-01-27 15:10 ` [PATCH v5 2/3] PCI: dwc: Clean up iATU index usage in dw_pcie_iatu_setup() Niklas Cassel
2026-01-27 18:14   ` Frank Li
2026-01-28  6:10   ` Hans Zhang
2026-01-28  6:37   ` Damien Le Moal
2026-02-04 16:05   ` Maciej W. Rozycki
2026-01-27 15:10 ` [PATCH v5 3/3] PCI: dwc: Fix missing iATU setup when ECAM is enabled Niklas Cassel
2026-01-27 18:16   ` Frank Li
2026-01-28  6:11   ` Hans Zhang
2026-01-28  6:40   ` Damien Le Moal
2026-02-04 16:05   ` Maciej W. Rozycki
2026-01-28  8:02 ` [PATCH v5 0/3] PCI: dwc: Clean up iATU index mess Manivannan Sadhasivam
2026-02-02 12:47   ` Niklas Cassel
2026-02-04 16:13     ` Maciej W. Rozycki
2026-02-05  8:04       ` Niklas Cassel
2026-02-05 12:56 ` Manivannan Sadhasivam
2026-02-05 13:23   ` Niklas Cassel [this message]

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=aYSZxCtACmQyC0XT@ryzen \
    --to=cassel@kernel.org \
    --cc=Frank.Li@nxp.com \
    --cc=bhelgaas@google.com \
    --cc=cmirabil@redhat.com \
    --cc=dlemoal@kernel.org \
    --cc=jingoohan1@gmail.com \
    --cc=krishna.chundru@oss.qualcomm.com \
    --cc=kwilczynski@kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=lpieralisi@kernel.org \
    --cc=macro@orcam.me.uk \
    --cc=manivannan.sadhasivam@oss.qualcomm.com \
    --cc=randolph@andestech.com \
    --cc=robh@kernel.org \
    --cc=samuel.holland@sifive.com \
    --cc=tim609@andestech.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