<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Dec 5, 2023, 15:01 Dan Carpenter <<a href="mailto:dan.carpenter@linaro.org">dan.carpenter@linaro.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Dec 05, 2023 at 02:48:26PM +0100, Robert Foss wrote:<br>
> On Mon, 4 Dec 2023 15:29:00 +0300, Dan Carpenter wrote:<br>
> > The i2c_master_send/recv() functions return negative error codes or the<br>
> > number of bytes that were able to be sent/received.  This code has<br>
> > two problems.  1)  Instead of checking if all the bytes were sent or<br>
> > received, it checks that at least one byte was sent or received.<br>
> > 2) If there was a partial send/receive then we should return a negative<br>
> > error code but this code returns success.<br>
> > <br>
> > [...]<br>
> <br>
> Applied, thanks!<br>
> <br>
> [1/1] drm/bridge: nxp-ptn3460: fix i2c_master_send() error checking<br>
>       <a href="https://cgit.freedesktop.org/drm/drm-misc/commit/?id=914437992876" rel="noreferrer noreferrer" target="_blank">https://cgit.freedesktop.org/drm/drm-misc/commit/?id=914437992876</a><br>
> <br>
<br>
Wait.  That was unexpected.  Neil's review comments were correct.  I was<br>
planning to send a v2 patch which was just a cleanup.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Sorry Dan, I was too quick on the draw. Can you send a fixup and I'll apply it too?</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
regards,<br>
dan carpenter<br>
<br>
</blockquote></div></div></div>