One question about this: I added spaces between the string and the PRIx32 macro because checkpatch generated a warning when I didn't. However most source files I've seen do not add spaces.
What is your preferred way? Does check patch needs to be updated to remove this specific warning, or should all the PRI.NN macros be padded with spaces? On 06/09/2015 10:06 AM, Nicolas Morey-Chaisemartin wrote: > Signed-off-by: Nicolas Morey-Chaisemartin <nmo...@kalray.eu> > --- > test/performance/odp_pktio_perf.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/test/performance/odp_pktio_perf.c > b/test/performance/odp_pktio_perf.c > index 17975bf..6d9b66c 100644 > --- a/test/performance/odp_pktio_perf.c > +++ b/test/performance/odp_pktio_perf.c > @@ -650,8 +650,10 @@ static int run_test(void) > printf("\tTransmit workers: \t%d\n", odp_cpumask_count(&txmask)); > printf("\tReceive workers: \t%d\n", odp_cpumask_count(&rxmask)); > printf("\tDuration (seconds): \t%d\n", gbl_args->args.duration); > - printf("\tTransmit batch length:\t%d\n", gbl_args->args.tx_batch_len); > - printf("\tReceive batch length: \t%d\n", gbl_args->args.rx_batch_len); > + printf("\tTransmit batch length:\t%" PRIu32 "\n", > + gbl_args->args.tx_batch_len); > + printf("\tReceive batch length: \t%" PRIu32 "\n", > + gbl_args->args.rx_batch_len); > printf("\tPacket receive method:\t%s\n", > gbl_args->args.schedule ? "schedule" : "poll"); > printf("\tInterface(s): \t"); > _______________________________________________ > lng-odp mailing list > lng-odp@lists.linaro.org > https://lists.linaro.org/mailman/listinfo/lng-odp _______________________________________________ lng-odp mailing list lng-odp@lists.linaro.org https://lists.linaro.org/mailman/listinfo/lng-odp