Assignment of nb_lfs variable with the available number of timer LFs was
not happening in the success case after the addition of the new label
"done". This patch fixes the same by moving the assignment under the
label.

Fixes: 44a9307c0908 ("common/cnxk: support mailbox locking")

Signed-off-by: Shijith Thotton <sthot...@marvell.com>
---
 drivers/common/cnxk/roc_tim.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/common/cnxk/roc_tim.c b/drivers/common/cnxk/roc_tim.c
index 6f256c60fa..f8607b2852 100644
--- a/drivers/common/cnxk/roc_tim.c
+++ b/drivers/common/cnxk/roc_tim.c
@@ -394,11 +394,11 @@ roc_tim_init(struct roc_tim *roc_tim)
        } else {
                goto done;
        }
-       roc_tim->nb_lfs = nb_lfs;
 
 fail:
        mbox_put(dev->mbox);
 done:
+       roc_tim->nb_lfs = nb_lfs;
        return nb_lfs;
 }
 
-- 
2.25.1

Reply via email to