habanalabs/gaudi: avoid resetting max power in hard reset
authorTomer Tayar <ttayar@habana.ai>
Wed, 26 Jan 2022 17:55:53 +0000 (19:55 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 22 May 2022 18:57:35 +0000 (20:57 +0200)
The default max power is deduced from the card type value in the CPU-CP
info. This value is then set in the max power variable of the device
structure.
Getting the CPU-CP info is done as part of the late init phase
which is called also during reset. This means that a max power value
which is modified via sysfs will be reset during hard reset back to the
default value.
As the max power is updated in any case during device init in
hl_sysfs_init(), this setting in late init can be removed, and the
overriding during reset is thus avoided.

Signed-off-by: Tomer Tayar <ttayar@habana.ai>
Reviewed-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/habanalabs/gaudi/gaudi.c

index 64cb195bc26e542b311319459aac07d681341732..9947e96d3baf2e8a27c789554511138e99771a5f 100644 (file)
@@ -8329,8 +8329,6 @@ static int gaudi_cpucp_info_get(struct hl_device *hdev)
 
        set_default_power_values(hdev);
 
-       hdev->max_power = prop->max_power_default;
-
        return 0;
 }