RE: [PATCHv2 0/3] crypto: inside-secure - broaden driver scope

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Antoine,

> -----Original Message-----
> From: antoine.tenart@xxxxxxxxxxx <antoine.tenart@xxxxxxxxxxx>
> Sent: Friday, July 26, 2019 2:02 PM
> To: Pascal Van Leeuwen <pvanleeuwen@xxxxxxxxxxxxxx>
> Cc: linux-crypto@xxxxxxxxxxxxxxx; antoine.tenart@xxxxxxxxxxx; herbert@xxxxxxxxxxxxxxxxxxx; davem@xxxxxxxxxxxxx
> Subject: Re: [PATCHv2 0/3] crypto: inside-secure - broaden driver scope
> 
> Hi Pascal,
> 
> On Fri, Jul 26, 2019 at 11:33:07AM +0000, Pascal Van Leeuwen wrote:
> >
> > Just a gentle ping to remind people that this patch set - which incorporates the feedback I
> > got on an earlier version thereof - has been pending for over a month now without
> > receiving any feedback on it whatsoever ...
> 
> I do not recall seeing this series and somehow I cannot find it in any
> of my mailboxes or in patchwork. Would you care to send it again ?
> 
> I'm not sure if the issue with this series is on my side or if there was
> an issue while sending it.
> 
I can resend them as is, but I'm afraid I will then I will face the same problem of 
them not being added to patchwork ...

> Thanks!
> Antoine
> 
> > > From: Pascal van Leeuwen <pvanleeuwen@xxxxxxxxxxxxxxxx>
> > >
> > > This is a first baby step towards making the inside-secure crypto driver
> > > more broadly useful. The current driver only works for Marvell Armada HW
> > > and requires proprietary firmware, only available under NDA from Marvell,
> > > to be installed. This patch set allows the driver to be used with other
> > > hardware and removes the dependence on that proprietary firmware.
> > >
> > > changes since v1:
> > > - changed dev_info's into dev_dbg to reduce normal verbosity
> > > - terminate all message strings with \n
> > > - use priv->version field strictly to enumerate device context
> > > - fixed some code & comment style issues
> > > - removed EIP97/197 references from messages
> > > - use #if(IS_ENABLED(CONFIG_PCI)) to remove all PCI related code
> > > - use #if(IS_ENABLED(CONFIG_OF)) to remove all device tree related code
> > > - do not inline the minifw but read it from /lib/firmware instead
> > >
> > > Pascal van Leeuwen (3):
> > >   crypto: inside-secure - make driver selectable for non-Marvell
> > >     hardware
> > >   crypto: inside-secure - add support for PCI based FPGA development
> > >     board
> > >   crypto: inside-secure - add support for using the EIP197 without
> > >     vendor firmware
> > >
> > >  drivers/crypto/Kconfig                         |  12 +-
> > >  drivers/crypto/inside-secure/safexcel.c        | 748 +++++++++++++++++--------
> > >  drivers/crypto/inside-secure/safexcel.h        |  36 +-
> > >  drivers/crypto/inside-secure/safexcel_cipher.c |  11 -
> > >  drivers/crypto/inside-secure/safexcel_hash.c   |  12 -
> > >  drivers/crypto/inside-secure/safexcel_ring.c   |   3 +-
> > >  6 files changed, 569 insertions(+), 253 deletions(-)
> > >
> > > --
> > > 1.8.3.1
> 
> --
> Antoine Ténart, Bootlin
> Embedded Linux and Kernel engineering
>

Regards,
Pascal van Leeuwen
Silicon IP Architect, Multi-Protocol Engines @ Verimatrix
www.insidesecure.com




[Index of Archives]     [Kernel]     [Gnu Classpath]     [Gnu Crypto]     [DM Crypt]     [Netfilter]     [Bugtraq]

  Powered by Linux