On Wed, Aug 14, 2013 at 12:45:26PM +0300, Dan Carpenter wrote: > The callers never check for negative but they sometimes check for zero > returns: > > int reg = ADT7462_REG_VOLT_MAX(data, i); > data->volt_max[i] = > (reg ? i2c_smbus_read_byte_data(client, reg) : 0); > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > --- > Static checker stuff. I don't have this hardware. I should have also said that not all the callers check for error. I don't know if that is a problem. regards, dan carpenter _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
