On Wed, 22 Jan 2020, Mika Westerberg wrote: > On Wed, Jan 22, 2020 at 01:27:57PM +0000, Lee Jones wrote: > > > Which type of device you suggest here? And which bus it should be > > > registered to? I think we can make this create a platform_device but > > > then we would need to do that from the PCI driver as well which seems > > > unnecessary since we already have the struct pci_dev. > > > > What kind of device is it? > > It is either part of an ACPI device (platform_device) or a PCI device > depending on the platform. > > > Refrain from using platform device, unless it is one please. > > OK. > > Greg suggested making the SCU IPC functionality a class and I think it > fits here nicely so I'm going to try that next if nobody objects. I'll > send the first cleanup patches separately. Sounds good. -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog