On 04/27/2018 09:48 AM, Tony Camuso wrote:
On 04/25/2018 08:29 AM, Corey Minyard wrote:
On 04/24/2018 01:13 PM, Tony Camuso wrote:
The following commit ...

    commit 0944d889a237b6107f9ceeee053fe7221cdd1089
    Author: Corey Minyard <cminy...@mvista.com>
    Date:   Wed Feb 3 10:45:28 2016 -0600
            ipmi: Convert DMI handling over to a platform device
            Now that the IPMI DMI code creates a platform device for IPMI 
devices
        in the firmware, use that instead of handling all the DMI work
        in the IPMI drivers themselves.
            Signed-off-by: Corey Minyard <cminy...@mvista.com>
        Cc: Andy Lutomirski <l...@kernel.org>

... moved the dmi detection of the ipmi_si hardware out of ipmi_si_intf.c
into ipmi_dmi.c

Not exactly.  ipmi_dmi.c finds IPMI devices in the DMI tables and creates
platform devices for them.  Before the linkage was more direct.

The point of this was to allow auto-loading of the IPMI driver based
upon the contents of the DMI tables, like it worked for ACPI.

The problem I am experiencing occurs in the platform_match code in
file drivers/base/platform.c

With some instrumentation, I can see that there can never be a match,
at least not based on the name strings.

** platform_match():802  pdev->name: dmi-ipmi-si == drv->name: ipmi_si
** platform_match():807 (strcmp(pdev->name, drv->name) == 0) NO MATCH

This code is identical in kernel.org v4.17-rc2 which works!

Never mind. I found the problem: driver_override

I did not backport this field for reasons I can't divulge.

I'll have to think of a workaround.

Thanks for "listening".

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openipmi-developer mailing list
Openipmi-developer@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openipmi-developer

Reply via email to