[lm-sensors] NCT6793D

[lm-sensors] NCT6793D

[Grazvydas Ignotas]

SGksCgpJIGhhdmUgTkNUNjc5M0QgWzFdIG9uIGEgTVNJICJaMTcwQSBQQyBNQVRFIChNUy03OTcx
KSIgYm9hcmQuIFRoZXJlCmRvZXNuJ3Qgc2VlbSB0byBiZSBhbnkgZHJpdmVyIHRoYXQgd291bGQg
d29yayBvdXQgb2YgdGhlIGJveCwgYW5kCm5jdDY3NzUgc2VlbXMgdG8gYmUgdGhlIGNsb3Nlc3Qg
b25lLiBDdXJpb3VzbHkgdGhpcyBjaGlwIGlzIG5vdCBsaXN0ZWQKb24gdGhlIG1hbnVmYWN0dXJl
cidzIHdlYnNpdGUuCgpMb2FkaW5nIHRoZSBuY3Q2Nzc1IGRyaXZlciB3aXRoIGRlYnVnIHJlc3Vs
dHMgaW46ClsgIDEyNS4xNTUyOTBdIG5jdDY3NzU6IHVuc3VwcG9ydGVkIGNoaXAgSUQ6IDB4ZDEy
MQoKUGF0Y2hpbmcgdGhhdCBjaGVjayB0byB0cmVhdCB0aGlzIGNoaXAgYXMgbmN0Njc5MiBwcm9k
dWNlcyBzZWVtaW5nbHkKd29ya2luZyBkcml2ZXIuIFRoZXJlIGFyZSB2YXJpb3VzIHZvbHRhZ2Vz
IHJlcG9ydGVkIChkb24ndCBrbm93IHdoYXQKdGhleSBjb3JyZXNwb25kIHRvKSwgY29vbGVyIGNv
bm5lY3RlZCB0byBDUFVGQU4xIGFwcGVhcnMgYXMgImZhbjIiIGFuZApnaXZlcyBjb3JyZWN0IGxv
b2tpbmcgUlBNLCAiUEVDSSBBZ2VudCAwIiByZWFkaW5nIGlzIGNsb3NlIHRvCmNvcmV0ZW1wLCBi
dXQgYWxsIG90aGVyIHRlbXBlcmF0dXJlIG91dHB1dHMgZG9uJ3Qgc2VlbSB0byBiZSB3b3JraW5n
CihqdXN0IHNob3dpbmcgc3RhdGljIHZhbHVlcykuIEkgd2FzIGFibGUgdG8gc3VjY2Vzc2Z1bGx5
IGNvbnRyb2wgdGhlCkNQVSBmYW4gdXNpbmcgcHdtIHN5c2ZzIGtub2JzLgoKQWxsIGluIGFsbCBp
dCBzZWVtcyB0byBiZSB1c2VmdWwgb24gdGhpcyBjaGlwLCBwcm9iYWJseSB3b3J0aCBlbmFibGlu
ZwppbiBtYWlubGluZS4gRmVlbCBmcmVlIHRvIENDIG1lIHdoZW4vaWYgdGhlcmUgYXJlIGFueSBw
YXRjaGVzIHRvIHRlc3QuCgpbMV0gaHR0cDovL25vdGF6LmdwMnguZGUvaW1nL21lc3MvaHcvTkNU
Njc5M0QuanBnCgpHcmHFvnZ5ZGFzCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fXwpsbS1zZW5zb3JzIG1haWxpbmcgbGlzdApsbS1zZW5zb3JzQGxtLXNlbnNv
cnMub3JnCmh0dHA6Ly9saXN0cy5sbS1zZW5zb3JzLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xtLXNl
bnNvcnM

Re: [lm-sensors] NCT6793D

[Guenter Roeck]

Hi,

