Re: [patch 08/12] thermal: add sanity check for the passive attribute

Frans Pop
2009-11-17T22:42:54Z

On Tuesday 17 November 2009, Matthew Garrett wrote:
> As noted before, I'm not a fan of this approach - I can't think of many
> reasons why it'd be necessary to use temperatures below 1 degree C, but
> this constraint isn't present anywhere else in the thermal code and,
> really, there's plenty of things that people can break if they just echo
> incorrect values into /sys.

Personally I'd be in favor of adding more sanity checks rather than
removing them. The cost of the check is virtually zero.

Cheers,
FJP