On 01/05/2016 06:35 PM, Martin Peres wrote: > On 04/01/16 18:42, Emil Velikov wrote: >> On 4 January 2016 at 14:56, Martin Peres <[email protected]> wrote: >>> On 17/12/15 19:18, Martin Peres wrote: >>>> On 29/11/15 16:10, Martin Peres wrote: >>>>> This is an oversight that made use of the trip-point-based fan >>>>> managenent >>>>> on >>>>> cards that never expose those. This led the fan to stay at fan_min. >>>>> >>>>> Fortunately, the emergency code would kick when the temperature would >>>>> reach >>>>> 90°C. >>>>> >>>>> Reported-by: Tom Englund <[email protected]> >>>>> Tested-by: Tom Englund <[email protected]> >>>> >>>> Tested-by: Daemon32 <[email protected]> >>>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92126 >>>> >>>>> Signed-off-by: Martin Peres <[email protected]> >>>>> --- >>>>> drm/nouveau/nvkm/subdev/bios/fan.c | 1 + >>>>> 1 file changed, 1 insertion(+) >>>>> >>>>> diff --git a/drm/nouveau/nvkm/subdev/bios/fan.c >>>>> b/drm/nouveau/nvkm/subdev/bios/fan.c >>>>> index 43006db..80fed7e 100644 >>>>> --- a/drm/nouveau/nvkm/subdev/bios/fan.c >>>>> +++ b/drm/nouveau/nvkm/subdev/bios/fan.c >>>>> @@ -83,6 +83,7 @@ nvbios_fan_parse(struct nvkm_bios *bios, struct >>>>> nvbios_therm_fan *fan) >>>>> fan->type = NVBIOS_THERM_FAN_UNK; >>>>> } >>>>> >>>>> + fan->fan_mode = NVBIOS_THERM_FAN_LINEAR; >>>>> fan->min_duty = nvbios_rd08(bios, data + 0x02); >>>>> fan->max_duty = nvbios_rd08(bios, data + 0x03); >>>>> >>>>> >>>> Ben, can you merge this patch? It is kind of critical :s And it >>>> should be >>>> CCed to stable too, without it, some kepler/maxwell get 0% fan power >>>> which >>>> is kind of dangerous, even without reclocking! >>> >>> Ping, this is really important :s >>> >> Not sure who picked it up, but it seems to be in the latest fixes >> batch [1]. >> >> [1] >> http://lists.freedesktop.org/archives/dri-devel/2015-December/097589.html > > Thanks Emil, Karol pointed it out to me on IRC too. Thanks to whoever > picked it up :) I did, my apologies for not communicating that better!
Ben.
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Nouveau mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/nouveau
