Maybe != 0 would be better? I don't have a strong opinion about it.
julia
---------- Forwarded message ----------
Date: Sat, 23 Jan 2016 02:33:48 +0800
From: kbuild test robot <fengguang.wu@xxxxxxxxx>
To: kbuild@xxxxxx
Cc: Julia Lawall <julia.lawall@xxxxxxx>
Subject: drivers/staging/media/lirc/lirc_parallel.c:163:22-33: WARNING: Unsigned
expression compared with zero: timeelapsed > 0
CC: kbuild-all@xxxxxx
CC: linux-kernel@xxxxxxxxxxxxxxx
TO: Arnd Bergmann <arnd@xxxxxxxx>
CC: Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx>
CC: linux-media@xxxxxxxxxxxxxxx
CC: Tapasweni Pathak <tapaswenipathak@xxxxxxxxx>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 3e1e21c7bfcfa9bf06c07f48a13faca2f62b3339
commit: 0dbf41a3c88e229009a9f5fd2a89835569fa3451 [media] staging: media: lirc: Replace timeval with ktime_t in lirc_parallel.c
date: 7 weeks ago
:::::: branch date: 15 hours ago
:::::: commit date: 7 weeks ago
>> drivers/staging/media/lirc/lirc_parallel.c:163:22-33: WARNING: Unsigned expression compared with zero: timeelapsed > 0
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git remote update linus
git checkout 0dbf41a3c88e229009a9f5fd2a89835569fa3451
vim +163 drivers/staging/media/lirc/lirc_parallel.c
0dbf41a3 drivers/staging/media/lirc/lirc_parallel.c Arnd Bergmann 2015-11-25 147 ktime_t kt, now, timeout;
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 148 unsigned int level, newlevel, timeelapsed, newtimer;
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 149 int count = 0;
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 150
0dbf41a3 drivers/staging/media/lirc/lirc_parallel.c Arnd Bergmann 2015-11-25 151 kt = ktime_get();
0dbf41a3 drivers/staging/media/lirc/lirc_parallel.c Arnd Bergmann 2015-11-25 152 /* wait max. 1 sec. */
0dbf41a3 drivers/staging/media/lirc/lirc_parallel.c Arnd Bergmann 2015-11-25 153 timeout = ktime_add_ns(kt, NSEC_PER_SEC);
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 154 level = lirc_get_timer();
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 155 do {
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 156 newlevel = lirc_get_timer();
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 157 if (level == 0 && newlevel != 0)
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 158 count++;
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 159 level = newlevel;
0dbf41a3 drivers/staging/media/lirc/lirc_parallel.c Arnd Bergmann 2015-11-25 160 now = ktime_get();
0dbf41a3 drivers/staging/media/lirc/lirc_parallel.c Arnd Bergmann 2015-11-25 161 } while (count < 1000 && (ktime_before(now, timeout)));
0dbf41a3 drivers/staging/media/lirc/lirc_parallel.c Arnd Bergmann 2015-11-25 162 timeelapsed = ktime_us_delta(now, kt);
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 @163 if (count >= 1000 && timeelapsed > 0) {
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 164 if (default_timer == 0) {
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 165 /* autodetect timer */
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 166 newtimer = (1000000*count)/timeelapsed;
cc38b8e9 drivers/staging/media/lirc/lirc_parallel.c YAMANE Toshiaki 2012-11-08 167 pr_info("%u Hz timer detected\n", newtimer);
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 168 return newtimer;
381d7f79 drivers/staging/media/lirc/lirc_parallel.c Zheng Di 2014-07-06 169 }
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 170 newtimer = (1000000*count)/timeelapsed;
805a8966 drivers/staging/lirc/lirc_parallel.c Jarod Wilson 2010-07-26 171 if (abs(newtimer - default_timer) > default_timer/10) {
:::::: The code at line 163 was first introduced by commit
:::::: 805a8966659563df68ea7bbd94241dafd645c725 V4L/DVB: staging/lirc: add lirc_parallel driver
:::::: TO: Jarod Wilson <jarod@xxxxxxxxxx>
:::::: CC: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
