On Fri, Mar 17, 2017 at 11:42:13AM -0700, Justin Maggard wrote: > Hi, > > We've started using an IT8613E chip in some hardware designs, and I > was planning to submit our it87 driver changes to support this new > chip. IT8613 supports 4 fans, but the fans it supports are fans 2-5. > One of our designs has 3 fans, with the third fan connected to FAN4; > however, I'm unable to control that fan using the current driver. > > The reason is, in the current driver, IT87_REG_PWM[3-4] is set to > 0x7f, 0xa7. But on IT8613E this should be 0x1e, 0x1f. I see that > these registers were defined when adding support for IT8620E. Can > anybody confirm that 0x7f, 0xa7 are correct on that chip? I'm sure > they weren't just pulled out of thin air, but it seems odd that ITE > would have changed only those register indexes. > > Assuming that those registers indexes are important for at least > IT8620E, what would be the desired way to proceed with adding IT8613E > support? > We are currently trying to deal with that problem. Here is a quick summary of what I know so far. IT8620E IT8622E IT8628E IT8665E IT8686E IT8732 PWM4_CTL 0x7f 0x1e 0x7f 0x1e 0x7f ? PWM5_CTL 0xa7 0x1f 0xa7 0x1f 0xa7 - PWM6_CTL 0xaf - 0xaf 0x92 0xaf - On top of that, not all chips share the same register definition. Only bit 7 is the same. This means the current code is wrong for at least IT8622E, won't work for IT8665E (and IT8613E), and we'll need to find a solution that works for all chips. Guenter -- To unsubscribe from this list: send the line "unsubscribe linux-hwmon" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html