RE: [Intel-wired-lan] [PATCH v2 14/21] net: dsa: sja1105: Switch to use %ptSp
From: Loktionov, Aleksandr <hidden>
Date: 2025-11-12 13:09:27
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 14/21] net: dsa: sja1105: 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/dsa/sja1105/sja1105_tas.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-)diff --git a/drivers/net/dsa/sja1105/sja1105_tas.cb/drivers/net/dsa/sja1105/sja1105_tas.c index d7818710bc02..d5949d2c3e71 100644--- a/drivers/net/dsa/sja1105/sja1105_tas.c +++ b/drivers/net/dsa/sja1105/sja1105_tas.c@@ -775,9 +775,8 @@ static void sja1105_tas_state_machine(structwork_struct *work) base_time_ts = ns_to_timespec64(base_time); now_ts = ns_to_timespec64(now); - dev_dbg(ds->dev, "OPER base time %lld.%09ld (now %lld.%09ld)\n", - base_time_ts.tv_sec, base_time_ts.tv_nsec, - now_ts.tv_sec, now_ts.tv_nsec); + dev_dbg(ds->dev, "OPER base time %ptSp (now %ptSp)\n", + &base_time_ts, &now_ts); break;@@ -798,8 +797,7 @@ static void sja1105_tas_state_machine(structwork_struct *work) if (now < tas_data->oper_base_time) { /* TAS has not started yet */ diff = ns_to_timespec64(tas_data->oper_base_time - now); - dev_dbg(ds->dev, "time to start: [%lld.%09ld]", - diff.tv_sec, diff.tv_nsec); + dev_dbg(ds->dev, "time to start: [%ptSp]", &diff); break; } -- 2.50.1
Reviewed-by: Aleksandr Loktionov <redacted>