Thread (28 messages) 28 messages, 5 authors, 2025-11-12

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.c
b/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(struct
work_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>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help