From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Mike Gorchak" Date: Sun, 22 Jan 2012 18:43:08 +0000 Subject: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: MIME-Version: 1 Content-Type: multipart/mixed; boundary="===============3947660060762313545==" List-Id: To: lm-sensors@vger.kernel.org This is a multi-part message in MIME format. --===============3947660060762313545== Content-Type: multipart/alternative; boundary="----=_NextPart_000_0000_01CCD946.7FEB0AB0" This is a multi-part message in MIME format. ------=_NextPart_000_0000_01CCD946.7FEB0AB0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Hi All, I've recently found that lm-sensors doesn't have support for Global Mixed-mode Technology Inc. (GMT) G781 and G7811 temperature sensors. Overview datasheets are available on manufacturer's site: http://www.gmt.com.tw/ , registers datasheet is available here: www.datasheetcatalog.org/datasheets2/35/350422_1.pdf It seems that this device is registers compatible with ADM1032. SMBus manufacturer id 0x47, device ids 0x01 and 0x03. Thanks! ------=_NextPart_000_0000_01CCD946.7FEB0AB0 Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Hi=20 All,

I've = recently found=20 that lm-sensors doesn't have support for Global Mixed-mode Technology = Inc. (GMT)=20 G781 and G7811 temperature sensors.

Overview datasheets=20 are available on manufacturer's site: http://www.gmt.com.tw/ , = registers=20 datasheet is available here: www.= datasheetcatalog.org/datasheets2/35/350422_1.pdf

It = seems that this=20 device is registers compatible with ADM1032. SMBus = manufacturer id=20 0x47, device ids 0x01 and 0x03.

Thanks!

