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 volt should not know about this and needs to take it as an input!


Signed-off-by: Karol Herbst <nouv...@karolherbst.de>
Do we really want to push reclocking logic to the volt subsystem?

To me, volt should just allow you to read back and set a voltage. All
the rest of the logic should be in clk.

Since this is my first NAK, here are my R-b for 3, 4 and 5:

Reviewed-by: Martin Peres <martin.pe...@free.fr>

With this fixed, this patch is Reviewed-by: Martin Peres <martin.pe...@free.fr>
_______________________________________________
Nouveau mailing list
Nouveau@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/nouveau

Reply via email to