All of lore.kernel.org
 help / color / mirror / Atom feed
* [lm-sensors] [PATCH] support tmp112
@ 2013-08-06 15:29 Felten, Lothar
  2013-08-06 16:01 ` Guenter Roeck
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Felten, Lothar @ 2013-08-06 15:29 UTC (permalink / raw)
  To: lm-sensors

Hello Guenter,

Sorry for the confusion, I modified the patch now it uses a higher speed.

This patch adds support for the tmp112 temperature sensor which is similar to the lm75 series.

Best regards,
Lothar

diff --git a/Documentation/hwmon/lm75 b/Documentation/hwmon/lm75
index 2560a9c..c6a5ff1 100644
--- a/Documentation/hwmon/lm75
+++ b/Documentation/hwmon/lm75
@@ -42,13 +42,14 @@ Supported chips:
     Addresses scanned: none
     Datasheet: Publicly available at the ST website
                http://www.st.com/internet/analog/product/121769.jsp
-  * Texas Instruments TMP100, TMP101, TMP105, TMP75, TMP175, TMP275
-    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp175', 'tmp75', 'tmp275'
+  * Texas Instruments TMP100, TMP101, TMP105, TMP112, TMP75, TMP175, TMP275
+    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp112', 'tmp175', 'tmp75', 'tmp275'
     Addresses scanned: none
     Datasheet: Publicly available at the Texas Instruments website
                http://www.ti.com/product/tmp100
                http://www.ti.com/product/tmp101
                http://www.ti.com/product/tmp105
+               http://www.ti.com/product/tmp112
                http://www.ti.com/product/tmp75
                http://www.ti.com/product/tmp175
                http://www.ti.com/product/tmp275
diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
index e989f7f..4316313 100644
--- a/drivers/hwmon/Kconfig
+++ b/drivers/hwmon/Kconfig
@@ -652,8 +652,8 @@ config SENSORS_LM75
                - NXP's LM75A
                - ST Microelectronics STDS75
                - TelCom (now Microchip) TCN75
-               - Texas Instruments TMP100, TMP101, TMP105, TMP75, TMP175,
-                 TMP275
+               - Texas Instruments TMP100, TMP101, TMP105, TMP112,
+                  TMP175, TMP275, TMP75

          This driver supports driver model based binding through board
          specific I2C device tables.
diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
index c03b490..3fe54ac 100644
--- a/drivers/hwmon/lm75.c
+++ b/drivers/hwmon/lm75.c
@@ -49,6 +49,7 @@ enum lm75_type {              /* keep sorted in alphabetical order */
        tmp100,
        tmp101,
        tmp105,
+       tmp112,
        tmp175,
        tmp275,
        tmp75,
@@ -236,6 +237,12 @@ lm75_probe(struct i2c_client *client, const struct i2c_device_id *id)
                data->sample_time = HZ;
                clr_mask |= 1 << 7;             /* not one-shot mode */
                break;
+       case tmp112:
+               set_mask |= 3 << 5;             /* 12-bit mode */
+               clr_mask |= 1 << 7;             /* not one-shot mode */
+               data->resolution = 12;
+               data->sample_time = HZ / 4;
+               break;
        case tmp105:
        case tmp175:
        case tmp275:
@@ -306,6 +313,7 @@ static const struct i2c_device_id lm75_ids[] = {
        { "tmp100", tmp100, },
        { "tmp101", tmp101, },
        { "tmp105", tmp105, },
+       { "tmp112", tmp112, },
        { "tmp175", tmp175, },
        { "tmp275", tmp275, },
        { "tmp75", tmp75, },

Signed-off-by: l-felten@ti.com

Texas Instruments Belgium SA, Rond Point Schuman 6- Bo?te 5, 1040 Bruxelles. BCE: 0414.207.123. RPM Bruxelles. IBAN: BE83570121895615. Swift: CITIBEBX. TVA BE 0414.207.123



_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [lm-sensors] [PATCH] support tmp112
  2013-08-06 15:29 [lm-sensors] [PATCH] support tmp112 Felten, Lothar
@ 2013-08-06 16:01 ` Guenter Roeck
  2013-08-06 16:47 ` Felten, Lothar
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Guenter Roeck @ 2013-08-06 16:01 UTC (permalink / raw)
  To: lm-sensors

On 08/06/2013 08:29 AM, Felten, Lothar wrote:
> Hello Guenter,
>
> Sorry for the confusion, I modified the patch now it uses a higher speed.
>
> This patch adds support for the tmp112 temperature sensor which is similar to the lm75 series.
>
> Best regards,
> Lothar
>
> diff --git a/Documentation/hwmon/lm75 b/Documentation/hwmon/lm75
> index 2560a9c..c6a5ff1 100644
> --- a/Documentation/hwmon/lm75
> +++ b/Documentation/hwmon/lm75
> @@ -42,13 +42,14 @@ Supported chips:
>       Addresses scanned: none
>       Datasheet: Publicly available at the ST website
>                  http://www.st.com/internet/analog/product/121769.jsp
> -  * Texas Instruments TMP100, TMP101, TMP105, TMP75, TMP175, TMP275
> -    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp175', 'tmp75', 'tmp275'
> +  * Texas Instruments TMP100, TMP101, TMP105, TMP112, TMP75, TMP175, TMP275
> +    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp112', 'tmp175', 'tmp75', 'tmp275'
>       Addresses scanned: none
>       Datasheet: Publicly available at the Texas Instruments website
>                  http://www.ti.com/product/tmp100
>                  http://www.ti.com/product/tmp101
>                  http://www.ti.com/product/tmp105
> +               http://www.ti.com/product/tmp112
>                  http://www.ti.com/product/tmp75
>                  http://www.ti.com/product/tmp175
>                  http://www.ti.com/product/tmp275
> diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
> index e989f7f..4316313 100644
> --- a/drivers/hwmon/Kconfig
> +++ b/drivers/hwmon/Kconfig
> @@ -652,8 +652,8 @@ config SENSORS_LM75
>                  - NXP's LM75A
>                  - ST Microelectronics STDS75
>                  - TelCom (now Microchip) TCN75
> -               - Texas Instruments TMP100, TMP101, TMP105, TMP75, TMP175,
> -                 TMP275
> +               - Texas Instruments TMP100, TMP101, TMP105, TMP112,
> +                  TMP175, TMP275, TMP75
>
>            This driver supports driver model based binding through board
>            specific I2C device tables.
> diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
> index c03b490..3fe54ac 100644
> --- a/drivers/hwmon/lm75.c
> +++ b/drivers/hwmon/lm75.c
> @@ -49,6 +49,7 @@ enum lm75_type {              /* keep sorted in alphabetical order */
>          tmp100,
>          tmp101,
>          tmp105,
> +       tmp112,
>          tmp175,
>          tmp275,
>          tmp75,
> @@ -236,6 +237,12 @@ lm75_probe(struct i2c_client *client, const struct i2c_device_id *id)
>                  data->sample_time = HZ;
>                  clr_mask |= 1 << 7;             /* not one-shot mode */
>                  break;
> +       case tmp112:
> +               set_mask |= 3 << 5;             /* 12-bit mode */
> +               clr_mask |= 1 << 7;             /* not one-shot mode */
> +               data->resolution = 12;
> +               data->sample_time = HZ / 4;
> +               break;
>          case tmp105:
>          case tmp175:
>          case tmp275:
> @@ -306,6 +313,7 @@ static const struct i2c_device_id lm75_ids[] = {
>          { "tmp100", tmp100, },
>          { "tmp101", tmp101, },
>          { "tmp105", tmp105, },
> +       { "tmp112", tmp112, },
>          { "tmp175", tmp175, },
>          { "tmp275", tmp275, },
>          { "tmp75", tmp75, },
>
> Signed-off-by: l-felten@ti.com
>

Hi Lothar,

Signed-off is still at the end :)

Also, looks like your mailer replaced all tabs with spaces, or your editor is set to do the same.
checkpatch result is 'total: 5 errors, 8 warnings, 52 lines checked'

Please fix.

Thanks,
Guenter


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [lm-sensors] [PATCH] support tmp112
  2013-08-06 15:29 [lm-sensors] [PATCH] support tmp112 Felten, Lothar
  2013-08-06 16:01 ` Guenter Roeck
