Browse Source

drm/nouveau/therm: always initialize alarm_program_lock

Fixes "BUG: spinlock bad magic" on module load for nva3+ cards.

Introduced in commit "drm/nouveau/therm: implement support for temperature
alarms".

Signed-off-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
Marcin Slusarz 12 years ago
parent
commit
3969f05bb8

+ 1 - 0
drivers/gpu/drm/nouveau/core/subdev/therm/base.c

@@ -299,6 +299,7 @@ nouveau_therm_create_(struct nouveau_object *parent,
 
 	nouveau_alarm_init(&priv->alarm, nouveau_therm_alarm);
 	spin_lock_init(&priv->lock);
+	spin_lock_init(&priv->sensor.alarm_program_lock);
 
 	priv->base.fan_get = nouveau_therm_fan_user_get;
 	priv->base.fan_set = nouveau_therm_fan_user_set;

+ 0 - 1
drivers/gpu/drm/nouveau/core/subdev/therm/nv50.c

@@ -279,7 +279,6 @@ nv50_therm_ctor(struct nouveau_object *parent,
 	priv->base.base.pwm_clock = nv50_fan_pwm_clock;
 	priv->base.base.temp_get = nv50_temp_get;
 	priv->base.sensor.program_alarms = nv50_therm_program_alarms;
-	spin_lock_init(&priv->base.sensor.alarm_program_lock);
 	nv_subdev(priv)->intr = nv50_therm_intr;
 
 	/* init the thresholds */