Karol Herbst
2017-Apr-23 18:05 UTC
[Nouveau] [PATCH v2] 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..43d304ce 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
Apparently Analagous Threads
- [PATCH] bios/iccsense: rails for power sensors have a mask of 0xf8 for version 0x10
- [PATCH v3] 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