drm/nouveau/fan: fix selection of fan speed when fan->get returns an error
authorMarcin Slusarz <marcin.slusarz@gmail.com>
Sun, 27 Jan 2013 14:51:09 +0000 (15:51 +0100)
committerBen Skeggs <bskeggs@redhat.com>
Wed, 20 Feb 2013 06:00:36 +0000 (16:00 +1000)
fan->get returns int, but we write it to unsigned variable, and then check
whether it's >= 0 (it always is)

Found by smatch:
drivers/gpu/drm/nouveau/core/subdev/therm/fan.c:61 nouveau_fan_update() warn: always true condition '(duty >= 0) => (0-u32max >= 0)'

Signed-off-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/core/subdev/therm/fan.c

index b179655e639cf3b643dbe394196bb954b42f5d44..c728380d3d625f4b6ff238f1363700fc324fe703 100644 (file)
@@ -39,7 +39,7 @@ nouveau_fan_update(struct nouveau_fan *fan, bool immediate, int target)
        struct nouveau_timer *ptimer = nouveau_timer(priv);
        unsigned long flags;
        int ret = 0;
-       u32 duty;
+       int duty;
 
        /* update target fan speed, restricting to allowed range */
        spin_lock_irqsave(&fan->lock, flags);
@@ -64,9 +64,9 @@ nouveau_fan_update(struct nouveau_fan *fan, bool immediate, int target)
                 * it is meant to bump the fan speed more incrementally
                 */
                if (duty < target)
-                       duty = min(duty + 3, (u32) target);
+                       duty = min(duty + 3, target);
                else if (duty > target)
-                       duty = max(duty - 3, (u32) target);
+                       duty = max(duty - 3, target);
        } else {
                duty = target;
        }