Re: [PATCH v2 1/1] lib/vsnprintf: Add %par specifier for sake of consistency

2017-01-12 Thread Andy Shevchenko
On Wed, 2017-01-11 at 16:59 -0800, Joe Perches wrote: > On Wed, 2017-01-11 at 15:57 -0800, Andrew Morton wrote: > > On Wed, 11 Jan 2017 18:28:07 +0200 Andy Shevchenko > o...@linux.intel.com> wrote: > > > > > While resource_size_t is repeating phys_addr_t, allocate %par > > > specifier for > > >

Re: [PATCH v2 1/1] lib/vsnprintf: Add %par specifier for sake of consistency

2017-01-12 Thread Andy Shevchenko
On Wed, 2017-01-11 at 16:59 -0800, Joe Perches wrote: > On Wed, 2017-01-11 at 15:57 -0800, Andrew Morton wrote: > > On Wed, 11 Jan 2017 18:28:07 +0200 Andy Shevchenko > o...@linux.intel.com> wrote: > > > > > While resource_size_t is repeating phys_addr_t, allocate %par > > > specifier for > > >

Re: [PATCH v2 1/1] lib/vsnprintf: Add %par specifier for sake of consistency

2017-01-11 Thread Joe Perches
On Wed, 2017-01-11 at 15:57 -0800, Andrew Morton wrote: > On Wed, 11 Jan 2017 18:28:07 +0200 Andy Shevchenko > wrote: > > > While resource_size_t is repeating phys_addr_t, allocate %par specifier for > > that type for sake of consistency. > > > > I'm

Re: [PATCH v2 1/1] lib/vsnprintf: Add %par specifier for sake of consistency

2017-01-11 Thread Joe Perches
On Wed, 2017-01-11 at 15:57 -0800, Andrew Morton wrote: > On Wed, 11 Jan 2017 18:28:07 +0200 Andy Shevchenko > wrote: > > > While resource_size_t is repeating phys_addr_t, allocate %par specifier for > > that type for sake of consistency. > > > > I'm struggling to see the value in this. A

Re: [PATCH v2 1/1] lib/vsnprintf: Add %par specifier for sake of consistency

2017-01-11 Thread Andrew Morton
On Wed, 11 Jan 2017 18:28:07 +0200 Andy Shevchenko wrote: > While resource_size_t is repeating phys_addr_t, allocate %par specifier for > that type for sake of consistency. > I'm struggling to see the value in this. A more detailed changelog would help,

Re: [PATCH v2 1/1] lib/vsnprintf: Add %par specifier for sake of consistency

2017-01-11 Thread Andrew Morton
On Wed, 11 Jan 2017 18:28:07 +0200 Andy Shevchenko wrote: > While resource_size_t is repeating phys_addr_t, allocate %par specifier for > that type for sake of consistency. > I'm struggling to see the value in this. A more detailed changelog would help, explaining why you think the kernel

[PATCH v2 1/1] lib/vsnprintf: Add %par specifier for sake of consistency

2017-01-11 Thread Andy Shevchenko
While resource_size_t is repeating phys_addr_t, allocate %par specifier for that type for sake of consistency. Signed-off-by: Andy Shevchenko --- In v2: - rebase on top of latest linux-next - include more people to Cc list Documentation/printk-formats.txt | 13

[PATCH v2 1/1] lib/vsnprintf: Add %par specifier for sake of consistency

2017-01-11 Thread Andy Shevchenko
While resource_size_t is repeating phys_addr_t, allocate %par specifier for that type for sake of consistency. Signed-off-by: Andy Shevchenko --- In v2: - rebase on top of latest linux-next - include more people to Cc list Documentation/printk-formats.txt | 13 ++--- lib/vsprintf.c