From mboxrd@z Thu Jan 1 00:00:00 1970 From: arno@natisbad.org (Arnaud Ebalard) Date: Sat, 09 Nov 2013 17:39:14 +0000 Subject: [lm-sensors] [PATCH] hwmon: add support for GMT G751 chip in lm75 driver Message-Id: <87bo1th3ul.fsf@natisbad.org> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Guenter Roeck Cc: Jean Delvare , Rob Herring , Pawel Moll , Mark Rutland , Stephen Warren , Ian Campbell , Rob Landley , Grant Likely , Linus Walleij , Arnd Bergmann , devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org ClRoaXMgd2FzIHRlc3RlZCBvbiBhIE5FVEdFQVIgUmVhZHlOQVMgMjEyMCBkZXZpY2UgKE1hcnZl bGwgQXJtYWRhIFhQCmJhc2VkIGJvYXJkLCB2aWEgRFQpLgoKU2lnbmVkLW9mZi1ieTogQXJuYXVk IEViYWxhcmQgPGFybm9AbmF0aXNiYWQub3JnPgotLS0KSGkgR3VlbnRlciwKCkFzIGEgc2lkZSBu b3RlLCBJIHJlbW92ZWQgdGhlIGh1bmsgdGhhdCB3YXMgcHJlc2VudCBpbiBwcmV2aW91cyBwYXRj aCB0bwphZGQgZ210IHRvIHRoZSBsaXN0IG9mIERUIHZlbmRvciBwcmVmaXhlcyBiZWNhdXNlIEkg bm90aWNlZCBzb21lb25lIGhhZAp0YWtlbiBjYXJlOiBodHRwczovL2xrbWwub3JnL2xrbWwvMjAx My85LzEyLzM2NQoKRm9yIHRoZSByZWNvcmRzLCBoZXJlIGlzIHdoYXQgSSBnZXQgb24gbXkgTkFT IHdpdGggdGhlIGF0dGFjaGVkIHBhdGNoOgoKcm9vdEB0aGluOi9zeXMjIHNlbnNvcnMKZzc2Mi1p MmMtMC0zZQpBZGFwdGVyOiBtdjY0eHh4X2kyYyBhZGFwdGVyCmZhbjE6ICAgICAgICA1NDYxIFJQ TSAgKGRpdiA9IDEpCgpnNzYyLWkyYy0wLTQ4CkFkYXB0ZXI6IG12NjR4eHhfaTJjIGFkYXB0ZXIK ZmFuMTogICAgICAgIDU0NjEgUlBNICAoZGl2ID0gMSkKCmc3NjItaTJjLTAtNDkKQWRhcHRlcjog bXY2NHh4eF9pMmMgYWRhcHRlcgpmYW4xOiAgICAgICAgNTQ2MSBSUE0gIChkaXYgPSAxKQoKZzc1 MS1pMmMtMC00YwpBZGFwdGVyOiBtdjY0eHh4X2kyYyBhZGFwdGVyCnRlbXAxOiAgICAgICAgKzMw LjXCsEMgIChoaWdoID0gKzgwLjDCsEMsIGh5c3QgPSArNzUuMMKwQykKCmFybWFkYV90aGVybWFs LXZpcnR1YWwtMApBZGFwdGVyOiBWaXJ0dWFsIGRldmljZQp0ZW1wMTogICAgICAgICszNC4ywrBD Cgpyb290QHRoaW46L3N5cy9idXMvaTJjL2RyaXZlcnMvbG03NS8wLTAwNGMjIGxzCmRyaXZlciAg bW9kYWxpYXMgIHN1YnN5c3RlbSAgICB0ZW1wMV9tYXggICAgICAgdWV2ZW50Cmh3bW9uICAgbmFt ZSAgICAgIHRlbXAxX2lucHV0ICB0ZW1wMV9tYXhfaHlzdAoKcm9vdEB0aGluOi9zeXMvYnVzL2ky Yy9kcml2ZXJzL2xtNzUvMC0wMDRjIyBkbWVzZyB8IGdyZXAgbG0KWyAgMTIwLjk2MDQ3MV0gbG03 NSAwLTAwNGM6IGh3bW9uMzogc2Vuc29yICdnNzUxJwoKCmp1c3QgYnkgYWRkaW5nIHRoZSBmb2xs b3dpbmcgdG8gdGhlIC5kdHM6CgoJZzc1MTogZzc1MUA0YyB7CgkJY29tcGF0aWJsZSA9ICJnbXQs Zzc1MSI7CgkJcmVnID0gPDB4NGM+OwoJfTsKCkRvIG5vdCBoZXNpdGF0ZSB0byB0ZWxsIG1lIGlm IEkgbWlzc2VkIHNvbWV0aGluZy4KCkNoZWVycywKCmErCgogRG9jdW1lbnRhdGlvbi9kZXZpY2V0 cmVlL2JpbmRpbmdzL2kyYy90cml2aWFsLWRldmljZXMudHh0IHwgMSArCiBkcml2ZXJzL2h3bW9u L0tjb25maWcgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgfCAxICsKIGRyaXZl cnMvaHdtb24vbG03NS5jICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICB8IDMg KysrCiAzIGZpbGVzIGNoYW5nZWQsIDUgaW5zZXJ0aW9ucygrKQoKZGlmZiAtLWdpdCBhL0RvY3Vt ZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9pMmMvdHJpdmlhbC1kZXZpY2VzLnR4dCBiL0Rv Y3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9pMmMvdHJpdmlhbC1kZXZpY2VzLnR4dApp bmRleCBhZDZhNzM4Li5jOThlNWEyIDEwMDY0NAotLS0gYS9Eb2N1bWVudGF0aW9uL2RldmljZXRy ZWUvYmluZGluZ3MvaTJjL3RyaXZpYWwtZGV2aWNlcy50eHQKKysrIGIvRG9jdW1lbnRhdGlvbi9k ZXZpY2V0cmVlL2JpbmRpbmdzL2kyYy90cml2aWFsLWRldmljZXMudHh0CkBAIC0zNSw2ICszNSw3 IEBAIGZzbCxtYzEzODkyCQlNQzEzODkyOiBQb3dlciBNYW5hZ2VtZW50IEludGVncmF0ZWQgQ2ly Y3VpdCAoUE1JQykgZm9yIGkuTVgzNS81MQogZnNsLG1tYTg0NTAJCU1NQTg0NTBROiBYdHJpbnNp YyBMb3ctcG93ZXIsIDMtYXhpcyBYdHJpbnNpYyBBY2NlbGVyb21ldGVyCiBmc2wsbXByMTIxCQlN UFIxMjE6IFByb3hpbWl0eSBDYXBhY2l0aXZlIFRvdWNoIFNlbnNvciBDb250cm9sbGVyCiBmc2ws c2d0bDUwMDAJCVNHVEw1MDAwOiBVbHRyYSBMb3ctUG93ZXIgQXVkaW8gQ29kZWMKK2dtdCxnNzUx CQlHNzUxOiBEaWdpdGFsIFRlbXBlcmF0dXJlIFNlbnNvciBhbmQgVGhlcm1hbCBXYXRjaGRvZyB3 aXRoIFR3by1XaXJlIEludGVyZmFjZQogaW5maW5lb24sc2xiOTYzNXR0CUluZmluZW9uIFNMQjk2 MzUgKFNvZnQtKSBJMkMgVFBNIChvbGQgcHJvdG9jb2wsIG1heCAxMDBraHopCiBpbmZpbmVvbixz bGI5NjQ1dHQJSW5maW5lb24gU0xCOTY0NSBJMkMgVFBNIChuZXcgcHJvdG9jb2wsIG1heCA0MDBr aHopCiBtYXhpbSxkczEwNTAJCTUgQml0IFByb2dyYW1tYWJsZSwgUHVsc2UtV2lkdGggTW9kdWxh dG9yCmRpZmYgLS1naXQgYS9kcml2ZXJzL2h3bW9uL0tjb25maWcgYi9kcml2ZXJzL2h3bW9uL0tj b25maWcKaW5kZXggYjNhYjlkNC4uNTJkNTQ4ZiAxMDA2NDQKLS0tIGEvZHJpdmVycy9od21vbi9L Y29uZmlnCisrKyBiL2RyaXZlcnMvaHdtb24vS2NvbmZpZwpAQCAtNjU2LDYgKzY1Niw3IEBAIGNv bmZpZyBTRU5TT1JTX0xNNzUKIAogCQktIEFuYWxvZyBEZXZpY2VzIEFEVDc1CiAJCS0gRGFsbGFz IFNlbWljb25kdWN0b3IgRFM3NSwgRFMxNzc1IGFuZCBEUzc1MDUKKwkJLSBHbG9iYWwgTWl4ZWQt bW9kZSBUZWNobm9sb2d5IChHTVQpIEc3NTEKIAkJLSBNYXhpbSBNQVg2NjI1IGFuZCBNQVg2NjI2 CiAJCS0gTWljcm9jaGlwIE1DUDk4MHgKIAkJLSBOYXRpb25hbCBTZW1pY29uZHVjdG9yIExNNzUs IExNNzVBCmRpZmYgLS1naXQgYS9kcml2ZXJzL2h3bW9uL2xtNzUuYyBiL2RyaXZlcnMvaHdtb24v bG03NS5jCmluZGV4IGMwM2I0OTAuLjdlM2VmMTMgMTAwNjQ0Ci0tLSBhL2RyaXZlcnMvaHdtb24v bG03NS5jCisrKyBiL2RyaXZlcnMvaHdtb24vbG03NS5jCkBAIC0zOSw2ICszOSw3IEBAIGVudW0g bG03NV90eXBlIHsJCS8qIGtlZXAgc29ydGVkIGluIGFscGhhYmV0aWNhbCBvcmRlciAqLwogCWRz MTc3NSwKIAlkczc1LAogCWRzNzUwNSwKKwlnNzUxLAogCWxtNzUsCiAJbG03NWEsCiAJbWF4NjYy NSwKQEAgLTIwOCw2ICsyMDksNyBAQCBsbTc1X3Byb2JlKHN0cnVjdCBpMmNfY2xpZW50ICpjbGll bnQsIGNvbnN0IHN0cnVjdCBpMmNfZGV2aWNlX2lkICppZCkKIAkJZGF0YS0+cmVzb2x1dGlvbiA9 IDEyOwogCQlkYXRhLT5zYW1wbGVfdGltZSA9IEhaIC8gNDsKIAkJYnJlYWs7CisJY2FzZSBnNzUx OgogCWNhc2UgbG03NToKIAljYXNlIGxtNzVhOgogCQlkYXRhLT5yZXNvbHV0aW9uID0gOTsKQEAg LTI5Niw2ICsyOTgsNyBAQCBzdGF0aWMgY29uc3Qgc3RydWN0IGkyY19kZXZpY2VfaWQgbG03NV9p ZHNbXSA9IHsKIAl7ICJkczE3NzUiLCBkczE3NzUsIH0sCiAJeyAiZHM3NSIsIGRzNzUsIH0sCiAJ eyAiZHM3NTA1IiwgZHM3NTA1LCB9LAorCXsgImc3NTEiLCBnNzUxLCB9LAogCXsgImxtNzUiLCBs bTc1LCB9LAogCXsgImxtNzVhIiwgbG03NWEsIH0sCiAJeyAibWF4NjYyNSIsIG1heDY2MjUsIH0s Ci0tIAoxLjguNC5yYzMKCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fXwpsbS1zZW5zb3JzIG1haWxpbmcgbGlzdApsbS1zZW5zb3JzQGxtLXNlbnNvcnMub3Jn Cmh0dHA6Ly9saXN0cy5sbS1zZW5zb3JzLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xtLXNlbnNvcnM From mboxrd@z Thu Jan 1 00:00:00 1970 From: arno@natisbad.org (Arnaud Ebalard) Subject: [PATCH] hwmon: add support for GMT G751 chip in lm75 driver Date: Sat, 09 Nov 2013 18:39:14 +0100 Message-ID: <87bo1th3ul.fsf@natisbad.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Sender: linux-doc-owner@vger.kernel.org To: Guenter Roeck Cc: Jean Delvare , Rob Herring , Pawel Moll , Mark Rutland , Stephen Warren , Ian Campbell , Rob Landley , Grant Likely , Linus Walleij , Arnd Bergmann , devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org List-Id: devicetree@vger.kernel.org This was tested on a NETGEAR ReadyNAS 2120 device (Marvell Armada XP based board, via DT). Signed-off-by: Arnaud Ebalard --- Hi Guenter, As a side note, I removed the hunk that was present in previous patch t= o add gmt to the list of DT vendor prefixes because I noticed someone had taken care: https://lkml.org/lkml/2013/9/12/365 =46or the records, here is what I get on my NAS with the attached patch= : root@thin:/sys# sensors g762-i2c-0-3e Adapter: mv64xxx_i2c adapter fan1: 5461 RPM (div =3D 1) g762-i2c-0-48 Adapter: mv64xxx_i2c adapter fan1: 5461 RPM (div =3D 1) g762-i2c-0-49 Adapter: mv64xxx_i2c adapter fan1: 5461 RPM (div =3D 1) g751-i2c-0-4c Adapter: mv64xxx_i2c adapter temp1: +30.5=C2=B0C (high =3D +80.0=C2=B0C, hyst =3D +75.0=C2=B0= C) armada_thermal-virtual-0 Adapter: Virtual device temp1: +34.2=C2=B0C root@thin:/sys/bus/i2c/drivers/lm75/0-004c# ls driver modalias subsystem temp1_max uevent hwmon name temp1_input temp1_max_hyst root@thin:/sys/bus/i2c/drivers/lm75/0-004c# dmesg | grep lm [ 120.960471] lm75 0-004c: hwmon3: sensor 'g751' just by adding the following to the .dts: g751: g751@4c { compatible =3D "gmt,g751"; reg =3D <0x4c>; }; Do not hesitate to tell me if I missed something. Cheers, a+ Documentation/devicetree/bindings/i2c/trivial-devices.txt | 1 + drivers/hwmon/Kconfig | 1 + drivers/hwmon/lm75.c | 3 +++ 3 files changed, 5 insertions(+) diff --git a/Documentation/devicetree/bindings/i2c/trivial-devices.txt = b/Documentation/devicetree/bindings/i2c/trivial-devices.txt index ad6a738..c98e5a2 100644 --- a/Documentation/devicetree/bindings/i2c/trivial-devices.txt +++ b/Documentation/devicetree/bindings/i2c/trivial-devices.txt @@ -35,6 +35,7 @@ fsl,mc13892 MC13892: Power Management Integrated Cir= cuit (PMIC) for i.MX35/51 fsl,mma8450 MMA8450Q: Xtrinsic Low-power, 3-axis Xtrinsic Acceleromet= er fsl,mpr121 MPR121: Proximity Capacitive Touch Sensor Controller fsl,sgtl5000 SGTL5000: Ultra Low-Power Audio Codec +gmt,g751 G751: Digital Temperature Sensor and Thermal Watchdog with T= wo-Wire Interface infineon,slb9635tt Infineon SLB9635 (Soft-) I2C TPM (old protocol, max= 100khz) infineon,slb9645tt Infineon SLB9645 I2C TPM (new protocol, max 400khz) maxim,ds1050 5 Bit Programmable, Pulse-Width Modulator diff --git a/drivers/hwmon/Kconfig b/drivers/hwmon/Kconfig index b3ab9d4..52d548f 100644 --- a/drivers/hwmon/Kconfig +++ b/drivers/hwmon/Kconfig @@ -656,6 +656,7 @@ config SENSORS_LM75 =20 - Analog Devices ADT75 - Dallas Semiconductor DS75, DS1775 and DS7505 + - Global Mixed-mode Technology (GMT) G751 - Maxim MAX6625 and MAX6626 - Microchip MCP980x - National Semiconductor LM75, LM75A diff --git a/drivers/hwmon/lm75.c b/drivers/hwmon/lm75.c index c03b490..7e3ef13 100644 --- a/drivers/hwmon/lm75.c +++ b/drivers/hwmon/lm75.c @@ -39,6 +39,7 @@ enum lm75_type { /* keep sorted in alphabetical orde= r */ ds1775, ds75, ds7505, + g751, lm75, lm75a, max6625, @@ -208,6 +209,7 @@ lm75_probe(struct i2c_client *client, const struct = i2c_device_id *id) data->resolution =3D 12; data->sample_time =3D HZ / 4; break; + case g751: case lm75: case lm75a: data->resolution =3D 9; @@ -296,6 +298,7 @@ static const struct i2c_device_id lm75_ids[] =3D { { "ds1775", ds1775, }, { "ds75", ds75, }, { "ds7505", ds7505, }, + { "g751", g751, }, { "lm75", lm75, }, { "lm75a", lm75a, }, { "max6625", max6625, }, --=20 1.8.4.rc3