Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-07-20 Thread Joe Perches
On Thu, 2017-07-20 at 10:57 -0700, Mark Salyzyn wrote: > It would probably need to take struct timespec64 as an argument. Pass by > structure might be difficult to swallow, so pass by pointer? Every %p extension is passed via a pointer.

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-07-20 Thread Mark Salyzyn
On 07/20/2017 03:33 AM, Andy Shevchenko wrote: On Tue, 2017-07-18 at 12:57 -0700, Mark Salyzyn wrote: On 07/18/2017 10:50 AM, Joe Perches wrote: On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: Recently I have noticed too many users of struct rtc_time that printing its content field b

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-07-20 Thread Andy Shevchenko
On Tue, 2017-07-18 at 12:57 -0700, Mark Salyzyn wrote: > On 07/18/2017 10:50 AM, Joe Perches wrote: > > On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: > > > Recently I have noticed too many users of struct rtc_time that > > > printing > > > its content field by field. > > > > > > In thi

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-07-18 Thread Mark Salyzyn
On 07/18/2017 10:50 AM, Joe Perches wrote: On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: Recently I have noticed too many users of struct rtc_time that printing its content field by field. In this series I introduce %pt[dt][rv] specifier to make life a bit easier. Hey Andy. I just

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-07-18 Thread Joe Perches
On Tue, 2017-07-18 at 20:55 +0300, Andy Shevchenko wrote: > On Tue, 2017-07-18 at 10:50 -0700, Joe Perches wrote: > > On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: > > > Recently I have noticed too many users of struct rtc_time that > > > printing > > > its content field by field. > > >

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-07-18 Thread Andy Shevchenko
On Tue, 2017-07-18 at 10:50 -0700, Joe Perches wrote: > On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: > > Recently I have noticed too many users of struct rtc_time that > > printing > > its content field by field. > > > > In this series I introduce %pt[dt][rv] specifier to make life a

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-07-18 Thread Joe Perches
On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: > Recently I have noticed too many users of struct rtc_time that printing > its content field by field. > > In this series I introduce %pt[dt][rv] specifier to make life a bit > easier. Hey Andy. I just saw a patch with a printk for rtc t

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-06-08 Thread Joe Perches
On Thu, 2017-06-08 at 18:02 +0300, Andy Shevchenko wrote: > On Thu, Jun 8, 2017 at 5:52 PM, Joe Perches wrote: > > On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: > > > Recently I have noticed too many users of struct rtc_time that printing > > > its content field by field. > > > > > >

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-06-08 Thread Andy Shevchenko
On Thu, Jun 8, 2017 at 5:52 PM, Joe Perches wrote: > On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: >> Recently I have noticed too many users of struct rtc_time that printing >> its content field by field. >> >> In this series I introduce %pt[dt][rv] specifier to make life a bit >> easi

Re: [PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-06-08 Thread Joe Perches
On Thu, 2017-06-08 at 16:47 +0300, Andy Shevchenko wrote: > Recently I have noticed too many users of struct rtc_time that printing > its content field by field. > > In this series I introduce %pt[dt][rv] specifier to make life a bit > easier. > > There are still users of detailed output of the s

[PATCH v1 00/25] lib, rtc: Print rtc_time via %pt[dt][rv]

2017-06-08 Thread Andy Shevchenko
Recently I have noticed too many users of struct rtc_time that printing its content field by field. In this series I introduce %pt[dt][rv] specifier to make life a bit easier. There are still users of detailed output of the struct rtc_time, but we can introduce an additional extension for them in