[coreboot-gerrit] New patch to review for coreboot: drivers/r8168: Read default MAC address from CBFS
Damien Zammit (damien@zamaudio.com)
gerrit at coreboot.org
Thu Dec 1 14:10:03 CET 2016
Damien Zammit (damien at zamaudio.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17672
-gerrit
commit f2bd9f182e88d1dfeadf93cd84b8782fb6a782b4
Author: Damien Zammit <damien at zamaudio.com>
Date: Thu Dec 1 23:29:34 2016 +1100
drivers/r8168: Read default MAC address from CBFS
This driver applies to 10ec:8168
Previously, this driver resetted the nic and set a hardcoded
MAC address. Now the driver reads a default MAC address
from CBFS in the form of a string:
echo -n "xx:xx:xx:xx:xx:xx" > macaddress
and store the macaddress file in CBFS with the same name.
Change-Id: If1af91120fa3efca3f1406334a83ed1e59fbdaf9
Signed-off-by: Damien Zammit <damien at zamaudio.com>
---
src/drivers/net/r8168.c | 115 ++++++++++++++++++++++++++++++++++++------------
1 file changed, 88 insertions(+), 27 deletions(-)
diff --git a/src/drivers/net/r8168.c b/src/drivers/net/r8168.c
index 1e6db57..4768b86 100644
--- a/src/drivers/net/r8168.c
+++ b/src/drivers/net/r8168.c
@@ -1,6 +1,7 @@
/*
* This file is part of the coreboot project.
*
+ * Copyright (C) 2012 Google Inc.
* Copyright (C) 2016 Damien Zammit <damien at zamaudio.com>
*
* This program is free software; you can redistribute it and/or modify
@@ -14,11 +15,14 @@
*/
/*
- * This driver forces the 10ec:8168 device to reset so that it goes
- * into a proper power state, also programs a default MAC address
- * so that if the EEPROM/efuse is unconfigured it still has a default MAC.
+ * This driver resets the 10ec:8168 NIC then tries to read
+ * "macaddress" string XX:XX:XX:XX:XX:XX from CBFS.
+ * If no MAC is found, it programs a default MAC address in the device
+ * so that if the EEPROM/efuse is unconfigured it still has a default MAC
*/
+#include <cbfs.h>
+#include <string.h>
#include <arch/io.h>
#include <device/device.h>
#include <device/pci.h>
@@ -36,26 +40,64 @@
#define CFG_9346_LOCK 0x00
#define CFG_9346_UNLOCK 0xc0
-static void r8168_init(struct device *dev)
+static unsigned char get_hex_digit(u8 *offset)
{
- u32 i;
- const u8 mac[6] = { 0x00, 0xe0, 0x4c, 0x00, 0xc0, 0xb0 };
+ unsigned char retval = 0;
- /* Get the resource of the NIC mmio */
- struct resource *nic_res = find_resource(dev, PCI_BASE_ADDRESS_0);
- u16 nic_port = (u16)nic_res->base;
+ retval = *offset - '0';
+ if (retval > 0x09) {
+ retval = *offset - 'A' + 0x0A;
+ if (retval > 0x0F)
+ retval = *offset - 'a' + 0x0a;
+ }
+ if (retval > 0x0F) {
+ printk(BIOS_DEBUG, "Error: Invalid Hex digit found: %c - 0x%02x\n",
+ *offset, *offset);
+ retval = 0;
+ }
- /* Ensble but do not set bus master. That's dangerous on a NIC. */
- pci_write_config16(dev, PCI_COMMAND,
- PCI_COMMAND_MEMORY | PCI_COMMAND_IO);
+ return retval;
+}
+
+static void get_mac_address(u8 *macaddr, u8 *cbfs_address)
+{
+ unsigned int offset = 0;
+ int i;
+
+ for (i = 0; i < 6; i++) {
+ macaddr[i] = 0;
+ macaddr[i] |= (get_hex_digit(cbfs_address + offset)
+ << (4 + (i * 8)));
+ macaddr[i] |= (get_hex_digit(cbfs_address + offset + 1)
+ << (i * 8));
+ offset += 3;
+ }
+}
+
+static void program_mac_address(struct device *dev, u16 io_base)
+{
+ void *cbfs_address = NULL;
+ size_t search_length = -1;
+ int i = 0;
+ u8 mac[6] = { 0x00, 0xe0, 0x4c, 0x00, 0xc0, 0xb0 };
+
+ cbfs_address = cbfs_boot_map_with_leak("macaddress",
+ CBFS_TYPE_RAW,
+ &search_length);
+
+ if (cbfs_address == NULL) {
+ printk(BIOS_ERR, "r8168: `macaddress` not found in CBFS, "
+ "using default 00:e0:4c:00:c0:b0\n");
+ } else {
+ get_mac_address(mac, cbfs_address);
+ }
/* Reset NIC */
printk(BIOS_DEBUG, "r8168: Resetting NIC...");
- outb(CMD_REG_RESET, nic_port + CMD_REG);
+ outb(CMD_REG_RESET, io_base + CMD_REG);
- i = 0;
- /* Poll for reset, with 1s timeout */
- while (i < NIC_TIMEOUT && (inb(nic_port + CMD_REG) & CMD_REG_RESET)) {
+ /* Poll for reset, with 1sec timeout */
+ while (i < NIC_TIMEOUT && (inb(io_base + CMD_REG) & CMD_REG_RESET)) {
udelay(1000);
if (++i >= NIC_TIMEOUT)
printk(BIOS_DEBUG, "timeout waiting for nic to reset\n");
@@ -63,20 +105,39 @@ static void r8168_init(struct device *dev)
if (i < NIC_TIMEOUT)
printk(BIOS_DEBUG, "done\n");
- /* Unlock config regs */
- outb(CFG_9346_UNLOCK, nic_port + CFG_9346);
-
- /* Set MAC address 00:e0:4c:00:c0:b0
- * NB: only 4-byte write accesses allowed
- */
- outl(mac[4] | mac[5] << 8, nic_port + 4);
- inl(nic_port + 4);
+ printk(BIOS_DEBUG, "r8168: Programming MAC Address...");
- outl(mac[0] | mac[1] << 8 | mac[2] << 16 | mac[3] << 24, nic_port);
- inl(nic_port);
+ /* Disable register protection */
+ outb(CFG_9346_UNLOCK, io_base + CFG_9346);
+ /* Set MAC address: only 4-byte write accesses allowed */
+ outl(mac[4] | mac[5] << 8, io_base + 4);
+ inl(io_base + 4);
+ outl(mac[0] | mac[1] << 8 | mac[2] << 16 | mac[3] << 24,
+ io_base);
+ inl(io_base);
/* Lock config regs */
- outb(CFG_9346_LOCK, nic_port + CFG_9346);
+ outb(CFG_9346_LOCK, io_base + CFG_9346);
+
+ printk(BIOS_DEBUG, "done\n");
+}
+
+static void r8168_init(struct device *dev)
+{
+ /* Get the resource of the NIC mmio */
+ struct resource *nic_res = find_resource(dev, PCI_BASE_ADDRESS_0);
+ u16 io_base = (u16)nic_res->base;
+
+ /* Enable but do not set bus master. That's dangerous on a NIC. */
+ pci_write_config16(dev, PCI_COMMAND,
+ PCI_COMMAND_MEMORY | PCI_COMMAND_IO);
+
+ /* Program MAC address based on CBFS "macaddress" containing
+ * a string AA:BB:CC:DD:EE:FF */
+ if (io_base)
+ program_mac_address(dev, io_base);
+ else
+ printk(BIOS_DEBUG, "r8168: Error cant find MMIO resource\n");
}
static struct device_operations r8168_ops = {
More information about the coreboot-gerrit
mailing list