On Tue, Jul 5, 2022 at 5:17 PM John Garry <john.garry@xxxxxxxxxx> wrote: > > > Next, I'd look at introducing something like > > > > acpi_create_platform_device_ops(struct acpi_device *adev, const struct > > property_entry *properties, const struct *platform_device_create_ops > > *ops); > > > > where ops would be a set of callbacks to invoke as a matter of customization. > > > > Then, acpi_create_platform_device() can be defined as a wrapper around > > the above. > > . > > ok, that seems easiest. But alternatively do you see any scope to have > that platform_device_create_ops * ops in the acpi_device struct (so that > we don't need to create this new API)? Well, ops and struct acpi_device have different life cycles (the former is only needed at the init time whereas the latter lives as long as the platform device based on it), so I'd rather keep them separate.