Re: [PATCH] staging: ccree: Fix format/argument mismatches

2017-07-31 Thread Joe Perches
On Mon, 2017-07-31 at 09:39 +0300, Gilad Ben-Yossef wrote: > On Sun, Jul 30, 2017 at 7:45 PM, Joe Perches wrote: > > By default, debug logging is disabled by CC_DEBUG not being defined. > > > > Convert SSI_LOG_DEBUG to use no_printk instead of an empty define > > to validate

Re: [PATCH] staging: ccree: Fix format/argument mismatches

2017-07-31 Thread Gilad Ben-Yossef
On Sun, Jul 30, 2017 at 7:45 PM, Joe Perches wrote: > By default, debug logging is disabled by CC_DEBUG not being defined. > > Convert SSI_LOG_DEBUG to use no_printk instead of an empty define > to validate formats and arguments. > > Fix fallout. > > Miscellanea: > > o One of

[PATCH] staging: ccree: Fix format/argument mismatches

2017-07-30 Thread Joe Perches
By default, debug logging is disabled by CC_DEBUG not being defined. Convert SSI_LOG_DEBUG to use no_printk instead of an empty define to validate formats and arguments. Fix fallout. Miscellanea: o One of the conversions now uses %pR instead of multiple uses of %pad Signed-off-by: Joe Perches