Re: [PATCH v3 23/25] bus: mhi: ep: Add support for queueing SKBs to the host

Alex Elder
2022-02-22T16:05:25Z

On 2/22/22 9:18 AM, Alex Elder wrote:
>
>> But anyway, there is no need to check for CHAIN flag while writing to
>> host.
>> CHAIN flag is only used or even make sense when host writes data to
>> device, so
>
> I'm not sure that's correct, but I don't want to get into that issue here.
> We can talk about that separately.

I just wanted to send a short followup here. My comments
were based on a misunderstanding, and Mani cleared it up
for me. For host receives, the MHI specification states
that a packet that requires more than the size of the
buffer in a single TRE leads to an overflow event being
generated from the device to the host. The buffer on
the TRE is filled, and subsequent packet data is written
to the next TRE's buffer (assuming it's present).

This differs from one feature of IPA and its GSI transfer
rings. I won't explain that here, to avoid any confusion.

Mani explained things to me, and he's going to send an
updated series, which I'll review.

-Alex