On Fri, Aug 26, 2022 at 11:15:55PM +0200, Marek Vasut wrote: > This reverts commit 8f8c04bf1ebbd2f72f1643e7ad9617dafa6e5409. > > The commit is largely wrong and breaks most of i2c command functionality. > The problem described in the aforementioned commit commit message is valid, > however the commit itself does many more changes unrelated to fixing that > one problem it describes. Those extra changes, namely the handling of i2c > device address length as unsigned instead of signed integer, breaks the > expectation that address length may be negative value. The negative value > is used by DM to indicate that address length of device does not change. > > The actual bug documented in commit 8f8c04bf1ebbd2f72f1643e7ad9617dafa6e5409 > can be fixed by extra sanitization in separate patch. > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Heiko Schocher <h...@denx.de> > Cc: Nicolas Iooss <nicolas.iooss+ub...@ledger.fr> > Cc: Simon Glass <s...@chromium.org> > Cc: Tim Harvey <thar...@gateworks.com> > Reviewed-by: Simon Glass <s...@chromium.org>
Applied to u-boot/master, thanks! -- Tom
signature.asc
Description: PGP signature
