Re: [PATCH 2/2] platform/x86: Add Intel Software Defined Silicon driver

On Sun, Sep 26, 2021 at 06:15:16PM -0700, David E. Box wrote:

quoted

quoted

+static struct platform_driver sdsi_driver = {
+       .driver = {
+               .name           = SDSI_DEV_NAME,
+               .dev_groups     = sdsi_groups,
+       },
+       .probe  = sdsi_probe,
+       .remove = sdsi_remove,
+};
+module_platform_driver(sdsi_driver);

What causes the platform to know to register, and enable, this platform
driver?  Shouldn't there be some hardware involved that is discoverable
to enable it to load dynamically?

Ah. The patch that adds the SDSi platform device string was added to a series for the intel_pmt MFD
driver and it's still waiting review. I see that complicates things. I can combine the two series
together.

Do you have a pointer to the lore.kernel.org location of that series?

Your code right here will bind to any system that it is loaded on, a
very dangerous thing...

thanks,

greg k-h