> > One question: Wouldn't it be more logical to have this patch first (fix > > pio) and then squash patches 1 and 3 as one on the top (add mx23 to now > > working pio)? I am not pushing too hard if this means a lot of work, but > > it sounds a bit more logical to me. > > I would prefer to keep Juergens' patch separate from mine if you don't mind > to > be clear on the authorship.
If you add both SoB, everything should be fine. We often work on someone
else's patches, no?
> > > if (msg->flags & I2C_M_RD) {
> > >
> > > + /*
> > > + * PIO READ transfer:
> > > + *
> > > + * This transfer MUST be limited to 4 bytes maximum. It is not
> > > + * possible to transfer more than four bytes via PIO, since we
> > > + * can not in any way make sure we can read the data from the
> > > + * DATA register fast enough. Besides, the RX FIFO is only four
> > > + * bytes deep, thus we can only really read up to four bytes at
> > > + * time. Finally, there is no bit indicating us that new data
> > > + * arrived at the FIFO and can thus be fetched from the DATA
> > > + * register.
> > > + */
> > > + BUG_ON(msg->len > 4);
> >
> > How could this happen? There is a check in mxs_i2c_xfer_msg.
>
> It cannot happen, I added the check here to make sure when someone becomes
> adventurous enough to start messing with these constants, it will stop him
> early
> enough.
Then, add the comment to the check so ppl will notice there? I like to
keep BUG_ON sparse, since it is hard to tell if the occasion is really
worth stopping the machine.
signature.asc
Description: Digital signature
