[PATCH 2/3] ARM: mxs: crypto: Add Freescale MXS DCP driver
Marek Vasut
marex at denx.de
Thu Sep 26 10:04:36 EDT 2013
Dear Lothar Waßmann,
> Hi Marek,
>
> some small comments below.
>
> Marek Vasut writes:
> > diff --git a/drivers/crypto/mxs-dcp.c b/drivers/crypto/mxs-dcp.c
> > new file mode 100644
> > index 0000000..c2b35c7
> > --- /dev/null
> > +++ b/drivers/crypto/mxs-dcp.c
>
> [...]
>
> > +/* AES 128 ECB and AES 128 CBC */
> > +static struct crypto_alg dcp_aes_algs[] = {
> > + [0] = {
> > + .cra_name = "ecb(aes)",
> > + .cra_driver_name = "ecb-aes-dcp",
> > + .cra_priority = 400,
> > + .cra_alignmask = 15,
> > + .cra_flags = CRYPTO_ALG_TYPE_ABLKCIPHER |
> > + CRYPTO_ALG_ASYNC |
> > + CRYPTO_ALG_NEED_FALLBACK,
> > + .cra_init = mxs_dcp_aes_fallback_init,
> > + .cra_exit = mxs_dcp_aes_fallback_exit,
> > + .cra_blocksize = AES_BLOCK_SIZE,
> > + .cra_ctxsize = sizeof(struct dcp_async_ctx),
> > + .cra_type = &crypto_ablkcipher_type,
> > + .cra_module = THIS_MODULE,
> > + .cra_u = {
> > + .ablkcipher = {
> > + .min_keysize = AES_MIN_KEY_SIZE,
> > + .max_keysize = AES_MAX_KEY_SIZE,
> > + .setkey = mxs_dcp_aes_setkey,
> > + .encrypt = mxs_dcp_aes_ecb_encrypt,
> > + .decrypt = mxs_dcp_aes_ecb_decrypt
> > + }
>
> missing ',' after '}'
Is this a problem? The last ',' is not needed by the C standard.
[...]
>
> > +static const struct of_device_id mxs_dcp_dt_ids[] = {
> > + {.compatible = "fsl,mxs-dcp", .data = NULL,},
>
> missing spaces after '{' and before '}'
Thanks!
Best regards,
Marek Vasut
More information about the linux-arm-kernel
mailing list