regex/octeontx2: fix driver name

Message ID 20201023174131.99585-1-thomas@monjalon.net (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series regex/octeontx2: fix driver name |

Checks

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

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 --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')