On Mon, Apr 23, 2018 at 10:48:39AM +0200, Rafael J. Wysocki wrote: > On Saturday, April 14, 2018 6:10:55 AM CEST Yu Chen wrote: > > From: Chen Yu <yu.c.c...@intel.com> > > > > There's a use case during test to only print specific round of iterations > > if --iterations is specified, for example, with this patch applied: > > > > turbostat -i 5 -r 4 > > will capture 4 samples with 5 seconds interval. > > > > Cc: Len Brown <l...@kernel.org> > > Cc: Rafael J Wysocki <r...@rjwysocki.net> > > Cc: Artem Bityutskiy <artem.bityuts...@intel.com> > > Cc: Doug Smythies <dsmyth...@telus.net> > > Cc: linux...@vger.kernel.org > > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: Chen Yu <yu.c.c...@intel.com> [cut]... > > + if (iterations && (++done_iters >= iterations)) > > The inner parens are not needed here (and similarly below). > > If you fix this little one, please feel free to add > > Reviewed-by: Rafael J. Wysocki <rafael.j.wyso...@intel.com> > > to the patch (FWIW). > OK, will do, thanks! Yu