On 10/12/2017 10:35 AM, Ajit Khaparde wrote:

This patch updates the HWRM APi to version 1.8.2
This patch is based on an autogenerated file.
Almost all the changes in the PMD are macro name changes.

Signed-off-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
---
 drivers/net/bnxt/bnxt.h                |     2 -
 drivers/net/bnxt/bnxt_ethdev.c         |     2 +-
 drivers/net/bnxt/bnxt_hwrm.c           |     2 +-
 drivers/net/bnxt/bnxt_hwrm.h           |     2 +
 drivers/net/bnxt/bnxt_irq.c            |     2 +-
 drivers/net/bnxt/bnxt_rxr.c            |     4 +-
 drivers/net/bnxt/bnxt_txr.c            |     2 +-
 drivers/net/bnxt/hsi_struct_def_dpdk.h | 30829 ++++++++++++++++++++++++-------

Only 30K lines of code :)

 8 files changed, 24374 insertions(+), 6471 deletions(-)

There are some replacement in hsi_struct_def_dpdk.h, but also there are lots of
new additions.
Are those new additions really used within the driver?

What do you think adding structures that are only used by driver?
I am aware code is auto-generated, but it may be possible to add some #ifdefs to
the parts DPDK uses and auto-generate file specific for DPDK?

Thanks,
ferruh