<div dir="ltr">On 25 January 2018 at 13:16, Tobias Klausmann <span dir="ltr"><<a href="mailto:tobias.johannes.klausmann@mni.thm.de" target="_blank">tobias.johannes.klausmann@mni.thm.de</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">This fixes wrong temperature outputs e.g. 511°C if the card is asleep.<br>
<br>
Signed-off-by: Tobias Klausmann <<a href="mailto:tobias.johannes.klausmann@mni.thm.de">tobias.johannes.klausmann@<wbr>mni.thm.de</a>><br></blockquote><div><br></div><div>LGTM,<br><br><span class="gmail-il"><span class="gmail-il">Reviewed</span>-<span class="gmail-il">by</span>: Rhys</span> Kidd <<a href="mailto:rhyskidd@gmail.com">rhyskidd@gmail.com</a>><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
drivers/gpu/drm/nouveau/nvkm/<wbr>subdev/therm/gp100.c | 4 +++-<br>
1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/drivers/gpu/drm/nouveau/<wbr>nvkm/subdev/therm/gp100.c b/drivers/gpu/drm/nouveau/<wbr>nvkm/subdev/therm/gp100.c<br>
index 9f0dea3f61dc..45d0ec632b5a 100644<br>
--- a/drivers/gpu/drm/nouveau/<wbr>nvkm/subdev/therm/gp100.c<br>
+++ b/drivers/gpu/drm/nouveau/<wbr>nvkm/subdev/therm/gp100.c<br>
@@ -32,8 +32,10 @@ gp100_temp_get(struct nvkm_therm *therm)<br>
u32 inttemp = (tsensor & 0x0001fff8);<br>
<br>
/* device SHADOWed */<br>
- if (tsensor & 0x40000000)<br>
+ if (tsensor & 0x40000000) {<br>
nvkm_trace(subdev, "reading temperature from SHADOWed sensor\n");<br>
+ return -ENODEV;<br>
+ }<br>
<br>
/* device valid */<br>
if (tsensor & 0x20000000)<br>
<span class="gmail-HOEnZb"><font color="#888888">--<br>
2.16.1<br>
<br>
______________________________<wbr>_________________<br>
Nouveau mailing list<br>
<a href="mailto:Nouveau@lists.freedesktop.org">Nouveau@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/nouveau" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/nouveau</a><br>
</font></span></blockquote></div><br></div></div>