On 8/5/25 09:14, Philippe Mathieu-Daudé wrote:
On 7/5/25 15:06, Corey Minyard wrote:
On Wed, May 07, 2025 at 02:40:40PM +0200, Bernhard Beschow wrote:
According to the i.MX 8M Plus reference manual, the status flag
I2C_I2SR[IIF]
continues to be set when an interrupt condition occurs even when I2C
interrupts
are disabled (I2C_I2CR[IIEN] is clear). However, the device model
only sets the
flag when I2C interrupts are enabled which causes U-Boot to loop
forever. Fix
the device model by always setting the flag and let I2C_I2CR[IIEN]
guard I2C
interrupts only.
Also remove the comment in the code since it merely stated the
obvious and would
be outdated now.
This looks good to me. I can give you an:
Acked-by: Corey Minyard <cminy...@mvista.com>
or I can take it into my tree.
I have to respin my hw-misc PR so I'll squeeze this.
Thanks!
Phil.
FWIW I noticed the patch subject is truncated to my default git-view
because it is over 72 chars. Since there is no enforcement on patch
subject / description lines length in checkpatch.pl I suppose nobody
really cares about that so I'll merge as is.
