[PATCH v4 3/6] common/cnxk: remove check around pthread_mutex_init()

Ariel Otilibili otilibil at eurecom.fr
Tue Jan 14 08:50:30 CET 2025


> pthread_mutex_init always returns 0. The other mutex functions
> return 0 on success and a non-zero error code on error.

Link: https://man7.org/linux/man-pages/man3/pthread_mutex_lock.3.html
Bugzilla ID: 1586
Cc: Nithin Dabilpuram <ndabilpuram at marvell.com>
Cc: Kiran Kumar K <kirankumark at marvell.com>
Cc: Sunil Kumar Kori <skori at marvell.com>
Cc: Satha Rao <skoteshwar at marvell.com>
Cc: Harman Kalra <hkalra at marvell.com>
Signed-off-by: Ariel Otilibili <otilibil at eurecom.fr>
Acked-by: Stephen Hemminger <stephen at networkplumber.org>
---
 drivers/common/cnxk/roc_bphy_cgx.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/common/cnxk/roc_bphy_cgx.c b/drivers/common/cnxk/roc_bphy_cgx.c
index 4f43605e1031..882cf654743b 100644
--- a/drivers/common/cnxk/roc_bphy_cgx.c
+++ b/drivers/common/cnxk/roc_bphy_cgx.c
@@ -189,14 +189,11 @@ int
 roc_bphy_cgx_dev_init(struct roc_bphy_cgx *roc_cgx)
 {
 	uint64_t val;
-	int ret;
 
 	if (!roc_cgx || !roc_cgx->bar0_va || !roc_cgx->bar0_pa)
 		return -EINVAL;
 
-	ret = pthread_mutex_init(&roc_cgx->lock, NULL);
-	if (ret)
-		return ret;
+	pthread_mutex_init(&roc_cgx->lock, NULL);
 
 	val = roc_bphy_cgx_read(roc_cgx, 0, CGX_CMRX_RX_LMACS);
 	val = FIELD_GET(CGX_CMRX_RX_LMACS_LMACS, val);
-- 
2.30.2



More information about the dev mailing list