Message-ID: <9b7161a7-2682-4824-8af0-39477b0938d8@bootlin.com>
Date: Mon, 23 Dec 2024 17:35:43 +0100
From: Olivier Benjamin <olivier.benjamin@...tlin.com>
To: Stanimir Varbanov <svarbanov@...e.de>, linux-kernel@...r.kernel.org,
 devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-rpi-kernel@...ts.infradead.org, linux-pci@...r.kernel.org,
 Broadcom internal kernel review list <bcm-kernel-feedback-list@...adcom.com>
Cc: Thomas Gleixner <tglx@...utronix.de>, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>, Florian Fainelli <florian.fainelli@...adcom.com>,
 Jim Quinlan <jim2101024@...il.com>,
 Nicolas Saenz Julienne <nsaenz@...nel.org>,
 Bjorn Helgaas <bhelgaas@...gle.com>,
 Lorenzo Pieralisi <lpieralisi@...nel.org>, kw@...ux.com,
 Philipp Zabel <p.zabel@...gutronix.de>,
 Andrea della Porta <andrea.porta@...e.com>,
 Phil Elwell <phil@...pberrypi.com>, Jonathan Bell <jonathan@...pberrypi.com>
Subject: Re: [PATCH v4 00/10] Add PCIe support for bcm2712

On 10/25/24 14:45, Stanimir Varbanov wrote:
> Here is v4 of the series which aims to add support for PCIe on bcm2712 SoC
> used by RPi5. Previous v3 can be found at [1].
> 
Hello Stanimir,

Thank you for you work on this!

> v3 -> v4 changes include:
>   - Addressed comments on the interrupt-controller driver (Thomas)
>   - Moved "Reuse config structure" patch earlier in the series (Bjorn)
>   - Merged "Avoid turn off of bridge reset" into "Add bcm2712 support" (Bjorn)
>   - Fixed DTB warnings on broadcom/bcm2712-rpi-5-b.dtb
> 
> For more detailed info check patches.
> 
I would simply like to report that I have (rather succintly) tested this 
series.
I have built a 6.13.0-rc4-v8 vanilla kernel and deployed it to a 
Raspberry Pi 5 equipped with a "Raspberry Pi SSD" NVMe Drive on an M.2 
Hat+ connected to the main board using PCIe.
This of course did not work, and I could not see my drive.
I then applied this series on top, rebuilt and deployed the kernel, and 
I could see the /dev/nvme0 device and mount the ext4 fs on the 1st 
partition.

> Comments are welcome!
> ~Stan
	
If you find it helpful, feel free to collect my Tested-By tag.

I'll be happy to do the same for future versions of the series and can 
do some additional testing if you want an extra pair of eyes.

Olivier
> 
> [1] https://patchwork.kernel.org/project/linux-pci/list/?series=898891
> 
> Stanimir Varbanov (10):
>    dt-bindings: interrupt-controller: Add bcm2712 MSI-X DT bindings
>    dt-bindings: PCI: brcmstb: Update bindings for PCIe on bcm2712
>    irqchip: Add Broadcom bcm2712 MSI-X interrupt controller
>    PCI: brcmstb: Reuse config structure
>    PCI: brcmstb: Expand inbound window size up to 64GB
>    PCI: brcmstb: Enable external MSI-X if available
>    PCI: brcmstb: Add bcm2712 support
>    PCI: brcmstb: Adjust PHY PLL setup to use a 54MHz input refclk
>    arm64: dts: broadcom: bcm2712: Add PCIe DT nodes
>    arm64: dts: broadcom: bcm2712-rpi-5-b: Enable PCIe DT nodes
> 
>   .../brcm,bcm2712-msix.yaml                    |  60 ++++
>   .../bindings/pci/brcm,stb-pcie.yaml           |  21 ++
>   .../boot/dts/broadcom/bcm2712-rpi-5-b.dts     |   8 +
>   arch/arm64/boot/dts/broadcom/bcm2712.dtsi     | 162 +++++++++
>   drivers/irqchip/Kconfig                       |  16 +
>   drivers/irqchip/Makefile                      |   1 +
>   drivers/irqchip/irq-bcm2712-mip.c             | 310 ++++++++++++++++++
>   drivers/pci/controller/pcie-brcmstb.c         | 204 +++++++++---
>   8 files changed, 735 insertions(+), 47 deletions(-)
>   create mode 100644 Documentation/devicetree/bindings/interrupt-controller/brcm,bcm2712-msix.yaml
>   create mode 100644 drivers/irqchip/irq-bcm2712-mip.c
> 

-- 
Olivier Benjamin, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Powered by blists - more mailing lists