[undi] Use meaningful driver and device names

Specify a driver name of "undionly" and a device name based on the
UNDI-reported underlying hardware device.  For example:

  net0: 52:54:00:12:34:56 using undionly on UNDI-PCI00:03.0 (open)

Signed-off-by: Michael Brown <mcb30@ipxe.org>

diff --git a/src/arch/i386/drivers/net/undionly.c b/src/arch/i386/drivers/net/undionly.c
index c38b574dbc433a8c1f7435d1bc7cd2839b1d9427..a0012c88fedbd9b4f8a029cb83ab507bc83dc3fd 100644 (file)
--- a/src/arch/i386/drivers/net/undionly.c
+++ b/src/arch/i386/drivers/net/undionly.c
@@ -20,9 +20,11 @@ FILE_LICENCE ( GPL2_OR_LATER );
 
 #include <stdint.h>
 #include <stdlib.h>
+#include <stdio.h>
 #include <string.h>
 #include <ipxe/device.h>
 #include <ipxe/init.h>
+#include <ipxe/pci.h>
 #include <undi.h>
 #include <undinet.h>
 #include <undipreload.h>
@@ -62,15 +64,21 @@ static int undibus_probe ( struct root_device *rootdev ) {
        }
 
        /* Add to device hierarchy */
-       strncpy ( undi->dev.name, "UNDI",
-                 ( sizeof ( undi->dev.name ) - 1 ) );
+       undi->dev.driver_name = "undionly";
        if ( undi->pci_busdevfn != UNDI_NO_PCI_BUSDEVFN ) {
                undi->dev.desc.bus_type = BUS_TYPE_PCI;
                undi->dev.desc.location = undi->pci_busdevfn;
                undi->dev.desc.vendor = undi->pci_vendor;
                undi->dev.desc.device = undi->pci_device;
+               snprintf ( undi->dev.name, sizeof ( undi->dev.name ),
+                          "UNDI-PCI%02x:%02x.%x",
+                          PCI_BUS ( undi->pci_busdevfn ),
+                          PCI_SLOT ( undi->pci_busdevfn ),
+                          PCI_FUNC ( undi->pci_busdevfn ) );
        } else if ( undi->isapnp_csn != UNDI_NO_ISAPNP_CSN ) {
                undi->dev.desc.bus_type = BUS_TYPE_ISAPNP;
+               snprintf ( undi->dev.name, sizeof ( undi->dev.name ),
+                          "UNDI-ISAPNP" );
        }
        undi->dev.parent = &rootdev->dev;
        list_add ( &undi->dev.siblings, &rootdev->dev.children);