On 7/2/2018 1:20 PM, Ferruh Yigit wrote: > On 6/28/2018 9:15 PM, Ajit Khaparde wrote: >> From: Scott Branden <scott.bran...@broadcom.com> >> >> Move check_zero_bytes into new bnxt_util.h file. >> >> Signed-off-by: Scott Branden <scott.bran...@broadcom.com> >> Reviewed-by: Ajit Khaparde <ajit.khapa...@broadcom.com> >> --- >> drivers/net/bnxt/Makefile | 1 + >> drivers/net/bnxt/bnxt_ethdev.c | 1 + >> drivers/net/bnxt/bnxt_filter.c | 9 --------- >> drivers/net/bnxt/bnxt_filter.h | 1 - >> drivers/net/bnxt/bnxt_util.c | 18 ++++++++++++++++++ >> drivers/net/bnxt/bnxt_util.h | 11 +++++++++++ >> 6 files changed, 31 insertions(+), 10 deletions(-) >> create mode 100644 drivers/net/bnxt/bnxt_util.c >> create mode 100644 drivers/net/bnxt/bnxt_util.h >> >> diff --git a/drivers/net/bnxt/Makefile b/drivers/net/bnxt/Makefile >> index fd0cb5235..80db03ea8 100644 >> --- a/drivers/net/bnxt/Makefile >> +++ b/drivers/net/bnxt/Makefile >> @@ -38,6 +38,7 @@ SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_txq.c >> SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_txr.c >> SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_vnic.c >> SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_irq.c >> +SRCS-$(CONFIG_RTE_LIBRTE_BNXT_PMD) += bnxt_util.c > > This breaks the meson build and similar change required for meson, same with > bnxt_flow.c in other patch, if there is no other issue I can fix this while > merging.
This patch also breaks the Makefile build, 'bnxt_filter.c' requires: #include "bnxt_util.h"
