From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Mon, 31 Aug 2015 23:06:03 +0000 Subject: Re: [lm-sensors] NCT6793D Message-Id: <55E4DDDB.8010404@roeck-us.net> 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 08/31/2015 03:33 PM, Grazvydas Ignotas wrote: > On Mon, Aug 31, 2015 at 5:59 AM, Guenter Roeck 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