On Thu, 14 Feb 2019 at 15:59, Avri Altman <Avri.Altman@xxxxxxx> wrote: > > > > +/* > > + * This is legacy code, which needs to be re-worked some day. Basically we > > need > > + * to take into account the properties of the host, as to enable the SDIO func > > + * driver layer to allocate optimal buffers. > > + */ > > +static inline unsigned int _sdio_align_size(unsigned int sz) > > +{ > > + /* > > + * FIXME: We don't have a system for the controller to tell > > + * the core about its problems yet, so for now we just 32-bit > > + * align the size. > > + */ > > + sz = ((sz + 3) / 4) * 4; > > + > > + return sz; > > +} > While at it, maybe just return ALIGN(sz, 4)? Good point, I do that! Thanks for reviewing! Kind regards Uffe