Adding a Kluge

I added a couple of things to my drivers. The big one is this addition to config.c.

{ PCI_VENDOR_ID_INTEL,    PCI_DEVICE_ID_INTEL_82546EB_FIBER,
  "Intel EtherExpressPro1000 82545EB Fiber", 0, 0, 0, 0},
{ PCI_VENDOR_ID_INTEL,    PCI_DEVICE_ID_INTEL_82546GB_COPPER,
  "Intel EtherExpressPro1000 82546GB Copper", 0, 0, 0, 0},
/* Mark start */
{ PCI_VENDOR_ID_INTEL,    PCI_DEVICE_ID_INTEL_MARKS_KLUGE,
  "Intel EtherExpressPro1000 Marks Kluge", 0, 0, 0, 0},
/* Mark end */

This makes it recognize my device by telling the pci.c to expect it. I saw the debug code print out that it found it, but I still got the Error 12: Invalid device.

So I looked and found another structure I hadn't updated yet in config.c:

static struct pci_dispatch_table PCI_NIC[] =
{
...
# ifdef    INCLUDE_E1000
 { PCI_VENDOR_ID_INTEL,    PCI_DEVICE_ID_INTEL_MARKS_KLUGE,     e1000_probe },
# endif /* INCLUDE_E1000 */
...

e1000_probe is the function that finds the device and actually assigns pointers to the correct driver functions (I hope they are correct enough) to the driver struct that holds pointers.

Baby steps. It's my first time working with drivers in Linux. It sure has its challenges.

So, I make. And as you might expect, I get build errors.

Seems the e1000.c uses pci_read_config_word whereas the rest of the files use pcibios_read_config_word. The same pattern applies to a few pci BIOS access routines. Those are easy enough with #defines for the time being. But there are 2 undefined functions--pci_bar_start and pci_bar_size. This will be a little tougher. There is code out there for VirtualBox. Is it the same thing?

Well, I guess I'll post this now and see what this bar address stuff is all about. Stay tuned.