Re: [dpdk-dev] [v4, 1/2] common/cnxk: add telemetry endpoints to npa
From: Bruce Richardson <hidden>
Date: 2021-08-26 18:06:46
On Thu, Aug 26, 2021 at 10:45:10PM +0530, Gowrishankar Muthukrishnan wrote:
Add telemetry endpoints to npa. Signed-off-by: Gowrishankar Muthukrishnan <redacted> --- drivers/common/cnxk/cnxk_telemetry.h | 26 +++ drivers/common/cnxk/cnxk_telemetry_npa.c | 227 +++++++++++++++++++++++ drivers/common/cnxk/meson.build | 4 + drivers/common/cnxk/roc_platform.h | 9 + 4 files changed, 266 insertions(+) create mode 100644 drivers/common/cnxk/cnxk_telemetry.h create mode 100644 drivers/common/cnxk/cnxk_telemetry_npa.c
<snip>
quoted hunk ↗ jump to hunk
diff --git a/drivers/common/cnxk/meson.build b/drivers/common/cnxk/meson.build index 6a7849f31c..3cc9a2ff37 100644 --- a/drivers/common/cnxk/meson.build +++ b/drivers/common/cnxk/meson.build@@ -60,5 +60,9 @@ sources = files( # Security common code sources += files('cnxk_security.c') +# Telemetry common code +sources += files('cnxk_telemetry_npa.c') + includes += include_directories('../../bus/pci') includes += include_directories('../../../lib/net') +includes += include_directories('../../../lib/telemetry')
Rather than giving the include directories, I'd recommend using "deps" variable to specify a direct dependency on those libraries. This should probably be: deps += ['bus_pci', 'net', 'telemetry'] Using these as dependencies means that the header paths will be added automatically, and the libraries linked against too if any functions from those are used by the code in this lib. Regards, /Bruce