Re: [Nouveau] [PATCH v2 07/22] volt: add min_id parameter to nvkm_volt_set_id

On 21/03/16 18:16, Karol Herbst wrote: min_id indicates a volt map entry which acts as a floor value, this will be used to set the lower voltage limit through pstates Signed-off-by: Karol Herbst Do we really want to push reclocking logic to the volt subsystem? To me,

Re: [Nouveau] [PATCH v2 06/22] volt: parse the both max voltage entries

On 21/03/16 18:16, Karol Herbst wrote: these entries specify a maximum voltage nvidia never exceeds, we shouldn't do that, too. Signed-off-by: Karol Herbst --- drm/nouveau/include/nvkm/subdev/bios/vmap.h | 2 ++ drm/nouveau/include/nvkm/subdev/volt.h | 2 ++

Re: [Nouveau] [PATCH v2 07/22] volt: add min_id parameter to nvkm_volt_set_id

On 28/03/16 23:52, Martin Peres wrote: On 21/03/16 18:16, Karol Herbst wrote: min_id indicates a volt map entry which acts as a floor value, this will be used to set the lower voltage limit through pstates Please state that this comes that this min_id is different for each pstate, hence why

Re: [Nouveau] [PATCH v2 06/22] volt: parse the both max voltage entries

On 28/03/16 23:49, Martin Peres wrote: On 21/03/16 18:16, Karol Herbst wrote: these entries specify a maximum voltage nvidia never exceeds, we shouldn't do that, too. Signed-off-by: Karol Herbst --- drm/nouveau/include/nvkm/subdev/bios/vmap.h | 2 ++

Re: [Nouveau] [PATCH v2 16/22] volt: don't require perfect fit

On 21/03/16 18:16, Karol Herbst wrote: if we calculate the voltage in the table right, we get all kinds of values, which never fit the hardware steps, so we use the closest higher value the hardware can do This is indeed the goal. Signed-off-by: Karol Herbst ---

Re: [Nouveau] [PATCH v2 22/22] debugfs: add boost interface to change the boost_mode

On 21/03/16 18:16, Karol Herbst wrote: Signed-off-by: Karol Herbst 21-22 are: Reviewed-by: Martin Peres ___ Nouveau mailing list Nouveau@lists.freedesktop.org

Re: [Nouveau] [PATCH v2 18/22] volt: add coefficients I found on my gpu

On 21/03/16 18:16, Karol Herbst wrote: I am sure that those are a bit different on other GPUs, but while testing the error range compared to nvidia was around 100%+-3%. Without this change we are most of the time around 10% below nvidias voltage, so this change causes no harm and improves the

[Nouveau] [Bug 94728] Nouveau doesn't work with Nvidia GTX 960

https://bugs.freedesktop.org/show_bug.cgi?id=94728 Martin Peres changed: What|Removed |Added Status|NEW |RESOLVED

[Nouveau] [Bug 91526] World of Warcraft (on Wine) has UI corruption with nouveau

https://bugs.freedesktop.org/show_bug.cgi?id=91526 --- Comment #14 from Karol Herbst --- (In reply to Ilia Mirkin from comment #13) > As I had originally suspected, this has nothing to actually do with BGRA4 > format. It was just an odd place in the command stream. >

[Nouveau] [Bug 94728] Nouveau doesn't work with Nvidia GTX 960

https://bugs.freedesktop.org/show_bug.cgi?id=94728 --- Comment #3 from Ilia Mirkin --- As you can see, it's falling back to modesetting, as expected. The xf86-video-nouveau DDX does not support any maxwell GPUs at the moment. Separately, modesetting isn't able to start up

[Nouveau] [Bug 91526] World of Warcraft (on Wine) has UI corruption with nouveau

https://bugs.freedesktop.org/show_bug.cgi?id=91526 Ilia Mirkin changed: What|Removed |Added Status|REOPENED|RESOLVED

[Nouveau] [PATCH v2 0/4] Configure Power Sensors

The power sensors can be configured to sample the readout values over time. Nvidia does this too, so nouveau should probably do that too. v2: use list_del and rework an error message Karol Herbst (4): iccsense: remove read function iccsense: convert to linked list iccsense: split sensor

[Nouveau] [PATCH v2 1/4] iccsense: remove read function

Signed-off-by: Karol Herbst Reviewed-by: Martin Peres --- drm/nouveau/include/nvkm/subdev/iccsense.h | 1 - drm/nouveau/nvkm/subdev/iccsense/base.c| 23 ++- 2 files changed, 10 insertions(+), 14 deletions(-) diff --git

[Nouveau] [PATCH v2 3/4] iccsense: split sensor into own struct

v2: add list_del call, reword error message Signed-off-by: Karol Herbst Reviewed-by: Martin Peres --- drm/nouveau/include/nvkm/subdev/iccsense.h | 1 + drm/nouveau/nvkm/subdev/iccsense/base.c| 142 -

[Nouveau] [PATCH v2 2/4] iccsense: convert to linked list

v2: add list_del calls Signed-off-by: Karol Herbst Reviewed-by: Martin Peres --- drm/nouveau/include/nvkm/subdev/iccsense.h | 4 +--- drm/nouveau/nouveau_hwmon.c| 2 +- drm/nouveau/nvkm/subdev/iccsense/base.c| 34

[Nouveau] [PATCH v2 4/4] iccsense: configure sensors like nvidia does

v2: rename ina209/ina219 read function Signed-off-by: Karol Herbst Reviewed-by: Martin Peres --- drm/nouveau/nvkm/subdev/iccsense/base.c | 68 + 1 file changed, 68 insertions(+) diff --git

[Nouveau] [Bug 94728] New: Nouveau doesn't work with Nvidia GTX 960

https://bugs.freedesktop.org/show_bug.cgi?id=94728 Bug ID: 94728 Summary: Nouveau doesn't work with Nvidia GTX 960 Product: xorg Version: unspecified Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW

Re: [Nouveau] [PATCH 3/4] iccsense: split sensor into own struct

On 25/03/16 13:19, Karol Herbst wrote: Signed-off-by: Karol Herbst --- drm/nouveau/include/nvkm/subdev/iccsense.h | 1 + drm/nouveau/nvkm/subdev/iccsense/base.c| 141 - drm/nouveau/nvkm/subdev/iccsense/priv.h| 15 ++- 3 files

Re: [Nouveau] [PATCH 4/4] iccsense: configure sensors like nvidia does

On 25/03/16 13:19, Karol Herbst wrote: Signed-off-by: Karol Herbst --- drm/nouveau/nvkm/subdev/iccsense/base.c | 68 + 1 file changed, 68 insertions(+) diff --git a/drm/nouveau/nvkm/subdev/iccsense/base.c

Re: [Nouveau] [PATCH 2/4] iccsense: convert to linked list

On 25/03/16 13:19, Karol Herbst wrote: Signed-off-by: Karol Herbst --- drm/nouveau/include/nvkm/subdev/iccsense.h | 4 +--- drm/nouveau/nouveau_hwmon.c| 2 +- drm/nouveau/nvkm/subdev/iccsense/base.c| 32 +-

[Nouveau] [Bug 94728] Nouveau doesn't work with Nvidia GTX 960

https://bugs.freedesktop.org/show_bug.cgi?id=94728 --- Comment #4 from neilmed...@gmail.com --- Makes sense. Thanks for the quick replies - and the good work in maintaining nouveau :) -- You are receiving this mail because: You are the assignee for the