Re: [PATCH v3 1/2] i.MX27: Add clock support for SAHARA2.

2013-03-10 Thread Herbert Xu
On Tue, Mar 05, 2013 at 08:48:19AM +0100, javier Martin wrote:
 Hi Sascha,
 
 On 4 March 2013 08:53, Sascha Hauer s.ha...@pengutronix.de wrote:
  On Fri, Mar 01, 2013 at 12:37:52PM +0100, Javier Martin wrote:
 
  Signed-off-by: Javier Martin javier.mar...@vista-silicon.com
  ---
   arch/arm/mach-imx/clk-imx27.c |2 ++
   1 file changed, 2 insertions(+)
 
  diff --git a/arch/arm/mach-imx/clk-imx27.c b/arch/arm/mach-imx/clk-imx27.c
  index 4c1d1e4..0b9664a 100644
  --- a/arch/arm/mach-imx/clk-imx27.c
  +++ b/arch/arm/mach-imx/clk-imx27.c
  @@ -253,6 +253,8 @@ int __init mx27_clocks_init(unsigned long fref)
clk_register_clkdev(clk[nfc_baud_gate], NULL, imx27-nand.0);
clk_register_clkdev(clk[vpu_baud_gate], per, coda-imx27.0);
clk_register_clkdev(clk[vpu_ahb_gate], ahb, coda-imx27.0);
  + clk_register_clkdev(clk[sahara_ahb_gate], ahb, sahara-imx27.0);
  + clk_register_clkdev(clk[sahara_ipg_gate], ipg, sahara-imx27.0);
 
  One of the first patches I want to push upstream for the next merge
  window is this one:
 
  https://patchwork.kernel.org/patch/1817741/
 
  So the change above becomes unnecessary
 
 Very good. Then this patch can be safely dropped.

So should I take this patch or not?

Thanks,
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/4 v4] ARM: AT91SAM9G45: same platform data structure for all crypto peripherals

2013-03-10 Thread Herbert Xu
On Wed, Feb 20, 2013 at 05:10:23PM +0100, Nicolas Royer wrote:
 Only AES use DMA in AT91SAM9G45 (TDES and SHA use PDC).
 
 However latest Atmel TDES and SHA IP releases use DMA instead of PDC.
   -- Atmel TDES and SHA drivers need DMA platform data for those IP releases.
 
 Goal of this patch is to use the same platform data structure for all Atmel
 crypto peripherals. This structure contains information about DMA interface.
 
 Signed-off-by: Nicolas Royer nico...@eukrea.com
 Acked-by: Nicolas Ferre nicolas.fe...@atmel.com
 Acked-by: Eric Bénard e...@eukrea.com
 Tested-by: Eric Bénard e...@eukrea.com

All applied.  Thanks!
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] Update the links to the white papers on CRC32C calculations with PCLMULQDQ instructions.

2013-03-10 Thread Herbert Xu
On Thu, Feb 21, 2013 at 11:04:22AM -0800, Tim Chen wrote:
 Herbert,
 
 The following patch update the stale link to the CRC32C white paper 
 that was referenced.

Patch applied.
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] crypto: user - constify netlink dispatch table

2013-03-10 Thread Herbert Xu
On Sun, Feb 24, 2013 at 02:09:12PM +0100, Mathias Krause wrote:
 There is no need to modify the netlink dispatch table at runtime and
 making it const even makes the resulting object file slightly smaller.
 
 Cc: Steffen Klassert steffen.klass...@secunet.com
 Signed-off-by: Mathias Krause mini...@googlemail.com

Patch applied.  Thanks Mathias!
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] crypto: bfin_crc: Fix possible NULL pointer dereference

2013-03-10 Thread Herbert Xu
On Mon, Feb 25, 2013 at 03:57:39AM +0530, Syam Sidhardhan wrote:
 If we define dev_dbg(), then there is a possible NULL pointer
 dereference.
 
 Signed-off-by: Syam Sidhardhan s.s...@samsung.com

Patch applied.  Thanks!
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 2/2] omap-aes: Use pm_runtime_put instead of pm_runtime_put_sync in tasklet

