Re: [PATCH v3 1/5] crypto: caam - move DMA mask selection into a function
From: Andrey Smirnov
Date: Wed Jul 03 2019 - 02:37:48 EST
On Thu, Jun 27, 2019 at 3:50 AM Horia Geanta <horia.geanta@xxxxxxx> wrote:
>
> 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.smirnov@xxxxxxxxx>
> > Cc: Chris Spencer <christopher.spencer@xxxxxxxxx>
> > Cc: Cory Tusar <cory.tusar@xxxxxxxx>
> > Cc: Chris Healy <cphealy@xxxxxxxxx>
> > Cc: Lucas Stach <l.stach@xxxxxxxxxxxxxx>
> > Cc: Horia GeantÄ <horia.geanta@xxxxxxx>
> > Cc: Aymen Sghaier <aymen.sghaier@xxxxxxx>
> > Cc: Leonard Crestez <leonard.crestez@xxxxxxx>
> > Cc: linux-crypto@xxxxxxxxxxxxxxx
> > Cc: linux-kernel@xxxxxxxxxxxxxxx
> Reviewed-by: Horia GeantÄ <horia.geanta@xxxxxxx>
>
> Being the 1st patch in the series and not i.MX8-specific, I'd say it should
> be merged separately.
>
Can it be done by cherry picking it from the series or does it have to
be submitted as a separate patch? I am hoping the former is possible
since it'd make life easier for me.
Thanks,
Andrey Smirnov