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