[PATCH 2/3] i2c/pxa: only define 'blue_murder'-function if DEBUG is #defined
u.kleine-koenig at pengutronix.de
Mon Mar 22 17:02:04 EDT 2010
On Sun, Feb 28, 2010 at 03:55:02PM +0000, Russell King - ARM Linux wrote:
> On Wed, Feb 24, 2010 at 12:01:45PM +0100, Uwe Kleine-König wrote:
> > From: Wolfram Sang <w.sang at pengutronix.de>
> > This talkative function is also called on timeouts. As timeouts can
> > happen on regular writes to EEPROMs (no error case), this creates false
> > positives. Giving lots of details is interesting only for developers
> > anyhow, so just use the function if DEBUG is #defined.
> Are you sure this is safe? If you time out the write before it completes,
> how do you know if the write was successful?
> I don't think this is "no error code" nor "false positive". If the timeout
> is too short for your EEPROMs, then the timeout needs to be increased.
any thoughts about this by you?
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
More information about the linux-arm-kernel