> -----Original Message----- > From: Stefan Berger <stefanb@xxxxxxxxxxxxx> > Sent: Thursday, February 15, 2024 5:14 PM > Subject: [PATCH v2 06/14] crypto: ecc - Add NIST P521 curve parameters > > Add the parameters for the NIST P521 curve and define a new curve ID > for it. Make the curve available in ecc_get_curve. > ... > diff --git a/crypto/ecc_curve_defs.h b/crypto/ecc_curve_defs.h ... > +static struct ecc_curve nist_p521 = { > + .name = "nist_521", Are the name fields in the ecc_curve structures used anywhere or exposed to userspace? It'd be nice if the strings for the nist_p192, nist_p256, and nist_p384 structures and this new nist_p521 structure included "p" before the number, better matching all the code and the NIST FIPS 186-4 names: .name = "nist_p192" .name = "nist_p256" .name = "nist_p384" .name = "nist_p521"