Looks good, tested and works on one such VOUT_MODE poor design... On 3/1/11 9:32 PM, "Guenter Roeck" <guenter.roeck@xxxxxxxxxxxx> wrote: > Some PMBus chips do not support the VOUT_MODE register. To make matters worse, > such chips may not return an error when reading the register either, but > instead > return 0xff. > > Check if the register exists using pmbus_check_byte_register() before reading > its value. In addition, validate the returned value and ignore it if it is > 0xff > (which is invalid). > > Reported-by: Greg Schnorr <gschnorr@xxxxxxxxx> > Signed-off-by: Guenter Roeck <guenter.roeck@xxxxxxxxxxxx> Acked-by: Greg Schnorr <gschnorr@xxxxxxxxx> _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors