On Tue, 2015-06-16 at 13:33 -0700, Jacob E Keller wrote: > On Tue, 2015-06-16 at 17:19 +0300, Sergei Shtylyov wrote: > > On 6/16/2015 4:47 PM, Jeff Kirsher wrote: > > > > > From: Jacob Keller <jacob.e.kel...@intel.com> > > > > > The value will never be negative, and we use the %i print format, > > > > > > use > > > > "%i" is the same as "%d", AFAIR. You need "%u" for the unsigned > > > > variables. > > > > > unsigned int for the loop counter. Issue found using cppcheck. > > > > > Signed-off-by: Jacob Keller <jacob.e.kel...@intel.com> > > > Tested-by: Krishneil Singh <krishneil.k.si...@intel.com> > > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirs...@intel.com> > > > > WBR, Sergei > > > > Oops, you are right. > > Regards, > Jake
The actual code does use %u, but I made a typo in the description. I am sending an rc2. Regards, JakeN�����r��y����b�X��ǧv�^�){.n�+���z�^�)����w*jg��������ݢj/���z�ޖ��2�ޙ����&�)ߡ�a�����G���h��j:+v���w��٥