linux-kernel - RE: [PATCH v5] mmc: dw_mmc: Add IDMAC 64-bit address mode support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Thu, 9 Oct 2014 07:37:40 +0000
From:	Prabu Thangamuthu <Prabu.T@...opsys.com>
To:	Alim Akhtar <alim.akhtar@...il.com>,
	Prabu Thangamuthu <Prabu.T@...opsys.com>
CC:	Seungwon Jeon <tgih.jun@...sung.com>,
	Jaehoon Chung <jh80.chung@...sung.com>,
	Chris Ball <chris@...ntf.net>,
	Ulf Hansson <ulf.hansson@...aro.org>,
	"linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Manjunath M Bettegowda <Manjunath.MB@...opsys.com>
Subject: RE: [PATCH v5] mmc: dw_mmc: Add IDMAC 64-bit address mode support

Hi Alim,

Thank you for reviewing and testing our Patch.

On Wed, Oct 8, 2014 at 7:17 PM, Alim Akhtar <alim.akhtar@...il.com > wrote:
> Hi Prabu,
> 
> On Tue, Oct 7, 2014 at 5:17 PM, Prabu Thangamuthu
> <Prabu.T@...opsys.com> wrote:
> > Synopsys DW_MMC IP core supports Internal DMA Controller with 64-bit
> address mode from IP version 2.70a onwards.
> > Updated the dw_mmc driver to support IDMAC 64-bit addressing mode.
> >
> > Tested the features in DW_MMC core v2.70a and v2.40a with HAPS-51
> setup and driver is working fine.
> >
> > Signed-off-by: Prabu Thangamuthu <prabu.t@...opsys.com>
> > ---
> With a minor nit below, this looks good to me.
> 
> So, Reviewed-by: Alim Akhtar <alim.akhtar@...sung.com>
> 
> Have tested this on exynos7 platform which has dw_mmc controller version
> 2.70a, Have tested this on exynos5800 Pi board and found no issue.
> 
> So, Tested-by: Alim Akhtar <alim.akhtar@...sung.com>
> 
> > Change log v5:
> >         - Recreated the patch against linux-next as this patch is
> > required for another patch
> > http://www.spinics.net/lists/arm-kernel/msg357985.html
> >
> yes, we are using this on exynos7 platform. Thanks for re-basing and re-
> posting .
> > +               /* Set the last descriptor as the end-of-ring descriptor */
> > +               p->des3 = host->sg_dma;
> > +               p->des0 = IDMAC_DES0_ER;
> > +       }
> > +
> > +       mci_writel(host, BMOD, SDMMC_IDMAC_SWRESET);
> Any reason to use this instead of earlier use of  dw_mci_idmac_reset(host)?
Thanks for pointing this. It's a typo. I will resend the patch with this modification.

Regards,
Prabu Thangamuthu.