drivers: net: add NXP ENETC ethernet driver
authorAlex Marginean <alexandru.marginean@nxp.com>
Wed, 3 Jul 2019 09:11:40 +0000 (12:11 +0300)
committerJoe Hershberger <joe.hershberger@ni.com>
Thu, 25 Jul 2019 18:13:30 +0000 (13:13 -0500)
Adds a driver for NXP ENETC ethernet controller currently integrated in
LS1028A.  ENETC is a fairly straight-forward BD ring device and interfaces
are presented as PCI EPs on the SoC ECAM.

Signed-off-by: Catalin Horghidan <catalin.horghidan@nxp.com>
Signed-off-by: Alex Marginean <alexm.osslist@gmail.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
drivers/net/Kconfig
drivers/net/Makefile
drivers/net/fsl_enetc.c [new file with mode: 0644]
drivers/net/fsl_enetc.h [new file with mode: 0644]

index 403df5e9600452e767ba4ac0d559a5cbe9a76ca4..802eadf99d7a21785a25999286540fb0c16321ce 100644 (file)
@@ -588,4 +588,11 @@ config HIGMACV300_ETH
          This driver supports HIGMACV300 Ethernet controller found on
          HiSilicon SoCs.
 
+config FSL_ENETC
+       bool "NXP ENETC Ethernet controller"
+       depends on DM_PCI && DM_ETH
+       help
+         This driver supports the NXP ENETC Ethernet controller found on some
+         of the NXP SoCs.
+
 endif # NETDEVICES
index 3c473b205d544c6e2b0936de76d75fbdef18e215..a894a42efb8d3aa92e8e09392fc0d6d1643caea0 100644 (file)
@@ -79,3 +79,4 @@ obj-$(CONFIG_MEDIATEK_ETH) += mtk_eth.o
 obj-y += mscc_eswitch/
 obj-$(CONFIG_HIGMACV300_ETH) += higmacv300.o
 obj-$(CONFIG_MDIO_SANDBOX) += mdio_sandbox.o
