From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754493AbbFSIY1 (ORCPT ); Fri, 19 Jun 2015 04:24:27 -0400 Received: from down.free-electrons.com ([37.187.137.238]:58753 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753740AbbFSIYP (ORCPT ); Fri, 19 Jun 2015 04:24:15 -0400 Date: Fri, 19 Jun 2015 10:24:07 +0200 From: Thomas Petazzoni To: Herbert Xu Cc: Boris Brezillon , "David S. Miller" , linux-crypto@vger.kernel.org, Arnaud Ebalard , Tawfik Bayouk , Lior Amsalem , Nadav Haklai , Eran Ben-Avi , Gregory CLEMENT , Jason Cooper , Sebastian Hesselbarth , Andrew Lunn , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Jason Gunthorpe , Imre Kaloz , Russell King - ARM Linux Subject: Re: [PATCH v7 04/14] crypto: add a new driver for Marvell's CESA Message-ID: <20150619102407.01187ff0@free-electrons.com> In-Reply-To: <20150619064256.GA10246@gondor.apana.org.au> References: <1434635190-27733-1-git-send-email-boris.brezillon@free-electrons.com> <1434635190-27733-5-git-send-email-boris.brezillon@free-electrons.com> <20150619064256.GA10246@gondor.apana.org.au> Organization: Free Electrons X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.25; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello Herbert, On Fri, 19 Jun 2015 14:42:56 +0800, Herbert Xu wrote: > On Thu, Jun 18, 2015 at 03:46:20PM +0200, Boris Brezillon wrote: > > > > + dram = mv_mbus_dram_info_nooverlap(); > > This doesn't compile because it hasn't made it into mainline yet. > I'm afraid this patch-set will have to wait until that happens. We have two options here: 1/ We can ask the mach-mvebu maintainers to provide a branch with just the commit implementing mv_mbus_dram_info_nooverlap(), and you merge that branch into your tree as a dependency. 2/ We temporarily revert back to using mv_mbus_dram_info(). For the vast majority of platforms, this is OK. It's only if you have 4 GB of RAM or more that it causes problem. And then we switch to mv_mbus_dram_info_nooverlap() as part of the -rc cycle. (2) is probably easier. Herbert, what do you think? Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Subject: Re: [PATCH v7 04/14] crypto: add a new driver for Marvell's CESA Date: Fri, 19 Jun 2015 10:24:07 +0200 Message-ID: <20150619102407.01187ff0@free-electrons.com> References: <1434635190-27733-1-git-send-email-boris.brezillon@free-electrons.com> <1434635190-27733-5-git-send-email-boris.brezillon@free-electrons.com> <20150619064256.GA10246@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Boris Brezillon , "David S. Miller" , linux-crypto@vger.kernel.org, Arnaud Ebalard , Tawfik Bayouk , Lior Amsalem , Nadav Haklai , Eran Ben-Avi , Gregory CLEMENT , Jason Cooper , Sebastian Hesselbarth , Andrew Lunn , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Jason Gunthorpe , Imre Kaloz , Russell King - ARM Linux Return-path: Received: from down.free-electrons.com ([37.187.137.238]:58753 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753740AbbFSIYP (ORCPT ); Fri, 19 Jun 2015 04:24:15 -0400 In-Reply-To: <20150619064256.GA10246@gondor.apana.org.au> Sender: linux-crypto-owner@vger.kernel.org List-ID: Hello Herbert, On Fri, 19 Jun 2015 14:42:56 +0800, Herbert Xu wrote: > On Thu, Jun 18, 2015 at 03:46:20PM +0200, Boris Brezillon wrote: > > > > + dram = mv_mbus_dram_info_nooverlap(); > > This doesn't compile because it hasn't made it into mainline yet. > I'm afraid this patch-set will have to wait until that happens. We have two options here: 1/ We can ask the mach-mvebu maintainers to provide a branch with just the commit implementing mv_mbus_dram_info_nooverlap(), and you merge that branch into your tree as a dependency. 2/ We temporarily revert back to using mv_mbus_dram_info(). For the vast majority of platforms, this is OK. It's only if you have 4 GB of RAM or more that it causes problem. And then we switch to mv_mbus_dram_info_nooverlap() as part of the -rc cycle. (2) is probably easier. Herbert, what do you think? Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Subject: Re: [PATCH v7 04/14] crypto: add a new driver for Marvell's CESA Date: Fri, 19 Jun 2015 10:24:07 +0200 Message-ID: <20150619102407.01187ff0@free-electrons.com> References: <1434635190-27733-1-git-send-email-boris.brezillon@free-electrons.com> <1434635190-27733-5-git-send-email-boris.brezillon@free-electrons.com> <20150619064256.GA10246@gondor.apana.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150619064256.GA10246@gondor.apana.org.au> Sender: linux-crypto-owner@vger.kernel.org To: Herbert Xu Cc: Boris Brezillon , "David S. Miller" , linux-crypto@vger.kernel.org, Arnaud Ebalard , Tawfik Bayouk , Lior Amsalem , Nadav Haklai , Eran Ben-Avi , Gregory CLEMENT , Jason Cooper , Sebastian Hesselbarth , Andrew Lunn , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Jason Gunthorpe , Imre Kaloz , Russell King - ARM Linux List-Id: devicetree@vger.kernel.org Hello Herbert, On Fri, 19 Jun 2015 14:42:56 +0800, Herbert Xu wrote: > On Thu, Jun 18, 2015 at 03:46:20PM +0200, Boris Brezillon wrote: > > > > + dram = mv_mbus_dram_info_nooverlap(); > > This doesn't compile because it hasn't made it into mainline yet. > I'm afraid this patch-set will have to wait until that happens. We have two options here: 1/ We can ask the mach-mvebu maintainers to provide a branch with just the commit implementing mv_mbus_dram_info_nooverlap(), and you merge that branch into your tree as a dependency. 2/ We temporarily revert back to using mv_mbus_dram_info(). For the vast majority of platforms, this is OK. It's only if you have 4 GB of RAM or more that it causes problem. And then we switch to mv_mbus_dram_info_nooverlap() as part of the -rc cycle. (2) is probably easier. Herbert, what do you think? Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: thomas.petazzoni@free-electrons.com (Thomas Petazzoni) Date: Fri, 19 Jun 2015 10:24:07 +0200 Subject: [PATCH v7 04/14] crypto: add a new driver for Marvell's CESA In-Reply-To: <20150619064256.GA10246@gondor.apana.org.au> References: <1434635190-27733-1-git-send-email-boris.brezillon@free-electrons.com> <1434635190-27733-5-git-send-email-boris.brezillon@free-electrons.com> <20150619064256.GA10246@gondor.apana.org.au> Message-ID: <20150619102407.01187ff0@free-electrons.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello Herbert, On Fri, 19 Jun 2015 14:42:56 +0800, Herbert Xu wrote: > On Thu, Jun 18, 2015 at 03:46:20PM +0200, Boris Brezillon wrote: > > > > + dram = mv_mbus_dram_info_nooverlap(); > > This doesn't compile because it hasn't made it into mainline yet. > I'm afraid this patch-set will have to wait until that happens. We have two options here: 1/ We can ask the mach-mvebu maintainers to provide a branch with just the commit implementing mv_mbus_dram_info_nooverlap(), and you merge that branch into your tree as a dependency. 2/ We temporarily revert back to using mv_mbus_dram_info(). For the vast majority of platforms, this is OK. It's only if you have 4 GB of RAM or more that it causes problem. And then we switch to mv_mbus_dram_info_nooverlap() as part of the -rc cycle. (2) is probably easier. Herbert, what do you think? Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com