------=_NextPart_000_0000_01CCD946.7FEB0AB0-- --===============3947660060762313545== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors --===============3947660060762313545==-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Sun, 22 Jan 2012 21:11:36 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <20120122211136.GB12747@ericsson.com> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Sun, Jan 22, 2012 at 01:43:08PM -0500, Mike Gorchak wrote: > Hi All, > > I've recently found that lm-sensors doesn't have support for Global Mixed-mode > Technology Inc. (GMT) G781 and G7811 temperature sensors. > > Overview datasheets are available on manufacturer's site: http://www.gmt.com.tw > / , registers datasheet is available here: www.datasheetcatalog.org/datasheets2 > /35/350422_1.pdf > > It seems that this device is registers compatible with ADM1032. SMBus > manufacturer id 0x47, device ids 0x01 and 0x03. > Should be safe to add support to the lm90 driver. Jean, any thoughts/comments ? I seem to be unable to find a datasheet for G7811, though. We would need that to know its I2C bus address, and for the correct device id (0x03 ?). Guenter _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Sun, 22 Jan 2012 23:02:44 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <20120122230244.GA12772@ericsson.com> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Sun, Jan 22, 2012 at 04:11:36PM -0500, Guenter Roeck wrote: > On Sun, Jan 22, 2012 at 01:43:08PM -0500, Mike Gorchak wrote: > > Hi All, > > > > I've recently found that lm-sensors doesn't have support for Global Mixed-mode > > Technology Inc. (GMT) G781 and G7811 temperature sensors. > > > > Overview datasheets are available on manufacturer's site: http://www.gmt.com.tw > > / , registers datasheet is available here: www.datasheetcatalog.org/datasheets2 > > /35/350422_1.pdf > > > > It seems that this device is registers compatible with ADM1032. SMBus > > manufacturer id 0x47, device ids 0x01 and 0x03. > > > Should be safe to add support to the lm90 driver. Jean, any thoughts/comments ? > > I seem to be unable to find a datasheet for G7811, though. We would need that to know > its I2C bus address, and for the correct device id (0x03 ?). > Hi Mike, never mind; no idea if G7811 ever really existed. I found version 1.3 of the datasheet which describes both G781 and G781-1. Can you test the driver and sensors-detect if I send you a pointer to the code ? Thanks, Guenter _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Mike Gorchak" Date: Mon, 23 Jan 2012 06:02:20 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <10D1F689E9954A0DA6125C87826FC674@mlv.local> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org Hello, Guenter! GR> never mind; no idea if G7811 ever really existed. I found version 1.3 GR> of the datasheet which describes both G781 and G781-1. Can you test the GR> driver and sensors-detect if I send you a pointer to the code ? It seems that this patch is ok, except for one thing. Let me explain what I did. I added support for ACPI SMBus connected to Embedded Controller to the lm-sensors (for a netbook without proper ACPI support). There are two G781 sensors connected to this bus, one to the address 0x4C and one to the 0x4D. Both sensors have 0x01 as device id. But patch assumes that only G781-1 can be connected to the 0x4D address. Thanks for the fast answer! :) With best regards, Mike Gorchak. E-mail: lestat@i.com.ua _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Mon, 23 Jan 2012 07:06:00 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <20120123070600.GA15210@ericsson.com> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Mon, Jan 23, 2012 at 01:02:20AM -0500, Mike Gorchak wrote: > Hello, Guenter! > > GR> never mind; no idea if G7811 ever really existed. I found version 1.3 > GR> of the datasheet which describes both G781 and G781-1. Can you test the > GR> driver and sensors-detect if I send you a pointer to the code ? > > It seems that this patch is ok, except for one thing. > > Let me explain what I did. I added support for ACPI SMBus connected to > Embedded Controller to the lm-sensors (for a netbook without proper ACPI > support). There are two G781 sensors connected to this bus, one to the > address 0x4C and one to the 0x4D. Both sensors have 0x01 as device id. But > patch assumes that only G781-1 can be connected to the 0x4D address. > > Thanks for the fast answer! :) > > With best regards, Mike Gorchak. E-mail: lestat@i.com.ua > Hi Mike, can you send me the output of i2cdump for both chips ? Thanks, Guenter _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Mike Gorchak" Date: Mon, 23 Jan 2012 07:13:21 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <9B2FC1AEC2BE42A2A588513075F1D090@mlv.local> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org Hello, Guenter! GR>>> never mind; no idea if G7811 ever really existed. I found version 1.3 GR>>> of the datasheet which describes both G781 and G781-1. Can you test GR>>> the driver and sensors-detect if I send you a pointer to the code ? ??>> ??>> It seems that this patch is ok, except for one thing. ??>> ??>> Let me explain what I did. I added support for ACPI SMBus connected to ??>> Embedded Controller to the lm-sensors (for a netbook without proper ??>> ACPI support). There are two G781 sensors connected to this bus, one ??>> to the address 0x4C and one to the 0x4D. Both sensors have 0x01 as ??>> device id. But patch assumes that only G781-1 can be connected to the ??>> 0x4D address. ??>> ??>> Thanks for the fast answer! :) ??>> ??>> With best regards, Mike Gorchak. E-mail: lestat@i.com.ua ??>> GR> Hi Mike, GR> can you send me the output of i2cdump for both chips ? Of course, but output is produced by another utility with similair behavior. Found device presence at address 0x4C | 00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F ------------------------------------------------------------------------ 00 | 14 23 00 00 04 73 C0 73 C0 00 04 73 C0 73 C0 14 .#...s.s...s.s.. 10 | 30 00 00 00 00 14 14 14 14 73 14 14 14 14 14 14 0........s...... 20 | 73 0A 00 14 14 14 14 14 14 14 14 14 14 14 14 14 s............... 30 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ 40 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ 50 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ 60 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ 70 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ 80 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ 90 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ A0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ B0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ C0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ D0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ E0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 ................ F0 | 14 06 FF 00 FF 00 00 FF 14 14 14 14 14 14 47 01 ..............G. Found device presence at address 0x4D | 00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F ------------------------------------------------------------------------ 00 | 11 0D 80 00 08 55 00 55 00 00 08 55 00 55 00 11 .....U.U...U.U.. 10 | D0 00 00 00 00 11 11 11 11 55 11 11 11 11 11 11 .........U...... 20 | 55 0A 00 11 11 11 11 11 11 11 11 11 11 11 11 11 U............... 30 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ 40 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ 50 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ 60 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ 70 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ 80 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ 90 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ A0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ B0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ C0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ D0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ E0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 ................ F0 | 11 06 FF 00 FF E0 00 FF 11 11 11 11 11 11 47 01 ..............G. Thanks! With best regards, Mike Gorchak. E-mail: lestat@i.com.ua _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Mike Gorchak" Date: Mon, 23 Jan 2012 07:16:26 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <32FEE972767F44CBBFD61EDD7ECFF3AE@mlv.local> MIME-Version: 1 Content-Type: multipart/mixed; boundary="----=_NextPart_000_0076_01CCD9AF.AF1C3F70" List-Id: References: In-Reply-To: To: lm-sensors@vger.kernel.org This is a multi-part message in MIME format. ------=_NextPart_000_0076_01CCD9AF.AF1C3F70 Content-Type: text/plain; format=flowed; charset="iso-8859-1"; reply-type=original Content-Transfer-Encoding: 7bit Hello, Guenter! GR> can you send me the output of i2cdump for both chips ? Sorry for the previous message, it looks like dump has been clobbered. The same in the attachment. With best regards, Mike Gorchak. E-mail: lestat@i.com.ua ------=_NextPart_000_0076_01CCD9AF.AF1C3F70 Content-Type: text/plain; format=flowed; name="ec_smbus.txt"; reply-type=original Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename="ec_smbus.txt" Found SMBUS0=0A= Found Smart Battery Charger at address 0x09=0A= | 00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F=0A= = ------------------------------------------------------------------------=0A= 00 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 10 | -- 01 -- 32 -- -- -- -- -- -- -- -- -- -- -- -- = -.-2------------=0A= 20 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 30 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 40 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 50 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 60 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 70 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 80 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 90 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= A0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= B0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= C0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= D0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= E0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= F0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= Found Smart Battery Selector/Manager at address 0x0A=0A= | 00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F=0A= = ------------------------------------------------------------------------=0A= 00 | -- 01 11 -- 11 -- -- -- -- -- -- -- -- -- -- -- = -..-.-----------=0A= 10 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 20 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 30 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 40 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 50 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 60 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 70 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 80 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 90 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= A0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= B0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= C0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= D0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= E0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= F0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= Found Smart Battery at address 0x0B=0A= | 00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F=0A= = ------------------------------------------------------------------------=0A= 00 | EF 53 0A 01 00 FF -- -- 41 8A 1D 25 0A 63 63 22 = .S....--A..%.cc"=0A= 10 | 3E 25 28 FF DC C0 47 2F 3E 30 10 B2 A2 -- -- -- = >%(...G/>0...---=0A= 20 | 0A 0B 04 03 -- -- -- -- -- -- -- -- -- -- -- 0D = ....-----------.=0A= 30 | 08 04 -- -- -- -- -- -- -- -- -- -- 00 00 02 C0 = ..----------....=0A= 40 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- FF = ---------------.=0A= 50 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 60 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 70 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 80 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= 90 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= A0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= B0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= C0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= D0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= E0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= F0 | -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- = ----------------=0A= Found Sensor Chip at address 0x4C=0A= | 00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F=0A= = ------------------------------------------------------------------------=0A= 00 | 14 23 00 00 04 73 C0 73 C0 00 04 73 C0 73 C0 14 = .#...s.s...s.s..=0A= 10 | 30 00 00 00 00 14 14 14 14 73 14 14 14 14 14 14 = 0........s......=0A= 20 | 73 0A 00 14 14 14 14 14 14 14 14 14 14 14 14 14 = s...............=0A= 30 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= 40 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= 50 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= 60 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= 70 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= 80 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= 90 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= A0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= B0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= C0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= D0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= E0 | 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 14 = ................=0A= F0 | 14 06 FF 00 FF 00 00 FF 14 14 14 14 14 14 47 01 = ..............G.=0A= Found Sensor Chip at address 0x4D=0A= | 00 01 02 03 04 05 06 07 08 09 0A 0B 0C 0D 0E 0F=0A= = ------------------------------------------------------------------------=0A= 00 | 11 0D 80 00 08 55 00 55 00 00 08 55 00 55 00 11 = .....U.U...U.U..=0A= 10 | D0 00 00 00 00 11 11 11 11 55 11 11 11 11 11 11 = .........U......=0A= 20 | 55 0A 00 11 11 11 11 11 11 11 11 11 11 11 11 11 = U...............=0A= 30 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= 40 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= 50 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= 60 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= 70 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= 80 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= 90 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= A0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= B0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= C0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= D0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= E0 | 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 11 = ................=0A= F0 | 11 06 FF 00 FF E0 00 FF 11 11 11 11 11 11 47 01 = ..............G.=0A= ------=_NextPart_000_0076_01CCD9AF.AF1C3F70 Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors ------=_NextPart_000_0076_01CCD9AF.AF1C3F70-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Date: Mon, 23 Jan 2012 07:55:03 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <20120123085503.7e4ea1b0@endymion.delvare> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Mon, 23 Jan 2012 08:02:20 +0200, Mike Gorchak wrote: > Hello, Guenter! > > GR> never mind; no idea if G7811 ever really existed. I found version 1.3 > GR> of the datasheet which describes both G781 and G781-1. Can you test the > GR> driver and sensors-detect if I send you a pointer to the code ? > > It seems that this patch is ok, except for one thing. > > Let me explain what I did. I added support for ACPI SMBus connected to > Embedded Controller to the lm-sensors (for a netbook without proper ACPI > support). There are two G781 sensors connected to this bus, one to the > address 0x4C and one to the 0x4D. Both sensors have 0x01 as device id. But > patch assumes that only G781-1 can be connected to the 0x4D address. If they followed a naming logic similar to National Semiconductor, G781 will have address 0x4C and G781-1 will have address 0x4D. They are basically the exact same chip except for the address, so the driver doesn't even need to know, other than the fact that both addresses 0x4C and 0x4D are possible. -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Mike Gorchak" Date: Mon, 23 Jan 2012 07:59:43 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <67FF5F74DE294890BC7E27A1432FB0AC@mlv.local> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org Hello, Jean! GR>>> never mind; no idea if G7811 ever really existed. I found version 1.3 GR>>> of the datasheet which describes both G781 and G781-1. Can you test GR>>> the driver and sensors-detect if I send you a pointer to the code ? ??>> It seems that this patch is ok, except for one thing. ??>> Let me explain what I did. I added support for ACPI SMBus connected to ??>> Embedded Controller to the lm-sensors (for a netbook without proper ??>> ACPI support). There are two G781 sensors connected to this bus, one ??>> to the address 0x4C and one to the 0x4D. Both sensors have 0x01 as ??>> device id. But patch assumes that only G781-1 can be connected to the ??>> 0x4D address. JD> If they followed a naming logic similar to National Semiconductor, G781 JD> will have address 0x4C and G781-1 will have address 0x4D. They are JD> basically the exact same chip except for the address, so the driver JD> doesn't even need to know, other than the fact that both addresses 0x4C JD> and 0x4D are possible. Now driver checks only for 0x03 device id at address 0x4D, which is not correct. 0x01 device id can be present at 0x4D address too. That's all I'd like to say :) With best regards, Mike Gorchak. E-mail: lestat@i.com.ua _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Date: Mon, 23 Jan 2012 08:36:49 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <20120123093649.6061e036@endymion.delvare> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Sun, 22 Jan 2012 13:11:36 -0800, Guenter Roeck wrote: > On Sun, Jan 22, 2012 at 01:43:08PM -0500, Mike Gorchak wrote: > > Hi All, > > > > I've recently found that lm-sensors doesn't have support for Global Mixed-mode > > Technology Inc. (GMT) G781 and G7811 temperature sensors. > > > > Overview datasheets are available on manufacturer's site: http://www.gmt.com.tw > > / , registers datasheet is available here: www.datasheetcatalog.org/datasheets2 > > /35/350422_1.pdf > > > > It seems that this device is registers compatible with ADM1032. SMBus > > manufacturer id 0x47, device ids 0x01 and 0x03. > > Should be safe to add support to the lm90 driver. Jean, any thoughts/comments ? Indeed the G781 seems compatible with the other chips supported by the lm90 driver, adding support to that driver sounds good. I'll gladly review any patch. > I seem to be unable to find a datasheet for G7811, though. We would need that to know > its I2C bus address, and for the correct device id (0x03 ?). GMT have a product sheet for the G7811 so it probably exists: http://www.gmt.com.tw/product/datasheet/EDS-7811.pdf Not enough to check for compatibility or write a proper driver though. But its lack of remote sensor means it can't be fully compatible with the ADM1032 anyway. Until we receive a formal support request and get a datasheet (or at the very least a register dump) I think we can just ignore the G7811. -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Date: Mon, 23 Jan 2012 09:11:07 +0000 Subject: Re: [lm-sensors] Support for SMBus based GMT G781 and G7811 temperature sensors. Message-Id: <20120123101107.4ba687eb@endymion.delvare> List-Id: References: In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: lm-sensors@vger.kernel.org On Mon, 23 Jan 2012 09:59:43 +0200, Mike Gorchak wrote: > Hello, Jean! > > GR>>> never mind; no idea if G7811 ever really existed. I found version 1.3 > GR>>> of the datasheet which describes both G781 and G781-1. Can you test > GR>>> the driver and sensors-detect if I send you a pointer to the code ? > ??>> It seems that this patch is ok, except for one thing. > ??>> Let me explain what I did. I added support for ACPI SMBus connected to > ??>> Embedded Controller to the lm-sensors (for a netbook without proper > ??>> ACPI support). There are two G781 sensors connected to this bus, one > ??>> to the address 0x4C and one to the 0x4D. Both sensors have 0x01 as > ??>> device id. But patch assumes that only G781-1 can be connected to the > ??>> 0x4D address. > JD> If they followed a naming logic similar to National Semiconductor, G781 > JD> will have address 0x4C and G781-1 will have address 0x4D. They are > JD> basically the exact same chip except for the address, so the driver > JD> doesn't even need to know, other than the fact that both addresses 0x4C > JD> and 0x4D are possible. > > Now driver checks only for 0x03 device id at address 0x4D, which is not > correct. 0x01 device id can be present at 0x4D address too. That's all I'd > like to say :) Yes, yes, I agree and I'm sure Guenter does as well. Just give me a moment to review his two patches, and then wait for him to wake up, get dressed, have coffee or whatever, and resubmit ;) -- Jean Delvare _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors