Specify I2C channels physically, not arbitrarilytest-4824.B

In board.c, we initialize this struct:

struct i2c_port_t {
	const char *name;  /* Port name */
	int port;          /* Port */
	int kbps;          /* Speed in kbps */
};
extern const struct i2c_port_t i2c_ports[];

The port field refers to the physical I2C bus on the EC.

Meanwhile, in board.h, we've identified the bus where each I2C device is
attached:

Up until this CL, we've been picking one of those device-to-bus macros to
initialize port fields of the i2c_ports[] array. That's wrong and confusing.

This change specifies the physical channel with the physical number.

BUG=chrome-os-partner:18343
BRANCH=none
TEST=manual

Renaming only. There should be no change in observed behavior.

Change-Id: I5427c26290572133f060b6cf0d9ebea5015adba1
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/176176
Reviewed-by: Yung-chieh Lo <yjlou@chromium.org>

1 files changed, 1 insertions, 1 deletions

diff --git a/board/bds/board.c b/board/bds/board.c
index e23b17c33a..90ac5eb946 100644
--- a/board/bds/board.c
+++ b/board/bds/board.c
@@ -34,7 +34,7 @@ BUILD_ASSERT(ARRAY_SIZE(adc_channels) == ADC_CH_COUNT);
 
 /* I2C ports */
 const struct i2c_port_t i2c_ports[] = {
-	{"lightbar", I2C_PORT_LIGHTBAR, 400},
+	{"lightbar", 5, 400},
 };
 const unsigned int i2c_ports_used = ARRAY_SIZE(i2c_ports);