Fix style in related functions.
Details
Details
Diff Detail
Diff Detail
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 10397 Build 10808: arc lint + arc unit
Event Timeline
| sys/compat/linuxkpi/common/include/linux/jiffies.h | ||
|---|---|---|
| 100 | These max checks are not sufficient for very large input values. You should also verify that the multiplication does not exceed 64-bits! const uint64_t nsec_max = -1ULL / (uint64_t)hz; if (nsec > nsec_max) nsec = nsec_max; | |
| 114 | passed jiffies argument should be "int" to avoid problems! | |
| sys/compat/linuxkpi/common/include/linux/jiffies.h | ||
|---|---|---|
| 99 | Can you add this check to the msecs_to_jiffies and usecs_to_jiffies aswell? | |
| sys/compat/linuxkpi/common/include/linux/jiffies.h | ||
|---|---|---|
| 114 | Do you need to cast j to (uint64_t) ?? | |
| sys/compat/linuxkpi/common/include/linux/jiffies.h | ||
|---|---|---|
| 117 | (uint64_t)(unsigned int)j ??? We don't like negative times?? Same below. Else the patch looks OK. | |