From 1c53e6845952fe45e7353a0bfb111e4bc92c02fb Mon Sep 17 00:00:00 2001 From: Alex Sergeev Date: Mon, 20 Sep 2021 23:59:24 -0700 Subject: [PATCH] drivers: ethernet: mcux: Bugfix ptp_clock_adjust ptp_clock_adjust() API call for mcux driver has a bug where increment gets compared with an unsigned int, causing it to always return -EINVAL. Signed-off-by: Alex Sergeev --- drivers/ethernet/eth_mcux.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/ethernet/eth_mcux.c b/drivers/ethernet/eth_mcux.c index 87249555f9c..084ed12e90a 100644 --- a/drivers/ethernet/eth_mcux.c +++ b/drivers/ethernet/eth_mcux.c @@ -1456,7 +1456,8 @@ static int ptp_clock_mcux_adjust(const struct device *dev, int increment) ARG_UNUSED(dev); - if ((increment <= -NSEC_PER_SEC) || (increment >= NSEC_PER_SEC)) { + if ((increment <= (int32_t)(-NSEC_PER_SEC)) || + (increment >= (int32_t)NSEC_PER_SEC)) { ret = -EINVAL; } else { key = irq_lock();