On 6/17/2019 7:04 PM, Andrey Smirnov wrote: > Exactly the same code to figure out DMA mask is repeated twice in the > driver code. To avoid repetition, move that logic into a standalone > subroutine in intern.h. While at it re-shuffle the code to make it > more readable with early returns. > > Signed-off-by: Andrey Smirnov <andrew.smir...@gmail.com> > Cc: Chris Spencer <christopher.spen...@sea.co.uk> > Cc: Cory Tusar <cory.tu...@zii.aero> > Cc: Chris Healy <cphe...@gmail.com> > Cc: Lucas Stach <l.st...@pengutronix.de> > Cc: Horia Geantă <horia.gea...@nxp.com> > Cc: Aymen Sghaier <aymen.sgha...@nxp.com> > Cc: Leonard Crestez <leonard.cres...@nxp.com> > Cc: linux-crypto@vger.kernel.org > Cc: linux-ker...@vger.kernel.org Reviewed-by: Horia Geantă <horia.gea...@nxp.com>
Being the 1st patch in the series and not i.MX8-specific, I'd say it should be merged separately. Thanks, Horia