[Nouveau] [PATCH v2 1/5] nouveau_hwmon: Add config for all sensors and their settings

[Oscar Salvador]

Got it, I'll make a v3 with that line in all patches.

Thanks!

On 19 April 2017 at 20:11, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> All kernel patches must have a Signed-off-by: $user. See
> https://www.kernel.org/doc/html/latest/process/submitting-patches.html#sign-your-work-the-developer-s-certificate-of-origin
>
> On Mon, Apr 17, 2017 at 3:47 AM, Oscar Salvador
> <osalvador.vilardaga at gmail.com> wrote:
>> This is a preparation for the next patches. It just adds the sensors with
>> their possible configurable settings and then fills the struct hwmon_channel_info
>> with all this information.
>> ---
>>  drivers/gpu/drm/nouveau/nouveau_hwmon.c | 72 +++++++++++++++++++++++++++++++++
>>  1 file changed, 72 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/nouveau/nouveau_hwmon.c b/drivers/gpu/drm/nouveau/nouveau_hwmon.c
>> index 23b1670..24b40c5 100644
>> --- a/drivers/gpu/drm/nouveau/nouveau_hwmon.c
>> +++ b/drivers/gpu/drm/nouveau/nouveau_hwmon.c
>> @@ -692,6 +692,78 @@ static const struct attribute_group hwmon_power_attrgroup = {
>>  static const struct attribute_group hwmon_power_caps_attrgroup = {
>>         .attrs = hwmon_power_caps_attributes,
>>  };
>> +
>> +static const u32 nouveau_config_chip[] = {
>> +       HWMON_C_UPDATE_INTERVAL,
>> +       0
>> +};
>> +
>> +static const u32 nouveau_config_in[] = {
>> +       HWMON_I_INPUT | HWMON_I_MIN | HWMON_I_MAX | HWMON_I_LABEL,
>> +       0
>> +};
>> +
>> +static const u32 nouveau_config_temp[] = {
>> +       HWMON_T_INPUT | HWMON_T_MAX | HWMON_T_MAX_HYST |
>> +       HWMON_T_CRIT | HWMON_T_CRIT_HYST | HWMON_T_EMERGENCY |
>> +       HWMON_T_EMERGENCY_HYST,
>> +       0
>> +};
>> +
>> +static const u32 nouveau_config_fan[] = {
>> +       HWMON_F_INPUT,
>> +       0
>> +};
>> +
>> +static const u32 nouveau_config_pwm[] = {
>> +       HWMON_PWM_INPUT | HWMON_PWM_ENABLE,
>> +       0
>> +};
>> +
>> +static const u32 nouveau_config_power[] = {
>> +       HWMON_P_INPUT | HWMON_P_CAP_MAX | HWMON_P_CRIT,
>> +       0
>> +};
>> +
>> +static const struct hwmon_channel_info nouveau_chip = {
>> +       .type = hwmon_chip,
>> +       .config = nouveau_config_chip,
>> +};
>> +
>> +static const struct hwmon_channel_info nouveau_temp = {
>> +       .type = hwmon_temp,
>> +       .config = nouveau_config_temp,
>> +};
>> +
>> +static const struct hwmon_channel_info nouveau_fan = {
>> +       .type = hwmon_fan,
>> +       .config = nouveau_config_fan,
>> +};
>> +
>> +static const struct hwmon_channel_info nouveau_in = {
>> +       .type = hwmon_in,
>> +       .config = nouveau_config_in,
>> +};
>> +
>> +static const struct hwmon_channel_info nouveau_pwm = {
>> +       .type = hwmon_pwm,
>> +       .config = nouveau_config_pwm,
>> +};
>> +
>> +static const struct hwmon_channel_info nouveau_power = {
>> +       .type = hwmon_power,
>> +       .config = nouveau_config_power,
>> +};
>> +
>> +static const struct hwmon_channel_info *nouveau_info[] = {
>> +       &nouveau_chip,
>> +       &nouveau_temp,
>> +       &nouveau_fan,
>> +       &nouveau_in,
>> +       &nouveau_pwm,
>> +       &nouveau_power,
>> +       NULL
>> +};
>>  #endif
>>
>>  int
>> --
>> 2.1.4
>>
>> _______________________________________________
>> Nouveau mailing list
>> Nouveau at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/nouveau