Re: [PATCH v2 3/4] hwmon: (lm75) Add new fields into lm75_params_

[Guenter Roeck]

On Thu, Aug 08, 2019 at 09:02:45AM +0100, Iker Perez wrote:
> From: Iker Perez del Palomar Sustatxa <iker.perez@xxxxxxxxxxxxxxx>
> 
> The new fields are included to prepare the driver for next patch. The
> fields are:
> 
> * *resolutions: Stores all the supported resolutions by the device.
> * num_sample_times: Stores the number of possible sample times.
> * *sample_times: Stores all the possible sample times to be set.
> * sample_set_masks: The set_masks for the possible sample times
> * sample_clr_mask: Clear mask to set the default sample time.
> 
> Signed-off-by: Iker Perez del Palomar Sustatxa <iker.perez@xxxxxxxxxxxxxxx>
> ---
> 
> Changes since v1:
> - In the lm75_params structure documentation there have been the next changes:
>         - @num_sample_times description has been extended.
>         - @sample_times description has been extended.
>         - @sample_set_masks description has been extended.
>         - @resolutions description has been included.
> 
>  drivers/hwmon/lm75.c | 36 +++++++++++++++++++++++++++++++-----
>  1 file changed, 31 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
> index a32d7952d799..ed72455bcfa3 100644
> --- a/drivers/hwmon/lm75.c
> +++ b/drivers/hwmon/lm75.c
> @@ -61,15 +61,34 @@ enum lm75_type {		/* keep sorted in alphabetical order */
>   * @resolution_limits:	Limit register resolution. Optional. Should be set if
>   *			the resolution of limit registers does not match the
>   *			resolution of the temperature register.
> + * @resolutions		List of resolutions associated with sample times.

@resolutions:

> + *			Optional. Should be set if num_sample_times is larger
> + *			than 1, and if the resolution changes with sample times.
> + *			If set, number of entries must match num_sample_times.
>   * default_sample_time:	Sample time to be set by default.

@default_sample_time:

No need to resend; I'll fix that up when applying.

> + * @num_sample_times:	Number of possible sample times to be set. Optional.
> + *			Should be set if the number of sample times is larger
> + *			than one.
> + * @sample_times:	All the possible sample times to be set. Mandatory if
> + *			num_sample_times is larger than 1. If set, number of
> + *			entries must match num_sample_times.
> + * @sample_set_masks:	All the set_masks for the possible sample times.
> + *			Mandatory if num_sample_times is larger than 1.
> + *			If set, number of entries must match num_sample_times.
> + * @sample_clr_mask:	Clear mask to set the default sample time.
>   */
>  
>  struct lm75_params {
> -	u8		set_mask;
> -	u8		clr_mask;
> -	u8		default_resolution;
> -	u8		resolution_limits;
> -	unsigned int	default_sample_time;
> +	u8			set_mask;
> +	u8			clr_mask;
> +	u8			default_resolution;
> +	u8			resolution_limits;
> +	const u8		*resolutions;
> +	unsigned int		default_sample_time;
> +	u8			num_sample_times;
> +	const unsigned int	*sample_times;
> +	const u8		*sample_set_masks;
> +	u8			sample_clr_mask;
>  };
>  
>  /* Addresses scanned */
> @@ -221,7 +240,14 @@ static const struct lm75_params device_params[] = {
>  	[tmp75b] = { /* not one-shot mode, Conversion rate 37Hz */
>  		.clr_mask = 1 << 7 | 3 << 5,
>  		.default_resolution = 12,
> +		.sample_set_masks = (u8 []){ 0 << 5, 1 << 5, 2 << 5,
> +			3 << 5 },
> +		.sample_clr_mask = 3 << 5,
>  		.default_sample_time = MSEC_PER_SEC / 37,
> +		.sample_times = (unsigned int []){ MSEC_PER_SEC / 37,
> +			MSEC_PER_SEC / 18,
> +			MSEC_PER_SEC / 9, MSEC_PER_SEC / 4 },
> +		.num_sample_times = 4,
>  	},
>  	[tmp75c] = {
>  		.clr_mask = 1 << 5,	/*not one-shot mode*/