Re: [patch] i2c-scmi: remove some bogus NULL checks

2013-10-31 Thread Wolfram Sang
On Sat, Oct 19, 2013 at 11:46:22AM +0300, Dan Carpenter wrote: obj can't be NULL here. We already know that pkg-package.elements gives us a valid pointer so the next pointer after that is also non-NULL. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com Applied to for-next, thanks!

Re: [patch] i2c-scmi: remove some bogus NULL checks

2013-10-30 Thread Wolfram Sang
On Sat, Oct 19, 2013 at 11:46:22AM +0300, Dan Carpenter wrote: obj can't be NULL here. We already know that pkg-package.elements gives us a valid pointer so the next pointer after that is also non-NULL. Why is that? Can't see it... Signed-off-by: Dan Carpenter dan.carpen...@oracle.com

Re: [patch] i2c-scmi: remove some bogus NULL checks

2013-10-30 Thread Dan Carpenter
On Wed, Oct 30, 2013 at 05:02:58PM +0100, Wolfram Sang wrote: --- a/drivers/i2c/busses/i2c-scmi.c +++ b/drivers/i2c/busses/i2c-scmi.c @@ -223,7 +223,7 @@ acpi_smbus_cmi_access(struct i2c_adapter *adap, u16 addr, unsigned short flags, goto out; obj =