From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Date: Wed, 14 Aug 2013 14:06:32 +0000 Subject: Re: [patch] hwmon: (adt7462) ADT7462_REG_VOLT_MAX() should return 0 Message-Id: <520B8EE8.5050703@roeck-us.net> List-Id: References: <20130814094526.GE521@elgon.mountain> In-Reply-To: <20130814094526.GE521@elgon.mountain> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: kernel-janitors@vger.kernel.org On 08/14/2013 03:53 AM, Dan Carpenter wrote: > 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 >> --- >> 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. > It is undesirable, but would be much more complex to fix. So your patch is the sensible approach to take care of the immediate problem, and we can leave the fancy solution to someone with enough time at hand. I'll apply it to -next. Thanks, Guenter