Hi Niklas,
On Fri, Mar 17, 2017 at 11:06 AM, Niklas Söderlund
<niklas.soderlund@xxxxxxxxxxxx> wrote:
> On 2017-03-07 20:55:33 +0100, Wolfram Sang wrote:
>> On Mon, Mar 06, 2017 at 09:03:59PM +0100, Niklas Söderlund wrote:
>> > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
>> > ---
>> > drivers/thermal/rcar_gen3_thermal.c | 137 ++++++++++++++++++++++++++++++++++++
>> > 1 file changed, 137 insertions(+)
>> >
>> > +static int rcar_gen3_thermal_set_trips(void *devdata, int low, int high)
>> > +{
>> > + struct rcar_gen3_thermal_tsc *tsc = devdata;
>> > +
>> > + if (low < -40000)
>> > + low = -40000;
>> > + if (high > 125000)
>> > + high = 125000;
>>
>> Use clamp_val()?
>
> Good idea, will use.
Why not clamp()? Does the type checking stand in the way?
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
