* Mark M. Hoffman <mhoffman at lightlink.com> [2003-06-20 00:14:07 -0400]: > > Remove this line from all set_foo(): > > data->foo = FOO_TO_REG(...); > > And add this to xxxx_write_value(): > > data->valid = 0; > > Hmmm, probably still need data->update_lock in xxxx_write_value(), but > that should take care of it no? *Sigh* no. There would still be races around the set_* functions that read the fan divisor. OK Phil, I'm done trying to improve on your patch. ;) Regards, -- Mark M. Hoffman mhoffman at lightlink.com
