[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <f04e466b-8b8b-466e-a67c-d7fbfea2fbfd@linux.dev>
Date: Mon, 29 Dec 2025 19:13:49 +0000
From: Vadim Fedorenko <vadim.fedorenko@...ux.dev>
To: Tinsae Tadesse <tinsaetadesse2015@...il.com>, richardcochran@...il.com,
andrew+netdev@...n.ch, davem@...emloft.net, edumazet@...gle.com,
kuba@...nel.org, pabeni@...hat.com
Cc: netdev@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Fix PTP driver warnings by removing settime64 check
On 29/12/2025 17:32, Tinsae Tadesse wrote:
> Signed-off-by: Tinsae Tadesse <tinsaetadesse2015@...il.com>
> ---
> drivers/ptp/ptp_clock.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/ptp/ptp_clock.c b/drivers/ptp/ptp_clock.c
> index b0e167c0b3eb..5374b3e9ad15 100644
> --- a/drivers/ptp/ptp_clock.c
> +++ b/drivers/ptp/ptp_clock.c
> @@ -323,8 +323,7 @@ struct ptp_clock *ptp_clock_register(struct ptp_clock_info *info,
> size_t size;
>
> if (WARN_ON_ONCE(info->n_alarm > PTP_MAX_ALARMS ||
> - (!info->gettimex64 && !info->gettime64) ||
> - !info->settime64))
> + (!info->gettimex64 && !info->gettime64)))
> return ERR_PTR(-EINVAL);
>
> /* Initialize a clock structure. */
Could you please tell us a bit more about reasoning for this patch?
Powered by blists - more mailing lists