Re: [PATCH 1/2] hwmon: lm75: Add TI TMP1075 support

[Guenter Roeck]

On Thu, Apr 29, 2021 at 02:11:49PM +0200, Robert Marko wrote:
> TI TMP1075 is a LM75 compatible sensor, so lets
> add support for it.
> 
> Signed-off-by: Robert Marko <robert.marko@xxxxxxxxxx>

Applied to hwmon-next.

Thanks,
Guenter

> ---
>  Documentation/hwmon/lm75.rst |  6 ++++--
>  drivers/hwmon/lm75.c         | 13 +++++++++++++
>  2 files changed, 17 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/hwmon/lm75.rst b/Documentation/hwmon/lm75.rst
> index 81257d5fc48f..8d0ab4ad5fb5 100644
> --- a/Documentation/hwmon/lm75.rst
> +++ b/Documentation/hwmon/lm75.rst
> @@ -93,9 +93,9 @@ Supported chips:
>  
>  	       https://www.st.com/resource/en/datasheet/stlm75.pdf
>  
> -  * Texas Instruments TMP100, TMP101, TMP105, TMP112, TMP75, TMP75B, TMP75C, TMP175, TMP275
> +  * Texas Instruments TMP100, TMP101, TMP105, TMP112, TMP75, TMP75B, TMP75C, TMP175, TMP275, TMP1075
>  
> -    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp112', 'tmp175', 'tmp75', 'tmp75b', 'tmp75c', 'tmp275'
> +    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp112', 'tmp175', 'tmp75', 'tmp75b', 'tmp75c', 'tmp275', 'tmp1075'
>  
>      Addresses scanned: none
>  
> @@ -119,6 +119,8 @@ Supported chips:
>  
>  	       https://www.ti.com/product/tmp275
>  
> +         https://www.ti.com/product/TMP1075
> +
>    * NXP LM75B, PCT2075
>  
>      Prefix: 'lm75b', 'pct2075'
> diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
> index e447febd121a..afdbb63237b9 100644
> --- a/drivers/hwmon/lm75.c
> +++ b/drivers/hwmon/lm75.c
> @@ -50,6 +50,7 @@ enum lm75_type {		/* keep sorted in alphabetical order */
>  	tmp75,
>  	tmp75b,
>  	tmp75c,
> +	tmp1075,
>  };
>  
>  /**
> @@ -293,6 +294,13 @@ static const struct lm75_params device_params[] = {
>  		.clr_mask = 1 << 5,	/*not one-shot mode*/
>  		.default_resolution = 12,
>  		.default_sample_time = MSEC_PER_SEC / 12,
> +	},
> +	[tmp1075] = { /* not one-shot mode, 27.5 ms sample rate */
> +		.clr_mask = 1 << 5 | 1 << 6 | 1 << 7,
> +		.default_resolution = 12,
> +		.default_sample_time = 28,
> +		.num_sample_times = 4,
> +		.sample_times = (unsigned int []){ 28, 55, 110, 220 },
>  	}
>  };
>  
> @@ -662,6 +670,7 @@ static const struct i2c_device_id lm75_ids[] = {
>  	{ "tmp75", tmp75, },
>  	{ "tmp75b", tmp75b, },
>  	{ "tmp75c", tmp75c, },
> +	{ "tmp1075", tmp1075, },
>  	{ /* LIST END */ }
>  };
>  MODULE_DEVICE_TABLE(i2c, lm75_ids);
> @@ -771,6 +780,10 @@ static const struct of_device_id __maybe_unused lm75_of_match[] = {
>  		.compatible = "ti,tmp75c",
>  		.data = (void *)tmp75c
>  	},
> +	{
> +		.compatible = "ti,tmp1075",
> +		.data = (void *)tmp1075
> +	},
>  	{ },
>  };
>  MODULE_DEVICE_TABLE(of, lm75_of_match);