regex/octeontx2: fix driver name
diff mbox series

Message ID 20201023174131.99585-1-thomas@monjalon.net
State Accepted
Delegated to: Thomas Monjalon
Headers show
Series
  • regex/octeontx2: fix driver name
Related show

Checks

Context Check Description
ci/iol-mellanox-Performance success Performance Testing PASS
ci/travis-robot success Travis build: passed
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-testing success Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/Intel-compilation success Compilation OK
ci/iol-broadcom-Performance success Performance Testing PASS
ci/checkpatch success coding style OK

Commit Message

Thomas Monjalon Oct. 23, 2020, 5:41 p.m. UTC
Following the recent alignment of all driver names,
this new driver get unaligned:
	librte_regex_octeontx2_regex.so

The 'fmt_name' must be "octeontx2_regex", and if not provided,
is taken from the 'name' variable.
But the variable 'name' should not be overwritten,
to keep the automatic value from the directory name.

The library name will be composed of the class directory
and the driver directory name:
	librte_regex_octeontx2.so

Reported-by: Christian Ehrhardt <christian.ehrhardt@canonical.com>
Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
Acked-by: David Marchand <david.marchand@redhat.com>
---
 drivers/regex/octeontx2/meson.build | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Monjalon Nov. 3, 2020, 10:51 a.m. UTC | #1
23/10/2020 19:41, Thomas Monjalon:
> Following the recent alignment of all driver names,
> this new driver get unaligned:
> 	librte_regex_octeontx2_regex.so
> 
> The 'fmt_name' must be "octeontx2_regex", and if not provided,
> is taken from the 'name' variable.
> But the variable 'name' should not be overwritten,
> to keep the automatic value from the directory name.
> 
> The library name will be composed of the class directory
> and the driver directory name:
> 	librte_regex_octeontx2.so
> 
> Reported-by: Christian Ehrhardt <christian.ehrhardt@canonical.com>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> Acked-by: David Marchand <david.marchand@redhat.com>

Applied

Patch
diff mbox series

diff --git a/drivers/regex/octeontx2/meson.build b/drivers/regex/octeontx2/meson.build
index 398a981c07..aada0b5601 100644
--- a/drivers/regex/octeontx2/meson.build
+++ b/drivers/regex/octeontx2/meson.build
@@ -38,7 +38,7 @@  foreach flag: extra_flags
 	endif
 endforeach
 
-name = 'octeontx2_regex'
+fmt_name = 'octeontx2_regex'
 deps += ['bus_pci', 'common_octeontx2', 'regexdev']
 
 includes += include_directories('../../common/octeontx2')