All of lore.kernel.org
 help / color / mirror / Atom feed
From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: Niklas Cassel <cassel@kernel.org>
Cc: "Damien Le Moal" <dlemoal@kernel.org>,
	"Lorenzo Pieralisi" <lpieralisi@kernel.org>,
	"Kishon Vijay Abraham I" <kishon@kernel.org>,
	"Shawn Lin" <shawn.lin@rock-chips.com>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	linux-pci@vger.kernel.org, "Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Heiko Stuebner" <heiko@sntech.de>,
	devicetree@vger.kernel.org, linux-rockchip@lists.infradead.org,
	"Rick Wertenbroek" <rick.wertenbroek@gmail.com>
Subject: Re: [PATCH v5 00/14] Fix and improve the Rockchip endpoint driver
Date: Mon, 16 Dec 2024 11:35:24 +0530	[thread overview]
Message-ID: <20241216060524.bc5fiwdyaxz4git7@thinkpad> (raw)
In-Reply-To: <45CC5230-1DFD-4F7E-A0E6-F4FAC5586038@kernel.org>

On Mon, Dec 16, 2024 at 07:00:16AM +0100, Niklas Cassel wrote:
> 
> 
> On 16 December 2024 06:49:53 CET, Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> wrote:
> >On Thu, Oct 17, 2024 at 10:58:35AM +0900, Damien Le Moal wrote:
> >> This patch series fix the PCI address mapping handling of the Rockchip
> >> PCI endpoint driver, refactor some of its code, improves link training
> >> and adds handling of the PERST# signal.
> >> 
> >> This series is organized as follows:
> >>  - Patch 1 fixes the rockchip ATU programming
> >>  - Patch 2, 3 and 4 introduce small code improvments
> >>  - Patch 5 implements the .align_addr() operation to make the RK3399
> >>    endpoint controller driver fully functional with the new
> >>    pci_epc_mem_map() function
> >>  - Patch 6 uses the new align_addr operation function to fix the ATU
> >>    programming for MSI IRQ data mapping
> >>  - Patch 7, 8, 9 and 10 refactor the driver code to make it more
> >>    readable
> >>  - Patch 11 introduces the .stop() endpoint controller operation to
> >>    correctly disable the endpopint controller after use
> >>  - Patch 12 improves link training
> >>  - Patch 13 implements handling of the #PERST signal
> >>  - Patch 14 adds a DT overlay file to enable EP mode and define the
> >>    PERST# GPIO (reset-gpios) property.
> >> 
> >
> >Damien, please wait for my review before spinning the next revision. Sorry for
> >the delay.
> 
> Mani,
> 
> This series has already been merged, and is in Torvalds tree.
> 

Doh! I was referring to the NVMe EPF series. Let me reply there (and also get
coffee).

- Mani

> Except for patch 14/14 which is a device tree overlay, so it should go via the rockchip tree.
> 
> 
> Kind regards,
> Niklas

-- 
மணிவண்ணன் சதாசிவம்

WARNING: multiple messages have this Message-ID (diff)
From: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
To: Niklas Cassel <cassel@kernel.org>
Cc: "Damien Le Moal" <dlemoal@kernel.org>,
	"Lorenzo Pieralisi" <lpieralisi@kernel.org>,
	"Kishon Vijay Abraham I" <kishon@kernel.org>,
	"Shawn Lin" <shawn.lin@rock-chips.com>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	linux-pci@vger.kernel.org, "Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Heiko Stuebner" <heiko@sntech.de>,
	devicetree@vger.kernel.org, linux-rockchip@lists.infradead.org,
	"Rick Wertenbroek" <rick.wertenbroek@gmail.com>
Subject: Re: [PATCH v5 00/14] Fix and improve the Rockchip endpoint driver
Date: Mon, 16 Dec 2024 11:35:24 +0530	[thread overview]
Message-ID: <20241216060524.bc5fiwdyaxz4git7@thinkpad> (raw)
In-Reply-To: <45CC5230-1DFD-4F7E-A0E6-F4FAC5586038@kernel.org>

On Mon, Dec 16, 2024 at 07:00:16AM +0100, Niklas Cassel wrote:
> 
> 
> On 16 December 2024 06:49:53 CET, Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> wrote:
> >On Thu, Oct 17, 2024 at 10:58:35AM +0900, Damien Le Moal wrote:
> >> This patch series fix the PCI address mapping handling of the Rockchip
> >> PCI endpoint driver, refactor some of its code, improves link training
> >> and adds handling of the PERST# signal.
> >> 
> >> This series is organized as follows:
> >>  - Patch 1 fixes the rockchip ATU programming
> >>  - Patch 2, 3 and 4 introduce small code improvments
> >>  - Patch 5 implements the .align_addr() operation to make the RK3399
> >>    endpoint controller driver fully functional with the new
> >>    pci_epc_mem_map() function
> >>  - Patch 6 uses the new align_addr operation function to fix the ATU
> >>    programming for MSI IRQ data mapping
> >>  - Patch 7, 8, 9 and 10 refactor the driver code to make it more
> >>    readable
> >>  - Patch 11 introduces the .stop() endpoint controller operation to
> >>    correctly disable the endpopint controller after use
> >>  - Patch 12 improves link training
> >>  - Patch 13 implements handling of the #PERST signal
> >>  - Patch 14 adds a DT overlay file to enable EP mode and define the
> >>    PERST# GPIO (reset-gpios) property.
> >> 
> >
> >Damien, please wait for my review before spinning the next revision. Sorry for
> >the delay.
> 
> Mani,
> 
> This series has already been merged, and is in Torvalds tree.
> 

