crypto: caam - Modify the interface layers to use JR API's
- Earlier interface layers - caamalg, caamhash, caamrng were directly using the Controller driver private structure to access the Job ring. - Changed the above to use alloc/free API's provided by Job Ring Drive Signed-off-by: Ruchika Gupta <ruchika.gupta@freescale.com> Reviewed-by: Garg Vakul-B16394 <vakul@freescale.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
This commit is contained in:

committed by
Herbert Xu

parent
07defbfb0f
commit
cfc6f11b76
@@ -86,6 +86,7 @@
|
||||
#else
|
||||
#define debug(format, arg...)
|
||||
#endif
|
||||
static struct list_head alg_list;
|
||||
|
||||
/* Set DK bit in class 1 operation if shared */
|
||||
static inline void append_dec_op1(u32 *desc, u32 type)
|
||||
@@ -2057,7 +2058,6 @@ static struct caam_alg_template driver_algs[] = {
|
||||
|
||||
struct caam_crypto_alg {
|
||||
struct list_head entry;
|
||||
struct device *ctrldev;
|
||||
int class1_alg_type;
|
||||
int class2_alg_type;
|
||||
int alg_op;
|
||||
@@ -2070,16 +2070,12 @@ static int caam_cra_init(struct crypto_tfm *tfm)
|
||||
struct caam_crypto_alg *caam_alg =
|
||||
container_of(alg, struct caam_crypto_alg, crypto_alg);
|
||||
struct caam_ctx *ctx = crypto_tfm_ctx(tfm);
|
||||
struct caam_drv_private *priv = dev_get_drvdata(caam_alg->ctrldev);
|
||||
struct platform_device *pdev;
|
||||
int tgt_jr = atomic_inc_return(&priv->tfm_count);
|
||||
|
||||
/*
|
||||
* distribute tfms across job rings to ensure in-order
|
||||
* crypto request processing per tfm
|
||||
*/
|
||||
pdev = priv->jrpdev[(tgt_jr / 2) % priv->total_jobrs];
|
||||
ctx->jrdev = &pdev->dev;
|
||||
ctx->jrdev = caam_jr_alloc();
|
||||
if (IS_ERR(ctx->jrdev)) {
|
||||
pr_err("Job Ring Device allocation for transform failed\n");
|
||||
return PTR_ERR(ctx->jrdev);
|
||||
}
|
||||
|
||||
/* copy descriptor header template value */
|
||||
ctx->class1_alg_type = OP_TYPE_CLASS1_ALG | caam_alg->class1_alg_type;
|
||||
@@ -2106,44 +2102,26 @@ static void caam_cra_exit(struct crypto_tfm *tfm)
|
||||
dma_unmap_single(ctx->jrdev, ctx->sh_desc_givenc_dma,
|
||||
desc_bytes(ctx->sh_desc_givenc),
|
||||
DMA_TO_DEVICE);
|
||||
|
||||
caam_jr_free(ctx->jrdev);
|
||||
}
|
||||
|
||||
static void __exit caam_algapi_exit(void)
|
||||
{
|
||||
|
||||
struct device_node *dev_node;
|
||||
struct platform_device *pdev;
|
||||
struct device *ctrldev;
|
||||
struct caam_drv_private *priv;
|
||||
struct caam_crypto_alg *t_alg, *n;
|
||||
|
||||
dev_node = of_find_compatible_node(NULL, NULL, "fsl,sec-v4.0");
|
||||
if (!dev_node) {
|
||||
dev_node = of_find_compatible_node(NULL, NULL, "fsl,sec4.0");
|
||||
if (!dev_node)
|
||||
return;
|
||||
}
|
||||
|
||||
pdev = of_find_device_by_node(dev_node);
|
||||
if (!pdev)
|
||||
if (!alg_list.next)
|
||||
return;
|
||||
|
||||
ctrldev = &pdev->dev;
|
||||
of_node_put(dev_node);
|
||||
priv = dev_get_drvdata(ctrldev);
|
||||
|
||||
if (!priv->alg_list.next)
|
||||
return;
|
||||
|
||||
list_for_each_entry_safe(t_alg, n, &priv->alg_list, entry) {
|
||||
list_for_each_entry_safe(t_alg, n, &alg_list, entry) {
|
||||
crypto_unregister_alg(&t_alg->crypto_alg);
|
||||
list_del(&t_alg->entry);
|
||||
kfree(t_alg);
|
||||
}
|
||||
}
|
||||
|
||||
static struct caam_crypto_alg *caam_alg_alloc(struct device *ctrldev,
|
||||
struct caam_alg_template
|
||||
static struct caam_crypto_alg *caam_alg_alloc(struct caam_alg_template
|
||||
*template)
|
||||
{
|
||||
struct caam_crypto_alg *t_alg;
|
||||
@@ -2151,7 +2129,7 @@ static struct caam_crypto_alg *caam_alg_alloc(struct device *ctrldev,
|
||||
|
||||
t_alg = kzalloc(sizeof(struct caam_crypto_alg), GFP_KERNEL);
|
||||
if (!t_alg) {
|
||||
dev_err(ctrldev, "failed to allocate t_alg\n");
|
||||
pr_err("failed to allocate t_alg\n");
|
||||
return ERR_PTR(-ENOMEM);
|
||||
}
|
||||
|
||||
@@ -2183,69 +2161,39 @@ static struct caam_crypto_alg *caam_alg_alloc(struct device *ctrldev,
|
||||
t_alg->class1_alg_type = template->class1_alg_type;
|
||||
t_alg->class2_alg_type = template->class2_alg_type;
|
||||
t_alg->alg_op = template->alg_op;
|
||||
t_alg->ctrldev = ctrldev;
|
||||
|
||||
return t_alg;
|
||||
}
|
||||
|
||||
static int __init caam_algapi_init(void)
|
||||
{
|
||||
struct device_node *dev_node;
|
||||
struct platform_device *pdev;
|
||||
struct device *ctrldev;
|
||||
struct caam_drv_private *priv;
|
||||
int i = 0, err = 0;
|
||||
|
||||
dev_node = of_find_compatible_node(NULL, NULL, "fsl,sec-v4.0");
|
||||
if (!dev_node) {
|
||||
dev_node = of_find_compatible_node(NULL, NULL, "fsl,sec4.0");
|
||||
if (!dev_node)
|
||||
return -ENODEV;
|
||||
}
|
||||
|
||||
pdev = of_find_device_by_node(dev_node);
|
||||
if (!pdev)
|
||||
return -ENODEV;
|
||||
|
||||
ctrldev = &pdev->dev;
|
||||
priv = dev_get_drvdata(ctrldev);
|
||||
of_node_put(dev_node);
|
||||
|
||||
/*
|
||||
* If priv is NULL, it's probably because the caam driver wasn't
|
||||
* properly initialized (e.g. RNG4 init failed). Thus, bail out here.
|
||||
*/
|
||||
if (!priv)
|
||||
return -ENODEV;
|
||||
|
||||
INIT_LIST_HEAD(&priv->alg_list);
|
||||
|
||||
atomic_set(&priv->tfm_count, -1);
|
||||
INIT_LIST_HEAD(&alg_list);
|
||||
|
||||
/* register crypto algorithms the device supports */
|
||||
for (i = 0; i < ARRAY_SIZE(driver_algs); i++) {
|
||||
/* TODO: check if h/w supports alg */
|
||||
struct caam_crypto_alg *t_alg;
|
||||
|
||||
t_alg = caam_alg_alloc(ctrldev, &driver_algs[i]);
|
||||
t_alg = caam_alg_alloc(&driver_algs[i]);
|
||||
if (IS_ERR(t_alg)) {
|
||||
err = PTR_ERR(t_alg);
|
||||
dev_warn(ctrldev, "%s alg allocation failed\n",
|
||||
driver_algs[i].driver_name);
|
||||
pr_warn("%s alg allocation failed\n",
|
||||
driver_algs[i].driver_name);
|
||||
continue;
|
||||
}
|
||||
|
||||
err = crypto_register_alg(&t_alg->crypto_alg);
|
||||
if (err) {
|
||||
dev_warn(ctrldev, "%s alg registration failed\n",
|
||||
pr_warn("%s alg registration failed\n",
|
||||
t_alg->crypto_alg.cra_driver_name);
|
||||
kfree(t_alg);
|
||||
} else
|
||||
list_add_tail(&t_alg->entry, &priv->alg_list);
|
||||
list_add_tail(&t_alg->entry, &alg_list);
|
||||
}
|
||||
if (!list_empty(&priv->alg_list))
|
||||
dev_info(ctrldev, "%s algorithms registered in /proc/crypto\n",
|
||||
(char *)of_get_property(dev_node, "compatible", NULL));
|
||||
if (!list_empty(&alg_list))
|
||||
pr_info("caam algorithms registered in /proc/crypto\n");
|
||||
|
||||
return err;
|
||||
}
|
||||
|
Reference in New Issue
Block a user