Re: [PATCH v3] OMAP: AES: Don't idle/start AES device between Encrypt operations

2013-05-29 Thread Santosh Shilimkar
On Wednesday 29 May 2013 02:59 PM, Joel A Fernandes wrote: > Calling runtime PM API for every block causes serious perf hit to > crypto operations that are done on a long buffer. > As crypto is performed on a page boundary, encrypting large buffers can > cause a series of crypto operations divided

[PATCH v3] OMAP: AES: Don't idle/start AES device between Encrypt operations

2013-05-29 Thread Joel A Fernandes
Calling runtime PM API for every block causes serious perf hit to crypto operations that are done on a long buffer. As crypto is performed on a page boundary, encrypting large buffers can cause a series of crypto operations divided by page. The runtime PM API is also called those many times. We ca

Re: [PATCH 20/39] usb: musb: ux500: move channel number knowledge into the driver

2013-05-29 Thread Felipe Balbi
On Wed, May 15, 2013 at 10:51:43AM +0100, Lee Jones wrote: > For all ux500 based platforms the maximum number of end-points are used. > Move this knowledge into the driver so we can relinquish the burden from > platform data. This also removes quite a bit of complexity from the driver > and will ai

Re: [PATCH v2] OMAP: AES: Don't idle/start AES device between Encrypt operations

2013-05-29 Thread Kevin Hilman
Joel A Fernandes writes: > Calling runtime PM API for every block causes serious perf hit to > crypto operations that are done on a long buffer. > As crypto is performed on a page boundary, encrypting large buffers can > cause a series of crypto operations divided by page. The runtime PM API > is

Re: [PATCH 20/39] usb: musb: ux500: move channel number knowledge into the driver

2013-05-29 Thread Linus Walleij
On Tue, May 28, 2013 at 6:27 PM, Felipe Balbi wrote: > On Wed, May 15, 2013 at 07:18:01PM +0200, Linus Walleij wrote: >> I can't merge any of this without Felipes ACKs in any >> case. > > Do you want to take this yourself ? I haven't fully read the series yet, > but seems like this depends on the

[PATCH 4/4] crypto: dcp: Staticize local symbols

2013-05-29 Thread Sachin Kamat
These symbols are referenced only in this file and hence should be static. Signed-off-by: Sachin Kamat --- drivers/crypto/dcp.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/crypto/dcp.c b/drivers/crypto/dcp.c index 49dd560..a8a7dd4 100644 --- a/drivers/c

[PATCH 3/4] crypto: dcp: Use NULL instead of 0

2013-05-29 Thread Sachin Kamat
Use NULL instead of 0 for pointer variables. Signed-off-by: Sachin Kamat --- drivers/crypto/dcp.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/dcp.c b/drivers/crypto/dcp.c index 9e91c2c..49dd560 100644 --- a/drivers/crypto/dcp.c +++ b/drivers/crypto/dcp.c

[PATCH 2/4] crypto: dcp: Use devm_* APIs

2013-05-29 Thread Sachin Kamat
devm_* APIs are device managed and make cleanup and exit code simpler. Signed-off-by: Sachin Kamat Cc: Tobias Rauter --- drivers/crypto/dcp.c | 32 ++-- 1 file changed, 10 insertions(+), 22 deletions(-) diff --git a/drivers/crypto/dcp.c b/drivers/crypto/dcp.c inde

[PATCH 1/4] crypto: dcp: Remove redundant platform_set_drvdata()

2013-05-29 Thread Sachin Kamat
Commit 0998d06310 (device-core: Ensure drvdata = NULL when no driver is bound) removes the need to set driver data field to NULL. Signed-off-by: Sachin Kamat Cc: Tobias Rauter --- This series compile tested against linux-next (20130529). --- drivers/crypto/dcp.c |1 - 1 file changed, 1