On Fri, Jul 12, 2024, at 15:11, Herve Codina wrote: > On Thu, 11 Jul 2024 14:33:26 -0600 Rob Herring <robh@xxxxxxxxxx> wrote: >> On Thu, Jul 11, 2024 at 1:08 PM Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> wrote: >> > > >> > > 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/ ? That sounds like a much worse fit than drivers/mfd: the code here does not actually run on the lan966x soc, it instead runs on whatever other machine you happen to plug it into as a PCI device. Arnd