Re: [RFT PATCH] crypto: ahash.c: Require export/import in ahash

2018-01-16 Thread Bartlomiej Zolnierkiewicz
,10 +457,6 @@ static int crypto_ahash_init_tfm(struct crypto_tfm *tfm) > hash->setkey = alg->setkey; > hash->has_setkey = true; > } > - if (alg->export) > - hash->export = alg->export; > - if (alg->import) > - hash->import = alg->import; > > return 0; > } Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics

Re: [PATCH v4 2/2] crypto: hw_random - Add new Exynos RNG driver

2017-04-10 Thread Bartlomiej Zolnierkiewicz
self with generated random data > periodically and during resuming from system suspend (previously driver > was re-seeding itself again with jiffies). > > Signed-off-by: Krzysztof Kozlowski > Reviewed-by: Stephan Müller > Reviewed-by: PrasannaKumar Muralidharan Reviewed-by

Re: [PATCH 1/3] crypto: hw_random - Add new Exynos RNG driver

2017-03-24 Thread Bartlomiej Zolnierkiewicz
On Friday, March 24, 2017 07:19:34 PM Krzysztof Kozlowski wrote: > On Fri, Mar 24, 2017 at 05:11:25PM +0100, Bartlomiej Zolnierkiewicz wrote: > > On Friday, March 24, 2017 06:46:00 PM Krzysztof Kozlowski wrote: > > > I really do not like global or file-scope variables. I do not

Re: [PATCH 1/3] crypto: hw_random - Add new Exynos RNG driver

2017-03-24 Thread Bartlomiej Zolnierkiewicz
On Friday, March 24, 2017 06:46:00 PM Krzysztof Kozlowski wrote: > On Fri, Mar 24, 2017 at 04:26:47PM +0100, Bartlomiej Zolnierkiewicz wrote: > > > > Hi, > > > > Firstly, thanks for working on this. > > > > The patch looks fine overall for me, some re

Re: [PATCH 1/3] crypto: hw_random - Add new Exynos RNG driver

2017-03-24 Thread Bartlomiej Zolnierkiewicz
ev) > +{ > + struct exynos_rng_dev *rng; > + struct resource *res; > + int ret; > + > + if (exynos_rng_dev) > + return -EEXIST; How this condition could ever happen? The probe function will never be called twice. Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics

Re: [PATCH 3/4] crypto: s5p-sss - Document the struct s5p_aes_dev

2017-03-17 Thread Bartlomiej Zolnierkiewicz
On Friday, March 17, 2017 04:49:21 PM Krzysztof Kozlowski wrote: > Add kernel-doc to s5p_aes_dev structure. > > Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics

Re: [PATCH 2/4] crypto: s5p-sss - Remove unused variant field from state container

2017-03-17 Thread Bartlomiej Zolnierkiewicz
On Friday, March 17, 2017 04:49:20 PM Krzysztof Kozlowski wrote: > The driver uses type of device (variant) only during probe so there is > no need to store it for later. > > Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Best regards, -- Bartlomiej Zo

Re: [PATCH 1/4] crypto: s5p-sss - Close possible race for completed requests

2017-03-17 Thread Bartlomiej Zolnierkiewicz
completion might try to operate on new crypto request. > > Cc: # v4.10.x > Fixes: 28b62b145868 ("crypto: s5p-sss - Fix spinlock recursion on LRW(AES)") > Signed-off-by: Krzysztof Kozlowski Reviewed-by: Bartlomiej Zolnierkiewicz Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics

Re: [PATCH 4/4] crypto: s5p-sss - Use mutex instead of spinlock

2017-03-17 Thread Bartlomiej Zolnierkiewicz
pto_dequeue_request(&dev->queue); > > if (!async_req) { > dev->busy = false; > - spin_unlock_irqrestore(&dev->lock, flags); > + mutex_unlock(&dev->lock); > return; > } > - spin_unlock_irqrestore(&