rawdev: fix device naming

Message ID 20230724133646.1385487-1-tduszynski@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: David Marchand
Headers
Series rawdev: fix device naming |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/Intel-compilation success Compilation OK
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-abi-testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-broadcom-Performance success Performance Testing PASS
ci/iol-broadcom-Functional success Functional Testing PASS
ci/iol-aarch-unit-testing success Testing PASS
ci/iol-unit-testing success Testing PASS
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/intel-Functional success Functional PASS
ci/intel-Testing success Testing PASS

Commit Message

Tomasz Duszynski July 24, 2023, 1:36 p.m. UTC
  Use proper naming when dealing with a raw device.

Fixes: c88b3f2558ed ("rawdev: introduce raw device library")
Cc: shreyansh.jain@nxp.com
Cc: stable@dpdk.org

Signed-off-by: Tomasz Duszynski <tduszynski@marvell.com>
---
 lib/rawdev/rte_rawdev.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
  

Comments

Hemant Agrawal July 24, 2023, 1:43 p.m. UTC | #1
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

> -----Original Message-----
> From: Tomasz Duszynski <tduszynski@marvell.com>
> Sent: Monday, July 24, 2023 7:07 PM
> To: dev@dpdk.org; Sachin Saxena <sachin.saxena@nxp.com>; Hemant
> Agrawal <hemant.agrawal@nxp.com>; Shreyansh Jain
> <shreyansh.jain@nxp.com>
> Cc: jerinj@marvell.com; thomas@monjalon.net; Tomasz Duszynski
> <tduszynski@marvell.com>; stable@dpdk.org
> Subject: [PATCH] rawdev: fix device naming
> Importance: High
> 
> Use proper naming when dealing with a raw device.
> 
> Fixes: c88b3f2558ed ("rawdev: introduce raw device library")
> Cc: shreyansh.jain@nxp.com
> Cc: stable@dpdk.org
> 
> Signed-off-by: Tomasz Duszynski <tduszynski@marvell.com>
> ---
>  lib/rawdev/rte_rawdev.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/lib/rawdev/rte_rawdev.c b/lib/rawdev/rte_rawdev.c index
> dacaa60e22..474bdc9540 100644
> --- a/lib/rawdev/rte_rawdev.c
> +++ b/lib/rawdev/rte_rawdev.c
> @@ -505,8 +505,7 @@ rte_rawdev_pmd_allocate(const char *name, size_t
> dev_priv_size, int socket_id)
>  	uint16_t dev_id;
> 
>  	if (rte_rawdev_pmd_get_named_dev(name) != NULL) {
> -		RTE_RDEV_ERR("Event device with name %s already
> allocated!",
> -			     name);
> +		RTE_RDEV_ERR("Raw device with name %s already
> allocated!", name);
>  		return NULL;
>  	}
> 
> --
> 2.34.1
  
David Marchand Oct. 2, 2023, 8:34 a.m. UTC | #2
On Mon, Jul 24, 2023 at 3:37 PM Tomasz Duszynski <tduszynski@marvell.com> wrote:
>
> Use proper naming when dealing with a raw device.
>
> Fixes: c88b3f2558ed ("rawdev: introduce raw device library")
> Cc: stable@dpdk.org
>
> Signed-off-by: Tomasz Duszynski <tduszynski@marvell.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

Applied, thanks.
  

Patch

diff --git a/lib/rawdev/rte_rawdev.c b/lib/rawdev/rte_rawdev.c
index dacaa60e22..474bdc9540 100644
--- a/lib/rawdev/rte_rawdev.c
+++ b/lib/rawdev/rte_rawdev.c
@@ -505,8 +505,7 @@  rte_rawdev_pmd_allocate(const char *name, size_t dev_priv_size, int socket_id)
 	uint16_t dev_id;
 
 	if (rte_rawdev_pmd_get_named_dev(name) != NULL) {
-		RTE_RDEV_ERR("Event device with name %s already allocated!",
-			     name);
+		RTE_RDEV_ERR("Raw device with name %s already allocated!", name);
 		return NULL;
 	}