Re: [PATCH v1 1/1] scsi: lpfc: Switch to use %ptTs
From: Andy Shevchenko
Date: Thu Feb 06 2025 - 13:58:34 EST
On Thu, Feb 06, 2025 at 05:52:16PM +0100, Thomas Weißschuh wrote:
> On Thu, Feb 06, 2025 at 05:58:22PM +0200, Andy Shevchenko wrote:
...
> > lpfc_printf_log(phba, KERN_INFO, LOG_CGN_MGMT,
> > - "2646 Updated CMF timestamp : "
> > - "%u/%u/%u %u:%u:%u\n",
> > - ts->day, ts->month,
> > - ts->year, ts->hour,
> > - ts->minute, ts->second);
> > + "2646 Updated CMF timestamp : %ptTs\n", cur_time);
>
> All %p<FOO> arguments need to be addresses.
> Also %ptT wants a time64_t, not a 'struct timespec64'.
> It would work by chance because tv_sec is the first member and time64_t.
>
> Correct: "&cur_time.tv_sec".
Indeed, thanks!
--
With Best Regards,
Andy Shevchenko