aboutsummaryrefslogtreecommitdiff
path: root/post
diff options
context:
space:
mode:
authorPeter Tyser <ptyser@xes-inc.com>2010-10-22 00:20:27 -0500
committerKumar Gala <galak@kernel.crashing.org>2010-10-22 02:18:30 -0500
commitb9b1bc8542db5f26453c45db843903dee7056244 (patch)
tree0b306669fe0ba7ac9ceab4af24dee4f6cef0b349 /post
parentc00ac259f60112bb263a73f211ce3bb8f529c2c0 (diff)
downloadu-boot-linaro-stable-b9b1bc8542db5f26453c45db843903dee7056244.tar.gz
post/i2c: General clean up
- Clean up ifdeffery - Update coding style No functional change should have occurred. Signed-off-by: Peter Tyser <ptyser@xes-inc.com> Acked-by: Heiko Schocher <hs@denx.de> Acked-by: Wolfgang Denk <wd@denx.de> Signed-off-by: Kumar Gala <galak@kernel.crashing.org>
Diffstat (limited to 'post')
-rw-r--r--post/drivers/i2c.c48
1 files changed, 23 insertions, 25 deletions
diff --git a/post/drivers/i2c.c b/post/drivers/i2c.c
index b152deaf6..84a603549 100644
--- a/post/drivers/i2c.c
+++ b/post/drivers/i2c.c
@@ -21,8 +21,6 @@
* MA 02111-1307 USA
*/
-#include <common.h>
-
/*
* I2C test
*
@@ -36,6 +34,7 @@
* #endif
*/
+#include <common.h>
#include <post.h>
#include <i2c.h>
@@ -44,42 +43,41 @@
int i2c_post_test (int flags)
{
unsigned int i;
+#ifndef I2C_ADDR_LIST
+ for (i = 0; i < 128; i++)
+ if (i2c_probe (i) == 0)
+ return 0;
+
+ /* No devices found */
+ return -1;
+#else
unsigned int good = 0;
-#ifdef I2C_ADDR_LIST
unsigned int bad = 0;
int j;
unsigned char i2c_addr_list[] = I2C_ADDR_LIST;
unsigned char i2c_miss_list[] = I2C_ADDR_LIST;
-#endif
for (i = 0; i < 128; i++) {
- if (i2c_probe (i) == 0) {
-#ifndef I2C_ADDR_LIST
- good++;
-#else /* I2C_ADDR_LIST */
- for (j=0; j<sizeof(i2c_addr_list); ++j) {
- if (i == i2c_addr_list[j]) {
- good++;
- i2c_miss_list[j] = 0xFF;
- break;
- }
- }
- if (j == sizeof(i2c_addr_list)) {
- bad++;
- post_log ("I2C: addr %02X not expected\n",
- i);
+ if (i2c_probe(i) != 0)
+ continue;
+ for (j = 0; j < sizeof(i2c_addr_list); ++j) {
+ if (i == i2c_addr_list[j]) {
+ good++;
+ i2c_miss_list[j] = 0xFF;
+ break;
}
-#endif /* I2C_ADDR_LIST */
+ }
+
+ if (j == sizeof(i2c_addr_list)) {
+ bad++;
+ post_log("I2C: addr %02X not expected\n", i);
}
}
-#ifndef I2C_ADDR_LIST
- return good > 0 ? 0 : -1;
-#else /* I2C_ADDR_LIST */
if (good != sizeof(i2c_addr_list)) {
- for (j=0; j<sizeof(i2c_miss_list); ++j) {
+ for (j = 0; j < sizeof(i2c_miss_list); ++j) {
if (i2c_miss_list[j] != 0xFF) {
- post_log ("I2C: addr %02X did not respond\n",
+ post_log("I2C: addr %02X did not respond\n",
i2c_miss_list[j]);
}
}