Doh! I was referring to the NVMe EPF series. Let me reply there (and also get
coffee).

- Mani

> Except for patch 14/14 which is a device tree overlay, so it should go via the rockchip tree.
> 
> 
> Kind regards,
> Niklas

-- 
மணிவண்ணன் சதாசிவம்

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

  reply	other threads:[~2024-12-16  6:05 UTC|newest]

Thread overview: 64+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-17  1:58 [PATCH v5 00/14] Fix and improve the Rockchip endpoint driver Damien Le Moal
2024-10-17  1:58 ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 01/14] PCI: rockchip-ep: Fix address translation unit programming Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-11-15 22:41   ` Bjorn Helgaas
2024-11-15 22:41     ` Bjorn Helgaas
2024-11-17  8:04     ` Damien Le Moal
2024-11-17  8:04       ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 02/14] PCI: rockchip-ep: Use a macro to define EP controller .align feature Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 03/14] PCI: rockchip-ep: Improve rockchip_pcie_ep_unmap_addr() Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 04/14] PCI: rockchip-ep: Improve rockchip_pcie_ep_map_addr() Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 05/14] PCI: rockchip-ep: Implement the pci_epc_ops::align_addr() operation Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 06/14] PCI: rockchip-ep: Fix MSI IRQ data mapping Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  9:52   ` Niklas Cassel
2024-10-17  9:52     ` Niklas Cassel
2024-10-17  1:58 ` [PATCH v5 07/14] PCI: rockchip-ep: Rename rockchip_pcie_parse_ep_dt() Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 08/14] PCI: rockchip-ep: Refactor rockchip_pcie_ep_probe() memory allocations Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 09/14] PCI: rockchip-ep: Refactor rockchip_pcie_ep_probe() MSI-X hiding Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 10/14] PCI: rockchip-ep: Refactor endpoint link training enable Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 11/14] PCI: rockship-ep: Implement the pci_epc_ops::stop_link() operation Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 12/14] PCI: rockchip-ep: Improve link training Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-11-15 23:03   ` Bjorn Helgaas
2024-11-15 23:03     ` Bjorn Helgaas
2024-11-17  8:00     ` Damien Le Moal
2024-11-17  8:00       ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 13/14] PCI: rockchip-ep: Handle PERST# signal in endpoint mode Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-12-15  0:13   ` Bjorn Helgaas
2024-12-15  0:13     ` Bjorn Helgaas
2024-12-15  2:09     ` Damien Le Moal
2024-12-15  2:09       ` Damien Le Moal
2024-10-17  1:58 ` [PATCH v5 14/14] arm64: dts: rockchip: Add rockpro64 overlay for PCIe " Damien Le Moal
2024-10-17  1:58   ` Damien Le Moal
2024-10-29 10:35 ` [PATCH v5 00/14] Fix and improve the Rockchip endpoint driver Damien Le Moal
2024-10-29 10:35   ` Damien Le Moal
2024-11-13 14:29   ` Damien Le Moal
2024-11-13 14:29     ` Damien Le Moal
2024-11-13 17:52     ` Manivannan Sadhasivam
2024-11-13 17:52       ` Manivannan Sadhasivam
2024-11-13 20:59       ` Krzysztof Wilczyński
2024-11-13 20:59         ` Krzysztof Wilczyński
2024-11-14  4:14         ` Damien Le Moal
2024-11-14  4:14           ` Damien Le Moal
2024-11-14 17:24           ` Krzysztof Wilczyński
2024-11-14 17:24             ` Krzysztof Wilczyński
2024-11-13 20:49 ` Krzysztof Wilczyński
2024-11-13 20:49   ` Krzysztof Wilczyński
2024-12-16  5:49 ` Manivannan Sadhasivam
2024-12-16  5:49   ` Manivannan Sadhasivam
2024-12-16  6:00   ` Niklas Cassel
2024-12-16  6:00     ` Niklas Cassel
2024-12-16  6:05     ` Manivannan Sadhasivam [this message]
2024-12-16  6:05       ` Manivannan Sadhasivam

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=20241216060524.bc5fiwdyaxz4git7@thinkpad \
    --to=manivannan.sadhasivam@linaro.org \
    --cc=bhelgaas@google.com \
    --cc=cassel@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dlemoal@kernel.org \
    --cc=heiko@sntech.de \
    --cc=kishon@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=kw@linux.com \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux-rockchip@lists.infradead.org \
    --cc=lpieralisi@kernel.org \
    --cc=rick.wertenbroek@gmail.com \
    --cc=robh@kernel.org \
    --cc=shawn.lin@rock-chips.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.