On 08/26/2015 05:57 PM, Grazvydas Ignotas wrote:
> Hi,
>
> I have NCT6793D [1] on a MSI "Z170A PC MATE (MS-7971)" board. There
> doesn't seem to be any driver that would work out of the box, and
> nct6775 seems to be the closest one. Curiously this chip is not listed
> on the manufacturer's website.
>
> Loading the nct6775 driver with debug results in:
> [  125.155290] nct6775: unsupported chip ID: 0xd121
>
> Patching that check to treat this chip as nct6792 produces seemingly
> working driver. There are various voltages reported (don't know what
> they correspond to), cooler connected to CPUFAN1 appears as "fan2" and
> gives correct looking RPM, "PECI Agent 0" reading is close to
> coretemp, but all other temperature outputs don't seem to be working
> (just showing static values). I was able to successfully control the
> CPU fan using pwm sysfs knobs.
>
> All in all it seems to be useful on this chip, probably worth enabling
> in mainline. Feel free to CC me when/if there are any patches to test.
>
> [1] http://notaz.gp2x.de/img/mess/hw/NCT6793D.jpg
>

We'll need a little more information. It may well be that the chip is
all the same as the NCT6792D, but who knows.

I'll try to get a datasheet. We'll see how it goes.

Thanks,
Guenter



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

Re: [lm-sensors] NCT6793D

[Guenter Roeck]

Hi,

On 08/26/2015 05:57 PM, Grazvydas Ignotas wrote:
> Hi,
>
> I have NCT6793D [1] on a MSI "Z170A PC MATE (MS-7971)" board. There
> doesn't seem to be any driver that would work out of the box, and
> nct6775 seems to be the closest one. Curiously this chip is not listed
> on the manufacturer's website.
>
> Loading the nct6775 driver with debug results in:
> [  125.155290] nct6775: unsupported chip ID: 0xd121
>
> Patching that check to treat this chip as nct6792 produces seemingly
> working driver. There are various voltages reported (don't know what
> they correspond to), cooler connected to CPUFAN1 appears as "fan2" and
> gives correct looking RPM, "PECI Agent 0" reading is close to
> coretemp, but all other temperature outputs don't seem to be working
> (just showing static values). I was able to successfully control the
> CPU fan using pwm sysfs knobs.
>
> All in all it seems to be useful on this chip, probably worth enabling
> in mainline. Feel free to CC me when/if there are any patches to test.
>

Turns out the NCT6793D is almost register compatible to NCT6792D. Only
difference I can find is that the NCT6792D has "3-wire enable" registers
for each fan. Those registers do not exist on NCT6793D.

Please try the driver at https://github.com/groeck/nct6775 and let me know
if it works.

Thanks,
Guenter


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

Re: [lm-sensors] NCT6793D

[Grazvydas Ignotas]

T24gTW9uLCBBdWcgMzEsIDIwMTUgYXQgNTo1OSBBTSwgR3VlbnRlciBSb2VjayA8bGludXhAcm9l
Y2stdXMubmV0PiB3cm90ZToKPiBUdXJucyBvdXQgdGhlIE5DVDY3OTNEIGlzIGFsbW9zdCByZWdp
c3RlciBjb21wYXRpYmxlIHRvIE5DVDY3OTJELiBPbmx5Cj4gZGlmZmVyZW5jZSBJIGNhbiBmaW5k
IGlzIHRoYXQgdGhlIE5DVDY3OTJEIGhhcyAiMy13aXJlIGVuYWJsZSIgcmVnaXN0ZXJzCj4gZm9y
IGVhY2ggZmFuLiBUaG9zZSByZWdpc3RlcnMgZG8gbm90IGV4aXN0IG9uIE5DVDY3OTNELgo+Cj4g
UGxlYXNlIHRyeSB0aGUgZHJpdmVyIGF0IGh0dHBzOi8vZ2l0aHViLmNvbS9ncm9lY2svbmN0Njc3
NSBhbmQgbGV0IG1lIGtub3cKPiBpZiBpdCB3b3Jrcy4KCkl0IGRvZXNuJ3QsIHN0aWxsIHNheXMg
InVuc3VwcG9ydGVkIGNoaXAgSUQ6IDB4ZDEyMSIuCgpXaXRoIHBhdGNoZWQgSUQgY2hlY2sgSSd2
ZSBhbHNvIG5vdGljZWQgdGhhdCBwd21YX3N0ZXBfZG93bl90aW1lIGFuZApwd21YX3N0ZXBfdXBf
dGltZSBhcmUgbWl4ZWQgdXAsIGRvbid0IGtub3cgaWYgdGhpcyBpcyBzb21ldGhpbmcKTkNUNjc5
M0Qgc3BlY2lmaWMsIGxvb2tzIG1vcmUgbGlrZSBnZW5lcmFsIGRyaXZlciBidWcuCgpHcmHFvnZ5
ZGFzCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsbS1z
ZW5zb3JzIG1haWxpbmcgbGlzdApsbS1zZW5zb3JzQGxtLXNlbnNvcnMub3JnCmh0dHA6Ly9saXN0
cy5sbS1zZW5zb3JzLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xtLXNlbnNvcnM

