All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory CLEMENT <gregory.clement@free-electrons.com>
To: Boris Brezillon <boris.brezillon@free-electrons.com>
Cc: Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
	Kevin Hilman <khilman@kernel.org>,
	linux-arm-kernel@lists.infradead.org,
	Herbert Xu <herbert@gondor.apana.org.au>,
	"David S. Miller" <davem@davemloft.net>,
	linux-crypto@vger.kernel.org,
	Russell King <linux@armlinux.org.uk>,
	Arnaud Ebalard <arno@natisbad.org>,
	Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
	Jason Cooper <jason@lakedaemon.net>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Andrew Lunn <andrew@lunn.ch>
Subject: Re: [PATCH 0/3] crypto: marvell - Remove the old CESA driver
Date: Thu, 12 Oct 2017 16:25:28 +0200	[thread overview]
Message-ID: <87zi8wa1ev.fsf@free-electrons.com> (raw)
In-Reply-To: <20171011131619.30196-1-boris.brezillon@free-electrons.com> (Boris Brezillon's message of "Wed, 11 Oct 2017 15:16:16 +0200")

Hi Boris,
 
 On mer., oct. 11 2017, Boris Brezillon <boris.brezillon@free-electrons.com> wrote:

> Hello,
>
> It's been several releases since we added a new driver to support the
> CESA IP (the new driver was introduced in 4.2). It seems most major
> bugs have been discovered and fixed and now is probably a good time to
> kill the old driver and force remaining users to switch to the new one.
>
> This series first add a platform_device_id table to the marvell CESA
> driver so that it can work on non-DT platforms. Then we patch all
> defconfigs to select the new driver instead of the old. And finally we
> remove the old driver.
> These patches have to be applied in this order to make things
> bisectable, so, if this series is accepted it will have to go through
> a single tree (either ARM or crypto).

For the whole series
Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>

And I am OK for this series going through crypto because I don't expect
any changes in the mvebu files so there won't be any merge conflict. But
I am also fine to take the series in my tree, it is up to Herbert Xu.

Thanks,

Gregory


>
> Regards,
>
> Boris
>
> Boris Brezillon (3):
>   crypto: marvell - Add a platform_device_id table
>   ARM: configs: Stop selecting the old CESA driver
>   crypto: marvell - Remove the old mv_cesa driver
>
>  arch/arm/configs/dove_defconfig     |    2 +-
>  arch/arm/configs/multi_v5_defconfig |    2 +-
>  arch/arm/configs/orion5x_defconfig  |    2 +-
>  drivers/crypto/Kconfig              |   22 +-
>  drivers/crypto/Makefile             |    1 -
>  drivers/crypto/marvell/cesa.c       |   13 +-
>  drivers/crypto/mv_cesa.c            | 1216 -----------------------------------
>  drivers/crypto/mv_cesa.h            |  150 -----
>  8 files changed, 12 insertions(+), 1396 deletions(-)
>  delete mode 100644 drivers/crypto/mv_cesa.c
>  delete mode 100644 drivers/crypto/mv_cesa.h
>
> -- 
> 2.11.0
>

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

WARNING: multiple messages have this Message-ID (diff)
From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/3] crypto: marvell - Remove the old CESA driver
Date: Thu, 12 Oct 2017 16:25:28 +0200	[thread overview]
Message-ID: <87zi8wa1ev.fsf@free-electrons.com> (raw)
In-Reply-To: <20171011131619.30196-1-boris.brezillon@free-electrons.com> (Boris Brezillon's message of "Wed, 11 Oct 2017 15:16:16 +0200")

Hi Boris,
 
 On mer., oct. 11 2017, Boris Brezillon <boris.brezillon@free-electrons.com> wrote:

> Hello,
>
> It's been several releases since we added a new driver to support the
> CESA IP (the new driver was introduced in 4.2). It seems most major
> bugs have been discovered and fixed and now is probably a good time to
> kill the old driver and force remaining users to switch to the new one.
>
> This series first add a platform_device_id table to the marvell CESA
> driver so that it can work on non-DT platforms. Then we patch all
> defconfigs to select the new driver instead of the old. And finally we
> remove the old driver.
> These patches have to be applied in this order to make things
> bisectable, so, if this series is accepted it will have to go through
> a single tree (either ARM or crypto).

For the whole series
Acked-by: Gregory CLEMENT <gregory.clement@free-electrons.com>

And I am OK for this series going through crypto because I don't expect
any changes in the mvebu files so there won't be any merge conflict. But
I am also fine to take the series in my tree, it is up to Herbert Xu.

Thanks,

Gregory


>
> Regards,
>
> Boris
>
> Boris Brezillon (3):
>   crypto: marvell - Add a platform_device_id table
>   ARM: configs: Stop selecting the old CESA driver
>   crypto: marvell - Remove the old mv_cesa driver
>
>  arch/arm/configs/dove_defconfig     |    2 +-
>  arch/arm/configs/multi_v5_defconfig |    2 +-
>  arch/arm/configs/orion5x_defconfig  |    2 +-
>  drivers/crypto/Kconfig              |   22 +-
>  drivers/crypto/Makefile             |    1 -
>  drivers/crypto/marvell/cesa.c       |   13 +-
>  drivers/crypto/mv_cesa.c            | 1216 -----------------------------------
>  drivers/crypto/mv_cesa.h            |  150 -----
>  8 files changed, 12 insertions(+), 1396 deletions(-)
>  delete mode 100644 drivers/crypto/mv_cesa.c
>  delete mode 100644 drivers/crypto/mv_cesa.h
>
> -- 
> 2.11.0
>

-- 
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

  parent reply	other threads:[~2017-10-12 14:25 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-11 13:16 [PATCH 0/3] crypto: marvell - Remove the old CESA driver Boris Brezillon
2017-10-11 13:16 ` Boris Brezillon
2017-10-11 13:16 ` [PATCH 1/3] crypto: marvell - Add a platform_device_id table Boris Brezillon
2017-10-11 13:16   ` Boris Brezillon
2017-10-11 13:16 ` [PATCH 2/3] ARM: configs: Stop selecting the old CESA driver Boris Brezillon
2017-10-11 13:16   ` Boris Brezillon
2017-10-11 13:16 ` [PATCH 3/3] crypto: marvell - Remove the old mv_cesa driver Boris Brezillon
2017-10-11 13:16   ` Boris Brezillon
2017-10-11 13:56 ` [PATCH 0/3] crypto: marvell - Remove the old CESA driver Jason Cooper
2017-10-11 13:56   ` Jason Cooper
2017-10-12 14:25 ` Gregory CLEMENT [this message]
2017-10-12 14:25   ` Gregory CLEMENT
2017-11-03 14:22 ` Herbert Xu
2017-11-03 14:22   ` Herbert Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87zi8wa1ev.fsf@free-electrons.com \
    --to=gregory.clement@free-electrons.com \
    --cc=andrew@lunn.ch \
    --cc=arnd@arndb.de \
    --cc=arno@natisbad.org \
    --cc=boris.brezillon@free-electrons.com \
    --cc=davem@davemloft.net \
    --cc=herbert@gondor.apana.org.au \
    --cc=jason@lakedaemon.net \
    --cc=khilman@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-crypto@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=olof@lixom.net \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=thomas.petazzoni@free-electrons.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.