From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Mon, 25 Nov 2013 08:46:06 +0100
Subject: [U-Boot] soft i2c behaviour on address not <ACK>ed
In-Reply-To: <E64115DDEB03174E9F77D3FC9D8DFEB506B703@srv214.gdsys.de>
References: <E64115DDEB03174E9F77D3FC9D8DFEB506B703@srv214.gdsys.de>
Message-ID: <5293003E.8070608@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Dirk,

Am 21.11.2013 13:34, schrieb Eibach, Dirk:
> While debugging an I2C problem I found in soft_i2c_read() and
> soft_i2c_write():
>
> if(write_byte(addr>>  shift)) {
> 	PRINTD("i2c_read, address not<ACK>ed\n");
> 	return(1);
> }
>
> and
>
> if(write_byte(addr>>  shift)) {
> 	PRINTD("i2c_write, address not<ACK>ed\n");
> 	return(1);
> }
>
> This means that these functions are left without sending a STOP
> condition to the bus and with SCL held low. Is that really intended?

This seems a bug to me ... could you send a patch?

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany