Thanks, this has been on my todo list for a while, but I didn't know exactly how to accomplish it.
Queued for 4.10, or whatever shows up next. I also added a similar change for the ipmi_ssif interface. -corey On 11/25/2016 03:55 AM, Martin Wilck wrote: > When a computer has an IPMI system interface, the device interface > is most probably also desired. Autoloading of ipmi_devintf currently > works only if ipmi_si has allocated a platform device. That doesn't > happen if the SI interface was detected e.g. via ACPI. But ACPI > detection is preferred these days, see e.g. kernel.org bug 46741. > > This patch introduces a softdep in place of the existing modalias > for ipmi_devintf. > > Signed-off-by: Martin Wilck <[email protected]> > Suggested-by: Takashi Iwai <[email protected]> > --- > drivers/char/ipmi/ipmi_devintf.c | 1 - > drivers/char/ipmi/ipmi_si_intf.c | 1 + > 2 files changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/char/ipmi/ipmi_devintf.c > b/drivers/char/ipmi/ipmi_devintf.c > index 1786574..a21407d 100644 > --- a/drivers/char/ipmi/ipmi_devintf.c > +++ b/drivers/char/ipmi/ipmi_devintf.c > @@ -989,4 +989,3 @@ module_exit(cleanup_ipmi); > MODULE_LICENSE("GPL"); > MODULE_AUTHOR("Corey Minyard <[email protected]>"); > MODULE_DESCRIPTION("Linux device interface for the IPMI message handler."); > -MODULE_ALIAS("platform:ipmi_si"); > diff --git a/drivers/char/ipmi/ipmi_si_intf.c > b/drivers/char/ipmi/ipmi_si_intf.c > index a112c01..20ebdf9 100644 > --- a/drivers/char/ipmi/ipmi_si_intf.c > +++ b/drivers/char/ipmi/ipmi_si_intf.c > @@ -3927,3 +3927,4 @@ MODULE_LICENSE("GPL"); > MODULE_AUTHOR("Corey Minyard <[email protected]>"); > MODULE_DESCRIPTION("Interface to the IPMI driver for the KCS, SMIC, and BT" > " system interfaces."); > +MODULE_SOFTDEP("post: ipmi_devintf"); ------------------------------------------------------------------------------ _______________________________________________ Openipmi-developer mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openipmi-developer
