On Tue, Jan 25, 2011 at 02:20:48PM +0000, Alan Cox wrote:
> From: Catalin Popescu <[email protected]>
> 
> The error messages printed from mrst_i2c_abort() didn't give slave address 
> info.
> 
> But I2C device driver developers always need this to check which slave device
> has the problem.

Erm, surely the driver itself should know what i2c device it is communicating
with? Secondly, does this swamp the kernel dmesg buffer when run with the
i2c-detect tool?
 
> This patch enhances the error message format by adding slave address info to
> each error message.

Why not use dev_err() on the i2c device's device state?
 
-- 
Ben Dooks, [email protected], http://www.fluff.org/ben/

Large Hadron Colada: A large Pina Colada that makes the universe disappear.

--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to