+obj-$(CONFIG_FSL_ENETC) += fsl_enetc.o
diff --git a/drivers/net/fsl_enetc.c b/drivers/net/fsl_enetc.c
new file mode 100644 (file)
index 0000000..58ba63c
--- /dev/null
@@ -0,0 +1,380 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * ENETC ethernet controller driver
+ * Copyright 2017-2019 NXP
+ */
+
+#include <common.h>
+#include <dm.h>
+#include <errno.h>
+#include <memalign.h>
+#include <asm/io.h>
+#include <pci.h>
+
+#include "fsl_enetc.h"
+
+/*
+ * Bind the device:
+ * - set a more explicit name on the interface
+ */
+static int enetc_bind(struct udevice *dev)
+{
+       char name[16];
+       static int eth_num_devices;
+
+       /*
+        * prefer using PCI function numbers to number interfaces, but these
+        * are only available if dts nodes are present.  For PCI they are
+        * optional, handle that case too.  Just in case some nodes are present
+        * and some are not, use different naming scheme - enetc-N based on
+        * PCI function # and enetc#N based on interface count
+        */
+       if (ofnode_valid(dev->node))
+               sprintf(name, "enetc-%u", PCI_FUNC(pci_get_devfn(dev)));
+       else
+               sprintf(name, "enetc#%u", eth_num_devices++);
+       device_set_name(dev, name);
+
+       return 0;
+}
+
+/*
+ * Probe ENETC driver:
+ * - initialize port and station interface BARs
+ */
+static int enetc_probe(struct udevice *dev)
+{
+       struct enetc_priv *priv = dev_get_priv(dev);
+
+       if (ofnode_valid(dev->node) && !ofnode_is_available(dev->node)) {
+               enetc_dbg(dev, "interface disabled\n");
+               return -ENODEV;
+       }
+
+       priv->enetc_txbd = memalign(ENETC_BD_ALIGN,
+                                   sizeof(struct enetc_tx_bd) * ENETC_BD_CNT);
+       priv->enetc_rxbd = memalign(ENETC_BD_ALIGN,
+                                   sizeof(union enetc_rx_bd) * ENETC_BD_CNT);
+
+       if (!priv->enetc_txbd || !priv->enetc_rxbd) {
+               /* free should be able to handle NULL, just free all pointers */
+               free(priv->enetc_txbd);
+               free(priv->enetc_rxbd);
+
+               return -ENOMEM;
+       }
+
+       /* initialize register */
+       priv->regs_base = dm_pci_map_bar(dev, PCI_BASE_ADDRESS_0, 0);
+       if (!priv->regs_base) {
+               enetc_dbg(dev, "failed to map BAR0\n");
+               return -EINVAL;
+       }
+       priv->port_regs = priv->regs_base + ENETC_PORT_REGS_OFF;
+
+       dm_pci_clrset_config16(dev, PCI_COMMAND, 0, PCI_COMMAND_MEMORY);
+
+       return 0;
+}
+
+/*
+ * Remove the driver from an interface:
+ * - free up allocated memory
+ */
+static int enetc_remove(struct udevice *dev)
+{
+       struct enetc_priv *priv = dev_get_priv(dev);
+
+       free(priv->enetc_txbd);
+       free(priv->enetc_rxbd);
+
+       return 0;
+}
+
+/* ENETC Port MAC address registers, accepts big-endian format */
+static void enetc_set_primary_mac_addr(struct enetc_priv *priv, const u8 *addr)
+{
+       u16 lower = *(const u16 *)(addr + 4);
+       u32 upper = *(const u32 *)addr;
+
+       enetc_write_port(priv, ENETC_PSIPMAR0, upper);
+       enetc_write_port(priv, ENETC_PSIPMAR1, lower);
+}
+
+/* Configure port parameters (# of rings, frame size, enable port) */
+static void enetc_enable_si_port(struct enetc_priv *priv)
+{
+       u32 val;
+
+       /* set Rx/Tx BDR count */
+       val = ENETC_PSICFGR_SET_TXBDR(ENETC_TX_BDR_CNT);
+       val |= ENETC_PSICFGR_SET_RXBDR(ENETC_RX_BDR_CNT);
+       enetc_write_port(priv, ENETC_PSICFGR(0), val);
+       /* set Rx max frame size */
+       enetc_write_port(priv, ENETC_PM_MAXFRM, ENETC_RX_MAXFRM_SIZE);
+       /* enable MAC port */
+       enetc_write_port(priv, ENETC_PM_CC, ENETC_PM_CC_RX_TX_EN);
+       /* enable port */
+       enetc_write_port(priv, ENETC_PMR, ENETC_PMR_SI0_EN);
+       /* set SI cache policy */
+       enetc_write(priv, ENETC_SICAR0,
+                   ENETC_SICAR_RD_CFG | ENETC_SICAR_WR_CFG);
+       /* enable SI */
+       enetc_write(priv, ENETC_SIMR, ENETC_SIMR_EN);
+}
+
+/* returns DMA address for a given buffer index */
+static inline u64 enetc_rxb_address(struct udevice *dev, int i)
+{
+       return cpu_to_le64(dm_pci_virt_to_mem(dev, net_rx_packets[i]));
+}
+
+/*
+ * Setup a single Tx BD Ring (ID = 0):
+ * - set Tx buffer descriptor address
+ * - set the BD count
+ * - initialize the producer and consumer index
+ */
+static void enetc_setup_tx_bdr(struct udevice *dev)
+{
+       struct enetc_priv *priv = dev_get_priv(dev);
+       struct bd_ring *tx_bdr = &priv->tx_bdr;
+       u64 tx_bd_add = (u64)priv->enetc_txbd;
+
+       /* used later to advance to the next Tx BD */
+       tx_bdr->bd_count = ENETC_BD_CNT;
+       tx_bdr->next_prod_idx = 0;
+       tx_bdr->next_cons_idx = 0;
+       tx_bdr->cons_idx = priv->regs_base +
+                               ENETC_BDR(TX, ENETC_TX_BDR_ID, ENETC_TBCIR);
+       tx_bdr->prod_idx = priv->regs_base +
+                               ENETC_BDR(TX, ENETC_TX_BDR_ID, ENETC_TBPIR);
+
+       /* set Tx BD address */
+       enetc_bdr_write(priv, TX, ENETC_TX_BDR_ID, ENETC_TBBAR0,
+                       lower_32_bits(tx_bd_add));
+       enetc_bdr_write(priv, TX, ENETC_TX_BDR_ID, ENETC_TBBAR1,
+                       upper_32_bits(tx_bd_add));
+       /* set Tx 8 BD count */
+       enetc_bdr_write(priv, TX, ENETC_TX_BDR_ID, ENETC_TBLENR,
+                       tx_bdr->bd_count);
+
+       /* reset both producer/consumer indexes */
+       enetc_write_reg(tx_bdr->cons_idx, tx_bdr->next_cons_idx);
+       enetc_write_reg(tx_bdr->prod_idx, tx_bdr->next_prod_idx);
+
+       /* enable TX ring */
+       enetc_bdr_write(priv, TX, ENETC_TX_BDR_ID, ENETC_TBMR, ENETC_TBMR_EN);
+}
+
+/*
+ * Setup a single Rx BD Ring (ID = 0):
+ * - set Rx buffer descriptors address (one descriptor per buffer)
+ * - set buffer size as max frame size
+ * - enable Rx ring
+ * - reset consumer and producer indexes
+ * - set buffer for each descriptor
+ */
+static void enetc_setup_rx_bdr(struct udevice *dev)
+{
+       struct enetc_priv *priv = dev_get_priv(dev);
+       struct bd_ring *rx_bdr = &priv->rx_bdr;
+       u64 rx_bd_add = (u64)priv->enetc_rxbd;
+       int i;
+
+       /* used later to advance to the next BD produced by ENETC HW */
+       rx_bdr->bd_count = ENETC_BD_CNT;
+       rx_bdr->next_prod_idx = 0;
+       rx_bdr->next_cons_idx = 0;
+       rx_bdr->cons_idx = priv->regs_base +
+                               ENETC_BDR(RX, ENETC_RX_BDR_ID, ENETC_RBCIR);
+       rx_bdr->prod_idx = priv->regs_base +
+                               ENETC_BDR(RX, ENETC_RX_BDR_ID, ENETC_RBPIR);
+
+       /* set Rx BD address */
+       enetc_bdr_write(priv, RX, ENETC_RX_BDR_ID, ENETC_RBBAR0,
+                       lower_32_bits(rx_bd_add));
+       enetc_bdr_write(priv, RX, ENETC_RX_BDR_ID, ENETC_RBBAR1,
+                       upper_32_bits(rx_bd_add));
+       /* set Rx BD count (multiple of 8) */
+       enetc_bdr_write(priv, RX, ENETC_RX_BDR_ID, ENETC_RBLENR,
+                       rx_bdr->bd_count);
+       /* set Rx buffer  size */
+       enetc_bdr_write(priv, RX, ENETC_RX_BDR_ID, ENETC_RBBSR, PKTSIZE_ALIGN);
+
+       /* fill Rx BD */
+       memset(priv->enetc_rxbd, 0,
+              rx_bdr->bd_count * sizeof(union enetc_rx_bd));
+       for (i = 0; i < rx_bdr->bd_count; i++) {
+               priv->enetc_rxbd[i].w.addr = enetc_rxb_address(dev, i);
+               /* each RX buffer must be aligned to 64B */
+               WARN_ON(priv->enetc_rxbd[i].w.addr & (ARCH_DMA_MINALIGN - 1));
+       }
+
+       /* reset producer (ENETC owned) and consumer (SW owned) index */
+       enetc_write_reg(rx_bdr->cons_idx, rx_bdr->next_cons_idx);
+       enetc_write_reg(rx_bdr->prod_idx, rx_bdr->next_prod_idx);
+
+       /* enable Rx ring */
+       enetc_bdr_write(priv, RX, ENETC_RX_BDR_ID, ENETC_RBMR, ENETC_RBMR_EN);
+}
+
+/*
+ * Start ENETC interface:
+ * - perform FLR
+ * - enable access to port and SI registers
+ * - set mac address
+ * - setup TX/RX buffer descriptors
+ * - enable Tx/Rx rings
+ */
+static int enetc_start(struct udevice *dev)
+{
+       struct eth_pdata *plat = dev_get_platdata(dev);
+       struct enetc_priv *priv = dev_get_priv(dev);
+
+       /* reset and enable the PCI device */
+       dm_pci_flr(dev);
+       dm_pci_clrset_config16(dev, PCI_COMMAND, 0,
+                              PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER);
+
+       if (!is_valid_ethaddr(plat->enetaddr)) {
+               enetc_dbg(dev, "invalid MAC address, generate random ...\n");
+               net_random_ethaddr(plat->enetaddr);
+       }
+       enetc_set_primary_mac_addr(priv, plat->enetaddr);
+
+       enetc_enable_si_port(priv);
+
+       /* setup Tx/Rx buffer descriptors */
+       enetc_setup_tx_bdr(dev);
+       enetc_setup_rx_bdr(dev);
+
+       return 0;
+}
+
+/*
+ * Stop the network interface:
+ * - just quiesce it, we can wipe all configuration as _start starts from
+ * scratch each time
+ */
+static void enetc_stop(struct udevice *dev)
+{
+       /* FLR is sufficient to quiesce the device */
+       dm_pci_flr(dev);
+}
+
+/*
+ * ENETC transmit packet:
+ * - check if Tx BD ring is full
+ * - set buffer/packet address (dma address)
+ * - set final fragment flag
+ * - try while producer index equals consumer index or timeout
+ */
+static int enetc_send(struct udevice *dev, void *packet, int length)
+{
+       struct enetc_priv *priv = dev_get_priv(dev);
+       struct bd_ring *txr = &priv->tx_bdr;
+       void *nv_packet = (void *)packet;
+       int tries = ENETC_POLL_TRIES;
+       u32 pi, ci;
+
+       pi = txr->next_prod_idx;
+       ci = enetc_read_reg(txr->cons_idx) & ENETC_BDR_IDX_MASK;
+       /* Tx ring is full when */
+       if (((pi + 1) % txr->bd_count) == ci) {
+               enetc_dbg(dev, "Tx BDR full\n");
+               return -ETIMEDOUT;
+       }
+       enetc_dbg(dev, "TxBD[%d]send: pkt_len=%d, buff @0x%x%08x\n", pi, length,
+                 upper_32_bits((u64)nv_packet), lower_32_bits((u64)nv_packet));
+
+       /* prepare Tx BD */
+       memset(&priv->enetc_txbd[pi], 0x0, sizeof(struct enetc_tx_bd));
+       priv->enetc_txbd[pi].addr =
+               cpu_to_le64(dm_pci_virt_to_mem(dev, nv_packet));
+       priv->enetc_txbd[pi].buf_len = cpu_to_le16(length);
+       priv->enetc_txbd[pi].frm_len = cpu_to_le16(length);
+       priv->enetc_txbd[pi].flags = cpu_to_le16(ENETC_TXBD_FLAGS_F);
+       dmb();
+       /* send frame: increment producer index */
+       pi = (pi + 1) % txr->bd_count;
+       txr->next_prod_idx = pi;
+       enetc_write_reg(txr->prod_idx, pi);
+       while ((--tries >= 0) &&
+              (pi != (enetc_read_reg(txr->cons_idx) & ENETC_BDR_IDX_MASK)))
+               udelay(10);
+
+       return tries > 0 ? 0 : -ETIMEDOUT;
+}
+
+/*
+ * Receive frame:
+ * - wait for the next BD to get ready bit set
+ * - clean up the descriptor
+ * - move on and indicate to HW that the cleaned BD is available for Rx
+ */
+static int enetc_recv(struct udevice *dev, int flags, uchar **packetp)
+{
+       struct enetc_priv *priv = dev_get_priv(dev);
+       struct bd_ring *rxr = &priv->rx_bdr;
+       int tries = ENETC_POLL_TRIES;
+       int pi = rxr->next_prod_idx;
+       int ci = rxr->next_cons_idx;
+       u32 status;
+       int len;
+       u8 rdy;
+
+       do {
+               dmb();
+               status = le32_to_cpu(priv->enetc_rxbd[pi].r.lstatus);
+               /* check if current BD is ready to be consumed */
+               rdy = ENETC_RXBD_STATUS_R(status);
+       } while (--tries >= 0 && !rdy);
+
+       if (!rdy)
+               return -EAGAIN;
+
+       dmb();
+       len = le16_to_cpu(priv->enetc_rxbd[pi].r.buf_len);
+       *packetp = (uchar *)enetc_rxb_address(dev, pi);
+       enetc_dbg(dev, "RxBD[%d]: len=%d err=%d pkt=0x%x%08x\n", pi, len,
+                 ENETC_RXBD_STATUS_ERRORS(status),
+                 upper_32_bits((u64)*packetp), lower_32_bits((u64)*packetp));
+
+       /* BD clean up and advance to next in ring */
+       memset(&priv->enetc_rxbd[pi], 0, sizeof(union enetc_rx_bd));
+       priv->enetc_rxbd[pi].w.addr = enetc_rxb_address(dev, pi);
+       rxr->next_prod_idx = (pi + 1) % rxr->bd_count;
+       ci = (ci + 1) % rxr->bd_count;
+       rxr->next_cons_idx = ci;
+       dmb();
+       /* free up the slot in the ring for HW */
+       enetc_write_reg(rxr->cons_idx, ci);
+
+       return len;
+}
+
+static const struct eth_ops enetc_ops = {
+       .start  = enetc_start,
+       .send   = enetc_send,
+       .recv   = enetc_recv,
+       .stop   = enetc_stop,
+};
+
+U_BOOT_DRIVER(eth_enetc) = {
+       .name   = "enetc_eth",
+       .id     = UCLASS_ETH,
+       .bind   = enetc_bind,
+       .probe  = enetc_probe,
+       .remove = enetc_remove,
+       .ops    = &enetc_ops,
+       .priv_auto_alloc_size = sizeof(struct enetc_priv),
+       .platdata_auto_alloc_size = sizeof(struct eth_pdata),
+};
+
+static struct pci_device_id enetc_ids[] = {
+       { PCI_DEVICE(PCI_VENDOR_ID_FREESCALE, PCI_DEVICE_ID_ENETC_ETH) },
+       {}
+};
+
+U_BOOT_PCI_DEVICE(eth_enetc, enetc_ids);
diff --git a/drivers/net/fsl_enetc.h b/drivers/net/fsl_enetc.h
new file mode 100644 (file)
index 0000000..155ecc8
--- /dev/null
@@ -0,0 +1,168 @@
+/* SPDX-License-Identifier: GPL-2.0+ */
+/*
+ * ENETC ethernet controller driver
+ * Copyright 2017-2019 NXP
+ */
+
+#ifndef _ENETC_H
+#define _ENETC_H
+
+#define enetc_dbg(dev, fmt, args...)   debug("%s:" fmt, dev->name, ##args)
+
+/* PCI function IDs */
+#define PCI_DEVICE_ID_ENETC_ETH                0xE100
+
+/* ENETC Ethernet controller registers */
+/* Station interface register offsets */
+#define ENETC_SIMR             0x000
+#define  ENETC_SIMR_EN         BIT(31)
+#define ENETC_SICAR0           0x040
+/* write cache cfg: snoop, no allocate, data & BD coherent */
+#define  ENETC_SICAR_WR_CFG    0x6767
+/* read cache cfg: coherent copy, look up, don't alloc in cache */
+#define  ENETC_SICAR_RD_CFG    0x27270000
+#define ENETC_SIROCT           0x300
+#define ENETC_SIRFRM           0x308
+#define ENETC_SITOCT           0x320
+#define ENETC_SITFRM           0x328
+
+/* Rx/Tx Buffer Descriptor Ring registers */
+enum enetc_bdr_type {TX, RX};
+#define ENETC_BDR(type, n, off)        (0x8000 + (type) * 0x100 + (n) * 0x200 + (off))
+#define ENETC_BDR_IDX_MASK     0xffff
+
+/* Rx BDR reg offsets */
+#define ENETC_RBMR             0x00
+#define  ENETC_RBMR_EN         BIT(31)
+#define ENETC_RBBSR            0x08
+/* initial consumer index for Rx BDR */
+#define ENETC_RBCIR            0x0c
+#define ENETC_RBBAR0           0x10
+#define ENETC_RBBAR1           0x14
+#define ENETC_RBPIR            0x18
+#define ENETC_RBLENR           0x20
+
+/* Tx BDR reg offsets */
+#define ENETC_TBMR             0x00
+#define  ENETC_TBMR_EN         BIT(31)
+#define ENETC_TBBAR0           0x10
+#define ENETC_TBBAR1           0x14
+#define ENETC_TBPIR            0x18
+#define ENETC_TBCIR            0x1c
+#define ENETC_TBLENR           0x20
+
+/* Port registers offset */
+#define ENETC_PORT_REGS_OFF            0x10000
+
+/* Port registers */
+#define ENETC_PMR                      0x0000
+#define  ENETC_PMR_SI0_EN              BIT(16)
+#define ENETC_PSIPMMR                  0x0018
+#define ENETC_PSIPMAR0                 0x0100
+#define ENETC_PSIPMAR1                 0x0104
+#define ENETC_PSICFGR(n)               (0x0940 + (n) * 0x10)
+#define  ENETC_PSICFGR_SET_TXBDR(val)  ((val) & 0xff)
+#define  ENETC_PSICFGR_SET_RXBDR(val)  (((val) & 0xff) << 16)
+/* MAC configuration */
+#define ENETC_PM_CC                    0x8008
+#define  ENETC_PM_CC_DEFAULT           0x0810
+#define  ENETC_PM_CC_RX_TX_EN          0x8813
+#define ENETC_PM_MAXFRM                        0x8014
+#define  ENETC_RX_MAXFRM_SIZE          PKTSIZE_ALIGN
+
+/* buffer descriptors count must be multiple of 8 and aligned to 128 bytes */
+#define ENETC_BD_CNT           CONFIG_SYS_RX_ETH_BUFFER
+#define ENETC_BD_ALIGN         128
+
+/* single pair of Rx/Tx rings */
+#define ENETC_RX_BDR_CNT       1
+#define ENETC_TX_BDR_CNT       1
+#define ENETC_RX_BDR_ID                0
+#define ENETC_TX_BDR_ID                0
+
+/* Tx buffer descriptor */
+struct enetc_tx_bd {
+       __le64 addr;
+       __le16 buf_len;
+       __le16 frm_len;
+       __le16 err_csum;
+       __le16 flags;
+};
+
+#define ENETC_TXBD_FLAGS_F     BIT(15)
+#define ENETC_POLL_TRIES       32000
+
+/* Rx buffer descriptor */
+union enetc_rx_bd {
+       /* SW provided BD format */
+       struct {
+               __le64 addr;
+               u8 reserved[8];
+       } w;
+
+       /* ENETC returned BD format */
+       struct {
+               __le16 inet_csum;
+               __le16 parse_summary;
+               __le32 rss_hash;
+               __le16 buf_len;
+               __le16 vlan_opt;
+               union {
+                       struct {
+                               __le16 flags;
+                               __le16 error;
+                       };
+                       __le32 lstatus;
+               };
+       } r;
+};
+
+#define ENETC_RXBD_STATUS_R(status)            (((status) >> 30) & 0x1)
+#define ENETC_RXBD_STATUS_F(status)            (((status) >> 31) & 0x1)
+#define ENETC_RXBD_STATUS_ERRORS(status)       (((status) >> 16) & 0xff)
+#define ENETC_RXBD_STATUS(flags)               ((flags) << 16)
+
+/* Tx/Rx ring info */
+struct bd_ring {
+       void *cons_idx;
+       void *prod_idx;
+       /* next BD index to use */
+       int next_prod_idx;
+       int next_cons_idx;
+       int bd_count;
+};
+
+/* ENETC private structure */
+struct enetc_priv {
+       struct enetc_tx_bd *enetc_txbd;
+       union enetc_rx_bd *enetc_rxbd;
+
+       void *regs_base; /* base ENETC registers */
+       void *port_regs; /* base ENETC port registers */
+
+       /* Rx/Tx buffer descriptor rings info */
+       struct bd_ring tx_bdr;
+       struct bd_ring rx_bdr;
+};
+
+/* register accessors */
+#define enetc_read_reg(x)      readl((x))
+#define enetc_write_reg(x, val)        writel((val), (x))
+#define enetc_read(priv, off)  enetc_read_reg((priv)->regs_base + (off))
+#define enetc_write(priv, off, v) \
+                       enetc_write_reg((priv)->regs_base + (off), v)
+
+/* port register accessors */
+#define enetc_port_regs(priv, off) ((priv)->port_regs + (off))
+#define enetc_read_port(priv, off) \
+                       enetc_read_reg(enetc_port_regs((priv), (off)))
+#define enetc_write_port(priv, off, v) \
+                       enetc_write_reg(enetc_port_regs((priv), (off)), v)
+
+/* BDR register accessors, see ENETC_BDR() */
+#define enetc_bdr_read(priv, t, n, off) \
+                       enetc_read(priv, ENETC_BDR(t, n, off))
+#define enetc_bdr_write(priv, t, n, off, val) \
+                       enetc_write(priv, ENETC_BDR(t, n, off), val)
+
+#endif /* _ENETC_H */