Hi Fabio,
On 03/06/2018 12:24 AM, Fabio Estevam wrote:
> Hi Vladimir,
>
> On Mon, Mar 5, 2018 at 7:21 PM, Vladimir Zapolskiy <v...@mleia.com> wrote:
>> The driver works well on i.MX31 powered boards with device description
>> taken from board device tree, the only
, apparently
the documentation file has no more potential for further extensions,
nevertheless generalize it by removing explicit RNGC specifics.
Signed-off-by: Vladimir Zapolskiy <v...@mleia.com>
Reviewed-by: Rob Herring <r...@kernel.org>
---
Changes from v1 to v2:
* added i.MX21 RNGA compat
The series is a trivial change to extend Freescale i.MX31 RNGA
driver to work on boards with device description taken from device
tree blob. The change was tested on a legacy LogicPD MX31Lite board.
Vladimir Zapolskiy (2):
dt-bindings: rng: Document Freescale i.MX21 and i.MX31 RNGA compatibles
On 02/27/2018 10:07 PM, Vladimir Zapolskiy wrote:
> On 02/27/2018 09:39 PM, Kim Phillips wrote:
>> On Tue, 27 Feb 2018 18:53:08 +0200
>> Vladimir Zapolskiy <v...@mleia.com> wrote:
>>
>>> On 02/27/2018 05:49 PM, Kim Phillips wrote:
>>>> On Mon, 26 Fe
On 02/27/2018 09:39 PM, Kim Phillips wrote:
> On Tue, 27 Feb 2018 18:53:08 +0200
> Vladimir Zapolskiy <v...@mleia.com> wrote:
>
>> On 02/27/2018 05:49 PM, Kim Phillips wrote:
>>> On Mon, 26 Feb 2018 20:38:49 +0200
>>> Vladimir Zapolskiy <v...@ml
On 02/27/2018 05:49 PM, Kim Phillips wrote:
> On Mon, 26 Feb 2018 20:38:49 +0200
> Vladimir Zapolskiy <v...@mleia.com> wrote:
>
>> +#ifdef CONFIG_OF
>> +static const struct of_device_id mxc_rnga_of_match[] = {
>> +{ .compatible = "fsl
the documentation file has no more potential for further extensions,
nevertheless generalize it by removing explicit RNGC specifics.
Signed-off-by: Vladimir Zapolskiy <v...@mleia.com>
---
.../devicetree/bindings/rng/{imx-rngc.txt => imx-rng.txt} | 10 --
1 file changed, 4 i
The driver works well on i.MX31 powered boards with device description
taken from board device tree, the only change to add to the driver is
the missing OF device id, the affected list of included headers and
platform driver struct are beautified a little.
Signed-off-by: Vladimir Zapolskiy &l
The series is a trivial change to extend Freescale i.MX31 RNGA
driver to work on boards with device description taken from device
tree blob. The change was tested on a legacy LogicPD MX31Lite board.
Vladimir Zapolskiy (2):
dt-bindings: rng: Document Freescale i.MX31 RNGA compatible
hwrng: mxc
ly almost all of your remarks,
> see answers below.
>
> On 22.10.2017 12:18, Vladimir Zapolskiy wrote:
>> Hi Kamil,
>>
>> thank you for updates, I have just a few more comments.
>>
[snip]
>>> +/**
>>> + * s5p_hash_import - import hash state
Hi Kamil,
On 10/24/2017 01:19 PM, Kamil Konieczny wrote:
> Hi Vladimir,
>
> Thank you for review.
>
> On 22.10.2017 12:18, Vladimir Zapolskiy wrote:
>> Hi Kamil,
>>
>> On 10/17/2017 02:28 PM, Kamil Konieczny wrote:
>>> change spaces into tabs in
Hi Kamil,
On 10/17/2017 02:28 PM, Kamil Konieczny wrote:
> change spaces into tabs in defines
Here a grammatically correct sentence in English is welcome.
>
> Signed-off-by: Kamil Konieczny <k.koniec...@partner.samsung.com>
> ---
Please feel free to add a tag
Acked-by: Vla
Hi Kamil,
thank you for updates, I have just a few more comments.
On 10/17/2017 02:28 PM, Kamil Konieczny wrote:
> Add support for MD5, SHA1, SHA256 hash algorithms for Exynos HW.
> It uses the crypto framework asynchronous hash api.
> It is based on omap-sham.c driver.
> S5P has some HW
Hello Kamil,
thank you for the change, please find below a number of minor
review comments.
On 10/09/2017 02:12 PM, Kamil Konieczny wrote:
> Add support for MD5, SHA1, SHA256 hash algorithms for Exynos HW.
> It uses the crypto framework asynchronous hash api.
> It is based on omap-sham.c driver.
On 03/11/2017 08:11 AM, Krzysztof Kozlowski wrote:
> Add Krzysztof Kozlowski and Vladimir Zapolskiy as maintainers of s5p-sss
> driver for handling reviews, testing and getting bug reports from the
> users.
>
> Cc: Vladimir Zapolskiy <v...@mleia.com>
> Cc: Herbert Xu <
Hi Krzysztof,
On 03/10/2017 09:10 PM, Krzysztof Kozlowski wrote:
> Beside developing of this driver recently, I handle also reviews and
> bug reports from users so having a maintainer entry will ensure that I
> will be CC-ed on important emails.
if you assume that the driver needs a special
as mixing usage of such indentation and lack of it.
>When removing indentation, reorder variables in
>reversed-christmas-tree order with first variables being initialized
>ones.
>
> Signed-off-by: Krzysztof Kozlowski <k.kozlow...@samsung.com>
it looks like a trivial and nice ch
rypto: s5p-sss - add S5PV210 advanced crypto engine
> support")
> Fixes: 9e4a1100a445 ("crypto: s5p-sss - Handle unaligned buffers")
> Cc: <sta...@vger.kernel.org>
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
Acked-by: Vladimir Zapolskiy &l
nterrupt bit.
>
> Driver does not provide hash functions so it is safe to remove the hash
> interrupt related code and to not require the interrupt in Device Tree.
>
> Signed-off-by: Krzysztof Kozlowski <k.kozlow...@samsung.com>
Thank you for the change.
Acked-by: V
ung.com>
I see that Herbert have just applied the changes, but anyway I reviewed
them and they are good in my opinion.
Acked-by: Vladimir Zapolskiy <v...@mleia.com>
> ---
> drivers/crypto/s5p-sss.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/cry
NOS
> + depends on ARCH_S5PV210 || ARCH_EXYNOS || COMPILE_TEST
> + depends on HAS_IOMEM && HAS_DMA
> select CRYPTO_AES
> select CRYPTO_BLKCIPHER
> help
>
Acked-by: Vladimir Zapolskiy <v...@mleia.com>
--
With best wishes,
Vlad
On 10.03.2016 03:31, Krzysztof Kozlowski wrote:
> Sort the headers alphabetically to improve readability and to spot
> duplications easier.
>
> Suggested-by: Vladimir Zapolskiy <v...@mleia.com>
> Signed-off-by: Krzysztof Kozlowski <k.kozlow...@samsung.com>
>
> --
On 11.03.2016 16:06, Steffen Trumtrar wrote:
> The driver is ported from Freescales Linux git and can be
> found in the
>
> vendor/freescale/imx_2.6.35_maintain
>
> branch.
>
> According to that code, the RNGC is found on Freescales i.MX3/5 SoCs.
> The i.MX2x actually has an RNGB, which
Hi Krzysztof,
On 07.03.2016 06:05, Krzysztof Kozlowski wrote:
> On 07.03.2016 10:01, Vladimir Zapolskiy wrote:
>> Hi Krzysztof,
>>
>> On 06.03.2016 12:17, Krzysztof Kozlowski wrote:
>>> Remove unneeded inclusion of delay.h and get rid of indentation from
>>
ss etc.
Anyway since it is a valid improvement, please feel free to add my
Acked-by: Vladimir Zapolskiy <v...@mleia.com>
> Signed-off-by: Krzysztof Kozlowski <k...@kernel.org>
> ---
> drivers/crypto/s5p-sss.c | 149
> +++
>
+ goto labels one day,
though it is better to have an officially agreed policy beforehand.
Anyway I do not strictly object against the change, see one my comment
below and feel free to add my
Acked-by: Vladimir Zapolskiy <v...@mleia.com>
> Signed-off-by: Krzysztof Kozlowski &
The change fixes potential oops while accessing iomem on invalid
address, if devm_ioremap_resource() fails due to some reason.
The devm_ioremap_resource() function returns ERR_PTR() and never
returns NULL, which makes useless a following check for NULL.
Signed-off-by: Vladimir Zapolskiy &l
The change fixes potential oops while accessing iomem on invalid
address, if devm_ioremap_resource() fails due to some reason.
The devm_ioremap_resource() function returns ERR_PTR() and never
returns NULL, which makes useless a following check for NULL.
Signed-off-by: Vladimir Zapolskiy &l
Hi James,
On 11.11.2014 16:59, James Hartley wrote:
Hi Vladimir, thanks for the review!
-Original Message-
From: Vladimir Zapolskiy [mailto:vladimir_zapols...@mentor.com]
Sent: 10 November 2014 15:10
To: James Hartley; herb...@gondor.apana.org.au; da...@davemloft.net;
grant.lik
Hello James,
On 10.11.2014 14:10, James Hartley wrote:
This adds support for the Imagination Technologies hash
accelerator that provides hardware acceleration for
SHA1 SHA224 SHA256 and MD5 Hashes.
Signed-off-by: James Hartley james.hart...@imgtec.com
---
[snip]
diff --git
Hi Corentin,
On 21.10.2014 19:25, Corentin LABBE wrote:
On 10/21/14 01:28, Vladimir Zapolskiy wrote:
Hello LABBE,
On 19.10.2014 17:16, LABBE Corentin wrote:
Add support for the Security System included in Allwinner SoC A20.
The Security System is a hardware cryptographic accelerator
Hello LABBE,
On 19.10.2014 17:16, LABBE Corentin wrote:
Add support for the Security System included in Allwinner SoC A20.
The Security System is a hardware cryptographic accelerator that support
AES/MD5/SHA1/DES/3DES/PRNG algorithms.
Signed-off-by: LABBE Corentin clabbe.montj...@gmail.com
merged in.
Its has got reviewed and re-based several times and got acked by a
couple of seniors.
Do you want me to pick these series up?
Cheers,
thank you for this changeset, in my opinion it is quite good. To all changes
Acked-by: Vladimir Zapolskiy v...@mleia.com
--
With best wishes
)
for which
1. AES register are at an offset of 0x200 and
2. hash interrupt is not available
Signed-off-by: Naveen Krishna Ch ch.nav...@samsung.com
CC: Herbert Xu herb...@gondor.apana.org.au
CC: David S. Miller da...@davemloft.net
CC: Vladimir Zapolskiy vzapols...@gmail.com
TO: linux-crypto
)
clk_disable(pdata-clk);
s5p_dev = NULL;
- platform_set_drvdata(pdev, NULL);
return 0;
}
Acked-by: Vladimir Zapolskiy v...@mleia.com
--
With best wishes,
Vladimir
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord
Wadekarvwade...@nvidia.com
Cc: Vladimir Zapolskiyvzapols...@gmail.com
Cc: Eric BĂ©narde...@eukrea.com
Signed-off-by: Jussi Kivilinnajussi.kivili...@mbnet.fi
Acked-by: Vladimir Zapolskiy vladimir_zapols...@mentor.com
Thanks,
Vladimir
--
To unsubscribe from this list: send the line unsubscribe linux
This change adds support for AES encrypting and decrypting using
advanced crypto engine found on Samsung S5PV210 and S5PC110 SoCs.
Signed-off-by: Vladimir Zapolskiy vzapols...@gmail.com
---
drivers/crypto/Kconfig | 11 +
drivers/crypto/Makefile |1 +
drivers/crypto/s5p-sss.c | 701
37 matches
Mail list logo