Re: [PATCH] scsi: megaraid_sas: Prevent future %p disaster

2015-02-10 Thread Joe Perches
Convert printks to pr_ Miscellaneous: o Coalesce formats Add and remove spaces where appropriate when coalescing o Add pr_fmt o Remove embedded prefixes o Convert embedded function names to "%s:", __func__ o Realign arguments o Outdent one block that was inappropriately indented o Use pr_cont w

Re: [PATCH] scsi: megaraid_sas: Prevent future %p disaster

2015-02-10 Thread Joe Perches
On Tue, 2015-02-10 at 22:16 +0530, Kashyap Desai wrote: > Acked-by: Kashyap Desai > > > -Original Message- > > From: Rasmus Villemoes [mailto:li...@rasmusvillemoes.dk] > > Sent: Friday, February 06, 2015 8:04 PM [] > > There is currently no %po format extension, so currently the letters >

RE: [PATCH] scsi: megaraid_sas: Prevent future %p disaster

2015-02-10 Thread Kashyap Desai
com; linux- > s...@vger.kernel.org; linux-ker...@vger.kernel.org > Subject: [PATCH] scsi: megaraid_sas: Prevent future %p disaster > > There is currently no %po format extension, so currently the letters "on" are > simply skipped and the pointer is printed as expected (whil

[PATCH] scsi: megaraid_sas: Prevent future %p disaster

2015-02-06 Thread Rasmus Villemoes
There is currently no %po format extension, so currently the letters "on" are simply skipped and the pointer is printed as expected (while missing the word on). However, it is probably only a matter of time before someone comes up with a %po extension, at which point this is likely to fail spectacu