@ 2013-08-06 16:47 ` Felten, Lothar
  2013-08-06 19:14 ` Guenter Roeck
  2013-08-17 14:48 ` Guenter Roeck
  3 siblings, 0 replies; 5+ messages in thread
From: Felten, Lothar @ 2013-08-06 16:47 UTC (permalink / raw)
  To: lm-sensors

> Hi Lothar,
>
> Signed-off is still at the end :)
That was me, I can fix that :)

>
> Also, looks like your mailer replaced all tabs with spaces, or your editor is set to do the same.
> checkpatch result is 'total: 5 errors, 8 warnings, 52 lines checked'
I ran checkpatch.pl, my editor is ok but outlook is probably not a good mail program.
I changed some settings... let's see...

>
> Please fix.
>
> Thanks,
> Guenter

Signed-off-by: l-felten@ti.com

diff --git a/Documentation/hwmon/lm75 b/Documentation/hwmon/lm75
index 2560a9c..c6a5ff1 100644
--- a/Documentation/hwmon/lm75
+++ b/Documentation/hwmon/lm75
@@ -42,13 +42,14 @@ Supported chips:
     Addresses scanned: none
     Datasheet: Publicly available at the ST website
                http://www.st.com/internet/analog/product/121769.jsp
-  * Texas Instruments TMP100, TMP101, TMP105, TMP75, TMP175, TMP275
-    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp175', 'tmp75', 'tmp275'
+  * Texas Instruments TMP100, TMP101, TMP105, TMP112, TMP75, TMP175, TMP275
+    Prefixes: 'tmp100', 'tmp101', 'tmp105', 'tmp112', 'tmp175', 'tmp75', 'tmp275'
     Addresses scanned: none
     Datasheet: Publicly available at the Texas Instruments website
                http://www.ti.com/product/tmp100
                http://www.ti.com/product/tmp101
                http://www.ti.com/product/tmp105
+               http://www.ti.com/product/tmp112
                http://www.ti.com/product/tmp75
                http://www.ti.com/product/tmp175
                http://www.ti.com/product/tmp275
diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig
index e989f7f..4316313 100644
--- a/drivers/hwmon/Kconfig
+++ b/drivers/hwmon/Kconfig
@@ -652,8 +652,8 @@ config SENSORS_LM75
                - NXP's LM75A
                - ST Microelectronics STDS75
                - TelCom (now Microchip) TCN75
-               - Texas Instruments TMP100, TMP101, TMP105, TMP75, TMP175,
-                 TMP275
+               - Texas Instruments TMP100, TMP101, TMP105, TMP112,
+                  TMP175, TMP275, TMP75

          This driver supports driver model based binding through board
          specific I2C device tables.
diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c
index c03b490..3fe54ac 100644
--- a/drivers/hwmon/lm75.c
+++ b/drivers/hwmon/lm75.c
@@ -49,6 +49,7 @@ enum lm75_type {              /* keep sorted in alphabetical order */
        tmp100,
        tmp101,
        tmp105,
+       tmp112,
        tmp175,
        tmp275,
        tmp75,
@@ -236,6 +237,12 @@ lm75_probe(struct i2c_client *client, const struct i2c_device_id *id)
                data->sample_time = HZ;
                clr_mask |= 1 << 7;             /* not one-shot mode */
                break;
+       case tmp112:
+               set_mask |= 3 << 5;             /* 12-bit mode */
+               clr_mask |= 1 << 7;             /* not one-shot mode */
+               data->resolution = 12;
+               data->sample_time = HZ / 4;
+               break;
        case tmp105:
        case tmp175:
        case tmp275:
@@ -306,6 +313,7 @@ static const struct i2c_device_id lm75_ids[] = {
        { "tmp100", tmp100, },
        { "tmp101", tmp101, },
        { "tmp105", tmp105, },
+       { "tmp112", tmp112, },
        { "tmp175", tmp175, },
        { "tmp275", tmp275, },
        { "tmp75", tmp75, },


Texas Instruments Belgium SA, Rond Point Schuman 6- Bo?te 5, 1040 Bruxelles. BCE: 0414.207.123. RPM Bruxelles. IBAN: BE83570121895615. Swift: CITIBEBX. TVA BE 0414.207.123



_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [lm-sensors] [PATCH] support tmp112
  2013-08-06 15:29 [lm-sensors] [PATCH] support tmp112 Felten, Lothar
  2013-08-06 16:01 ` Guenter Roeck
  2013-08-06 16:47 ` Felten, Lothar
@ 2013-08-06 19:14 ` Guenter Roeck
  2013-08-17 14:48 ` Guenter Roeck
  3 siblings, 0 replies; 5+ messages in thread
From: Guenter Roeck @ 2013-08-06 19:14 UTC (permalink / raw)
  To: lm-sensors

On Tue, Aug 06, 2013 at 04:47:45PM +0000, Felten, Lothar wrote:
> > Hi Lothar,
> >
> > Signed-off is still at the end :)
> That was me, I can fix that :)
> 
> >
> > Also, looks like your mailer replaced all tabs with spaces, or your editor is set to do the same.
> > checkpatch result is 'total: 5 errors, 8 warnings, 52 lines checked'
> I ran checkpatch.pl, my editor is ok but outlook is probably not a good mail program.
> I changed some settings... let's see...
> 
Same problem. In general, trying to send patches through outlook is a really bad
idea; outlook always believes to know better and screws up e-mail. Actually, if
your outgoing e-mail is through an exchange server, the exchange server may
be the culprit (same reason as with outlook - it believes that it knows better
than its users).

Best is really to send it with "git send-email". If that doesn't work, try to
send it through another mailer. Last resort might be to send it as attachment.
The latter may be neccesary if the exchange server causes the problem.

Thanks,
Guenter

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [lm-sensors] [PATCH] support tmp112
  2013-08-06 15:29 [lm-sensors] [PATCH] support tmp112 Felten, Lothar
                   ` (2 preceding siblings ...)
  2013-08-06 19:14 ` Guenter Roeck
@ 2013-08-17 14:48 ` Guenter Roeck
  3 siblings, 0 replies; 5+ messages in thread
From: Guenter Roeck @ 2013-08-17 14:48 UTC (permalink / raw)
  To: lm-sensors

On 08/06/2013 09:47 AM, Felten, Lothar wrote:
>> Hi Lothar,
>>
>> Signed-off is still at the end :)
> That was me, I can fix that :)
>

Lothar,

any update ?

Thanks,
Guenter


_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-08-17 14:48 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-08-06 15:29 [lm-sensors] [PATCH] support tmp112 Felten, Lothar
2013-08-06 16:01 ` Guenter Roeck
2013-08-06 16:47 ` Felten, Lothar
2013-08-06 19:14 ` Guenter Roeck
2013-08-17 14:48 ` Guenter Roeck

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.