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> -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