Re: [lm-sensors] NCT6793D

[Guenter Roeck]

On 08/31/2015 03:33 PM, Grazvydas Ignotas wrote:
> On Mon, Aug 31, 2015 at 5:59 AM, Guenter Roeck <linux@roeck-us.net> wrote:
>> Turns out the NCT6793D is almost register compatible to NCT6792D. Only
>> difference I can find is that the NCT6792D has "3-wire enable" registers
>> for each fan. Those registers do not exist on NCT6793D.
>>
>> Please try the driver at https://github.com/groeck/nct6775 and let me know
>> if it works.
>
> It doesn't, still says "unsupported chip ID: 0xd121".
>
Oops - sorry, now fixed.

> With patched ID check I've also noticed that pwmX_step_down_time and
> pwmX_step_up_time are mixed up, don't know if this is something
> NCT6793D specific, looks more like general driver bug.
>
Thanks for noticing. I'll look into it.

Thanks,
Guenter


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

Re: [lm-sensors] NCT6793D

[Guenter Roeck]

On 08/31/2015 03:33 PM, Grazvydas Ignotas wrote:
> On Mon, Aug 31, 2015 at 5:59 AM, Guenter Roeck <linux@roeck-us.net> wrote:
>> Turns out the NCT6793D is almost register compatible to NCT6792D. Only
>> difference I can find is that the NCT6792D has "3-wire enable" registers
>> for each fan. Those registers do not exist on NCT6793D.
>>
>> Please try the driver at https://github.com/groeck/nct6775 and let me know
>> if it works.
>
> It doesn't, still says "unsupported chip ID: 0xd121".
>
> With patched ID check I've also noticed that pwmX_step_down_time and
> pwmX_step_up_time are mixed up, don't know if this is something
> NCT6793D specific, looks more like general driver bug.
>

This is interesting. Datasheet for NCT6775 says that up registers are at 0xX04,
and down registers at 0xX03. All other datasheets, starting with NCT6776,
say the opposite.

Makes me wonder if the NCT6775 datasheet is wrong. I'll try to find out.

Again, thanks for noticing.

Guenter


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

Re: [lm-sensors] NCT6793D

[Guenter Roeck]

On 08/31/2015 03:33 PM, Grazvydas Ignotas wrote:
> On Mon, Aug 31, 2015 at 5:59 AM, Guenter Roeck <linux@roeck-us.net> wrote:
>> Turns out the NCT6793D is almost register compatible to NCT6792D. Only
>> difference I can find is that the NCT6792D has "3-wire enable" registers
>> for each fan. Those registers do not exist on NCT6793D.
>>
>> Please try the driver at https://github.com/groeck/nct6775 and let me know
>> if it works.
>
> It doesn't, still says "unsupported chip ID: 0xd121".
>
> With patched ID check I've also noticed that pwmX_step_down_time and
> pwmX_step_up_time are mixed up, don't know if this is something
> NCT6793D specific, looks more like general driver bug.
>

Should now be fixed. Unfortunately, I don't have a system with nct6775
anymore, so we'll have to assume that the datasheets are correct.

