* Ronald G Minnich rminnich@lanl.gov [060307 23:41]:
Do you want this controlled by an option in the chip struct for that part? You're sure it does no harm to turn it on? it looks fine to me.
It does not harm on any of the agami boards we've tried, no matter what Winbond revision was used. On the 0x17 parts the problem went away.
From what I can tell it looks quite safe generally.
How would such an option in the chip struct look like? Should we have the revision and vendor of the SuperIO chip stored in the device tree's superio entry and enable the workaround if we find an adequate device in the tree?
Yinghai, can we switch this on unconditionally or might that hurt some motherboards?
Stefan