Thread (9 messages) 9 messages, 5 authors, 2021-04-18

Re: [PATCH v4 2/2] iio: temperature: add driver support for ti tmp117

From: Jonathan Cameron <jic23@kernel.org>
Date: 2021-04-18 09:48:11
Also in: linux-iio, lkml

On Sun, 11 Apr 2021 21:08:29 +0300
Andy Shevchenko [off-list ref] wrote:
On Sun, Apr 11, 2021 at 9:07 PM Andy Shevchenko
[off-list ref] wrote:
quoted
On Sun, Apr 11, 2021 at 5:53 PM Jonathan Cameron [off-list ref] wrote:  
quoted
On Wed,  7 Apr 2021 23:51:47 +0530
Puranjay Mohan [off-list ref] wrote:  
quoted
Good point, but better is to use clamp_t(s16, ...) rather than explicit casting.  
Sorry, I meant clamp_t(int, ...) of course, otherwise it will give wrong values.
I've switched it over to this which is indeed nicer.

Jonathan
quoted
I always consider explicit casting in C (and esp. in Linux kernel) is
a red flag. Should be really rarely needed.  
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help