On Wed, Sep 11, 2019 at 03:37:25PM +0000, Pascal Van Leeuwen wrote:
> > On Tue, Sep 10, 2019 at 04:38:13PM +0200, Pascal van Leeuwen wrote:
> > > @@ -43,8 +44,8 @@ struct safexcel_cipher_ctx {
> > >
> > > u32 mode;
> > > enum safexcel_cipher_alg alg;
> > > - bool aead;
> > > - int xcm; /* 0=authenc, 1=GCM, 2 reserved for CCM */
> > > + char aead; /* !=0=AEAD, 2=IPSec ESP AEAD */
> > > + char xcm; /* 0=authenc, 1=GCM, 2 reserved for CCM */
> >
> > You could use an u8 instead. It also seems the aead comment has an
> > issue, I'll let you check that.
> >
> I don't see what's wrong with the comment though?
> Anything unequal to 0 is AEAD, with value 2 being the ESP variant.
OK, that wasn't clear to me when I first read it. Maybe you could say
that 1: AEAD, 2: IPsec ESP AEAD; and then of course the check of this
value being > 0 would mean it's one of the two.
Thanks!
Antoine
--
Antoine Ténart, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
