> > +static inline bool ufshpb_is_support_chunk(int transfer_len) > > +{ > > + return transfer_len <= HPB_MULTI_CHUNK_HIGH; > > +} > > The names used in the above function are mysterious. What is a support > chunk? What does "multi chunk high" mean? Please add a comment. HPB1.0 limits transfer_len to be at most 1. HPB2.0, which is in its final draft, allows transfer_len to be at most 128, But introduce some new behavior depends on transfer_len. This is just preparing for that. Thanks, Avri