Guenter



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

Re: [lm-sensors] NCT6793D

[Grazvydas Ignotas]

T24gVHVlLCBTZXAgMSwgMjAxNSBhdCAyOjE3IEFNLCBHdWVudGVyIFJvZWNrIDxsaW51eEByb2Vj
ay11cy5uZXQ+IHdyb3RlOgo+IE9uIDA4LzMxLzIwMTUgMDM6MzMgUE0sIEdyYXp2eWRhcyBJZ25v
dGFzIHdyb3RlOgo+Pgo+PiBPbiBNb24sIEF1ZyAzMSwgMjAxNSBhdCA1OjU5IEFNLCBHdWVudGVy
IFJvZWNrIDxsaW51eEByb2Vjay11cy5uZXQ+IHdyb3RlOgo+Pj4KPj4+IFR1cm5zIG91dCB0aGUg
TkNUNjc5M0QgaXMgYWxtb3N0IHJlZ2lzdGVyIGNvbXBhdGlibGUgdG8gTkNUNjc5MkQuIE9ubHkK
Pj4+IGRpZmZlcmVuY2UgSSBjYW4gZmluZCBpcyB0aGF0IHRoZSBOQ1Q2NzkyRCBoYXMgIjMtd2ly
ZSBlbmFibGUiIHJlZ2lzdGVycwo+Pj4gZm9yIGVhY2ggZmFuLiBUaG9zZSByZWdpc3RlcnMgZG8g
bm90IGV4aXN0IG9uIE5DVDY3OTNELgo+Pj4KPj4+IFBsZWFzZSB0cnkgdGhlIGRyaXZlciBhdCBo
dHRwczovL2dpdGh1Yi5jb20vZ3JvZWNrL25jdDY3NzUgYW5kIGxldCBtZQo+Pj4ga25vdwo+Pj4g
aWYgaXQgd29ya3MuCj4+Cj4+Cj4+IEl0IGRvZXNuJ3QsIHN0aWxsIHNheXMgInVuc3VwcG9ydGVk
IGNoaXAgSUQ6IDB4ZDEyMSIuCj4+Cj4+IFdpdGggcGF0Y2hlZCBJRCBjaGVjayBJJ3ZlIGFsc28g
bm90aWNlZCB0aGF0IHB3bVhfc3RlcF9kb3duX3RpbWUgYW5kCj4+IHB3bVhfc3RlcF91cF90aW1l
IGFyZSBtaXhlZCB1cCwgZG9uJ3Qga25vdyBpZiB0aGlzIGlzIHNvbWV0aGluZwo+PiBOQ1Q2Nzkz
RCBzcGVjaWZpYywgbG9va3MgbW9yZSBsaWtlIGdlbmVyYWwgZHJpdmVyIGJ1Zy4KPj4KPgo+IFNo
b3VsZCBub3cgYmUgZml4ZWQuIFVuZm9ydHVuYXRlbHksIEkgZG9uJ3QgaGF2ZSBhIHN5c3RlbSB3
aXRoIG5jdDY3NzUKPiBhbnltb3JlLCBzbyB3ZSdsbCBoYXZlIHRvIGFzc3VtZSB0aGF0IHRoZSBk
YXRhc2hlZXRzIGFyZSBjb3JyZWN0LgoKQWxsIGdvb2Qgbm93IG9uIE5DVDY3OTNELCBGV0lXOgpU
ZXN0ZWQtYnk6IEdyYXp2eWRhcyBJZ25vdGFzIDxub3Rhc2FzQGdtYWlsLmNvbT4KCkdyYcW+dnlk
YXMKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxtLXNl
bnNvcnMgbWFpbGluZyBsaXN0CmxtLXNlbnNvcnNAbG0tc2Vuc29ycy5vcmcKaHR0cDovL2xpc3Rz
LmxtLXNlbnNvcnMub3JnL21haWxtYW4vbGlzdGluZm8vbG0tc2Vuc29ycw=