RE: [Intel-wired-lan] [PATCH v2 09/21] e1000e: Switch to use %ptSp
From: Loktionov, Aleksandr <hidden>
Date: 2025-11-12 13:08:02
Also in:
amd-gfx, ceph-devel, dri-devel, intel-wired-lan, intel-xe, linux-arm-msm, linux-doc, linux-media, linux-mmc, linux-pci, linux-s390, linux-scsi, linux-staging, linux-trace-kernel, lkml
quoted hunk ↗ jump to hunk
-----Original Message----- From: Intel-wired-lan <redacted> On Behalf Of Andy Shevchenko Sent: Tuesday, November 11, 2025 1:20 PM To: Corey Minyard <corey@minyard.net>; Christian König [off-list ref]; Dr. David Alan Gilbert [off-list ref]; Alex Deucher [off-list ref]; Thomas Zimmermann [off-list ref]; Dmitry Baryshkov [off-list ref]; Rob Clark [off-list ref]; Brost, Matthew [off-list ref]; Ulf Hansson [off-list ref]; Andy Shevchenko [off-list ref]; Lifshits, Vitaly [off-list ref]; Manivannan Sadhasivam [off-list ref]; Niklas Cassel [off-list ref]; Calvin Owens [off-list ref]; Vadim Fedorenko [off-list ref]; Sagi Maimon [off-list ref]; Martin K. Petersen [off-list ref]; Karan Tilak Kumar [off-list ref]; Hans Verkuil [off-list ref]; Casey Schaufler [off-list ref]; Steven Rostedt [off-list ref]; Petr Mladek [off-list ref]; Viacheslav Dubeyko [off-list ref]; Max Kellermann [off-list ref]; linux-doc@vger.kernel.org; linux-kernel@vger.kernel.org; openipmi- developer@lists.sourceforge.net; linux-media@vger.kernel.org; dri- devel@lists.freedesktop.org; linaro-mm-sig@lists.linaro.org; amd- gfx@lists.freedesktop.org; linux-arm-msm@vger.kernel.org; freedreno@lists.freedesktop.org; intel-xe@lists.freedesktop.org; linux-mmc@vger.kernel.org; netdev@vger.kernel.org; intel-wired- lan@lists.osuosl.org; linux-pci@vger.kernel.org; linux- s390@vger.kernel.org; linux-scsi@vger.kernel.org; linux- staging@lists.linux.dev; ceph-devel@vger.kernel.org; linux-trace- kernel@vger.kernel.org Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk>; Sergey Senozhatsky [off-list ref]; Jonathan Corbet [off-list ref]; Sumit Semwal [off-list ref]; Gustavo Padovan [off-list ref]; David Airlie [off-list ref]; Simona Vetter [off-list ref]; Maarten Lankhorst [off-list ref]; Maxime Ripard [off-list ref]; Dmitry Baryshkov [off-list ref]; Abhinav Kumar [off-list ref]; Jessica Zhang [off-list ref]; Sean Paul [off-list ref]; Marijn Suijten [off-list ref]; Konrad Dybcio [off-list ref]; De Marchi, Lucas [off-list ref]; Thomas Hellström [off-list ref]; Vivi, Rodrigo [off-list ref]; Vladimir Oltean [off-list ref]; Andrew Lunn [off-list ref]; David S. Miller [off-list ref]; Eric Dumazet [off-list ref]; Jakub Kicinski [off-list ref]; Paolo Abeni [off-list ref]; Nguyen, Anthony L [off-list ref]; Kitszel, Przemyslaw [off-list ref]; Krzysztof Wilczyński [off-list ref]; Kishon Vijay Abraham I [off-list ref]; Bjorn Helgaas [off-list ref]; Rodolfo Giometti [off-list ref]; Richard Cochran [off-list ref]; Jonathan Lemon [off-list ref]; Stefan Haberland [off-list ref]; Jan Hoeppner [off-list ref]; Heiko Carstens [off-list ref]; Vasily Gorbik [off-list ref]; Alexander Gordeev [off-list ref]; Christian Borntraeger [off-list ref]; Sven Schnelle [off-list ref]; Satish Kharat [off-list ref]; Baddela, Sesidhar [off-list ref]; James E.J. Bottomley [off-list ref]; Mauro Carvalho Chehab [off-list ref]; Greg Kroah-Hartman [off-list ref]; Xiubo Li [off-list ref]; Ilya Dryomov [off-list ref]; Masami Hiramatsu [off-list ref]; Mathieu Desnoyers [off-list ref]; Andrew Morton <akpm@linux- foundation.org> Subject: [Intel-wired-lan] [PATCH v2 09/21] e1000e: Switch to use %ptSp Use %ptSp instead of open coded variants to print content of struct timespec64 in human readable format. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- drivers/net/ethernet/intel/e1000e/ptp.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)diff --git a/drivers/net/ethernet/intel/e1000e/ptp.cb/drivers/net/ethernet/intel/e1000e/ptp.c index ea3c3eb2ef20..ec39e35f3857 100644--- a/drivers/net/ethernet/intel/e1000e/ptp.c +++ b/drivers/net/ethernet/intel/e1000e/ptp.c@@ -229,14 +229,11 @@ static void e1000e_systim_overflow_work(structwork_struct *work) systim_overflow_work.work); struct e1000_hw *hw = &adapter->hw; struct timespec64 ts; - u64 ns; /* Update the timecounter */ - ns = timecounter_read(&adapter->tc); + ts = ns_to_timespec64(timecounter_read(&adapter->tc)); - ts = ns_to_timespec64(ns); - e_dbg("SYSTIM overflow check at %lld.%09lu\n", - (long long) ts.tv_sec, ts.tv_nsec); + e_dbg("SYSTIM overflow check at %ptSp\n", &ts); schedule_delayed_work(&adapter->systim_overflow_work, E1000_SYSTIM_OVERFLOW_PERIOD); -- 2.50.1
Reviewed-by: Aleksandr Loktionov <redacted>