On Saturday 14 August 2021 16:36:37 W_Armin@xxxxxx wrote:
> From: Armin Wolf <W_Armin@xxxxxx>
>
> Mark function i8k_get_fan_nominal_speed() as __init since
> it is only used in code also marked as __init.
>
> Signed-off-by: Armin Wolf <W_Armin@xxxxxx>
Looks good,
Reviewed-by: Pali Rohár <pali@xxxxxxxxxx>
> ---
> drivers/hwmon/dell-smm-hwmon.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/hwmon/dell-smm-hwmon.c b/drivers/hwmon/dell-smm-hwmon.c
> index 0e229e3dae33..8bbeeda13faf 100644
> --- a/drivers/hwmon/dell-smm-hwmon.c
> +++ b/drivers/hwmon/dell-smm-hwmon.c
> @@ -303,7 +303,7 @@ static int i8k_get_fan_type(struct dell_smm_data *data, int fan)
> /*
> * Read the fan nominal rpm for specific fan speed.
> */
> -static int i8k_get_fan_nominal_speed(const struct dell_smm_data *data, int fan, int speed)
> +static int __init i8k_get_fan_nominal_speed(const struct dell_smm_data *data, int fan, int speed)
> {
> struct smm_regs regs = { .eax = I8K_SMM_GET_NOM_SPEED, };
>
> --
> 2.20.1
>
