diff options
| author |  Aya Levin <ayal@nvidia.com>
| 2021-03-01 17:54:21 +0200 |
|---|---|---|
| committer |  Saeed Mahameed <saeedm@nvidia.com>
| 2021-03-10 11:01:54 -0800 |
| commit | 55affa97d6758b6aeab0bc68f4884c4b5a6828af (patch) | |
| tree | 5238c8086efd0618ff2431f3d3ce520b504d8612 /drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c | |
| parent | net/mlx5e: Don't match on Geneve options in case option masks are all zero (diff) | |
| download | linux-55affa97d6758b6aeab0bc68f4884c4b5a6828af.tar.gz linux-55affa97d6758b6aeab0bc68f4884c4b5a6828af.tar.bz2 linux-55affa97d6758b6aeab0bc68f4884c4b5a6828af.zip | |
net/mlx5: Fix turn-off PPS command
Fix a bug of uninitialized pin index when trying to turn off PPS out.
Fixes: de19cd6cc977 ("net/mlx5: Move some PPS logic into helper functions")
Signed-off-by: Aya Levin <ayal@nvidia.com>
Reviewed-by: Eran Ben Elisha <eranbe@nvidia.com>
Signed-off-by: Saeed Mahameed <saeedm@nvidia.com>
Diffstat (limited to 'drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c')
| -rw-r--r-- | drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c b/drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c index b0e129d0f6d8..1e7f26b240de 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c @@ -495,15 +495,15 @@ static int mlx5_perout_configure(struct ptp_clock_info *ptp, return -EINVAL; field_select = MLX5_MTPPS_FS_ENABLE; + pin = ptp_find_pin(clock->ptp, PTP_PF_PEROUT, rq->perout.index); + if (pin < 0) + return -EBUSY; + if (on) { bool rt_mode = mlx5_real_time_mode(mdev); u32 nsec; s64 sec; - pin = ptp_find_pin(clock->ptp, PTP_PF_PEROUT, rq->perout.index); - if (pin < 0) - return -EBUSY; - pin_mode = MLX5_PIN_MODE_OUT; pattern = MLX5_OUT_PATTERN_PERIODIC; ts.tv_sec = rq->perout.period.sec; |