On Sat, Feb 23, 2008 at 12:10:48AM +0100, Carl-Daniel Hailfinger wrote:
/* set the correct sensor types 1,2 thermistor; 3 diode */ pnp_write_index(base, 0x51, 0x1c); /* set the 'zero' voltage for diode type sensor */ pnp_write_index(base, 0x5c, 0x80); pnp_write_index(base, 0x56, 0xe5); pnp_write_index(base, 0x57, 0xe5); pnp_write_index(base, 0x59, 0xe5); pnp_write_index(base, 0x5c, 0x00); /* fan1 (controlled by temp3) control parameters */ /* fan off limit */ pnp_write_index(base, 0x60, 0xff); /* fan start limit */ pnp_write_index(base, 0x61, 0x14); /* ???? */ pnp_write_index(base, 0x62, 0x00); /* start PWM */ pnp_write_index(base, 0x63, 0x27); /* smooth and slope PWM */ pnp_write_index(base, 0x64, 0x90); /* direct-down and interval */ pnp_write_index(base, 0x65, 0x03); /* fan1 auto controlled by temp3 */ pnp_write_index(base, 0x15, 0x82); /* fan2 soft controlled, max speed */ pnp_write_index(base, 0x16, 0x7f); /* fan3 soft controlled, 75% speed */ pnp_write_index(base, 0x17, 0x60); /* all fans enable, fan1 ctl smart */ pnp_write_index(base, 0x13, 0x71);
Then the fan is properly controlled. BUT this stuff is far too mainboard specific (as opposed to superio specific) to be in this file. It should be in a file in the mainboard/gigabyte/m57sli directory (probably even V2.0 specific). But I wouldn't know how to link it in so it gets called in the ec_init routine. Do you have some suggestions? Maybe there is a way to put it in the Config.lb file?
Ronald,
you did great work tracking this down. Congratulations! We have to make sure we can handle such settings in v3 easily.
There is another possible problem with fan settings: Do they belong to (in the sense of copyright) the mainboard vendor or the BIOS vendor? I don't want to get nasty letters from anyone.
I think that's going too far. This stuff comes straight from the superio datasheet (check out the comments). There's no way anyone can claim copyright on a bunch of configuration settings imo.
Thanks, Ward.