Re: [PATCH] Fix PTP driver warnings by removing settime64 check

[TINSAE TADESSE GUTEMA]

On Tue, Dec 30, 2025 at 1:59 PM Andrew Lunn <andrew@xxxxxxx> wrote:
>
> > This patch resolves warnings triggered when either gettimex64 or settime64 are
> > NULL.
> > The CONFIG_PTP_1588_CLOCK=y option enables PTP support for all Ethernet
> > devices.
> > In systems with Intel-based network devices, both the iwlwifi and e1000e
> > modules attempt to register clocks, resulting in calls to ptp_clock_register in
> > drivers/ptp/ptp_clock.c that produce warnings if any function pointers are
> > uninitialized.
> >
> > Without this patch, the following warning is logged during registration:
> >
> >     info->n_alarm > PTP_MAX_ALARMS || (!info->gettimex64 && !info->gettime64) |
> >     | !info->settime64
> >     WARNING: drivers/ptp/ptp_clock.c:325 at ptp_clock_register+0x54/0xb70, CPU#
> >     2: NetworkManager/1102
> >     ...
> >     iwlwifi 0000:00:14.3: Failed to register PHC clock (-22)
>
> It seems this patch is no longer needed. But FYI, this explanation
> should of been in the commit message. The commit message should
> explain why a change is needed.
>
>
>     Andrew
>
> ---
> pw-bot: cr

Hi Andrew,

Your suggestion is well noted, thank you!