[Nouveau] [PATCH 0/4] nouveau_hwmon: migrate to hwmon_device_register_with_info

[Roy Spliet]

Just a non-technical (but important!) point: if you send four patches, 
make sure their title (the first line of your git commit messages) 
describes the content. Given you re-used the same title four times, I 
dare to say it's too generic.
Also, I noticed that this is not the standard git patch format as 
produced by git format-patch. I don't know whether this is a concern (or 
whether you tweaked the format yourself), but in the off chance that you 
are relatively new to git, I recommend you to look up the "git 
format-patch" and "git send-email" commands. They make life a lot easier 
in our mailing-list based work-flow!
This being said, thanks for the work! I'm sure with a bit of minor 
tweaking this is suitable for upstream.

Roy

Op 13-04-17 om 10:07 schreef Oscar Salvador:
> Hi again,
>
> I've split the patches as Karol Herbst suggested.
> I hope now it's fine.
>
> This series of patches introduce the new hwmon_device_register_with_info
> and gets rid of the old hwmon_device_register.
>
> This patch adds the default sensors with their possible config values.
> Just to prepare for the next patches
>
>
> --- linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c.orig	2017-04-12 19:18:09.638073562 +0200
> +++ linux/drivers/gpu/drm/nouveau/nouveau_hwmon.c	2017-04-12 19:19:44.244797202 +0200
> @@ -692,6 +692,78 @@ static const struct attribute_group hwmo
>   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
> _______________________________________________
> Nouveau mailing list
> Nouveau at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/nouveau

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/nouveau/attachments/20170413/b3935d54/attachment.html>