From mboxrd@z Thu Jan 1 00:00:00 1970 From: Santosh Shilimkar Subject: Re: [PATCH 0/7] crypto: omap-sham updates Date: Sun, 21 Oct 2012 11:58:36 +0530 Message-ID: <50839614.1080607@ti.com> References: <1350683640-15044-1-git-send-email-mgreer@animalcreek.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from devils.ext.ti.com ([198.47.26.153]:33563 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750806Ab2JUG3M (ORCPT ); Sun, 21 Oct 2012 02:29:12 -0400 In-Reply-To: <1350683640-15044-1-git-send-email-mgreer@animalcreek.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Mark A. Greer" Cc: linux-omap@vger.kernel.org, linux-arm-kernel@listse.infradead.orgp Mark, On Saturday 20 October 2012 03:23 AM, Mark A. Greer wrote: > From: "Mark A. Greer" > > This series updates the crypto omap-sham driver and supporting > infrastructure. > > Notes: > > a) Based on current k.o. c9623de (Merge branch 'v4l_for_linus' > of git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media) > > b) These have only been tested on an omap2420 h4 and an am37x evm. If you > have different hardware available and a few minutes, please test them. > A quick and easy test is to enable tcrypt as a module > (CONFIG_CRYPTO_TEST=m), boot, then run 'modprobe tcrypt sec=2 mode=403'. > 'CONFIG_CRYPTO_SHA1' and 'CONFIG_CRYPTO_DEV_OMAP_SHAM' also have to be > enabled. A quick 'grep omap-sham /proc/interrupts' will tell you if > the omap-sham driver was really used. > > c) To test these patches, you will likely need... > i) The patch included here: > http://marc.info/?l=kernel-janitors&m=134910841909057&w=2 > ii) This patch from linux-omap/master: > 27615a9 (ARM: OMAP: Trivial driver changes to remove include > plat/cpu.h) > iii) This patch from Paul Walmsley: > http://www.spinics.net/lists/linux-omap/msg79436.html > > d) If you prefer, a version you can test is available at > git@github.com:mgreeraz/linux-mag.git mag/wip/crypto-test > > e) There is a reduction in DMA performance after switching to dmaengine > (see http://www.spinics.net/lists/linux-omap/msg79855.html) > > f) Many thanks to Jon Hunter for testing on his omap2420 h4. > > Mark A. Greer (7): > ARM: OMAP2xxx: hwmod: Convert SHAM crypto device data to hwmod > ARM: OMAP2xxx: hwmod: Add DMA information for SHAM module > ARM: OMAP3xxx: hwmod: Convert SHAM crypto device data to hwmod > ARM: OMAP2+: Remove unnecessary message when no SHA IP is present > crypto: omap-sham: Convert to use pm_runtime API > crypto: omap-sham: Add code to use dmaengine API > crypto: omap_sham: Remove usage of private DMA API > Thanks for the series and oveall its looks pretty good to me. The DMA performance is understandable with need of PREFETCH support in OMAP DMA engine driver. IIRC, crypto was the only driver used this feature. From the history mostly we need to restrict the prefetch usage to applicable client driver like crypto and hence adding such parameter/api support might be the direction to go forward. Lets see what Peter reports for audio as discussed on other thread. Regards Santosh