From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Fri, 28 Apr 2017 06:00:02 +0200 Subject: [U-Boot] [PATCH v2 4/8] ti: common: board_detect: Set alen to expected value before i2c read In-Reply-To: <20170420152549.28031-5-fcooper@ti.com> References: <20170420152549.28031-1-fcooper@ti.com> <20170420152549.28031-5-fcooper@ti.com> Message-ID: <5902BE42.3020407@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Franklin, Am 20.04.2017 um 17:25 schrieb Franklin S Cooper Jr: > In non DM I2C read operations the address length passed in during a read > operation will be used automatically. However, in DM I2C the address length > is set to a default value of one which causes problems when trying to > perform a read with a differing alen. Therefore, before the first read in a > series of read operations set the alen to the correct value. > > Signed-off-by: Franklin S Cooper Jr > Reviewed-by: Tom Rini > --- > board/ti/common/board_detect.c | 24 ++++++++++++++++++++++-- > 1 file changed, 22 insertions(+), 2 deletions(-) Reviewed-by: Heiko Schocher Applied to u-boot-i2c/next bye, Heiko > > diff --git a/board/ti/common/board_detect.c b/board/ti/common/board_detect.c > index 1e695f4..6fdcb61 100644 > --- a/board/ti/common/board_detect.c > +++ b/board/ti/common/board_detect.c > @@ -83,7 +83,17 @@ static int __maybe_unused ti_i2c_eeprom_init(int i2c_bus, int dev_addr) > static int __maybe_unused ti_i2c_eeprom_read(int dev_addr, int offset, > uchar *ep, int epsize) > { > - return i2c_read(dev_addr, offset, 2, ep, epsize); > + int bus_num, rc, alen; > + > + bus_num = i2c_get_bus_num(); > + > + alen = 2; > + > + rc = ti_i2c_set_alen(bus_num, dev_addr, alen); > + if (rc) > + return rc; > + > + return i2c_read(dev_addr, offset, alen, ep, epsize); > } > > /** > @@ -125,6 +135,11 @@ static int __maybe_unused ti_i2c_eeprom_get(int bus_addr, int dev_addr, > * Read the header first then only read the other contents. > */ > byte = 2; > + > + rc = ti_i2c_set_alen(bus_addr, dev_addr, byte); > + if (rc) > + return rc; > + > rc = i2c_read(dev_addr, 0x0, byte, (uint8_t *)&hdr_read, 4); > if (rc) > return rc; > @@ -137,9 +152,14 @@ static int __maybe_unused ti_i2c_eeprom_get(int bus_addr, int dev_addr, > * 1 byte address (some legacy boards need this..) > */ > byte = 1; > - if (rc) > + if (rc) { > + rc = ti_i2c_set_alen(bus_addr, dev_addr, byte); > + if (rc) > + return rc; > + > rc = i2c_read(dev_addr, 0x0, byte, (uint8_t *)&hdr_read, > 4); > + } > if (rc) > return rc; > } > -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany