On Mon, Feb 24, 2025 at 05:56:18PM +0000, Simon Horman wrote: > On Wed, Feb 19, 2025 at 08:32:57PM +0100, Peter Hilber wrote: > > ... > > > +/** > > + * viortc_ptp_gettimex64() - PTP clock gettimex64 op > > + * > > Hi Peter, > > Tooling recognises this as a kernel doc, and complains > that there is no documentation present for the function's > parameters: ptp, ts, and sts. > > Flagged by W=1 builds. >
Thanks, I will change the offending documentation to non kernel-doc. I was not aware that these warnings are always considered a problem. Peter > > + * Context: Process context. > > + */ > > +static int viortc_ptp_gettimex64(struct ptp_clock_info *ptp, > > + struct timespec64 *ts, > > + struct ptp_system_timestamp *sts) > > ...
