SOLVED: Ticket #1409

[Mark Studebaker]

We already have similar super i/o detection in sensors-detect
and in 3 drivers: vt1211, smsc47m1, and w83627hf.
There's also super i/o detection in sensors-detect for the it8705, but
it is
commented out because the "exit" value is different than for other
chips and I didn't want to leave other chips stuck in PnP mode
(although I wonder if your 8712 gets stuck in PnP mode when running
sensors-detect??).

If you port over the super i/o code from one of the 3 drivers listed
above
and send us a patch we'll include it.

thanks
mds


Bjørn Mork wrote:
> 
> I didn't find any way of updating the ticket, so I'm sending this via
> mail.  After writing the original ticket I downloaded and read the
> IT8712F data sheet, which I of course should have done first...  This
> enabled me to write a small program to retrieve the non-default
> address of the IT8712F on the Asus A7N8X-VM:
> 
> #include <stdio.h>
> #include <unistd.h>
> #include <sys/io.h>
> #include <string.h>
> 
> int main(int argc, char **argv) {
>         if (ioperm(0x0, 0x3ff, 1)) {
>                 perror("Set i/o permission");
>                 exit(0);
>         }
> 
>         /* enter MB PnP mode */
>         outb(0x87, 0x2e);
>         outb(0x01, 0x2e);
>         outb(0x55, 0x2e);
>         outb(0x55, 0x2e);
> 
>         /* get chip ID */
>         outb(0x20, 0x2e);
>         printf("chip id = 0x%02x", inb(0x2f));
>         outb(0x21, 0x2e);
>         printf("%02x\n", inb(0x2f));
> 
>         /* select GPIO */
>         outb(0x07, 0x2e);
>         outb(0x04, 0x2f);
> 
>         outb(0x60, 0x2e); /* address MSB */
>         printf("LDN 4, reg 0x60 = 0x%02x\n", inb(0x2f));  /* default 0x02 */
>         outb(0x61, 0x2e); /* address LSB */
>         printf("LDN 4, reg 0x61 = 0x%02x\n", inb(0x2f));  /* default 0x90 */
>         outb(0x30, 0x2e); /* enable */
>         printf("LDN 4, reg 0x30 = 0x%02x\n", inb(0x2f));  /* default 0x00, should be >0 */
> 
>         /* exit from MB PnP mode */
>         outb(0x02, 0x2e);
>         outb(0x02, 0x2f);
>         return 0;
> }
> 
> Which gave the following output right after a reboot:
> 
> chip id = 0x8712
> LDN 4, reg 0x60 = 0x0d
> LDN 4, reg 0x61 = 0x00
> LDN 4, reg 0x30 = 0x01
> 
> I.e. Asus have changed the base address from 0x290 to 0xd00.  Using
> this address with it87 worked perfectly:
> 
>   modprobe it87 probe=9191,0x0d00
> 
> I wonder however, would it be possible to include a refined version of
> the address detection code in the module?  I am willing to try to
> write it myself, but I won't do it unless I know it has some remote
> possibility of being included.
> 
> Bjørn