2013-03-10 Thread Herbert Xu
On Tue, Feb 26, 2013 at 04:06:36PM -0700, Mark A. Greer wrote:
 On Tue, Feb 26, 2013 at 10:04:32AM -0600, agnel.j...@gmail.com wrote:
  From: Joel A Fernandes joelag...@ti.com
  
  After DMA is complete, the omap_aes_finish_req function is called as
  a part of the done_task tasklet. During this its atomic and any calls
  to pm functions should not assume they wont sleep.
  
  The patch replaces a call to pm_runtime_put_sync (which can sleep) with
  pm_runtime_put thus fixing a kernel panic observed on AM33xx SoC during
  AES operation.
  
  Tested on an AM33xx SoC device (beaglebone board).
  To reproduce the problem, I used the tcrypt kernel module as:
  modprobe tcrypt sec=2 mode=500
  
  Signed-off-by: Joel A Fernandes joelag...@ti.com
  Cc: Herbert Xu herb...@gondor.apana.org.au
  Cc: David S. Miller da...@davemloft.net
  Cc: Mark A. Greer mgr...@animalcreek.com
  ---
   drivers/crypto/omap-aes.c |2 +-
   1 files changed, 1 insertions(+), 1 deletions(-)
  
  diff --git a/drivers/crypto/omap-aes.c b/drivers/crypto/omap-aes.c
  index cf57866..8450bfd2 100644
  --- a/drivers/crypto/omap-aes.c
  +++ b/drivers/crypto/omap-aes.c
  @@ -636,7 +636,7 @@ static void omap_aes_finish_req(struct omap_aes_dev 
  *dd, int err)
   
  pr_debug(err: %d\n, err);
   
  -   pm_runtime_put_sync(dd-dev);
  +   pm_runtime_put(dd-dev);
  dd-flags = ~FLAGS_BUSY;
   
  req-base.complete(req-base, err);
 
 Acked-by: Mark A. Greer mgr...@animalcreek.com

Both patches applied.  Thanks!
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/2] crypto: omap-aes: Use module_platform_driver macro

2013-03-10 Thread Herbert Xu
On Mon, Mar 04, 2013 at 03:09:42PM +0530, Sachin Kamat wrote:
 module_platform_driver() makes the code simpler by eliminating boilerplate
 code.
 
 Signed-off-by: Sachin Kamat sachin.ka...@linaro.org

Bot pathces applied.
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] crypto: caam - fix typo CRYPTO_AHASH

2013-03-10 Thread Herbert Xu
On Thu, Mar 07, 2013 at 06:51:27PM -0600, Kim Phillips wrote:
 On Tue, 5 Mar 2013 14:33:16 +0100
 Paul Bolle pebo...@tiscali.nl wrote:
 
  The Kconfig entry for CAAM's hash algorithm implementations has always
  selected CRYPTO_AHASH. But there's no corresponding Kconfig symbol.
  
  It seems it was intended to select CRYPTO_HASH, like other crypto
  drivers do. That would apparently (indirectly) select CRYPTO_HASH2,
  which would enable the ahash functionality this driver uses.
  
  Signed-off-by: Paul Bolle pebo...@tiscali.nl
  ---
 
 Reviewed-by: Kim Phillips kim.phill...@freescale.com

Patch applied.
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 1/2] i.MX27: Add clock support for SAHARA2.

2013-03-10 Thread Herbert Xu
On Mon, Mar 11, 2013 at 12:08:56AM +0100, Sascha Hauer wrote:
 On Sun, Mar 10, 2013 at 04:34:01PM +0800, Herbert Xu wrote:
   
https://patchwork.kernel.org/patch/1817741/
   
So the change above becomes unnecessary
   
   Very good. Then this patch can be safely dropped.
  
  So should I take this patch or not?
 
 This clk patch, no. The sahara patch, yes, if it is fine for you.

But will the second patch work fine without the first?

Thanks,
-- 
Email: Herbert Xu herb...@gondor.apana.org.au
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line unsubscribe linux-crypto in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html