On 6/25/15, 6:45 AM, "dev on behalf of Stephen Hemminger" <dev-bounces at dpdk.org on behalf of stephen at networkplumber.org> wrote:
>On Wed, 17 Jun 2015 18:22:13 -0400 >Liang-Min Larry Wang <liang-min.wang at intel.com> wrote: > >> + >> +static reg_info ixgbe_regs_general[] = { >> + {IXGBE_CTRL, 1, 1, "IXGBE_CTRL"}, >> + {IXGBE_STATUS, 1, 1, "IXGBE_STATUS"}, >> + {IXGBE_CTRL_EXT, 1, 1, "IXGBE_CTRL_EXT"}, >> + {IXGBE_ESDP, 1, 1, "IXGBE_ESDP"}, >> + {IXGBE_EODSDP, 1, 1, "IXGBE_EODSDP"}, >> + {IXGBE_LEDCTL, 1, 1, "IXGBE_LEDCTL"}, >> + {IXGBE_FRTIMER, 1, 1, "IXGBE_FRTIMER"}, >> + {IXGBE_TCPTIMER, 1, 1, "IXGBE_TCPTIMER"}, >> + {0, 0, 0, ""} >> +}; >> + >> +static reg_info ixgbevf_regs_general[] = { >> + {IXGBE_CTRL, 1, 1, "IXGBE_CTRL"}, >> + {IXGBE_STATUS, 1, 1, "IXGBE_STATUS"}, >> + {IXGBE_VFLINKS, 1, 1, "IXGBE_VFLINKS"}, >> + {IXGBE_FRTIMER, 1, 1, "IXGBE_FRTIMER"}, >> + {IXGBE_VFMAILBOX, 1, 1, "IXGBE_VFMAILBOX"}, >> + {IXGBE_VFMBMEM, 16, 4, "IXGBE_VFMBMEM"}, >> + {IXGBE_VFRXMEMWRAP, 1, 1, "IXGBE_VFRXMEMWRAP"}, >> + {0, 0, 0, ""} >> +}; >> + > >All these tables should be const >and API may need to change. Good point, I have sent a patch to Larry for inclusion in the next version. Thanks ? Andy