Re: [PATCH] I2C Part 1 - Remove some redundancy from the i2c-core.cfile
From: Jean Delvare
Date: Wed Mar 23 2005 - 16:45:42 EST
Hi Corey,
> Call i2c_transfer() from i2c_master_send() and i2c_master_recv()
> to avoid the redundant code that was in all three functions.
I like this. You're right, there is code duplication here, which we can
get rid of, so let's do so. I'd only have one comments about your patch:
You can get rid of the dev_dbg calls in i2c_master_send() and
i2c_master_recv() altogether IMHO. I recently updated i2c_transfer() to
make it more verbose in debug mode [1], so the debug messages in
i2c_master_send() and i2c_master_recv() are mostly redundant now as far
as I can see.
[1] http://archives.andrew.net.au/lm-sensors/msg29859.html
Thanks,
--
Jean Delvare
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/