Hi Rob, Conor,

On Thu, 11 Jul 2024 14:33:26 -0600
Rob Herring [off-list ref] wrote:

On Thu, Jul 11, 2024 at 1:08 PM Greg Kroah-Hartman
[off-list ref] wrote:

quoted

On Thu, Jul 11, 2024 at 06:44:38PM +0200, Herve Codina wrote:  

quoted

Hi Lee,

On Thu, 11 Jul 2024 16:29:52 +0100
Lee Jones [off-list ref] wrote:
 

quoted

On Thu, 27 Jun 2024, Herve Codina wrote:
 

quoted

Add a PCI driver that handles the LAN966x PCI device using a device-tree
overlay. This overlay is applied to the PCI device DT node and allows to
describe components that are present in the device.

The memory from the device-tree is remapped to the BAR memory thanks to
"ranges" properties computed at runtime by the PCI core during the PCI
enumeration.

The PCI device itself acts as an interrupt controller and is used as the
parent of the internal LAN966x interrupt controller to route the
interrupts to the assigned PCI INTx interrupt.  

Not entirely sure why this is in MFD.  

This PCI driver purpose is to instanciate many other drivers using a DT
overlay. I think MFD is the right subsystem.  

It is a Multi-function Device, but it doesn't appear to use any of the
MFD subsystem. So maybe drivers/soc/? Another dumping ground, but it
is a driver for an SoC exposed as a PCI device.

In drivers/soc, drivers/soc/microchip/ could be the right place.

Conor, are you open to have the PCI LAN966x device driver in
drivers/soc/microchip/ ?

Best regards,
Hervé