Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751467AbdFHRQr (ORCPT ); Thu, 8 Jun 2017 13:16:47 -0400 Received: from mail-it0-f66.google.com ([209.85.214.66]:35059 "EHLO mail-it0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750725AbdFHRQq (ORCPT ); Thu, 8 Jun 2017 13:16:46 -0400 MIME-Version: 1.0 In-Reply-To: <20170608134811.60786-26-andriy.shevchenko@linux.intel.com> References: <20170608134811.60786-1-andriy.shevchenko@linux.intel.com> <20170608134811.60786-26-andriy.shevchenko@linux.intel.com> From: "Rafael J. Wysocki" Date: Thu, 8 Jun 2017 19:16:45 +0200 X-Google-Sender-Auth: eQ8ABCGPGAAJ4yGcHQwGL7x5B18 Message-ID: Subject: Re: [PATCH v1 25/25] PM: Switch to use %pt To: Andy Shevchenko Cc: Rasmus Villemoes , Greg Kroah-Hartman , Andrew Morton , Linux Kernel Mailing List , Alessandro Zummo , Alexandre Belloni , linux-rtc@vger.kernel.org, "Rafael J. Wysocki" , Linux PM Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1219 Lines: 33 On Thu, Jun 8, 2017 at 3:48 PM, Andy Shevchenko wrote: > Use %pt instead of open coded variant to print content of > struct rtc_time in human readable format. > > Cc: "Rafael J. Wysocki" > Cc: linux-pm@vger.kernel.org > Signed-off-by: Andy Shevchenko Acked-by: Rafael J. Wysocki > --- > drivers/base/power/trace.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/base/power/trace.c b/drivers/base/power/trace.c > index 1cda505d6a85..5e32a4f28a49 100644 > --- a/drivers/base/power/trace.c > +++ b/drivers/base/power/trace.c > @@ -118,9 +118,7 @@ static unsigned int read_magic_time(void) > unsigned int val; > > mc146818_get_time(&time); > - pr_info("RTC time: %2d:%02d:%02d, date: %02d/%02d/%02d\n", > - time.tm_hour, time.tm_min, time.tm_sec, > - time.tm_mon + 1, time.tm_mday, time.tm_year % 100); > + pr_info("RTC time: %ptt, date: %ptd\n", &time, &time); > val = time.tm_year; /* 100 years */ > if (val > 100) > val -= 100; > -- > 2.11.0 >