* [lm-sensors] [PATCH] Allow it87.c to handle IT8720
@ 2008-10-06 17:31 Jean-Marc Spaggiari
2008-10-06 17:33 ` Jean-Marc Spaggiari
` (3 more replies)
0 siblings, 4 replies; 10+ messages in thread
From: Jean-Marc Spaggiari @ 2008-10-06 17:31 UTC (permalink / raw)
To: lm-sensors
The goal of this patch is to allow it87.c to handle IT8720 chipset
like IT8718 in order to retreive voltage, temperatures and fans speed
from sensors tools.
JMS
--- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02
09:04:44.000000000 -0400
+++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06
13:27:08.000000000 -0400
@@ -14,6 +14,7 @@
IT8712F Super I/O chip w/LPC interface
IT8716F Super I/O chip w/LPC interface
IT8718F Super I/O chip w/LPC interface
+ IT8720F Super I/O chip w/LPC interface
IT8726F Super I/O chip w/LPC interface
Sis950 A clone of the IT8705F
@@ -50,7 +51,7 @@
#define DRVNAME "it87"
-enum chips { it87, it8712, it8716, it8718 };
+enum chips { it87, it8712, it8716, it8718, it8720 };
static unsigned short force_id;
module_param(force_id, ushort, 0);
@@ -112,6 +113,7 @@ superio_exit(void)
#define IT8716F_DEVID 0x8716
#define IT8718F_DEVID 0x8718
#define IT8726F_DEVID 0x8726
+#define IT8720F_DEVID 0x8720
#define IT87_ACT_REG 0x30
#define IT87_BASE_REG 0x60
@@ -278,7 +280,8 @@ static inline int has_16bit_fans(const s
return (data->type = it87 && data->revision >= 0x03)
|| (data->type = it8712 && data->revision >= 0x08)
|| data->type = it8716
- || data->type = it8718;
+ || data->type = it8718
+ || data->type = it8720;
}
static int it87_probe(struct platform_device *pdev);
@@ -982,6 +985,9 @@ static int __init it87_find(unsigned sho
case IT8718F_DEVID:
sio_data->type = it8718;
break;
+ case IT8720F_DEVID:
+ sio_data->type = it8720;
+ break;
case 0xffff: /* No device at all */
goto exit;
default:
@@ -1040,6 +1046,7 @@ static int __devinit it87_probe(struct p
"it8712",
"it8716",
"it8718",
+ "it8720",
};
res = platform_get_resource(pdev, IORESOURCE_IO, 0);
@@ -1190,7 +1197,7 @@ static int __devinit it87_probe(struct p
}
if (data->type = it8712 || data->type = it8716
- || data->type = it8718) {
+ || data->type = it8718 || data->type = it8720) {
data->vrm = vid_which_vrm();
/* VID reading from Super-I/O config space if available */
data->vid = sio_data->vid_value;
@@ -1571,7 +1578,7 @@ static void __exit sm_it87_exit(void)
MODULE_AUTHOR("Chris Gauthron, "
"Jean Delvare <khali@linux-fr.org>");
-MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8726F, SiS950 driver");
+MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8720F/8726F, SiS950 driver");
module_param(update_vbat, bool, 0);
MODULE_PARM_DESC(update_vbat, "Update vbat if set else return powerup value");
module_param(fix_pwm_polarity, bool, 0);
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 10+ messages in thread* [lm-sensors] [PATCH] Allow it87.c to handle IT8720 @ 2008-10-06 17:33 ` Jean-Marc Spaggiari 0 siblings, 0 replies; 10+ messages in thread From: Jean-Marc Spaggiari @ 2008-10-06 17:33 UTC (permalink / raw) To: LKML, LM Sensors [-- Attachment #1: Type: text/plain, Size: 2915 bytes --] The goal of this patch is to allow it87.c to handle IT8720 chipset like IT8718 in order to retreive voltage, temperatures and fans speed from sensors tools. JMS Patch also attached. --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 09:04:44.000000000 -0400 +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 13:27:08.000000000 -0400 @@ -14,6 +14,7 @@ IT8712F Super I/O chip w/LPC interface IT8716F Super I/O chip w/LPC interface IT8718F Super I/O chip w/LPC interface + IT8720F Super I/O chip w/LPC interface IT8726F Super I/O chip w/LPC interface Sis950 A clone of the IT8705F @@ -50,7 +51,7 @@ #define DRVNAME "it87" -enum chips { it87, it8712, it8716, it8718 }; +enum chips { it87, it8712, it8716, it8718, it8720 }; static unsigned short force_id; module_param(force_id, ushort, 0); @@ -112,6 +113,7 @@ superio_exit(void) #define IT8716F_DEVID 0x8716 #define IT8718F_DEVID 0x8718 #define IT8726F_DEVID 0x8726 +#define IT8720F_DEVID 0x8720 #define IT87_ACT_REG 0x30 #define IT87_BASE_REG 0x60 @@ -278,7 +280,8 @@ static inline int has_16bit_fans(const s return (data->type == it87 && data->revision >= 0x03) || (data->type == it8712 && data->revision >= 0x08) || data->type == it8716 - || data->type == it8718; + || data->type == it8718 + || data->type == it8720; } static int it87_probe(struct platform_device *pdev); @@ -982,6 +985,9 @@ static int __init it87_find(unsigned sho case IT8718F_DEVID: sio_data->type = it8718; break; + case IT8720F_DEVID: + sio_data->type = it8720; + break; case 0xffff: /* No device at all */ goto exit; default: @@ -1040,6 +1046,7 @@ static int __devinit it87_probe(struct p "it8712", "it8716", "it8718", + "it8720", }; res = platform_get_resource(pdev, IORESOURCE_IO, 0); @@ -1190,7 +1197,7 @@ static int __devinit it87_probe(struct p } if (data->type == it8712 || data->type == it8716 - || data->type == it8718) { + || data->type == it8718 || data->type == it8720) { data->vrm = vid_which_vrm(); /* VID reading from Super-I/O config space if available */ data->vid = sio_data->vid_value; @@ -1571,7 +1578,7 @@ static void __exit sm_it87_exit(void) MODULE_AUTHOR("Chris Gauthron, " "Jean Delvare <khali@linux-fr.org>"); -MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8726F, SiS950 driver"); +MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8720F/8726F, SiS950 driver"); module_param(update_vbat, bool, 0); MODULE_PARM_DESC(update_vbat, "Update vbat if set else return powerup value"); module_param(fix_pwm_polarity, bool, 0); [-- Attachment #2: patch --] [-- Type: application/octet-stream, Size: 2490 bytes --] --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 09:04:44.000000000 -0400 +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 13:27:08.000000000 -0400 @@ -14,6 +14,7 @@ IT8712F Super I/O chip w/LPC interface IT8716F Super I/O chip w/LPC interface IT8718F Super I/O chip w/LPC interface + IT8720F Super I/O chip w/LPC interface IT8726F Super I/O chip w/LPC interface Sis950 A clone of the IT8705F @@ -50,7 +51,7 @@ #define DRVNAME "it87" -enum chips { it87, it8712, it8716, it8718 }; +enum chips { it87, it8712, it8716, it8718, it8720 }; static unsigned short force_id; module_param(force_id, ushort, 0); @@ -112,6 +113,7 @@ superio_exit(void) #define IT8716F_DEVID 0x8716 #define IT8718F_DEVID 0x8718 #define IT8726F_DEVID 0x8726 +#define IT8720F_DEVID 0x8720 #define IT87_ACT_REG 0x30 #define IT87_BASE_REG 0x60 @@ -278,7 +280,8 @@ static inline int has_16bit_fans(const s return (data->type == it87 && data->revision >= 0x03) || (data->type == it8712 && data->revision >= 0x08) || data->type == it8716 - || data->type == it8718; + || data->type == it8718 + || data->type == it8720; } static int it87_probe(struct platform_device *pdev); @@ -982,6 +985,9 @@ static int __init it87_find(unsigned sho case IT8718F_DEVID: sio_data->type = it8718; break; + case IT8720F_DEVID: + sio_data->type = it8720; + break; case 0xffff: /* No device at all */ goto exit; default: @@ -1040,6 +1046,7 @@ static int __devinit it87_probe(struct p "it8712", "it8716", "it8718", + "it8720", }; res = platform_get_resource(pdev, IORESOURCE_IO, 0); @@ -1190,7 +1197,7 @@ static int __devinit it87_probe(struct p } if (data->type == it8712 || data->type == it8716 - || data->type == it8718) { + || data->type == it8718 || data->type == it8720) { data->vrm = vid_which_vrm(); /* VID reading from Super-I/O config space if available */ data->vid = sio_data->vid_value; @@ -1571,7 +1578,7 @@ static void __exit sm_it87_exit(void) MODULE_AUTHOR("Chris Gauthron, " "Jean Delvare <khali@linux-fr.org>"); -MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8726F, SiS950 driver"); +MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8720F/8726F, SiS950 driver"); module_param(update_vbat, bool, 0); MODULE_PARM_DESC(update_vbat, "Update vbat if set else return powerup value"); module_param(fix_pwm_polarity, bool, 0); [-- Attachment #3: Type: text/plain, Size: 153 bytes --] _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] Allow it87.c to handle IT8720 @ 2008-10-06 17:33 ` Jean-Marc Spaggiari 0 siblings, 0 replies; 10+ messages in thread From: Jean-Marc Spaggiari @ 2008-10-06 17:33 UTC (permalink / raw) To: LKML, LM Sensors [-- Attachment #1: Type: text/plain, Size: 2915 bytes --] The goal of this patch is to allow it87.c to handle IT8720 chipset like IT8718 in order to retreive voltage, temperatures and fans speed from sensors tools. JMS Patch also attached. --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 09:04:44.000000000 -0400 +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 13:27:08.000000000 -0400 @@ -14,6 +14,7 @@ IT8712F Super I/O chip w/LPC interface IT8716F Super I/O chip w/LPC interface IT8718F Super I/O chip w/LPC interface + IT8720F Super I/O chip w/LPC interface IT8726F Super I/O chip w/LPC interface Sis950 A clone of the IT8705F @@ -50,7 +51,7 @@ #define DRVNAME "it87" -enum chips { it87, it8712, it8716, it8718 }; +enum chips { it87, it8712, it8716, it8718, it8720 }; static unsigned short force_id; module_param(force_id, ushort, 0); @@ -112,6 +113,7 @@ superio_exit(void) #define IT8716F_DEVID 0x8716 #define IT8718F_DEVID 0x8718 #define IT8726F_DEVID 0x8726 +#define IT8720F_DEVID 0x8720 #define IT87_ACT_REG 0x30 #define IT87_BASE_REG 0x60 @@ -278,7 +280,8 @@ static inline int has_16bit_fans(const s return (data->type == it87 && data->revision >= 0x03) || (data->type == it8712 && data->revision >= 0x08) || data->type == it8716 - || data->type == it8718; + || data->type == it8718 + || data->type == it8720; } static int it87_probe(struct platform_device *pdev); @@ -982,6 +985,9 @@ static int __init it87_find(unsigned sho case IT8718F_DEVID: sio_data->type = it8718; break; + case IT8720F_DEVID: + sio_data->type = it8720; + break; case 0xffff: /* No device at all */ goto exit; default: @@ -1040,6 +1046,7 @@ static int __devinit it87_probe(struct p "it8712", "it8716", "it8718", + "it8720", }; res = platform_get_resource(pdev, IORESOURCE_IO, 0); @@ -1190,7 +1197,7 @@ static int __devinit it87_probe(struct p } if (data->type == it8712 || data->type == it8716 - || data->type == it8718) { + || data->type == it8718 || data->type == it8720) { data->vrm = vid_which_vrm(); /* VID reading from Super-I/O config space if available */ data->vid = sio_data->vid_value; @@ -1571,7 +1578,7 @@ static void __exit sm_it87_exit(void) MODULE_AUTHOR("Chris Gauthron, " "Jean Delvare <khali@linux-fr.org>"); -MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8726F, SiS950 driver"); +MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8720F/8726F, SiS950 driver"); module_param(update_vbat, bool, 0); MODULE_PARM_DESC(update_vbat, "Update vbat if set else return powerup value"); module_param(fix_pwm_polarity, bool, 0); [-- Attachment #2: patch --] [-- Type: application/octet-stream, Size: 2490 bytes --] --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 09:04:44.000000000 -0400 +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 13:27:08.000000000 -0400 @@ -14,6 +14,7 @@ IT8712F Super I/O chip w/LPC interface IT8716F Super I/O chip w/LPC interface IT8718F Super I/O chip w/LPC interface + IT8720F Super I/O chip w/LPC interface IT8726F Super I/O chip w/LPC interface Sis950 A clone of the IT8705F @@ -50,7 +51,7 @@ #define DRVNAME "it87" -enum chips { it87, it8712, it8716, it8718 }; +enum chips { it87, it8712, it8716, it8718, it8720 }; static unsigned short force_id; module_param(force_id, ushort, 0); @@ -112,6 +113,7 @@ superio_exit(void) #define IT8716F_DEVID 0x8716 #define IT8718F_DEVID 0x8718 #define IT8726F_DEVID 0x8726 +#define IT8720F_DEVID 0x8720 #define IT87_ACT_REG 0x30 #define IT87_BASE_REG 0x60 @@ -278,7 +280,8 @@ static inline int has_16bit_fans(const s return (data->type == it87 && data->revision >= 0x03) || (data->type == it8712 && data->revision >= 0x08) || data->type == it8716 - || data->type == it8718; + || data->type == it8718 + || data->type == it8720; } static int it87_probe(struct platform_device *pdev); @@ -982,6 +985,9 @@ static int __init it87_find(unsigned sho case IT8718F_DEVID: sio_data->type = it8718; break; + case IT8720F_DEVID: + sio_data->type = it8720; + break; case 0xffff: /* No device at all */ goto exit; default: @@ -1040,6 +1046,7 @@ static int __devinit it87_probe(struct p "it8712", "it8716", "it8718", + "it8720", }; res = platform_get_resource(pdev, IORESOURCE_IO, 0); @@ -1190,7 +1197,7 @@ static int __devinit it87_probe(struct p } if (data->type == it8712 || data->type == it8716 - || data->type == it8718) { + || data->type == it8718 || data->type == it8720) { data->vrm = vid_which_vrm(); /* VID reading from Super-I/O config space if available */ data->vid = sio_data->vid_value; @@ -1571,7 +1578,7 @@ static void __exit sm_it87_exit(void) MODULE_AUTHOR("Chris Gauthron, " "Jean Delvare <khali@linux-fr.org>"); -MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8726F, SiS950 driver"); +MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8720F/8726F, SiS950 driver"); module_param(update_vbat, bool, 0); MODULE_PARM_DESC(update_vbat, "Update vbat if set else return powerup value"); module_param(fix_pwm_polarity, bool, 0); ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [lm-sensors] [PATCH] Allow it87.c to handle IT8720 2008-10-06 17:33 ` Jean-Marc Spaggiari @ 2008-10-07 20:36 ` Jean Delvare -1 siblings, 0 replies; 10+ messages in thread From: Jean Delvare @ 2008-10-07 20:36 UTC (permalink / raw) To: Jean-Marc Spaggiari; +Cc: LKML, LM Sensors Hi Jean-Marc, On Mon, 6 Oct 2008 13:33:56 -0400, Jean-Marc Spaggiari wrote: > The goal of this patch is to allow it87.c to handle IT8720 chipset > like IT8718 in order to retreive voltage, temperatures and fans speed > from sensors tools. > > JMS > > Patch also attached. > > --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 > 09:04:44.000000000 -0400 > +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 > 13:27:08.000000000 -0400 > @@ -14,6 +14,7 @@ > IT8712F Super I/O chip w/LPC interface > IT8716F Super I/O chip w/LPC interface > IT8718F Super I/O chip w/LPC interface > + IT8720F Super I/O chip w/LPC interface > IT8726F Super I/O chip w/LPC interface > Sis950 A clone of the IT8705F > > @@ -50,7 +51,7 @@ > > #define DRVNAME "it87" > > -enum chips { it87, it8712, it8716, it8718 }; > +enum chips { it87, it8712, it8716, it8718, it8720 }; > > static unsigned short force_id; > module_param(force_id, ushort, 0); > @@ -112,6 +113,7 @@ superio_exit(void) > #define IT8716F_DEVID 0x8716 > #define IT8718F_DEVID 0x8718 > #define IT8726F_DEVID 0x8726 > +#define IT8720F_DEVID 0x8720 You have an interesting notion of numeric sorting ;) > #define IT87_ACT_REG 0x30 > #define IT87_BASE_REG 0x60 > > @@ -278,7 +280,8 @@ static inline int has_16bit_fans(const s > return (data->type = it87 && data->revision >= 0x03) > || (data->type = it8712 && data->revision >= 0x08) > || data->type = it8716 > - || data->type = it8718; > + || data->type = it8718 > + || data->type = it8720; > } > > static int it87_probe(struct platform_device *pdev); > @@ -982,6 +985,9 @@ static int __init it87_find(unsigned sho > case IT8718F_DEVID: > sio_data->type = it8718; > break; > + case IT8720F_DEVID: > + sio_data->type = it8720; > + break; > case 0xffff: /* No device at all */ > goto exit; > default: > @@ -1040,6 +1046,7 @@ static int __devinit it87_probe(struct p > "it8712", > "it8716", > "it8718", > + "it8720", > }; > > res = platform_get_resource(pdev, IORESOURCE_IO, 0); > @@ -1190,7 +1197,7 @@ static int __devinit it87_probe(struct p > } > > if (data->type = it8712 || data->type = it8716 > - || data->type = it8718) { > + || data->type = it8718 || data->type = it8720) { > data->vrm = vid_which_vrm(); > /* VID reading from Super-I/O config space if available */ > data->vid = sio_data->vid_value; > @@ -1571,7 +1578,7 @@ static void __exit sm_it87_exit(void) > > MODULE_AUTHOR("Chris Gauthron, " > "Jean Delvare <khali@linux-fr.org>"); > -MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8726F, SiS950 driver"); > +MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8720F/8726F, SiS950 driver"); > module_param(update_vbat, bool, 0); > MODULE_PARM_DESC(update_vbat, "Update vbat if set else return powerup value"); > module_param(fix_pwm_polarity, bool, 0); It seems that you missed one occurrence of it8718-specific code (in function it87_find). /* Read GPIO config and VID value from LDN 7 (GPIO) */ if (chip_type != IT8705F_DEVID) { int reg; superio_select(GPIO); if (chip_type = it8718) sio_data->vid_value = superio_inb(IT87_SIO_VID_REG); reg = superio_inb(IT87_SIO_PINX2_REG); if (reg & (1 << 0)) pr_info("it87: in3 is VCC (+5V)\n"); if (reg & (1 << 1)) pr_info("it87: in7 is VCCH (+5V Stand-By)\n"); } You also need to update Documentation/hwmon/it87 to mention the IT8720F as supported, as well ad drivers/hwmon/Kconfig. Other than that - and the lack of Signed-off-by line - your patch looks OK to me. -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [lm-sensors] [PATCH] Allow it87.c to handle IT8720 @ 2008-10-07 20:36 ` Jean Delvare 0 siblings, 0 replies; 10+ messages in thread From: Jean Delvare @ 2008-10-07 20:36 UTC (permalink / raw) To: Jean-Marc Spaggiari; +Cc: LKML, LM Sensors Hi Jean-Marc, On Mon, 6 Oct 2008 13:33:56 -0400, Jean-Marc Spaggiari wrote: > The goal of this patch is to allow it87.c to handle IT8720 chipset > like IT8718 in order to retreive voltage, temperatures and fans speed > from sensors tools. > > JMS > > Patch also attached. > > --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 > 09:04:44.000000000 -0400 > +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 > 13:27:08.000000000 -0400 > @@ -14,6 +14,7 @@ > IT8712F Super I/O chip w/LPC interface > IT8716F Super I/O chip w/LPC interface > IT8718F Super I/O chip w/LPC interface > + IT8720F Super I/O chip w/LPC interface > IT8726F Super I/O chip w/LPC interface > Sis950 A clone of the IT8705F > > @@ -50,7 +51,7 @@ > > #define DRVNAME "it87" > > -enum chips { it87, it8712, it8716, it8718 }; > +enum chips { it87, it8712, it8716, it8718, it8720 }; > > static unsigned short force_id; > module_param(force_id, ushort, 0); > @@ -112,6 +113,7 @@ superio_exit(void) > #define IT8716F_DEVID 0x8716 > #define IT8718F_DEVID 0x8718 > #define IT8726F_DEVID 0x8726 > +#define IT8720F_DEVID 0x8720 You have an interesting notion of numeric sorting ;) > #define IT87_ACT_REG 0x30 > #define IT87_BASE_REG 0x60 > > @@ -278,7 +280,8 @@ static inline int has_16bit_fans(const s > return (data->type == it87 && data->revision >= 0x03) > || (data->type == it8712 && data->revision >= 0x08) > || data->type == it8716 > - || data->type == it8718; > + || data->type == it8718 > + || data->type == it8720; > } > > static int it87_probe(struct platform_device *pdev); > @@ -982,6 +985,9 @@ static int __init it87_find(unsigned sho > case IT8718F_DEVID: > sio_data->type = it8718; > break; > + case IT8720F_DEVID: > + sio_data->type = it8720; > + break; > case 0xffff: /* No device at all */ > goto exit; > default: > @@ -1040,6 +1046,7 @@ static int __devinit it87_probe(struct p > "it8712", > "it8716", > "it8718", > + "it8720", > }; > > res = platform_get_resource(pdev, IORESOURCE_IO, 0); > @@ -1190,7 +1197,7 @@ static int __devinit it87_probe(struct p > } > > if (data->type == it8712 || data->type == it8716 > - || data->type == it8718) { > + || data->type == it8718 || data->type == it8720) { > data->vrm = vid_which_vrm(); > /* VID reading from Super-I/O config space if available */ > data->vid = sio_data->vid_value; > @@ -1571,7 +1578,7 @@ static void __exit sm_it87_exit(void) > > MODULE_AUTHOR("Chris Gauthron, " > "Jean Delvare <khali@linux-fr.org>"); > -MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8726F, SiS950 driver"); > +MODULE_DESCRIPTION("IT8705F/8712F/8716F/8718F/8720F/8726F, SiS950 driver"); > module_param(update_vbat, bool, 0); > MODULE_PARM_DESC(update_vbat, "Update vbat if set else return powerup value"); > module_param(fix_pwm_polarity, bool, 0); It seems that you missed one occurrence of it8718-specific code (in function it87_find). /* Read GPIO config and VID value from LDN 7 (GPIO) */ if (chip_type != IT8705F_DEVID) { int reg; superio_select(GPIO); if (chip_type == it8718) sio_data->vid_value = superio_inb(IT87_SIO_VID_REG); reg = superio_inb(IT87_SIO_PINX2_REG); if (reg & (1 << 0)) pr_info("it87: in3 is VCC (+5V)\n"); if (reg & (1 << 1)) pr_info("it87: in7 is VCCH (+5V Stand-By)\n"); } You also need to update Documentation/hwmon/it87 to mention the IT8720F as supported, as well ad drivers/hwmon/Kconfig. Other than that - and the lack of Signed-off-by line - your patch looks OK to me. -- Jean Delvare ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [lm-sensors] [PATCH] Allow it87.c to handle IT8720 2008-10-07 20:36 ` Jean Delvare @ 2008-10-22 9:59 ` Jean Delvare -1 siblings, 0 replies; 10+ messages in thread From: Jean Delvare @ 2008-10-22 9:59 UTC (permalink / raw) To: Jean-Marc Spaggiari; +Cc: LKML, LM Sensors Hi Jean-Marc, On Tue, 7 Oct 2008 22:36:23 +0200, Jean Delvare wrote: > Hi Jean-Marc, > > On Mon, 6 Oct 2008 13:33:56 -0400, Jean-Marc Spaggiari wrote: > > The goal of this patch is to allow it87.c to handle IT8720 chipset > > like IT8718 in order to retreive voltage, temperatures and fans speed > > from sensors tools. > > > > JMS > > > > Patch also attached. > > > > --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 09:04:44.000000000 -0400 > > +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 13:27:08.000000000 -0400 > > @@ -14,6 +14,7 @@ <snip, me reviewing the patch> > Other than that - and the lack of Signed-off-by line - your patch looks > OK to me. Any chance to have an updated patch from you by the end of the day, so that it makes it into kernel 2.6.28? The merge window is closing really soon now. -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [lm-sensors] [PATCH] Allow it87.c to handle IT8720 @ 2008-10-22 9:59 ` Jean Delvare 0 siblings, 0 replies; 10+ messages in thread From: Jean Delvare @ 2008-10-22 9:59 UTC (permalink / raw) To: Jean-Marc Spaggiari; +Cc: LKML, LM Sensors Hi Jean-Marc, On Tue, 7 Oct 2008 22:36:23 +0200, Jean Delvare wrote: > Hi Jean-Marc, > > On Mon, 6 Oct 2008 13:33:56 -0400, Jean-Marc Spaggiari wrote: > > The goal of this patch is to allow it87.c to handle IT8720 chipset > > like IT8718 in order to retreive voltage, temperatures and fans speed > > from sensors tools. > > > > JMS > > > > Patch also attached. > > > > --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 09:04:44.000000000 -0400 > > +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 13:27:08.000000000 -0400 > > @@ -14,6 +14,7 @@ <snip, me reviewing the patch> > Other than that - and the lack of Signed-off-by line - your patch looks > OK to me. Any chance to have an updated patch from you by the end of the day, so that it makes it into kernel 2.6.28? The merge window is closing really soon now. -- Jean Delvare ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [lm-sensors] [PATCH] Allow it87.c to handle IT8720 2008-10-06 17:31 [lm-sensors] [PATCH] Allow it87.c to handle IT8720 Jean-Marc Spaggiari 2008-10-06 17:33 ` Jean-Marc Spaggiari @ 2008-10-07 13:55 ` Frank Myhr 2008-10-07 20:21 ` Jean Delvare 2008-10-22 12:11 ` Jean-Marc Spaggiari 3 siblings, 0 replies; 10+ messages in thread From: Frank Myhr @ 2008-10-07 13:55 UTC (permalink / raw) To: lm-sensors Hi Jean-Marc, Your patch looks good to me with a few niggles: > To: > LKML <linux-kernel@vger.kernel.org>, "LM Sensors" > <lm-sensors@lm-sensors.org> I don't know if it's necessary/recommended to submit directly to LKML in addition to the lm-sensors list. (Maybe somebody else can comment on that.) > The goal of this patch is to allow it87.c to handle IT8720 chipset > like IT8718 in order to retreive voltage, temperatures and fans speed > from sensors tools. > > JMS > > Patch also attached. Good description but the "Patch also attached" is superfluous; your [PATCH] subject line already says that. I believe these comments are copied verbatim into the kernel git changelog, so it's important to keep them succinct. More importantly, you need to sign your patch with: Signed-off-by: [your real name] [your email address] --- > --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 > 09:04:44.000000000 -0400 > +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 > 13:27:08.000000000 -0400 Please make sure your mailer is not wrapping lines, it looks here like it is. Regards, -Frank _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [lm-sensors] [PATCH] Allow it87.c to handle IT8720 2008-10-06 17:31 [lm-sensors] [PATCH] Allow it87.c to handle IT8720 Jean-Marc Spaggiari 2008-10-06 17:33 ` Jean-Marc Spaggiari 2008-10-07 13:55 ` Frank Myhr @ 2008-10-07 20:21 ` Jean Delvare 2008-10-22 12:11 ` Jean-Marc Spaggiari 3 siblings, 0 replies; 10+ messages in thread From: Jean Delvare @ 2008-10-07 20:21 UTC (permalink / raw) To: lm-sensors Hi Jean-Marc, Frank, On Tue, 07 Oct 2008 09:55:29 -0400, Frank Myhr wrote: > Hi Jean-Marc, > > Your patch looks good to me with a few niggles: > > > > To: > > LKML <linux-kernel@vger.kernel.org>, "LM Sensors" > > <lm-sensors@lm-sensors.org> > > I don't know if it's necessary/recommended to submit directly to LKML in > addition to the lm-sensors list. (Maybe somebody else can comment on that.) My personal opinion is that flooding LKML with patches which can be handled on specific mailing lists is pointless. I don't expect anyone on LKML to pay attention to this patch, so it's only adding to the 160k+ mails that go to the LKML every year. But I've been bashed in the past for asking people to not post to LKML, so now I'm keeping quiet. > > > > The goal of this patch is to allow it87.c to handle IT8720 chipset > > like IT8718 in order to retreive voltage, temperatures and fans speed > > from sensors tools. > > > > JMS > > > > Patch also attached. > > Good description but the "Patch also attached" is superfluous; your > [PATCH] subject line already says that. I believe these comments are > copied verbatim into the kernel git changelog, so it's important to keep > them succinct. > > More importantly, you need to sign your patch with: > > Signed-off-by: [your real name] [your email address] > --- > > > > --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 > > 09:04:44.000000000 -0400 > > +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 > > 13:27:08.000000000 -0400 > > Please make sure your mailer is not wrapping lines, it looks here like > it is. And it is also converting tabs to space, so I can't apply the patch. At least the attached copy doesn't have this problem, but if you can get your e-mail client to behave so that you no longer have to provide the patches as attachments, it's preferred. I second everything else Frank said above. -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [lm-sensors] [PATCH] Allow it87.c to handle IT8720 2008-10-06 17:31 [lm-sensors] [PATCH] Allow it87.c to handle IT8720 Jean-Marc Spaggiari ` (2 preceding siblings ...) 2008-10-07 20:21 ` Jean Delvare @ 2008-10-22 12:11 ` Jean-Marc Spaggiari 3 siblings, 0 replies; 10+ messages in thread From: Jean-Marc Spaggiari @ 2008-10-22 12:11 UTC (permalink / raw) To: lm-sensors [-- Attachment #1.1: Type: text/plain, Size: 1009 bytes --] Hi Jean, I will do it this afternoon (EST). JM 2008/10/22 Jean Delvare <khali@linux-fr.org> > Hi Jean-Marc, > > On Tue, 7 Oct 2008 22:36:23 +0200, Jean Delvare wrote: > > Hi Jean-Marc, > > > > On Mon, 6 Oct 2008 13:33:56 -0400, Jean-Marc Spaggiari wrote: > > > The goal of this patch is to allow it87.c to handle IT8720 chipset > > > like IT8718 in order to retreive voltage, temperatures and fans speed > > > from sensors tools. > > > > > > JMS > > > > > > Patch also attached. > > > > > > --- linux-2.6.27-rc8/drivers/hwmon/it87.c.orig 2008-10-02 > 09:04:44.000000000 -0400 > > > +++ linux-2.6.27-rc8/drivers/hwmon/it87.c 2008-10-06 > 13:27:08.000000000 -0400 > > > @@ -14,6 +14,7 @@ > > <snip, me reviewing the patch> > > > Other than that - and the lack of Signed-off-by line - your patch looks > > OK to me. > > Any chance to have an updated patch from you by the end of the day, so > that it makes it into kernel 2.6.28? The merge window is closing really > soon now. > > -- > Jean Delvare > [-- Attachment #1.2: Type: text/html, Size: 1522 bytes --] [-- Attachment #2: Type: text/plain, Size: 153 bytes --] _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2008-10-22 12:11 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2008-10-06 17:31 [lm-sensors] [PATCH] Allow it87.c to handle IT8720 Jean-Marc Spaggiari 2008-10-06 17:33 ` Jean-Marc Spaggiari 2008-10-06 17:33 ` Jean-Marc Spaggiari 2008-10-07 20:36 ` [lm-sensors] " Jean Delvare 2008-10-07 20:36 ` Jean Delvare 2008-10-22 9:59 ` Jean Delvare 2008-10-22 9:59 ` Jean Delvare 2008-10-07 13:55 ` Frank Myhr 2008-10-07 20:21 ` Jean Delvare 2008-10-22 12:11 ` Jean-Marc Spaggiari
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.