* Re: [lm-sensors] NCT6793D
2015-08-27 0:57 [lm-sensors] NCT6793D Grazvydas Ignotas
@ 2015-08-27 3:06 ` Guenter Roeck
2015-08-31 2:59 ` Guenter Roeck
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Guenter Roeck @ 2015-08-27 3:06 UTC (permalink / raw)
To: lm-sensors
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
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [lm-sensors] NCT6793D
2015-08-27 0:57 [lm-sensors] NCT6793D Grazvydas Ignotas
2015-08-27 3:06 ` Guenter Roeck
@ 2015-08-31 2:59 ` Guenter Roeck
2015-08-31 22:33 ` Grazvydas Ignotas
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Guenter Roeck @ 2015-08-31 2:59 UTC (permalink / raw)
To: lm-sensors
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
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [lm-sensors] NCT6793D
2015-08-27 0:57 [lm-sensors] NCT6793D Grazvydas Ignotas
2015-08-27 3:06 ` Guenter Roeck
2015-08-31 2:59 ` Guenter Roeck
@ 2015-08-31 22:33 ` Grazvydas Ignotas
2015-08-31 22:55 ` Guenter Roeck
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Grazvydas Ignotas @ 2015-08-31 22:33 UTC (permalink / raw)
To: lm-sensors
T24gTW9uLCBBdWcgMzEsIDIwMTUgYXQgNTo1OSBBTSwgR3VlbnRlciBSb2VjayA8bGludXhAcm9l
Y2stdXMubmV0PiB3cm90ZToKPiBUdXJucyBvdXQgdGhlIE5DVDY3OTNEIGlzIGFsbW9zdCByZWdp
c3RlciBjb21wYXRpYmxlIHRvIE5DVDY3OTJELiBPbmx5Cj4gZGlmZmVyZW5jZSBJIGNhbiBmaW5k
IGlzIHRoYXQgdGhlIE5DVDY3OTJEIGhhcyAiMy13aXJlIGVuYWJsZSIgcmVnaXN0ZXJzCj4gZm9y
IGVhY2ggZmFuLiBUaG9zZSByZWdpc3RlcnMgZG8gbm90IGV4aXN0IG9uIE5DVDY3OTNELgo+Cj4g
UGxlYXNlIHRyeSB0aGUgZHJpdmVyIGF0IGh0dHBzOi8vZ2l0aHViLmNvbS9ncm9lY2svbmN0Njc3
NSBhbmQgbGV0IG1lIGtub3cKPiBpZiBpdCB3b3Jrcy4KCkl0IGRvZXNuJ3QsIHN0aWxsIHNheXMg
InVuc3VwcG9ydGVkIGNoaXAgSUQ6IDB4ZDEyMSIuCgpXaXRoIHBhdGNoZWQgSUQgY2hlY2sgSSd2
ZSBhbHNvIG5vdGljZWQgdGhhdCBwd21YX3N0ZXBfZG93bl90aW1lIGFuZApwd21YX3N0ZXBfdXBf
dGltZSBhcmUgbWl4ZWQgdXAsIGRvbid0IGtub3cgaWYgdGhpcyBpcyBzb21ldGhpbmcKTkNUNjc5
M0Qgc3BlY2lmaWMsIGxvb2tzIG1vcmUgbGlrZSBnZW5lcmFsIGRyaXZlciBidWcuCgpHcmHFvnZ5
ZGFzCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsbS1z
ZW5zb3JzIG1haWxpbmcgbGlzdApsbS1zZW5zb3JzQGxtLXNlbnNvcnMub3JnCmh0dHA6Ly9saXN0
cy5sbS1zZW5zb3JzLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xtLXNlbnNvcnM
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [lm-sensors] NCT6793D
2015-08-27 0:57 [lm-sensors] NCT6793D Grazvydas Ignotas
` (2 preceding siblings ...)
2015-08-31 22:33 ` Grazvydas Ignotas
@ 2015-08-31 22:55 ` Guenter Roeck
2015-08-31 23:06 ` Guenter Roeck
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Guenter Roeck @ 2015-08-31 22:55 UTC (permalink / raw)
To: lm-sensors
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
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [lm-sensors] NCT6793D
2015-08-27 0:57 [lm-sensors] NCT6793D Grazvydas Ignotas
` (3 preceding siblings ...)
2015-08-31 22:55 ` Guenter Roeck
@ 2015-08-31 23:06 ` Guenter Roeck
2015-08-31 23:17 ` Guenter Roeck
2015-08-31 23:45 ` Grazvydas Ignotas
6 siblings, 0 replies; 8+ messages in thread
From: Guenter Roeck @ 2015-08-31 23:06 UTC (permalink / raw)
To: lm-sensors
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
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [lm-sensors] NCT6793D
2015-08-27 0:57 [lm-sensors] NCT6793D Grazvydas Ignotas
` (4 preceding siblings ...)
2015-08-31 23:06 ` Guenter Roeck
@ 2015-08-31 23:17 ` Guenter Roeck
2015-08-31 23:45 ` Grazvydas Ignotas
6 siblings, 0 replies; 8+ messages in thread
From: Guenter Roeck @ 2015-08-31 23:17 UTC (permalink / raw)
To: lm-sensors
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
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [lm-sensors] NCT6793D
2015-08-27 0:57 [lm-sensors] NCT6793D Grazvydas Ignotas
` (5 preceding siblings ...)
2015-08-31 23:17 ` Guenter Roeck
@ 2015-08-31 23:45 ` Grazvydas Ignotas
6 siblings, 0 replies; 8+ messages in thread
From: Grazvydas Ignotas @ 2015-08-31 23:45 UTC (permalink / raw)
To: lm-sensors
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=
^ permalink raw reply [flat|nested] 8+ messages in thread
