[PATCH v1] common/cnxk: fix initialization of timer LF count

[Pavan Nikhilesh Bhagavatula]

> -----Original Message-----
> From: Shijith Thotton <sthotton at marvell.com>
> Sent: Tuesday, March 14, 2023 9:17 AM
> To: Jerin Jacob Kollanukkaran <jerinj at marvell.com>
> Cc: Shijith Thotton <sthotton at marvell.com>; dev at dpdk.org; Pavan
> Nikhilesh Bhagavatula <pbhagavatula at marvell.com>
> Subject: [PATCH v1] common/cnxk: fix initialization of timer LF count
> 
> 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 <sthotton at marvell.com>

Acked-by: Pavan Nikhilesh <pbhagavatula at 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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: winmail.dat
Type: application/ms-tnef
Size: 23601 bytes
Desc: not available
URL: <http://mails.dpdk.org/archives/dev/attachments/20230314/5a4a01d8/attachment-0001.bin>