Re: Fwd: [PATCH 1/2] hwmon: (lm75) Add support for TMP75B

[Guenter Roeck <linux@roeck-us.net>]

On 4/23/19 3:11 AM, Iker Perez del Palomar wrote:
> The TMP75B has a different control register, supports 12-bit
> resolution and the default conversion rate is 37 Hz.
> 
> Signed-off-by: Iker Perez del Palomar Sustatxa <iker.perez@codethink.co.uk>

This patch is corrupted. It replaces tabs with spaces, and there are line wraps.

> ---
>   Documentation/hwmon/lm75 |  5 +++--
>   drivers/hwmon/lm75.c     | 13 +++++++++++++
>   2 files changed, 16 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/hwmon/lm75 b/Documentation/hwmon/lm75
> index 010583608f12..baf052935761 100644
> --- a/Documentation/hwmon/lm75
> +++ b/Documentation/hwmon/lm75
> @@ -47,8 +47,8 @@ Supported chips:
>       Addresses scanned: none
>       Datasheet: Publicly available at the ST website
>              https://www.st.com/resource/en/datasheet/stlm75.pdf
> -  * Texas Instruments TMP100, TMP101, TMP105, TMP112, TMP75, TMP75C, TMP175, TMP275
> -    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp112', 'tmp175', 'tmp75', 'tmp75c', 'tmp275'
> +  * Texas Instruments TMP100, TMP101, TMP105, TMP112, TMP75, TMP75B, TMP75C, TMP175, TMP275
> +    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp112', 'tmp175', 'tmp75', 'tmp75b', 'tmp75c', 'tmp275'
>       Addresses scanned: none
>       Datasheet: Publicly available at the Texas Instruments website
>                  http://www.ti.com/product/tmp100
> @@ -56,6 +56,7 @@ Supported chips:
>                  http://www.ti.com/product/tmp105
>                  http://www.ti.com/product/tmp112
>                  http://www.ti.com/product/tmp75
> +               http://www.ti.com/product/tmp75b
>                  http://www.ti.com/product/tmp75c
>                  http://www.ti.com/product/tmp175
>                  http://www.ti.com/product/tmp275
> diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
> index 62acb9f16ec5..1b5c32cbdc62 100644
> --- a/drivers/hwmon/lm75.c
> +++ b/drivers/hwmon/lm75.c
> @@ -32,6 +32,7 @@
>   #include "lm75.h"
> 
> 
> +

I don't mind if you'd remove one of the two empty lines above, but add yet another one ?
Please don't.

>   /*
>    * This driver handles the LM75 and compatible digital temperature sensors.
>    */
> @@ -59,6 +60,7 @@ enum lm75_type {        /* keep sorted in alphabetical order */
>       tmp175,
>       tmp275,
>       tmp75,
> +    tmp75b,
>       tmp75c,
>   };
> 
> @@ -378,6 +380,12 @@ lm75_probe(struct i2c_client *client, const struct i2c_device_id *id)
>           data->resolution = 12;
>           data->sample_time = MSEC_PER_SEC / 2;
>           break;
> +    case tmp75b:
> +        clr_mask |= 1 << 15;        /* not one-shot mode */
> +        data->resolution = 12;
> +        data->sample_time = MSEC_PER_SEC / 32;
> +        break;
> +
>       case tmp75c:
>           clr_mask |= 1 << 5;        /* not one-shot mode */
>           data->resolution = 12;
> @@ -438,6 +446,7 @@ static const struct i2c_device_id lm75_ids[] = {
>       { "tmp175", tmp175, },
>       { "tmp275", tmp275, },
>       { "tmp75", tmp75, },
> +    { "tmp75b", tmp75b,},
>       { "tmp75c", tmp75c, },
>       { /* LIST END */ }
>   };
> @@ -537,6 +546,10 @@ static const struct of_device_id lm75_of_match[] = {
>           .data = (void *)tmp75
>       },
>       {
> +        .compatible = "ti,tmp75b",
> +        .data = (void *)tmp75b
> +    },
> +    {
>           .compatible = "ti,tmp75c",
>           .data = (void *)tmp75c
>       },
>