Karol Herbst
2017-Apr-23 18:06 UTC
[Nouveau] [PATCH v3] bios/iccsense: rails for power sensors have a mask of 0xf8 for version 0x10
I only saw those values inside the vbios: 0xff, 0xfd, 0xfc, 0xfa for valid
rails.
No idea what the lower value does, but at least we get power readings on
a lot of Fermi GPUs with that.
v2: add missing parentheses
Signed-off-by: Karol Herbst <karolherbst at gmail.com>
---
drm/nouveau/nvkm/subdev/bios/iccsense.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drm/nouveau/nvkm/subdev/bios/iccsense.c
b/drm/nouveau/nvkm/subdev/bios/iccsense.c
index 3953d118..23caef8d 100644
--- a/drm/nouveau/nvkm/subdev/bios/iccsense.c
+++ b/drm/nouveau/nvkm/subdev/bios/iccsense.c
@@ -87,7 +87,10 @@ nvbios_iccsense_parse(struct nvkm_bios *bios, struct
nvbios_iccsense *iccsense)
switch(ver) {
case 0x10:
- rail->mode = nvbios_rd08(bios, entry + 0x1);
+ if ((nvbios_rd08(bios, entry + 0x1) & 0xf8) == 0xf8)
+ rail->mode = 1;
+ else
+ rail->mode = 0;
rail->extdev_id = nvbios_rd08(bios, entry + 0x2);
res_start = 0x3;
break;
--
2.12.2
Seemingly Similar Threads
- [PATCH] bios/iccsense: rails for power sensors have a mask of 0xf8 for version 0x10
- [PATCH v2] bios/iccsense: rails for power sensors have a mask of 0xf8 for version 0x10
- [PATCH] nvkm/iccsense: Parse the resistors and config the right way
- [PATCH v4 2/6] nvbios/iccsense: add parsing of the SENSE table
- [PATCH 1/2] power sensor support
